From: Antti Palosaari <crope@iki.fi>
To: "Till Dörges" <till@doerges.net>
Cc: linux-media@vger.kernel.org
Subject: Re: PATCH: Added device (0ccd:00b4) to DVB_USB_RTL28XXU media driver
Date: Tue, 11 Feb 2014 22:53:13 +0200 [thread overview]
Message-ID: <52FA8DB9.3070300@iki.fi> (raw)
In-Reply-To: <52FA87CD.2030206@doerges.net>
Moikka Till!
Thanks for the 'patch' :)
I am not sure how I should handle that as the code itself is valid, but
patch is not as it should.
If you could make a proper patch, using git commit & git format-patch,
it will be nice. But as I understand it could be quite much of learning
many things, I am willing to take that and apply half manually. I still
need you signed-off-by tag as documented [1]. Please reply with
signed-off-by or even better if you could make formally correct patch.
[1] https://www.kernel.org/doc/Documentation/SubmittingPatches
regards
Antti
On 11.02.2014 22:27, Till Dörges wrote:
> Hi all,
>
> I've got the following DAB USB stick that also works fine with the DVB_USB_RTL28XXU
> driver after I added its USB ID:
>
> --- snip ---
> user@box:~> lsusb -d 0ccd:00b4
> Bus 001 Device 009: ID 0ccd:00b4 TerraTec Electronic GmbH
> --- snap ---
>
>
> I tried it on a recent openSUSE 13.1 with this kernel/architecture
>
> --- snip ---
> user@box:~> uname -a
> Linux box 3.11.10-7-desktop #1 SMP PREEMPT Mon Feb 3 09:41:24 UTC 2014 (750023e)
> x86_64 x86_64 x86_64 GNU/Linux
> --- snap ---
>
>
> The patches itself are trivial:
>
> --- ./drivers/media/dvb-core/dvb-usb-ids.h.orig 2014-02-09 22:36:35.266625484 +0100
> +++ ./drivers/media/dvb-core/dvb-usb-ids.h 2014-02-09 22:38:00.128199957 +0100
> @@ -256,6 +256,7 @@
> #define USB_PID_TERRATEC_T5 0x10a1
> #define USB_PID_NOXON_DAB_STICK 0x00b3
> #define USB_PID_NOXON_DAB_STICK_REV2 0x00e0
> +#define USB_PID_NOXON_DAB_STICK_REV3 0x00b4
> #define USB_PID_PINNACLE_EXPRESSCARD_320CX 0x022e
> #define USB_PID_PINNACLE_PCTV2000E 0x022c
> #define USB_PID_PINNACLE_PCTV_DVB_T_FLASH 0x0228
>
>
> --- ./drivers/media/usb/dvb-usb-v2/rtl28xxu.c.orig 2014-02-03 10:41:24.000000000
> +0100
> +++ ./drivers/media/usb/dvb-usb-v2/rtl28xxu.c 2014-02-09 22:37:53.464154845 +0100
> @@ -1362,6 +1362,8 @@ static const struct usb_device_id rtl28x
> &rtl2832u_props, "TerraTec NOXON DAB Stick", NULL) },
> { DVB_USB_DEVICE(USB_VID_TERRATEC, USB_PID_NOXON_DAB_STICK_REV2,
> &rtl2832u_props, "TerraTec NOXON DAB Stick (rev 2)", NULL) },
> + { DVB_USB_DEVICE(USB_VID_TERRATEC, USB_PID_NOXON_DAB_STICK_REV3,
> + &rtl2832u_props, "TerraTec NOXON DAB Stick (rev 3)", NULL) },
> { DVB_USB_DEVICE(USB_VID_GTEK, USB_PID_TREKSTOR_TERRES_2_0,
> &rtl2832u_props, "Trekstor DVB-T Stick Terres 2.0", NULL) },
> { DVB_USB_DEVICE(USB_VID_DEXATEK, 0x1101,
>
> HTH -- Till
>
--
http://palosaari.fi/
next prev parent reply other threads:[~2014-02-11 20:53 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-11 20:27 PATCH: Added device (0ccd:00b4) to DVB_USB_RTL28XXU media driver Till Dörges
2014-02-11 20:53 ` Antti Palosaari [this message]
2014-02-12 1:16 ` Till Dörges
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=52FA8DB9.3070300@iki.fi \
--to=crope@iki.fi \
--cc=linux-media@vger.kernel.org \
--cc=till@doerges.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox