From: Timothy Lee <timothy.lee@siriushk.com>
To: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: Devin Heitmueller <devin.heitmueller@gmail.com>,
David Wong <davidtlwong@gmail.com>,
linux-media@vger.kernel.org
Subject: Re: [PATCH] Support for Legend Silicon LGS8913/LGS8GL5/LGS8GXX China DMB-TH digital demodulator
Date: Tue, 07 Apr 2009 08:54:04 +0800 [thread overview]
Message-ID: <49DAA42C.8070302@siriushk.com> (raw)
In-Reply-To: <20090406144408.63b2ef71@pedra.chehab.org>
Dear Mauro,
I wrote the original lgs8gl5 driver by reverse-engineering my USB TV
stick using UsbSnoop.
I've been working together with David to make sure his lgs8gxx driver
works with my TV stick, so yes, the generic driver actually works. :)
Regards,
Timothy Lee
On 04/07/2009 01:44 AM, Mauro Carvalho Chehab wrote:
> On Tue, 31 Mar 2009 00:54:49 -0400
> Devin Heitmueller<devin.heitmueller@gmail.com> wrote:
>
>
>> On Tue, Mar 17, 2009 at 11:55 AM, David Wong<davidtlwong@gmail.com> wrote:
>>
>>> This patch contains the unified driver for Legend Silicon LGS8913 and
>>> LGS8GL5. It should replace lgs8gl5.c in media/dvb/frontends
>>>
>>> David T.L. Wong
>>>
>> David,
>>
>> The questions you posed tonight on a separate thread about making the
>> xc5000 work with this device prompts the question:
>>
>> Do you know that this driver you submitted actually works? Have you
>> successfully achieved lock with this driver and been able to view the
>> stream?
>>
>> It is great to see the improvements and more generic support, but if
>> you don't have it working in at least one device, then it probably
>> shouldn't be submitted upstream yet, and it definitely should not be
>> replacing an existing driver.
>>
> We need to do some tests before replacing the existing one. Yet, it is better
> to have a generic device than specific ones. Do you have any card with lg8gl5?
> If so, could you please test the new driver for us?
>
> Anyway, I'm applying what we currently have.
>
> Cheers,
> Mauro
> --
> 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
>
next prev parent reply other threads:[~2009-04-07 1:34 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-03-17 15:55 [PATCH] Support for Legend Silicon LGS8913/LGS8GL5/LGS8GXX China DMB-TH digital demodulator David Wong
2009-03-27 9:57 ` Mauro Carvalho Chehab
2009-03-27 13:02 ` David Wong
2009-04-01 7:35 ` David Wong
2009-03-31 4:54 ` Devin Heitmueller
2009-03-31 5:19 ` David Wong
2009-04-06 17:44 ` Mauro Carvalho Chehab
2009-04-07 0:54 ` Timothy Lee [this message]
2009-04-07 2:17 ` Mauro Carvalho Chehab
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=49DAA42C.8070302@siriushk.com \
--to=timothy.lee@siriushk.com \
--cc=davidtlwong@gmail.com \
--cc=devin.heitmueller@gmail.com \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@infradead.org \
/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