Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Linux: Kernel

Re: [PATCH v3] staging: add Broadcom Crystal HD driver

 

 

Linux kernel RSS feed   Index | Next | Previous | View Threaded


greg at kroah

Jan 4, 2010, 3:19 PM

Post #1 of 8 (2740 views)
Permalink
Re: [PATCH v3] staging: add Broadcom Crystal HD driver

On Mon, Jan 04, 2010 at 06:02:27PM -0500, Jarod Wilson wrote:
> This patch supersedes the earlier ones sent by Manu Abraham to add
> the Broadcom Crystal HD driver to the staging tree, per discussion
> with him about it. I've been working with Broadcom's Naren Sankar
> on this driver for a number of months, and had already talked Naren
> about submitting this on Broadcom's behalf, didn't expect anyone
> else to jump on submitting it as quickly as Manu did. ;)

Looks good, I'll queue it up tomorrow.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


jarod at redhat

Jan 4, 2010, 3:34 PM

Post #2 of 8 (2655 views)
Permalink
Re: [PATCH v3] staging: add Broadcom Crystal HD driver [In reply to]

On 1/4/10 6:19 PM, Greg KH wrote:
> On Mon, Jan 04, 2010 at 06:02:27PM -0500, Jarod Wilson wrote:
>> This patch supersedes the earlier ones sent by Manu Abraham to add
>> the Broadcom Crystal HD driver to the staging tree, per discussion
>> with him about it. I've been working with Broadcom's Naren Sankar
>> on this driver for a number of months, and had already talked Naren
>> about submitting this on Broadcom's behalf, didn't expect anyone
>> else to jump on submitting it as quickly as Manu did. ;)
>
> Looks good, I'll queue it up tomorrow.

Nb: it looks like the thing is so massive (~935k), its not showing up on
lkml (at least, not in a timely fashion), so for the interested:

<http://people.redhat.com/jwilson/misc/broadcom-crystalhd-decoder-driver-staging.patch>

Thanks much!

--
Jarod Wilson
jarod [at] redhat
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


hancockrwd at gmail

Jan 4, 2010, 6:21 PM

Post #3 of 8 (2651 views)
Permalink
Re: [PATCH v3] staging: add Broadcom Crystal HD driver [In reply to]

On 01/04/2010 05:34 PM, Jarod Wilson wrote:
> On 1/4/10 6:19 PM, Greg KH wrote:
>> On Mon, Jan 04, 2010 at 06:02:27PM -0500, Jarod Wilson wrote:
>>> This patch supersedes the earlier ones sent by Manu Abraham to add
>>> the Broadcom Crystal HD driver to the staging tree, per discussion
>>> with him about it. I've been working with Broadcom's Naren Sankar
>>> on this driver for a number of months, and had already talked Naren
>>> about submitting this on Broadcom's behalf, didn't expect anyone
>>> else to jump on submitting it as quickly as Manu did. ;)
>>
>> Looks good, I'll queue it up tomorrow.
>
> Nb: it looks like the thing is so massive (~935k), its not showing up on
> lkml (at least, not in a timely fashion), so for the interested:
>
> <http://people.redhat.com/jwilson/misc/broadcom-crystalhd-decoder-driver-staging.patch>
>
> Thanks much!

Literally the majority of the code is #defines for what looks like
register bits, etc - is all that really necessary?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


nsankar at broadcom

Jan 4, 2010, 6:36 PM

Post #4 of 8 (2646 views)
Permalink
RE: [PATCH v3] staging: add Broadcom Crystal HD driver [In reply to]

The register bits come straight from the HW RTL. So this way there are no errors or typos. It would be easier for maintenance from our side since this is generated automagically. And when we add new HW, again it is a script that spews this out.

Not absolutely necessary, but worried that cleanups may cause more problems than fix.


Naren Sankar (+1 408 218 6327)
Architect/PLM
Media PC, Broadband Communications Group
Broadcom Corp.

