[media] az6007: support Technisat Cablestar Combo HDCI (minus remote)

Message ID 1383421772-28243-1-git-send-email-rscheidegger_lists@hispeed.ch (mailing list archive)
State Accepted, archived
Headers

Commit Message

Roland Scheidegger Nov. 2, 2013, 7:49 p.m. UTC
  This is similar to the Terratec H7. It works with the same az6007 firmware as
the former, however the drx-k firmware of the H7 will NOT work. Hence use
a different firmware name. The firmware does not need to exist as the one in
the eeprom is just fine as long as the h7 one doesn't get loaded, but maybe
some day someone wants to load it (the one from the h5 would work too).
Also since the config entry is now different anyway disable support for rc.
AFAIK the Technisat remote (TS35) is RC5 and the code (which a code comment
claims doesn't work anyway) only would handle NEC hence it's pointless creating
a device and polling it if we already know it can't work.
CI is untested.
Originally based on idea found on
http://www.linuxtv.org/wiki/index.php/TechniSat_CableStar_Combo_HD_CI claiming
only id needs to be added (but failed to mention it only worked because the
driver couldn't find the h7 drx-k firmware...).

Signed-off-by: Roland Scheidegger <rscheidegger_lists@hispeed.ch>
---
 drivers/media/dvb-core/dvb-usb-ids.h  |  1 +
 drivers/media/usb/dvb-usb-v2/az6007.c | 59 +++++++++++++++++++++++++++++++++++
 2 files changed, 60 insertions(+)
  

Comments

Janusz S. Bien Nov. 5, 2013, 6:14 a.m. UTC | #1
Thank you very much for the patch.

Quote/Cytat - Roland Scheidegger <rscheidegger_lists@hispeed.ch> (Sat  
02 Nov 2013 08:49:32 PM CET):

[...]

> Originally based on idea found on
> http://www.linuxtv.org/wiki/index.php/TechniSat_CableStar_Combo_HD_CI  
> claiming
> only id needs to be added (but failed to mention it only worked because the
> driver couldn't find the h7 drx-k firmware...).

Together with Tobias Wessel, another user of the device, we have  
updated and extended the wiki entry.

The problem is that although I use the same system as Tobias (Debian  
wheezy) and it seems we installed media_build in the identical way, it  
works OK for Tobias but not for me, as the required drivers are not  
loaded. The device itself is not a problem because it works perfectly  
on Windows.

Looks like udev may be the culprit, I enclose a log fragment from my  
rather chaotic experiments - I'm just a Debian user and my knowledge  
of the system internals is rudimentary.

So my questions are:

Do you have any comments or suggestions regarding the modified wiki entry?

Where to look for help with my problem?

How to uninstall media_build to start experimenting from the scratch?

Best regards

Janusz
  
Janusz S. Bien Nov. 5, 2013, 12:05 p.m. UTC | #2
Quote/Cytat - "Janusz S. Bien" <jsbien@mimuw.edu.pl> (Tue 05 Nov 2013  
07:14:22 AM CET):

> Thank you very much for the patch.
>
> Quote/Cytat - Roland Scheidegger <rscheidegger_lists@hispeed.ch>  
> (Sat  02 Nov 2013 08:49:32 PM CET):
>
> [...]
>
>> Originally based on idea found on
>> http://www.linuxtv.org/wiki/index.php/TechniSat_CableStar_Combo_HD_CI   
>> claiming
>> only id needs to be added (but failed to mention it only worked because the
>> driver couldn't find the h7 drx-k firmware...).
>
> Together with Tobias Wessel, another user of the device, we have   
> updated and extended the wiki entry.
>
> The problem is that although I use the same system as Tobias (Debian  
>  wheezy) and it seems we installed media_build in the identical way,  
> it  works OK for Tobias but not for me,

My problem was solved by Tobias - thank you very much! As could be  
expected, I've installed the software differently...

I have erroneusly assumed that a patch posted on the list is  
immediately included  http://git.linuxtv.org/media_build.git.

I see it now at

https://patchwork.linuxtv.org/patch/20558/

together with my previous misleading letter...

Regards

Janusz
  
Roland Scheidegger Nov. 27, 2013, 11:08 p.m. UTC | #3
Any chances this could get applied?

Thanks,

Roland

Am 02.11.2013 20:49, schrieb linux-media-owner@vger.kernel.org:
> This is similar to the Terratec H7. It works with the same az6007 firmware as
> the former, however the drx-k firmware of the H7 will NOT work. Hence use
> a different firmware name. The firmware does not need to exist as the one in
> the eeprom is just fine as long as the h7 one doesn't get loaded, but maybe
> some day someone wants to load it (the one from the h5 would work too).
> Also since the config entry is now different anyway disable support for rc.
> AFAIK the Technisat remote (TS35) is RC5 and the code (which a code comment
> claims doesn't work anyway) only would handle NEC hence it's pointless creating
> a device and polling it if we already know it can't work.
> CI is untested.
> Originally based on idea found on
> http://www.linuxtv.org/wiki/index.php/TechniSat_CableStar_Combo_HD_CI claiming
> only id needs to be added (but failed to mention it only worked because the
> driver couldn't find the h7 drx-k firmware...).
> 
> Signed-off-by: Roland Scheidegger <rscheidegger_lists@hispeed.ch>
> ---
>  drivers/media/dvb-core/dvb-usb-ids.h  |  1 +
>  drivers/media/usb/dvb-usb-v2/az6007.c | 59 +++++++++++++++++++++++++++++++++++
>  2 files changed, 60 insertions(+)
> 
> diff --git a/drivers/media/dvb-core/dvb-usb-ids.h b/drivers/media/dvb-core/dvb-usb-ids.h
> index 419a2d6..4a53454 100644
> --- a/drivers/media/dvb-core/dvb-usb-ids.h
> +++ b/drivers/media/dvb-core/dvb-usb-ids.h
> @@ -365,6 +365,7 @@
>  #define USB_PID_TERRATEC_DVBS2CI_V2			0x10ac
>  #define USB_PID_TECHNISAT_USB2_HDCI_V1			0x0001
>  #define USB_PID_TECHNISAT_USB2_HDCI_V2			0x0002
> +#define USB_PID_TECHNISAT_USB2_CABLESTAR_HDCI		0x0003
>  #define USB_PID_TECHNISAT_AIRSTAR_TELESTICK_2		0x0004
>  #define USB_PID_TECHNISAT_USB2_DVB_S2			0x0500
>  #define USB_PID_CPYTO_REDI_PC50A			0xa803
> diff --git a/drivers/media/usb/dvb-usb-v2/az6007.c b/drivers/media/usb/dvb-usb-v2/az6007.c
> index 44c64ef3..c1051c3 100644
> --- a/drivers/media/usb/dvb-usb-v2/az6007.c
> +++ b/drivers/media/usb/dvb-usb-v2/az6007.c
> @@ -68,6 +68,19 @@ static struct drxk_config terratec_h7_drxk = {
>  	.microcode_name = "dvb-usb-terratec-h7-drxk.fw",
>  };
>  
> +static struct drxk_config cablestar_hdci_drxk = {
> +	.adr = 0x29,
> +	.parallel_ts = true,
> +	.dynamic_clk = true,
> +	.single_master = true,
> +	.enable_merr_cfg = true,
> +	.no_i2c_bridge = false,
> +	.chunk_size = 64,
> +	.mpeg_out_clk_strength = 0x02,
> +	.qam_demod_parameter_count = 2,
> +	.microcode_name = "dvb-usb-technisat-cablestar-hdci-drxk.fw",
> +};
> +
>  static int drxk_gate_ctrl(struct dvb_frontend *fe, int enable)
>  {
>  	struct az6007_device_state *st = fe_to_priv(fe);
> @@ -630,6 +643,27 @@ static int az6007_frontend_attach(struct dvb_usb_adapter *adap)
>  	return 0;
>  }
>  
> +static int az6007_cablestar_hdci_frontend_attach(struct dvb_usb_adapter *adap)
> +{
> +	struct az6007_device_state *st = adap_to_priv(adap);
> +	struct dvb_usb_device *d = adap_to_d(adap);
> +
> +	pr_debug("attaching demod drxk\n");
> +
> +	adap->fe[0] = dvb_attach(drxk_attach, &cablestar_hdci_drxk,
> +				 &d->i2c_adap);
> +	if (!adap->fe[0])
> +		return -EINVAL;
> +
> +	adap->fe[0]->sec_priv = adap;
> +	st->gate_ctrl = adap->fe[0]->ops.i2c_gate_ctrl;
> +	adap->fe[0]->ops.i2c_gate_ctrl = drxk_gate_ctrl;
> +
> +	az6007_ci_init(adap);
> +
> +	return 0;
> +}
> +
>  static int az6007_tuner_attach(struct dvb_usb_adapter *adap)
>  {
>  	struct dvb_usb_device *d = adap_to_d(adap);
> @@ -868,6 +902,29 @@ static struct dvb_usb_device_properties az6007_props = {
>  	}
>  };
>  
> +static struct dvb_usb_device_properties az6007_cablestar_hdci_props = {
> +	.driver_name         = KBUILD_MODNAME,
> +	.owner               = THIS_MODULE,
> +	.firmware            = AZ6007_FIRMWARE,
> +
> +	.adapter_nr          = adapter_nr,
> +	.size_of_priv        = sizeof(struct az6007_device_state),
> +	.i2c_algo            = &az6007_i2c_algo,
> +	.tuner_attach        = az6007_tuner_attach,
> +	.frontend_attach     = az6007_cablestar_hdci_frontend_attach,
> +	.streaming_ctrl      = az6007_streaming_ctrl,
> +/* ditch get_rc_config as it can't work (TS35 remote, I believe it's rc5) */
> +	.get_rc_config       = NULL,
> +	.read_mac_address    = az6007_read_mac_addr,
> +	.download_firmware   = az6007_download_firmware,
> +	.identify_state	     = az6007_identify_state,
> +	.power_ctrl          = az6007_power_ctrl,
> +	.num_adapters        = 1,
> +	.adapter             = {
> +		{ .stream = DVB_USB_STREAM_BULK(0x02, 10, 4096), }
> +	}
> +};
> +
>  static struct usb_device_id az6007_usb_table[] = {
>  	{DVB_USB_DEVICE(USB_VID_AZUREWAVE, USB_PID_AZUREWAVE_6007,
>  		&az6007_props, "Azurewave 6007", RC_MAP_EMPTY)},
> @@ -875,6 +932,8 @@ static struct usb_device_id az6007_usb_table[] = {
>  		&az6007_props, "Terratec H7", RC_MAP_NEC_TERRATEC_CINERGY_XS)},
>  	{DVB_USB_DEVICE(USB_VID_TERRATEC, USB_PID_TERRATEC_H7_2,
>  		&az6007_props, "Terratec H7", RC_MAP_NEC_TERRATEC_CINERGY_XS)},
> +	{DVB_USB_DEVICE(USB_VID_TECHNISAT, USB_PID_TECHNISAT_USB2_CABLESTAR_HDCI,
> +		&az6007_cablestar_hdci_props, "Technisat CableStar Combo HD CI", RC_MAP_EMPTY)},
>  	{0},
>  };
>  
> 

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

Patch

diff --git a/drivers/media/dvb-core/dvb-usb-ids.h b/drivers/media/dvb-core/dvb-usb-ids.h
index 419a2d6..4a53454 100644
--- a/drivers/media/dvb-core/dvb-usb-ids.h
+++ b/drivers/media/dvb-core/dvb-usb-ids.h
@@ -365,6 +365,7 @@ 
 #define USB_PID_TERRATEC_DVBS2CI_V2			0x10ac
 #define USB_PID_TECHNISAT_USB2_HDCI_V1			0x0001
 #define USB_PID_TECHNISAT_USB2_HDCI_V2			0x0002
+#define USB_PID_TECHNISAT_USB2_CABLESTAR_HDCI		0x0003
 #define USB_PID_TECHNISAT_AIRSTAR_TELESTICK_2		0x0004
 #define USB_PID_TECHNISAT_USB2_DVB_S2			0x0500
 #define USB_PID_CPYTO_REDI_PC50A			0xa803
diff --git a/drivers/media/usb/dvb-usb-v2/az6007.c b/drivers/media/usb/dvb-usb-v2/az6007.c
index 44c64ef3..c1051c3 100644
--- a/drivers/media/usb/dvb-usb-v2/az6007.c
+++ b/drivers/media/usb/dvb-usb-v2/az6007.c
@@ -68,6 +68,19 @@  static struct drxk_config terratec_h7_drxk = {
 	.microcode_name = "dvb-usb-terratec-h7-drxk.fw",
 };
 
+static struct drxk_config cablestar_hdci_drxk = {
+	.adr = 0x29,
+	.parallel_ts = true,
+	.dynamic_clk = true,
+	.single_master = true,
+	.enable_merr_cfg = true,
+	.no_i2c_bridge = false,
+	.chunk_size = 64,
+	.mpeg_out_clk_strength = 0x02,
+	.qam_demod_parameter_count = 2,
+	.microcode_name = "dvb-usb-technisat-cablestar-hdci-drxk.fw",
+};
+
 static int drxk_gate_ctrl(struct dvb_frontend *fe, int enable)
 {
 	struct az6007_device_state *st = fe_to_priv(fe);
@@ -630,6 +643,27 @@  static int az6007_frontend_attach(struct dvb_usb_adapter *adap)
 	return 0;
 }
 
+static int az6007_cablestar_hdci_frontend_attach(struct dvb_usb_adapter *adap)
+{
+	struct az6007_device_state *st = adap_to_priv(adap);
+	struct dvb_usb_device *d = adap_to_d(adap);
+
+	pr_debug("attaching demod drxk\n");
+
+	adap->fe[0] = dvb_attach(drxk_attach, &cablestar_hdci_drxk,
+				 &d->i2c_adap);
+	if (!adap->fe[0])
+		return -EINVAL;
+
+	adap->fe[0]->sec_priv = adap;
+	st->gate_ctrl = adap->fe[0]->ops.i2c_gate_ctrl;
+	adap->fe[0]->ops.i2c_gate_ctrl = drxk_gate_ctrl;
+
+	az6007_ci_init(adap);
+
+	return 0;
+}
+
 static int az6007_tuner_attach(struct dvb_usb_adapter *adap)
 {
 	struct dvb_usb_device *d = adap_to_d(adap);
@@ -868,6 +902,29 @@  static struct dvb_usb_device_properties az6007_props = {
 	}
 };
 
+static struct dvb_usb_device_properties az6007_cablestar_hdci_props = {
+	.driver_name         = KBUILD_MODNAME,
+	.owner               = THIS_MODULE,
+	.firmware            = AZ6007_FIRMWARE,
+
+	.adapter_nr          = adapter_nr,
+	.size_of_priv        = sizeof(struct az6007_device_state),
+	.i2c_algo            = &az6007_i2c_algo,
+	.tuner_attach        = az6007_tuner_attach,
+	.frontend_attach     = az6007_cablestar_hdci_frontend_attach,
+	.streaming_ctrl      = az6007_streaming_ctrl,
+/* ditch get_rc_config as it can't work (TS35 remote, I believe it's rc5) */
+	.get_rc_config       = NULL,
+	.read_mac_address    = az6007_read_mac_addr,
+	.download_firmware   = az6007_download_firmware,
+	.identify_state	     = az6007_identify_state,
+	.power_ctrl          = az6007_power_ctrl,
+	.num_adapters        = 1,
+	.adapter             = {
+		{ .stream = DVB_USB_STREAM_BULK(0x02, 10, 4096), }
+	}
+};
+
 static struct usb_device_id az6007_usb_table[] = {
 	{DVB_USB_DEVICE(USB_VID_AZUREWAVE, USB_PID_AZUREWAVE_6007,
 		&az6007_props, "Azurewave 6007", RC_MAP_EMPTY)},
@@ -875,6 +932,8 @@  static struct usb_device_id az6007_usb_table[] = {
 		&az6007_props, "Terratec H7", RC_MAP_NEC_TERRATEC_CINERGY_XS)},
 	{DVB_USB_DEVICE(USB_VID_TERRATEC, USB_PID_TERRATEC_H7_2,
 		&az6007_props, "Terratec H7", RC_MAP_NEC_TERRATEC_CINERGY_XS)},
+	{DVB_USB_DEVICE(USB_VID_TECHNISAT, USB_PID_TECHNISAT_USB2_CABLESTAR_HDCI,
+		&az6007_cablestar_hdci_props, "Technisat CableStar Combo HD CI", RC_MAP_EMPTY)},
 	{0},
 };