kernel.git/drivers/media/dvb/dvb-core, branch linux-3.1.y

Revert "[media] DVB: dvb_frontend: off by one in dtv_property_dump()"

2011-07-27T20:56:02Z

This reverts commit a3e4adf274f86b2363fedaa964297cb38526cef0. As pointed by Andread Oberritter : That's wrong, because the array size is DTV_MAX_COMMAND + 1. Using the ARRAY_SIZE macro instead might reduce the confusion. Also, changeset 3995223038 already fixed this issue. Reported-by: Andread Oberritter Signed-off-by: Mauro Carvalho Chehab

[media] DVB: dvb_frontend: off by one in dtv_property_dump()

2011-07-27T20:55:55Z

If the tvp->cmd == DTV_MAX_COMMAND then we read past the end of the array. Signed-off-by: Dan Carpenter Signed-off-by: Mauro Carvalho Chehab

[media] DVB: dvb_frontend: fix dtv_property_dump for DTV_DVBT2_PLP_ID

2011-07-27T20:52:24Z

- Add missing entry to array "dtv_cmds". - Set array size to DTV_MAX_COMMAND + 1 to avoid future off-by-ones. Reported-by: Dan Carpenter Signed-off-by: Andreas Oberritter Signed-off-by: Mauro Carvalho Chehab

dvb_net: Simplify the code if DVB NET is not defined

2011-07-27T20:52:20Z

Signed-off-by: Mauro Carvalho Chehab

[media] Make DVB NET configurable in the kernel

2011-07-27T20:52:20Z

Signed-off-by: Hans Petter Selasky Signed-off-by: Mauro Carvalho Chehab

[media] dvb_frontend: fix race condition in stopping/starting frontend

2011-07-13T22:50:12Z

Attached is a patch which addresses a race condition in the DVB core related to closing/reopening the DVB frontend device in quick succession. This is the reason that devices such as the HVR-1300, HVR-3000, and HVR-4000 have been failing to scan properly under MythTV and w_scan. The gory details of the race are described in the patch. Devin There is a race condition exhibited when channel scanners such as w_scan and MythTV quickly close and then reopen the frontend device node. Under normal conditions, the behavior is as follows: 1. Application closes the device node 2. DVB frontend ioctl calls dvb_frontend_release which sets fepriv->release_jiffies 3. DVB frontend thread *eventually* calls dvb_frontend_is_exiting() which compares fepriv->release_jiffies, and shuts down the thread if timeout has expired 4. Thread goes away 5. Application opens frontend device 6. DVB frontend ioctl() calls ts_bus_ctrl(1) 7. DVB frontend ioctl() creates new frontend thread, which calls dvb_frontend_init(), which has demod driver init() routine setup initial register state for demod chip. 8. Tuning request is issued. The race occurs when the application in step 5 performs the new open() call before the frontend thread is shutdown. In this case the ts_bus_ctrl() call is made, which strobes the RESET pin on the demodulator, but the dvb_frontend_init() function never gets called because the frontend thread hasn't gone away yet. As a result, the initial register config for the demod is *never* setup, causing subsequent tuning requests to fail. If there is time between the close and open (enough for the dvb frontend thread to be torn down), then in that case the new frontend thread is created and thus the dvb_frontend_init() function does get called. The fix is to set the flag which forces reinitialization if we did in fact call ts_bus_ctrl(). This problem has been seen on the HVR-1300, HVR-3000, and HVR-4000, and is likely occuring on other designs as well where ts_bus_ctrl() actually strobes the reset pin on the demodulator. Note that this patch should supercede any patches submitted for the 1300/3000/4000 which remove the code that removes GPIO code in cx8802_dvb_advise_acquire(), which have been circulating by users for some time now... Canonical tracking this issue in Launchpad 439163: Thanks to Jon Sayers from Hauppauge and Florent Audebert from Anevia S.A. for providing hardware to test/debug with. Signed-off-by: Devin Heitmueller Cc: Jon Sayers Cc: Florent Audebert Signed-off-by: Mauro Carvalho Chehab

[media] DVB: allow to read back of detected parameters through S2API

2011-05-20T22:53:31Z

Signed-off-by: Andreas Oberritter Signed-off-by: Mauro Carvalho Chehab

[media] DVB: dvb_frontend: add parameters_out

2011-05-20T22:53:00Z

- Holds the parameters detected by the demod. - Updated on every call to get_frontend, either through ioctl or when a frontend event occurs. - Reset to input parameters after every call to set_frontend, tune or search/track. Signed-off-by: Andreas Oberritter Signed-off-by: Mauro Carvalho Chehab

[media] DVB: dvb_frontend: use shortcut to access fe->dtv_property_cache

2011-05-20T22:52:32Z

Signed-off-by: Andreas Oberritter Signed-off-by: Mauro Carvalho Chehab

[media] DVB: dvb_frontend: remove unused assignments

2011-05-20T22:52:15Z

Signed-off-by: Andreas Oberritter Signed-off-by: Mauro Carvalho Chehab