-----Original Message-----
From: Robert Hancock [mailto:hancockrwd [at] gmail]
Sent: Monday, January 04, 2010 6:22 PM
To: Jarod Wilson
Cc: Greg KH; linux-kernel [at] vger; Scott Davilla; Naren (Narendra) Sankar; Manu Abraham
Subject: Re: [PATCH v3] staging: add Broadcom Crystal HD driver

On 01/04/2010 05:34 PM, Jarod Wilson wrote:
> On 1/4/10 6:19 PM, Greg KH wrote:
>> On Mon, Jan 04, 2010 at 06:02:27PM -0500, Jarod Wilson wrote:
>>> This patch supersedes the earlier ones sent by Manu Abraham to add
>>> the Broadcom Crystal HD driver to the staging tree, per discussion
>>> with him about it. I've been working with Broadcom's Naren Sankar
>>> on this driver for a number of months, and had already talked Naren
>>> about submitting this on Broadcom's behalf, didn't expect anyone
>>> else to jump on submitting it as quickly as Manu did. ;)
>>
>> Looks good, I'll queue it up tomorrow.
>
> Nb: it looks like the thing is so massive (~935k), its not showing up on
> lkml (at least, not in a timely fashion), so for the interested:
>
> <http://people.redhat.com/jwilson/misc/broadcom-crystalhd-decoder-driver-staging.patch>
>
> Thanks much!

Literally the majority of the code is #defines for what looks like
register bits, etc - is all that really necessary?


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


jarodwilson at gmail

Jan 4, 2010, 10:58 PM

Post #5 of 8 (2655 views)
Permalink
Re: [PATCH v3] staging: add Broadcom Crystal HD driver [In reply to]

On Mon, Jan 4, 2010 at 9:36 PM, Naren (Narendra) Sankar
<nsankar [at] broadcom> wrote:
> The register bits come straight from the HW RTL. So this way there are no errors or typos. It would be easier for maintenance from our side since this is generated automagically. And when we add new HW, again it is a script that spews this out.
>
> Not absolutely necessary, but worried that cleanups may cause more problems than fix.

I think the automation is worth it to a point, but this header does
seem to be a wee bit excessive. I doubt if we're even using 1% of the
~12k defines in that one header, so we're bloating the code by a huge
amount for relatively little benefit. Its probably worth maintaining
the full register header *somewhere* (in with the userspace lib
somewhere?), but maybe we just do a trimmed down version for the
kernel module that contains only the bits we actually need.

> -----Original Message-----
> From: Robert Hancock [mailto:hancockrwd [at] gmail]
> Sent: Monday, January 04, 2010 6:22 PM
> To: Jarod Wilson
> Cc: Greg KH; linux-kernel [at] vger; Scott Davilla; Naren (Narendra) Sankar; Manu Abraham
> Subject: Re: [PATCH v3] staging: add Broadcom Crystal HD driver
>
> On 01/04/2010 05:34 PM, Jarod Wilson wrote:
>> On 1/4/10 6:19 PM, Greg KH wrote:
>>> On Mon, Jan 04, 2010 at 06:02:27PM -0500, Jarod Wilson wrote:
>>>> This patch supersedes the earlier ones sent by Manu Abraham to add
>>>> the Broadcom Crystal HD driver to the staging tree, per discussion
>>>> with him about it. I've been working with Broadcom's Naren Sankar
>>>> on this driver for a number of months, and had already talked Naren
>>>> about submitting this on Broadcom's behalf, didn't expect anyone
>>>> else to jump on submitting it as quickly as Manu did. ;)
>>>
>>> Looks good, I'll queue it up tomorrow.
>>
>> Nb: it looks like the thing is so massive (~935k), its not showing up on
>> lkml (at least, not in a timely fashion), so for the interested:
>>
>> <http://people.redhat.com/jwilson/misc/broadcom-crystalhd-decoder-driver-staging.patch>
>>
>> Thanks much!
>
> Literally the majority of the code is #defines for what looks like
> register bits, etc - is all that really necessary?
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo [at] vger
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>



