linux-kernel - RE: [PATCH v3] staging: add Broadcom Crystal HD driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <80A6ED01A151614884EE6C5D6AF0F2E90460D23460@SJEXCHCCR01.corp.ad.broadcom.com>
Date:	Mon, 4 Jan 2010 18:36:01 -0800
From:	"Naren (Narendra) Sankar" <nsankar@...adcom.com>
To:	"Robert Hancock" <hancockrwd@...il.com>,
	"Jarod Wilson" <jarod@...hat.com>
cc:	"Greg KH" <greg@...ah.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"Scott Davilla" <davilla@....com>,
	"Manu Abraham" <abraham.manu@...il.com>
Subject: RE: [PATCH v3] staging: add Broadcom Crystal HD driver

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@...il.com] 
Sent: Monday, January 04, 2010 6:22 PM
To: Jarod Wilson
Cc: Greg KH; linux-kernel@...r.kernel.org; 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/