pctv452e: hm.. tidy bogus code up

  Currently, usb_register calls two times with cloned structures, but for 
 	.devices = {
@@ -1055,22 +1055,9 @@ static struct usb_driver pctv452e_usb_driver = {
 	.id_table   = pctv452e_usb_table,
 };
 
-static struct usb_driver tt_connects2_3600_usb_driver = {
-	.name       = "dvb-usb-tt-connect-s2-3600-01.fw",
-	.probe      = pctv452e_usb_probe,
-	.disconnect = pctv452e_usb_disconnect,
-	.id_table   = pctv452e_usb_table,
-};
-
 static int __init pctv452e_usb_init(void)
 {
 	int ret = usb_register(&pctv452e_usb_driver);
-
-	if (ret) {
-		err("%s: usb_register failed! Error %d", __FILE__, ret);
-		return ret;
-	}
-	ret = usb_register(&tt_connects2_3600_usb_driver);
 	if (ret)
 		err("%s: usb_register failed! Error %d", __FILE__, ret);
 
@@ -1080,7 +1067,6 @@ static int __init pctv452e_usb_init(void)
 static void __exit pctv452e_usb_exit(void)
 {
 	usb_deregister(&pctv452e_usb_driver);
-	usb_deregister(&tt_connects2_3600_usb_driver);
 }
 
 module_init(pctv452e_usb_init);
  

Hi Igor,

On 30.09.2011 22:58, Igor M. Liplianin wrote:
> Currently, usb_register calls two times with cloned structures, but for
> different driver names. Let's remove it.
>
> Signed-off-by: Igor M. Liplianin<liplianin@me.by>

Well spotted... The cloned struct should have been removed a long time 
go. The final version of patch I submitted for the tt-connect S2-3600, 
did not contain it anymore: 
http://www.linuxtv.org/pipermail/linux-dvb/2008-March/024233.html

Acked-by: André Weideamm<Andre.Weidemann@web.de>

Regards
  André
--
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
  

On 03.10.2011 14:30, André Weidemann wrote:
> Hi Igor,
>
> On 30.09.2011 22:58, Igor M. Liplianin wrote:
>> Currently, usb_register calls two times with cloned structures, but for
>> different driver names. Let's remove it.
>>
>> Signed-off-by: Igor M. Liplianin<liplianin@me.by>
>
> Well spotted... The cloned struct should have been removed a long time
> go. The final version of patch I submitted for the tt-connect S2-3600,
> did not contain it anymore:
> http://www.linuxtv.org/pipermail/linux-dvb/2008-March/024233.html
>
> Acked-by: André Weideamm<Andre.Weidemann@web.de>

This should read:
Acked-by: André Weidemann<Andre.Weidemann@web.de>

;-)

Regards
  André
--
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
  

Hi Mauro,

On 30.09.2011 22:58, Igor M. Liplianin wrote:
> Currently, usb_register calls two times with cloned structures, but for
> different driver names. Let's remove it.

It looks like the comments and the patch under 
http://patchwork.linuxtv.org/patch/8042/ got mixed up.

Regards,
  André
--
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