Support for Sveon STV22 (IT9137)

Message ID CABb1zhvUMZ1bSqz1X5qCzOArKYsGG4EHthK-OrbAWRLn+q_+Sg@mail.gmail.com (mailing list archive)
State Changes Requested, archived
Headers

Commit Message

Lord_LT Oct. 9, 2011, 5:09 a.m. UTC
This device identifies has IdProduct 0xe411 and is a clone of KWorld
UB499-2T T09(IT9137).

This patch simply adds support for this device.
  

Comments

Mauro Carvalho Chehab Oct. 14, 2011, 1:27 p.m. UTC | #1
Em 09-10-2011 02:09, Leandro Terrés escreveu:
> This device identifies has IdProduct 0xe411 and is a clone of KWorld
> UB499-2T T09(IT9137).
> 
> This patch simply adds support for this device.

Patch applies ok, with just one small whitespace issue. However, you
forgot to add your signed-off-by: on it. Also, it helps if you copy the
driver's maintainer (Malcolm).

Patchwork: http://patchwork.linuxtv.org/patch/8099/

WARNING: please, no space before tabs
#24: FILE: drivers/media/dvb/dvb-usb/dvb-usb-ids.h:323:
+#define USB_PID_SVEON_STV22_IT9137     ^I^I0xe411$

ERROR: Missing Signed-off-by: line(s)

total: 1 errors, 1 warnings, 29 lines checked
--
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
  
Lord_LT Oct. 14, 2011, 5:01 p.m. UTC | #2
2011/10/14 Mauro Carvalho Chehab <mchehab@redhat.com>:
> Em 09-10-2011 02:09, Leandro Terrés escreveu:
>> This device identifies has IdProduct 0xe411 and is a clone of KWorld
>> UB499-2T T09(IT9137).
>>
>> This patch simply adds support for this device.
>
> Patch applies ok, with just one small whitespace issue. However, you
> forgot to add your signed-off-by: on it. Also, it helps if you copy the
> driver's maintainer (Malcolm).
>
> Patchwork: http://patchwork.linuxtv.org/patch/8099/
>
> WARNING: please, no space before tabs
> #24: FILE: drivers/media/dvb/dvb-usb/dvb-usb-ids.h:323:
> +#define USB_PID_SVEON_STV22_IT9137     ^I^I0xe411$
>
> ERROR: Missing Signed-off-by: line(s)
>
> total: 1 errors, 1 warnings, 29 lines checked
>

This is my first contribution and I don't know who to do that.

Sorry.
--
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/dvb-usb/dvb-usb-ids.h b/drivers/media/dvb/dvb-usb/dvb-usb-ids.h
--- a/drivers/media/dvb/dvb-usb/dvb-usb-ids.h	2011-09-24 05:45:14.000000000 +0200
+++ b/drivers/media/dvb/dvb-usb/dvb-usb-ids.h	2011-10-08 08:23:15.789883877 +0200
@@ -320,6 +320,7 @@ 
 #define USB_PID_TVWAY_PLUS				0x0002
 #define USB_PID_SVEON_STV20				0xe39d
 #define USB_PID_SVEON_STV22				0xe401
+#define USB_PID_SVEON_STV22_IT9137     		0xe411
 #define USB_PID_AZUREWAVE_AZ6027			0x3275
 #define USB_PID_TERRATEC_DVBS2CI_V1			0x10a4
 #define USB_PID_TERRATEC_DVBS2CI_V2			0x10ac
diff --git a/drivers/media/dvb/dvb-usb/it913x.c b/drivers/media/dvb/dvb-usb/it913x.c
--- a/drivers/media/dvb/dvb-usb/it913x.c	2011-09-24 05:45:14.000000000 +0200
+++ b/drivers/media/dvb/dvb-usb/it913x.c	2011-10-08 08:32:00.388191986 +0200
@@ -533,6 +533,7 @@ 
 
 static struct usb_device_id it913x_table[] = {
 	{ USB_DEVICE(USB_VID_KWORLD_2, USB_PID_KWORLD_UB499_2T_T09) },
+	{ USB_DEVICE(USB_VID_KWORLD_2, USB_PID_SVEON_STV22_IT9137) },
 	{}		/* Terminating entry */
 };
 
@@ -608,11 +609,14 @@ 
 		.rc_codes	= RC_MAP_KWORLD_315U,
 	},
 	.i2c_algo         = &it913x_i2c_algo,
-	.num_device_descs = 1,
+	.num_device_descs = 2,
 	.devices = {
 		{   "Kworld UB499-2T T09(IT9137)",
 			{ &it913x_table[0], NULL },
 			},
+		{   "Sveon STV22 Dual DVB-T HDTV(IT9137)",
+			{ &it913x_table[1], NULL },
+			},
 
 	}
 };