Home
last modified time | relevance | path

Searched hist:"2 fcc82d8831a74afd55c3cb898beb9fde5f2a1fd" (Results 1 – 2 of 2) sorted by relevance

/openbmc/linux/drivers/media/usb/em28xx/
H A Dem28xx-i2c.cdiff 2fcc82d8831a74afd55c3cb898beb9fde5f2a1fd Thu Jan 03 11:27:03 CST 2013 Frank Schaefer <fschaefer.oss@googlemail.com> [media] em28xx: fix two severe bugs in function em2800_i2c_recv_bytes()

Function em2800_i2c_recv_bytes() has 2 severe bugs:
1) It does not wait for the i2c read to complete before reading the received message content from the bridge registers.
2) Reading more than 1 byte doesn't work
The former can result in data corruption, the latter always does.
The rewritten code also superseds the content of function
em2800_i2c_check_for_device().
Tested with device "Terratec Cinergy 200 USB".

[mchehab@redhat.com: Fix CodingStyle issues]
Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com>

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
H A Dem28xx.hdiff 2fcc82d8831a74afd55c3cb898beb9fde5f2a1fd Thu Jan 03 11:27:03 CST 2013 Frank Schaefer <fschaefer.oss@googlemail.com> [media] em28xx: fix two severe bugs in function em2800_i2c_recv_bytes()

Function em2800_i2c_recv_bytes() has 2 severe bugs:
1) It does not wait for the i2c read to complete before reading the received message content from the bridge registers.
2) Reading more than 1 byte doesn't work
The former can result in data corruption, the latter always does.
The rewritten code also superseds the content of function
em2800_i2c_check_for_device().
Tested with device "Terratec Cinergy 200 USB".

[mchehab@redhat.com: Fix CodingStyle issues]
Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com>

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>