arkfb: misplaced parentheses?
PATCH 1/1 arkfb: misplaced parentheses?
I think below is what was intended? otherwise we could as well have written:
regval || (code[0] & 4) ? 0x80 : 0)
But it doesn't seem likely that regval should be considered as boolean
please review. -------------------------->8------------------8<--------------------------- fix misplaced parentheses
---
drivers/video/arkfb.c +2 -2
@@ -470,7 +470,7 @@ static void ark_dac_read_regs(void *data, u8 *code, int count)while (count !=0 ) { -vga_wseq (NULL ,0x1C , regval | (code[0 ] &4 ) ?0x80 :0 ); +vga_wseq (NULL ,0x1C , regval | (code[0 ] &4 ?0x80 :0 )); code[1 ] =vga_r (NULL , dac_regs[code[0 ] &3 ]); count--; code +=2 ;
@@ -485,7 +485,7 @@ static void ark_dac_write_regs(void *data, u8 *code, int count)while (count !=0 ) { -vga_wseq (NULL ,0x1C , regval | (code[0 ] &4 ) ?0x80 :0 ); +vga_wseq (NULL ,0x1C , regval | (code[0 ] &4 ?0x80 :0 ));vga_w (NULL , dac_regs[code[0 ] &3 ], code[1 ]); count--; code +=2 ;
------------------------------------------------------------------------------
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
Quoted 1 line · Roel Kluin
I think below is what was intended?
Yes. By coincidence, the curent code also works, but it is definitely better to fix it.
Quoted 1 line
Signed-off-by Roel Kluin <[email protected]>
Signature · 5 lines
Elen sila lumenn' omentielvo Ondrej 'SanTiago' Zajicek (email: [email protected]) OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, wwwkeys.pgp.net) "To err is human -- to blame it on a computer is even more so."
------------------------------------------------------------------------------
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
_______________________________________________
Linux-fbdev-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-fbdev-devel