linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: tskd08@gmail.com
To: linux-media@vger.kernel.org
Cc: crope@iki.fi, sean@mess.org, mchehab@kernel.org,
	Akihiro Tsukada <tskd08@gmail.com>
Subject: [PATCH v2 0/4] dvb-usb-gl861, tc90522: remove device-specific i2c algo
Date: Thu, 29 Aug 2019 17:38:18 +0900	[thread overview]
Message-ID: <20190829083822.3670-1-tskd08@gmail.com> (raw)

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 algo (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.

Changes since v1:
- include change to demod driver (tc90522)
In the previous version, I omitted this, but it was wrong.
This change IS required even now,
as the tuner driver of Friio device (dvb-pll) issues a read on probe.
I made mistakes in building my test environment again,
missed that some modules were not of the right version during the tests of v1.
sorry for my hasty v1 post.

Akihiro Tsukada (4):
  dvb-usb-v2/gl861: remove device-specific i2c algo
  dvb-usb-v2/gl861: remove un-used header file
  dvb-frontends/tc90522: extend i2c algo to support some devices
  dvb-usb-v2/gl861: support I2C read from tuner via demod

 drivers/media/dvb-frontends/tc90522.c |  25 ++-
 drivers/media/dvb-frontends/tc90522.h |   3 +
 drivers/media/usb/dvb-usb-v2/gl861.c  | 237 +++-----------------------
 drivers/media/usb/dvb-usb-v2/gl861.h  |  14 --
 4 files changed, 55 insertions(+), 224 deletions(-)
 delete mode 100644 drivers/media/usb/dvb-usb-v2/gl861.h

-- 
2.23.0


             reply	other threads:[~2019-08-29  8:38 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-29  8:38 tskd08 [this message]
2019-08-29  8:38 ` [PATCH v2 1/4] dvb-usb-v2/gl861: remove device-specific i2c algo tskd08
2019-08-29  8:38 ` [PATCH v2 2/4] dvb-usb-v2/gl861: remove an un-used header file tskd08
2019-08-29  8:38 ` [PATCH v2 3/4] dvb-frontends/tc90522: extend i2c algo to support some devices tskd08
2019-08-29  8:38 ` [PATCH v2 4/4] dvb-usb-gl861: support I2C read from tuner via demod tskd08

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190829083822.3670-1-tskd08@gmail.com \
    --to=tskd08@gmail.com \
    --cc=crope@iki.fi \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=sean@mess.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).