[coreboot] [PATCH] flashrom: add SST49LF020 support

Eric W. Biederman ebiederm at xmission.com
Thu Jan 8 17:54:31 CET 2009


Uwe Hermann <uwe at hermann-uwe.de> writes:

> On Wed, Jan 07, 2009 at 01:55:26PM +0100, Sven Schnelle wrote:
>> Hi Uwe,
>> 
>> Uwe Hermann <uwe at hermann-uwe.de> writes:
>> 
>> > Btw, can we mark all the new stuff from your patches as supported in
>> > the wiki, i.e. is all of it tested on hardware?
>> 
>> Yes, they are tested on hardware - i have three Boards here,
>
> OK, wiki updated.
>
>
>> 2 with the Intel 5000 Chipset,
>
> Is this chipset related to or compatible with the new 631xESB/632xESB/3100
> stuff you added in flashrom? Did you test 5000, or 631xESB/632xESB/3100,
> or both?
>
>
>> 1 with AMD 760 Chipset.
>
> Do you mean 780 here?
>
>  
>> The reason for adding the Flash/Chipset support is that i want to port/use
>> coreboot on these platforms.
>
> Nice! There's some old AMD-768 code in coreboot-v1, maybe you can use
> that as inspiration (no idea if or how well that worked back then).
> Those files don't have license headers unfortunately, so you can either

Not necessary the entire project was GPLv2.  Saying it in every file
is redundant and useful, but as I understand it not strictly necessary,
and in certain cases very very annoying.

> start from scratch with a v2 or even v3 code, or find out who wrote that
> code and add the proper license headers (judging from the svn commit log
> it should be Eric Biederman, GPL2)...

Yep.  I wrote it.  The code worked well, there is a cluster or two running it.
It was released late in the life cycle of the hardware because it took a while
to get the last critical bits out of NDA with AMD.

Eric




More information about the coreboot mailing list