[flashrom] [PATCH] Add Broadcom OSB4 chipset enable / parallel flash chips

Joshua Roys roysjosh at gmail.com
Wed Sep 15 16:34:26 CEST 2010


Hello,

Awesome!  I'm glad you got it working.

On 09/15/2010 06:19 AM, Carl-Daniel Hailfinger wrote:
> Funny. The 0xc6f access looks very similar to parts of the SB400 chipset
> enable. (Side note: we should ask AMD if the SB400 chipset enable is
> correct.)
> 

Yes, I noticed the same thing- hence the placement in chipset_enable.c.

> Joshua, I would like this in 0.9.3. Would it be OK for you to split the
> patch in a chipset enable and a flash chip addition? If possible,
> coordinate with Mattias to avoid conflicts in flashchips.[ch]
> 

OK, attached are two separate patches and a diff...  if the chip #define
fixup gets applied first, then use the post-prefix-change.diff.txt and
squash it in with the flash chip patch.  Otherwise that patch will have
to be updated in order to account for these chips.  (I'll try to figure
out on IRC what order these will be in, and if needs be I can spin
another patch set.)

Thanks!

Signed-off-by: Joshua Roys <roysjosh at gmail.com>

Josh
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-Add-some-parallel-flash-chips.patch.txt
URL: <http://www.flashrom.org/pipermail/flashrom/attachments/20100915/0645764b/attachment.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-Add-chipset-enable-for-Broadcom-OSB4.patch.txt
URL: <http://www.flashrom.org/pipermail/flashrom/attachments/20100915/0645764b/attachment-0001.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: post-prefix-change.diff.txt
URL: <http://www.flashrom.org/pipermail/flashrom/attachments/20100915/0645764b/attachment-0002.txt>


More information about the flashrom mailing list