--
Jarod Wilson
jarodwilson [at] gmail
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


nsankar at broadcom

Jan 4, 2010, 11:05 PM

Post #6 of 8 (2646 views)
Permalink
Re: [PATCH v3] staging: add Broadcom Crystal HD driver [In reply to]

That is ok by us.

The driver does only use a small subset.

Naren

----- Original Message -----
From: Jarod Wilson <jarodwilson [at] gmail>
To: Naren (Narendra) Sankar
Cc: Robert Hancock <hancockrwd [at] gmail>; Jarod Wilson <jarod [at] redhat>; Greg KH <greg [at] kroah>; linux-kernel [at] vger <linux-kernel [at] vger>; Scott Davilla <davilla [at] 4pi>; Manu Abraham <abraham.manu [at] gmail>
Sent: Mon Jan 04 22:58:29 2010
Subject: Re: [PATCH v3] staging: add Broadcom Crystal HD driver

On Mon, Jan 4, 2010 at 9:36 PM, Naren (Narendra) Sankar
<nsankar [at] broadcom> wrote:
> The register bits come straight from the HW RTL. So this way there are no errors or typos. It would be easier for maintenance from our side since this is generated automagically. And when we add new HW, again it is a script that spews this out.
>
> Not absolutely necessary, but worried that cleanups may cause more problems than fix.

I think the automation is worth it to a point, but this header does
seem to be a wee bit excessive. I doubt if we're even using 1% of the
~12k defines in that one header, so we're bloating the code by a huge
amount for relatively little benefit. Its probably worth maintaining
the full register header *somewhere* (in with the userspace lib
somewhere?), but maybe we just do a trimmed down version for the
kernel module that contains only the bits we actually need.

> -----Original Message-----
> From: Robert Hancock [mailto:hancockrwd [at] gmail]
> Sent: Monday, January 04, 2010 6:22 PM
> To: Jarod Wilson
> Cc: Greg KH; linux-kernel [at] vger; Scott Davilla; Naren (Narendra) Sankar; Manu Abraham
> Subject: Re: [PATCH v3] staging: add Broadcom Crystal HD driver
>
> On 01/04/2010 05:34 PM, Jarod Wilson wrote:
>> On 1/4/10 6:19 PM, Greg KH wrote:
>>> On Mon, Jan 04, 2010 at 06:02:27PM -0500, Jarod Wilson wrote:
>>>> This patch supersedes the earlier ones sent by Manu Abraham to add
>>>> the Broadcom Crystal HD driver to the staging tree, per discussion
>>>> with him about it. I've been working with Broadcom's Naren Sankar
>>>> on this driver for a number of months, and had already talked Naren
>>>> about submitting this on Broadcom's behalf, didn't expect anyone
>>>> else to jump on submitting it as quickly as Manu did. ;)
>>>
>>> Looks good, I'll queue it up tomorrow.
>>
>> Nb: it looks like the thing is so massive (~935k), its not showing up on
>> lkml (at least, not in a timely fashion), so for the interested:
>>
>> <http://people.redhat.com/jwilson/misc/broadcom-crystalhd-decoder-driver-staging.patch>
>>
>> Thanks much!
>
> Literally the majority of the code is #defines for what looks like
> register bits, etc - is all that really necessary?
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo [at] vger
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>



--
Jarod Wilson
jarodwilson [at] gmail

칻&~&+-ݶw˛mbdzޖ)w*jgݢj/zޖ2ޙ&)ߡaGhj:+vw٥>WiaxPjm -+d_


jarodwilson at gmail

Jan 4, 2010, 11:10 PM

Post #7 of 8 (2646 views)
Permalink
Re: [PATCH v3] staging: add Broadcom Crystal HD driver [In reply to]

Okay, cool. Greg, would you like an updated trimmed header before
adding this to staging, or can I just follow up later with a patch to
reduce the heft?

--jarod


