diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-10-07 03:33:00 -0300 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-10-21 01:17:46 -0200 |
commit | bae94dc39e9530842379208c4406512d34dc7ef3 (patch) | |
tree | 309b10639370e4bee3036ad960c2041abeecacb3 /drivers/media/video/cx231xx/cx231xx-cards.c | |
parent | 955e6ed843ddddb57cf599584574c505175cd86f (diff) |
[media] cx231xx-417: Fix a gcc warning
gcc didn't like to have i++ inside a complex operation:
drivers/media/video/cx231xx/cx231xx-417.c: In function ‘cx231xx_load_firmware’:
drivers/media/video/cx231xx/cx231xx-417.c:1059: warning: operation on ‘i’ may be undefined
drivers/media/video/cx231xx/cx231xx-417.c:1061: warning: operation on ‘i’ may be undefined
drivers/media/video/cx231xx/cx231xx-417.c:1063: warning: operation on ‘i’ may be undefined
Btw, I agree with gcc, as we're using i and i++ at the same operation and,
depending on how optimization may occur, it may produce a wrong code.
While here, fix CodingStyle issues on the changed code.
Acked-by: Sri Deevi <Srinivasa.Deevi@conexant.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/cx231xx/cx231xx-cards.c')
0 files changed, 0 insertions, 0 deletions