]> Zhao Yanbai Git Server - minix.git/commitdiff
Fix some incorrect testing of bit flags.
authorKees van Reeuwijk <reeuwijk@few.vu.nl>
Tue, 2 Feb 2010 12:24:35 +0000 (12:24 +0000)
committerKees van Reeuwijk <reeuwijk@few.vu.nl>
Tue, 2 Feb 2010 12:24:35 +0000 (12:24 +0000)
drivers/audio/sb16/mixer.c
drivers/fxp/fxp.c

index 770af03ea90f94829c2006b63229e04017248871..f276b1e8776392bc3388540f34cf602d7d99351c 100644 (file)
@@ -172,8 +172,8 @@ PRIVATE int get_set_input(struct inout_ctrl *input, int flag, int channel) {
                mixer_set(input_cmd, mask);
        } else {        /* Get input */
                if (shift > 0) {
-                       input->left = ((mask >> (shift+1)) & 1 == 1 ? ON : OFF);
-                       input->right = ((mask >> shift) & 1 == 1 ? ON : OFF);
+                       input->left = (((mask >> (shift+1)) & 1) == 1 ? ON : OFF);
+                       input->right = (((mask >> shift) & 1) == 1 ? ON : OFF);
                } else {
                        input->left = ((mask & 1) == 1 ? ON : OFF);
                }
@@ -220,8 +220,8 @@ PRIVATE int get_set_output(struct inout_ctrl *output, int flag) {
                mixer_set(MIXER_OUTPUT_CTRL, mask);
        } else {    /* Get input */
                if (shift > 0) {
-                       output->left = ((mask >> (shift+1)) & 1 == 1 ? ON : OFF);
-                       output->right = ((mask >> shift) & 1 == 1 ? ON : OFF);
+                       output->left = (((mask >> (shift+1)) & 1) == 1 ? ON : OFF);
+                       output->right = (((mask >> shift) & 1) == 1 ? ON : OFF);
                } else {
                        output->left = ((mask & 1) == 1 ? ON : OFF);
                }
@@ -250,4 +250,4 @@ PUBLIC int mixer_get(int reg) {
        sb16_outb(MIXER_REG, reg);
        for(i = 0; i < 100; i++);
        return sb16_inb(MIXER_DATA) & 0xff;
-}  
\ No newline at end of file
+}  
index 0f8ca0e603e1d3fbf11f513eaf0f4f835cf26520..e806bda21193f8f946a9323e8931aa59b2084940 100644 (file)
@@ -1482,7 +1482,7 @@ int vectored;
                goto suspend;
        }
 
-       if (!rfd_status & RFDS_OK)
+       if (!(rfd_status & RFDS_OK))
        {
                /* Not OK? What happened? */
                assert(0);
@@ -1662,7 +1662,7 @@ int from_int;
                goto suspend;
        }
 
-       if (!rfd_status & RFDS_OK)
+       if (!(rfd_status & RFDS_OK))
        {
                /* Not OK? What happened? */
                assert(0);