On Tue, Jan 5, 2010 at 2:05 AM, Naren (Narendra) Sankar
<nsankar [at] broadcom> wrote:
> That is ok by us.
>
> The driver does only use a small subset.
>
> Naren
>
> ----- Original Message -----
> From: Jarod Wilson <jarodwilson [at] gmail>
> To: Naren (Narendra) Sankar
> Cc: Robert Hancock <hancockrwd [at] gmail>; Jarod Wilson <jarod [at] redhat>; Greg KH <greg [at] kroah>; linux-kernel [at] vger <linux-kernel [at] vger>; Scott Davilla <davilla [at] 4pi>; Manu Abraham <abraham.manu [at] gmail>
> Sent: Mon Jan 04 22:58:29 2010
> Subject: Re: [PATCH v3] staging: add Broadcom Crystal HD driver
>
> On Mon, Jan 4, 2010 at 9:36 PM, Naren (Narendra) Sankar
> <nsankar [at] broadcom> wrote:
>> The register bits come straight from the HW RTL. So this way there are no errors or typos. It would be easier for maintenance from our side since this is generated automagically. And when we add new HW, again it is a script that spews this out.
>>
>> Not absolutely necessary, but worried that cleanups may cause more problems than fix.
>
> I think the automation is worth it to a point, but this header does
> seem to be a wee bit excessive. I doubt if we're even using 1% of the
> ~12k defines in that one header, so we're bloating the code by a huge
> amount for relatively little benefit. Its probably worth maintaining
> the full register header *somewhere* (in with the userspace lib
> somewhere?), but maybe we just do a trimmed down version for the
> kernel module that contains only the bits we actually need.
>
>> -----Original Message-----
>> From: Robert Hancock [mailto:hancockrwd [at] gmail]
>> Sent: Monday, January 04, 2010 6:22 PM
>> To: Jarod Wilson
>> Cc: Greg KH; linux-kernel [at] vger; Scott Davilla; Naren (Narendra) Sankar; Manu Abraham
>> Subject: Re: [PATCH v3] staging: add Broadcom Crystal HD driver
>>
>> On 01/04/2010 05:34 PM, Jarod Wilson wrote:
>>> On 1/4/10 6:19 PM, Greg KH wrote:
>>>> On Mon, Jan 04, 2010 at 06:02:27PM -0500, Jarod Wilson wrote:
>>>>> This patch supersedes the earlier ones sent by Manu Abraham to add
>>>>> the Broadcom Crystal HD driver to the staging tree, per discussion
>>>>> with him about it. I've been working with Broadcom's Naren Sankar
>>>>> on this driver for a number of months, and had already talked Naren
>>>>> about submitting this on Broadcom's behalf, didn't expect anyone
>>>>> else to jump on submitting it as quickly as Manu did. ;)
>>>>
>>>> Looks good, I'll queue it up tomorrow.
>>>
>>> Nb: it looks like the thing is so massive (~935k), its not showing up on
>>> lkml (at least, not in a timely fashion), so for the interested:
>>>
>>> <http://people.redhat.com/jwilson/misc/broadcom-crystalhd-decoder-driver-staging.patch>
>>>
>>> Thanks much!
>>
>> Literally the majority of the code is #defines for what looks like
>> register bits, etc - is all that really necessary?
>>
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>> the body of a message to majordomo [at] vger
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>> Please read the FAQ at http://www.tux.org/lkml/
>>
>
>
>
> --
> Jarod Wilson
> jarodwilson [at] gmail
>
>



--
Jarod Wilson
jarodwilson [at] gmail
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


greg at kroah

Jan 5, 2010, 6:45 AM

Post #8 of 8 (2632 views)
Permalink
Re: [PATCH v3] staging: add Broadcom Crystal HD driver [In reply to]

On Tue, Jan 05, 2010 at 02:10:56AM -0500, Jarod Wilson wrote:
> Okay, cool. Greg, would you like an updated trimmed header before
> adding this to staging, or can I just follow up later with a patch to
> reduce the heft?

Later patches cleaning stuff up are best.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Linux kernel RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.