All of lore.kernel.org
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: linux-media@vger.kernel.org
Cc: Antti Palosaari <crope@iki.fi>
Subject: [REVIEW PATCH 09/41] af9035: USB1.1 support (== PID filters)
Date: Sun, 10 Mar 2013 04:03:01 +0200	[thread overview]
Message-ID: <1362881013-5271-9-git-send-email-crope@iki.fi> (raw)
In-Reply-To: <1362881013-5271-1-git-send-email-crope@iki.fi>

Signed-off-by: Antti Palosaari <crope@iki.fi>
---
 drivers/media/usb/dvb-usb-v2/af9035.c | 85 +++++++++++++++++++++++++++++++++--
 1 file changed, 82 insertions(+), 3 deletions(-)

diff --git a/drivers/media/usb/dvb-usb-v2/af9035.c b/drivers/media/usb/dvb-usb-v2/af9035.c
index 1e1cee6..42ed0f7 100644
--- a/drivers/media/usb/dvb-usb-v2/af9035.c
+++ b/drivers/media/usb/dvb-usb-v2/af9035.c
@@ -894,7 +894,12 @@ static int af9035_frontend_callback(void *adapter_priv, int component,
 static int af9035_get_adapter_count(struct dvb_usb_device *d)
 {
 	struct state *state = d_to_priv(d);
-	return state->dual_mode + 1;
+
+	/* disable 2nd adapter as we don't have PID filters implemented */
+	if (d->udev->speed == USB_SPEED_FULL)
+		return 1;
+	else
+		return state->dual_mode + 1;
 }
 
 static int af9035_frontend_attach(struct dvb_usb_adapter *adap)
@@ -1201,8 +1206,8 @@ static int af9035_init(struct dvb_usb_device *d)
 {
 	struct state *state = d_to_priv(d);
 	int ret, i;
-	u16 frame_size = 87 * 188 / 4;
-	u8  packet_size = 512 / 4;
+	u16 frame_size = (d->udev->speed == USB_SPEED_FULL ? 5 : 87) * 188 / 4;
+	u8 packet_size = (d->udev->speed == USB_SPEED_FULL ? 64 : 512) / 4;
 	struct reg_val_mask tab[] = {
 		{ 0x80f99d, 0x01, 0x01 },
 		{ 0x80f9a4, 0x01, 0x01 },
@@ -1328,6 +1333,72 @@ err:
 	#define af9035_get_rc_config NULL
 #endif
 
+static int af9035_get_stream_config(struct dvb_frontend *fe, u8 *ts_type,
+		struct usb_data_stream_properties *stream)
+{
+	struct dvb_usb_device *d = fe_to_d(fe);
+	dev_dbg(&d->udev->dev, "%s: adap=%d\n", __func__, fe_to_adap(fe)->id);
+
+	if (d->udev->speed == USB_SPEED_FULL)
+		stream->u.bulk.buffersize = 5 * 188;
+
+	return 0;
+}
+
+/*
+ * FIXME: PID filter is property of demodulator and should be moved to the
+ * correct driver. Also we support only adapter #0 PID filter and will
+ * disable adapter #1 if USB1.1 is used.
+ */
+static int af9035_pid_filter_ctrl(struct dvb_usb_adapter *adap, int onoff)
+{
+	struct dvb_usb_device *d = adap_to_d(adap);
+	int ret;
+
+	dev_dbg(&d->udev->dev, "%s: onoff=%d\n", __func__, onoff);
+
+	ret = af9035_wr_reg_mask(d, 0x80f993, onoff, 0x01);
+	if (ret < 0)
+		goto err;
+
+	return 0;
+
+err:
+	dev_dbg(&d->udev->dev, "%s: failed=%d\n", __func__, ret);
+
+	return ret;
+}
+
+static int af9035_pid_filter(struct dvb_usb_adapter *adap, int index, u16 pid,
+		int onoff)
+{
+	struct dvb_usb_device *d = adap_to_d(adap);
+	int ret;
+	u8 wbuf[2] = {(pid >> 0) & 0xff, (pid >> 8) & 0xff};
+
+	dev_dbg(&d->udev->dev, "%s: index=%d pid=%04x onoff=%d\n",
+			__func__, index, pid, onoff);
+
+	ret = af9035_wr_regs(d, 0x80f996, wbuf, 2);
+	if (ret < 0)
+		goto err;
+
+	ret = af9035_wr_reg(d, 0x80f994, onoff);
+	if (ret < 0)
+		goto err;
+
+	ret = af9035_wr_reg(d, 0x80f995, index);
+	if (ret < 0)
+		goto err;
+
+	return 0;
+
+err:
+	dev_dbg(&d->udev->dev, "%s: failed=%d\n", __func__, ret);
+
+	return ret;
+}
+
 static int af9035_probe(struct usb_interface *intf,
 		const struct usb_device_id *id)
 {
@@ -1385,10 +1456,18 @@ static const struct dvb_usb_device_properties af9035_props = {
 	.tuner_attach = af9035_tuner_attach,
 	.init = af9035_init,
 	.get_rc_config = af9035_get_rc_config,
+	.get_stream_config = af9035_get_stream_config,
 
 	.get_adapter_count = af9035_get_adapter_count,
 	.adapter = {
 		{
+			.caps = DVB_USB_ADAP_HAS_PID_FILTER |
+				DVB_USB_ADAP_PID_FILTER_CAN_BE_TURNED_OFF,
+
+			.pid_filter_count = 32,
+			.pid_filter_ctrl = af9035_pid_filter_ctrl,
+			.pid_filter = af9035_pid_filter,
+
 			.stream = DVB_USB_STREAM_BULK(0x84, 6, 87 * 188),
 		}, {
 			.stream = DVB_USB_STREAM_BULK(0x85, 6, 87 * 188),
-- 
1.7.11.7


  parent reply	other threads:[~2013-03-10  2:04 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-10  2:02 [REVIEW PATCH 01/41] ITE IT913X silicon tuner driver Antti Palosaari
2013-03-10  2:02 ` [REVIEW PATCH 02/41] af9033: support for it913x tuners Antti Palosaari
2013-03-10  2:02 ` [REVIEW PATCH 03/41] af9035: add support for 1st gen it9135 Antti Palosaari
2013-03-10  2:02 ` [REVIEW PATCH 04/41] af9035: add auto configuration heuristic for it9135 Antti Palosaari
2013-03-10  2:02 ` [REVIEW PATCH 05/41] af9035: fix af9033 demod sampling frequency Antti Palosaari
2013-03-10  2:02 ` [REVIEW PATCH 06/41] af9015: reject device TerraTec Cinergy T Stick Dual RC (rev. 2) Antti Palosaari
2013-03-10  2:02 ` [REVIEW PATCH 07/41] af9035: [0ccd:0099] " Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 08/41] af9035: constify clock tables Antti Palosaari
2013-03-10  2:03 ` Antti Palosaari [this message]
2013-03-10  2:03 ` [REVIEW PATCH 10/41] af9035: merge af9035 and it9135 eeprom read routines Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 11/41] af9035: basic support for IT9135 v2 chips Antti Palosaari
2013-03-21 21:54   ` Mauro Carvalho Chehab
2013-03-21 23:45     ` Antti Palosaari
2013-03-22  9:30       ` Mauro Carvalho Chehab
2013-03-22 10:45         ` Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 12/41] af9033: IT9135 v2 supported related changes Antti Palosaari
2013-03-10  2:03 ` =?y?q?=5BREVIEW=20PATCH=2013/41=5D=20af9035=3A=20IT9135=20dual=20tuner=20related=20changes?= Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 14/41] it913x: merge it913x_fe_start() to it913x_init_tuner() Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 15/41] it913x: merge it913x_fe_suspend() to it913x_fe_sleep() Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 16/41] it913x: rename functions and variables Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 17/41] it913x: tuner power up routines Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 18/41] it913x: get rid of it913x config struct Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 19/41] it913x: remove unused variables Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 20/41] it913x: include tuner IDs from af9033.h Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 21/41] it913x: use dev_foo() logging Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 22/41] af9033: add IT9135 demod reg init tables Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 23/41] it913x: remove demod init reg tables Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 24/41] af9035: select firmware loader according to firmware Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 25/41] af9035: use already detected eeprom base addr Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 26/41] af9035: set demod TS mode config in read_config() Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 27/41] af9035: enable remote controller for IT9135 too Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 28/41] af9035: change dual mode boolean to bit field Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 29/41] af9033: add IT9135 tuner config "38" init table Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 30/41] af9033: add IT9135 tuner config "51" " Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 31/41] af9033: add IT9135 tuner config "52" " Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 32/41] af9033: add IT9135 tuner config "60" " Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 33/41] af9033: add IT9135 tuner config "61" " Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 34/41] af9033: add IT9135 tuner config "62" " Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 35/41] it913x: remove unused af9033 demod tuner config inits Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 36/41] af9033: move code from it913x to af9033 Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 37/41] af9033: sleep on attach() Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 38/41] af9033: implement i/o optimized reg table writer Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 39/41] af9035: check I/O errors on IR polling Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 40/41] af9035: style changes for remote controller polling Antti Palosaari
2013-03-10  2:03 ` [REVIEW PATCH 41/41] MAINTAINERS: add drivers/media/tuners/it913x* Antti Palosaari

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=1362881013-5271-9-git-send-email-crope@iki.fi \
    --to=crope@iki.fi \
    --cc=linux-media@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.