[linux-dvb] Re: Patch for Yuan PD300 mini

Ben Woods woodsb02 at gmail.com
Tue Oct 10 06:41:21 CEST 2006


Just in case you were wondering, I have updated my hg source and
recreated the patches, which are once again attached. Also, this time
i didnt forget to add the Subject and Signed off parts to the top of
yuan2.patch.

Both patches successfully compile and work with my device.

Cheers

On 10/10/06, Ben Woods <woodsb02 at gmail.com> wrote:
> Ok,
>
> I think I have created a successful patch now.
>
> There are 2 versions... one which defines a new device in dt200u.c in the
> static struct dvb_usb_device_properties wt220u_properties
> struct... and one which just adds an additional cold id to the already
> existing device.
>
> Which is a better method?
>
> Could the better patch please be tested and applied if ok?
>
> Cheers
>
> On 10/10/06, Michael Krufky <mkrufky at linuxtv.org> wrote:
> > On 10/8/06, Gunther Mayer <gunther.mayer at gmx.net> wrote:
> > > Ben Woods wrote:
> > > > This patch fixes the cold boot id for the Yuan PD300 mini USB pen.
> > > > It can be applied with the following command from within the v4l-dvb
> > > > directory:
> > > > patch -p1 < /path/to/yuan.patch
> > > >
> > > > When using this card, you need to have the dvb-usb-wt220u-02.fw
> > > > firmware file in the correct directory (eg. /lib/firmware/`uname
> > > > -r`/).
> > > >
> > > > Can this patch be tested by folk and then hopefully applied?
> > > No. Your patch breaks cards with ID 0x222.
> >
> > Ben Woods wrote:
> > > Ok - i see.
> > >
> > > Is there anyway we can include the correct cold boot id for my device
> > > without breaking it for other devices? mkrufky - you were quite
> > > helpful with this once before... any ideas?
> >
> > It's rather simple, actually.  Rather than replacing the current pid with yours, what you need to do
> > is add an additional pid to dvb-usb-ids.h.  Then add the new id's to the driver's usb table, reference
> > them in the device properties, and increase num_device_descs.
> >
> > For example, the changeset in the following link does just what you're trying to do now, but with a
> > different device:
> >
> > http://www.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=ac9ffb90b8d0d28e3d9e5b5a045ee870a6269b5f
> >
> > In your case, you are creating a new entry, using 0x220 as the cold id, and 0x221 as the warm id.  Since
> > that warm idea is already recognized by the driver, you only have to add the missing cold id.
> >
> > Your patch should look very similar to the one in the link above.
> >
> > Hope this helps,
> >
> > Mike
> >
>
>
> --
> From: Benjamin Woods
> woodsb02 at gmail.com
> Registered Linux User #372573
> Registered Ubuntu User # 6395
>
>
>


-- 
From: Benjamin Woods
woodsb02 at gmail.com
Registered Linux User #372573
Registered Ubuntu User # 6395
-------------- next part --------------
A non-text attachment was scrubbed...
Name: yuan1.patch
Type: text/x-patch
Size: 1801 bytes
Desc: not available
Url : http://www.linuxtv.org/pipermail/linux-dvb/attachments/20061010/bc49006c/yuan1.bin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: yuan2.patch
Type: text/x-patch
Size: 1674 bytes
Desc: not available
Url : http://www.linuxtv.org/pipermail/linux-dvb/attachments/20061010/bc49006c/yuan2.bin


More information about the linux-dvb mailing list