Re: [PATCH] Support for Asus MyCinema U3100Mini Plus (attempt 2)

Antti Palosaari <crope@xxxxxx> · Wed, 19 Sep 2012 01:42:58 +0300

On 09/19/2012 01:22 AM, oliver@xxxxxxxxxxx wrote:
From: Oliver Schinagl <oliver@xxxxxxxxxxx>

This is initial support for the Asus MyCinema U3100Mini Plus. The driver
in its current form gets detected and loads properly. It uses the
af9035 USB Bridge chip, with an af9033 demodulator. The tuner used is
the FCI FC2580.

I have only done a quick dvb scan, but it failed to tune to anything.
Using dvbv5-scan -I CHANNEL <channelfile> It did show 'signal 100%' but
failed to tune to anything, so I don't think signal strength works at
all. Since I have really bad reception where my dev PC is, I may simple
not receive anything here.

Signal strength is very worst indicator. It should not be 100% in any 
case. Switch off stupid % meter your are using and look plain numbers. 
It is should be something between 0-0xffff (0xffff == 100% ?).

For me successful tzap reports (af9035 + tua9001):
status 1f | signal 5eb7 | snr 010e | ber 00000000 | unc 00000000 | 
FE_HAS_LOCK

FE_HAS_LOCK is most important, it says demodulator is locked to channel 
and likely device is 100% working.

Biggest problem of your patch is fc2580 frontend callback. fc2580 driver 
does not use any callback and that code is simple dead. It is never called.

Otherwise it looks quite correct.

regards
Antti


Signed-off-by: Oliver Schinagl <oliver@xxxxxxxxxxx>
---
  drivers/media/dvb-core/dvb-usb-ids.h      |  1 +
  drivers/media/dvb-frontends/af9033.c      |  4 +++
  drivers/media/dvb-frontends/af9033.h      |  1 +
  drivers/media/dvb-frontends/af9033_priv.h | 38 +++++++++++++++++++++++
  drivers/media/tuners/fc2580.c             |  4 ++-
  drivers/media/tuners/fc2580.h             | 10 +++++++
  drivers/media/usb/dvb-usb-v2/Kconfig      |  1 +
  drivers/media/usb/dvb-usb-v2/af9035.c     | 50 +++++++++++++++++++++++++++++++
  drivers/media/usb/dvb-usb-v2/af9035.h     |  1 +
  9 files changed, 109 insertions(+), 1 deletion(-)

diff --git a/drivers/media/dvb-core/dvb-usb-ids.h b/drivers/media/dvb-core/dvb-usb-ids.h
index d572307..58e0220 100644
--- a/drivers/media/dvb-core/dvb-usb-ids.h
+++ b/drivers/media/dvb-core/dvb-usb-ids.h
@@ -329,6 +329,7 @@
  #define USB_PID_ASUS_U3000				0x171f
  #define USB_PID_ASUS_U3000H				0x1736
  #define USB_PID_ASUS_U3100				0x173f
+#define USB_PID_ASUS_U3100MINI_PLUS			0x1779
  #define USB_PID_YUAN_EC372S				0x1edc
  #define USB_PID_YUAN_STK7700PH				0x1f08
  #define USB_PID_YUAN_PD378S				0x2edc
diff --git a/drivers/media/dvb-frontends/af9033.c b/drivers/media/dvb-frontends/af9033.c
index 0979ada..b40f5a0 100644
--- a/drivers/media/dvb-frontends/af9033.c
+++ b/drivers/media/dvb-frontends/af9033.c
@@ -314,6 +314,10 @@ static int af9033_init(struct dvb_frontend *fe)
  		len = ARRAY_SIZE(tuner_init_tda18218);
  		init = tuner_init_tda18218;
  		break;
+	case AF9033_TUNER_FC2580:
+		len = ARRAY_SIZE(tuner_init_fc2580);
+		init = tuner_init_fc2580;
+		break;
  	default:
  		dev_dbg(&state->i2c->dev, "%s: unsupported tuner ID=%d\n",
  				__func__, state->cfg.tuner);
diff --git a/drivers/media/dvb-frontends/af9033.h b/drivers/media/dvb-frontends/af9033.h
index 288622b..739137e 100644
--- a/drivers/media/dvb-frontends/af9033.h
+++ b/drivers/media/dvb-frontends/af9033.h
@@ -42,6 +42,7 @@ struct af9033_config {
  #define AF9033_TUNER_FC0011      0x28 /* Fitipower FC0011 */
  #define AF9033_TUNER_MXL5007T    0xa0 /* MaxLinear MxL5007T */
  #define AF9033_TUNER_TDA18218    0xa1 /* NXP TDA 18218HN */
+#define AF9033_TUNER_FC2580      0x32 /* FIC FC2580 */
  	u8 tuner;

  	/*
diff --git a/drivers/media/dvb-frontends/af9033_priv.h b/drivers/media/dvb-frontends/af9033_priv.h
index 0b783b9..d2c9ae6 100644
--- a/drivers/media/dvb-frontends/af9033_priv.h
+++ b/drivers/media/dvb-frontends/af9033_priv.h
@@ -466,5 +466,43 @@ static const struct reg_val tuner_init_tda18218[] = {
  	{0x80f1e6, 0x00},
  };

+/* FIC FC2580 tuner init
+   AF9033_TUNER_FC2580      = 0x32 */
+static const struct reg_val tuner_init_fc2580[] = {
+	{ 0x800046, AF9033_TUNER_FC2580 },
+	{ 0x800057, 0x01 },
+	{ 0x800058, 0x00 },
+	{ 0x80005f, 0x00 },
+	{ 0x800060, 0x00 },
+	{ 0x800071, 0x05 },
+	{ 0x800072, 0x02 },
+	{ 0x800074, 0x01 },
+	{ 0x800079, 0x01 },
+	{ 0x800093, 0x00 },
+	{ 0x800094, 0x00 },
+	{ 0x800095, 0x00 },
+	{ 0x800096, 0x05 },
+	{ 0x8000b3, 0x01 },
+	{ 0x8000c3, 0x01 },
+	{ 0x8000c4, 0x00 },
+	{ 0x80f007, 0x00 },
+	{ 0x80f00c, 0x19 },
+	{ 0x80f00d, 0x1A },
+	{ 0x80f00e, 0x00 },
+	{ 0x80f00f, 0x02 },
+	{ 0x80f010, 0x00 },
+	{ 0x80f011, 0x02 },
+	{ 0x80f012, 0x00 },
+	{ 0x80f013, 0x02 },
+	{ 0x80f014, 0x00 },
+	{ 0x80f015, 0x02 },
+	{ 0x80f01f, 0x96 },
+	{ 0x80f020, 0x00 },
+	{ 0x80f029, 0x96 },
+	{ 0x80f02a, 0x00 },
+	{ 0x80f077, 0x01 },
+	{ 0x80f1e6, 0x01 },
+};
+
  #endif /* AF9033_PRIV_H */

diff --git a/drivers/media/tuners/fc2580.c b/drivers/media/tuners/fc2580.c
index afc0491..4e7c802 100644
--- a/drivers/media/tuners/fc2580.c
+++ b/drivers/media/tuners/fc2580.c
@@ -498,8 +498,10 @@ struct dvb_frontend *fc2580_attach(struct dvb_frontend *fe,

  	dev_dbg(&priv->i2c->dev, "%s: chip_id=%02x\n", __func__, chip_id);

-	if (chip_id != 0x56)
+	if ((chip_id != 0x56) && (chip_id != 0x5a)) {
  		goto err;
+	}
+

  	dev_info(&priv->i2c->dev,
  			"%s: FCI FC2580 successfully identified\n",
diff --git a/drivers/media/tuners/fc2580.h b/drivers/media/tuners/fc2580.h
index 222601e..952513d 100644
--- a/drivers/media/tuners/fc2580.h
+++ b/drivers/media/tuners/fc2580.h
@@ -36,6 +36,16 @@ struct fc2580_config {
  	u32 clock;
  };

+/** enum fc2580_fe_callback_commands - Frontend callbacks
+ *
+ * @FC2580_FE_CALLBACK_POWER: Power on tuner hardware.
+ */
+enum fc2580_fe_callback_commands {
+	FC2580_FE_CALLBACK_POWER,
+};
+
+
+
  #if defined(CONFIG_MEDIA_TUNER_FC2580) || \
  	(defined(CONFIG_MEDIA_TUNER_FC2580_MODULE) && defined(MODULE))
  extern struct dvb_frontend *fc2580_attach(struct dvb_frontend *fe,
diff --git a/drivers/media/usb/dvb-usb-v2/Kconfig b/drivers/media/usb/dvb-usb-v2/Kconfig
index e09930c..834bfec 100644
--- a/drivers/media/usb/dvb-usb-v2/Kconfig
+++ b/drivers/media/usb/dvb-usb-v2/Kconfig
@@ -40,6 +40,7 @@ config DVB_USB_AF9035
  	select MEDIA_TUNER_FC0011 if MEDIA_SUBDRV_AUTOSELECT
  	select MEDIA_TUNER_MXL5007T if MEDIA_SUBDRV_AUTOSELECT
  	select MEDIA_TUNER_TDA18218 if MEDIA_SUBDRV_AUTOSELECT
+	select MEDIA_TUNER_FC2580 if MEDIA_SUBDRV_AUTOSELECT
  	help
  	  Say Y here to support the Afatech AF9035 based DVB USB receiver.

diff --git a/drivers/media/usb/dvb-usb-v2/af9035.c b/drivers/media/usb/dvb-usb-v2/af9035.c
index 89cc901..f6ca30e 100644
--- a/drivers/media/usb/dvb-usb-v2/af9035.c
+++ b/drivers/media/usb/dvb-usb-v2/af9035.c
@@ -513,6 +513,7 @@ static int af9035_read_config(struct dvb_usb_device *d)
  		case AF9033_TUNER_FC0011:
  		case AF9033_TUNER_MXL5007T:
  		case AF9033_TUNER_TDA18218:
+		case AF9033_TUNER_FC2580:
  			state->af9033_config[i].spec_inv = 1;
  			break;
  		default:
@@ -648,6 +649,41 @@ err:
  	return ret;
  }

+static int af9035_fc2580_tuner_callback(struct dvb_usb_device *d,
+		int cmd, int arg)
+{
+	int ret;
+
+	switch (cmd) {
+	case FC2580_FE_CALLBACK_POWER:
+		/* Tuner enable */
+		ret = af9035_wr_reg_mask(d, 0xd8eb, 1, 1);
+		if (ret < 0)
+			goto err;
+
+		ret = af9035_wr_reg_mask(d, 0xd8ec, 1, 1);
+		if (ret < 0)
+			goto err;
+
+		ret = af9035_wr_reg_mask(d, 0xd8ed, 1, 1);
+		if (ret < 0)
+			goto err;
+
+		usleep_range(10000, 50000);
+		break;
+	default:
+		ret = -EINVAL;
+		goto err;
+	}
+
+	return 0;
+
+err:
+	dev_dbg(&d->udev->dev, "%s: failed=%d\n", __func__, ret);
+
+	return ret;
+}
+
  static int af9035_tuner_callback(struct dvb_usb_device *d, int cmd, int arg)
  {
  	struct state *state = d_to_priv(d);
@@ -655,6 +691,8 @@ static int af9035_tuner_callback(struct dvb_usb_device *d, int cmd, int arg)
  	switch (state->af9033_config[0].tuner) {
  	case AF9033_TUNER_FC0011:
  		return af9035_fc0011_tuner_callback(d, cmd, arg);
+	case AF9033_TUNER_FC2580:
+		return af9035_fc2580_tuner_callback(d, cmd, arg);
  	default:
  		break;
  	}
@@ -750,6 +788,11 @@ static struct tda18218_config af9035_tda18218_config = {
  	.i2c_wr_max = 21,
  };

+static struct fc2580_config af9035_fc2580_config = {
+	.i2c_addr = 0x56,
+	.clock = 16384000,
+};
+
  static int af9035_tuner_attach(struct dvb_usb_adapter *adap)
  {
  	struct state *state = adap_to_priv(adap);
@@ -851,6 +894,11 @@ static int af9035_tuner_attach(struct dvb_usb_adapter *adap)
  		fe = dvb_attach(tda18218_attach, adap->fe[0],
  				&d->i2c_adap, &af9035_tda18218_config);
  		break;
+	case AF9033_TUNER_FC2580:
+		/* attach tuner */
+		fe = dvb_attach(fc2580_attach, adap->fe[0],
+				&d->i2c_adap, &af9035_fc2580_config);
+		break;
  	default:
  		fe = NULL;
  	}
@@ -1075,6 +1123,8 @@ static const struct usb_device_id af9035_id_table[] = {
  		&af9035_props, "AVerMedia HD Volar (A867)", NULL) },
  	{ DVB_USB_DEVICE(USB_VID_AVERMEDIA, USB_PID_AVERMEDIA_TWINSTAR,
  		&af9035_props, "AVerMedia Twinstar (A825)", NULL) },
+	{ DVB_USB_DEVICE(USB_VID_ASUS, USB_PID_ASUS_U3100MINI_PLUS,
+		&af9035_props, "Asus U3100Mini Plus", NULL) },
  	{ }
  };
  MODULE_DEVICE_TABLE(usb, af9035_id_table);
diff --git a/drivers/media/usb/dvb-usb-v2/af9035.h b/drivers/media/usb/dvb-usb-v2/af9035.h
index de8e761..75ef1ec 100644
--- a/drivers/media/usb/dvb-usb-v2/af9035.h
+++ b/drivers/media/usb/dvb-usb-v2/af9035.h
@@ -28,6 +28,7 @@
  #include "fc0011.h"
  #include "mxl5007t.h"
  #include "tda18218.h"
+#include "fc2580.h"

  struct reg_val {
  	u32 reg;



--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html





