mbox series

[0/2] dvb-usb-gl861: remove device-specific i2c algo

Message ID 20190827150141.10240-1-tskd08@gmail.com (mailing list archive)
Headers show
Series dvb-usb-gl861: remove device-specific i2c algo | expand

Message

Akihiro TSUKADA Aug. 27, 2019, 3:01 p.m. UTC
From: Akihiro Tsukada <tskd08@gmail.com>

This patch series removes and integrates the device(Friio)-specific
i2c algo with the other, generic one that was recently patched
( https://patchwork.linuxtv.org/patch/58340/ )
to support the i2c messages used in Friio, for reading/writing to tuner.

To fully complete the integration,
users of this i2c alog (in this case, i2c adapter in demod tc90522)
must be modified to divide each tuner read transaction into two.
I had implemented this modification (and the config option to enable it)
in tc90522 (and in gl861, to use the config option), and tested it,
which verified that a Friio device can properly read tuner status,
(and write/set frequency as well) without problems.
Although there was a concern over small in-compatibility
of the generic i2c algo with the old devcie-specific algo in tuner reads,
(as described in https://patchwork.linuxtv.org/patch/58340/#111873 )
it had been cleared by the testing.

But for now I do not include the patch to tc90522,
because it just adds tuner read capablity,
and the tuner in Friio just supports reading its status (PLL lock status)
which is not particularly useful.
Besides, the current tuner driver (dvb-pll) does not implement any readings.
So I would rather spare the modification in exchange for simplicity,
until future devices that use gl861, tc90522 and a tuner with RSSI reading
come up.

Akihiro Tsukada (2):
  dvb-usb-v2/gl861: remove device-specific i2c algo and use generic one
  dvb-usb-v2/gl861: remove un-used header file

 drivers/media/usb/dvb-usb-v2/gl861.c | 236 +++------------------------
 drivers/media/usb/dvb-usb-v2/gl861.h |  14 --
 2 files changed, 27 insertions(+), 223 deletions(-)
 delete mode 100644 drivers/media/usb/dvb-usb-v2/gl861.h