[LinuxBIOS] [PATCH] flashrom patch for iwill dk8_htx
Mondrian Nuessle
nuessle at uni-mannheim.de
Wed May 2 18:45:42 CEST 2007
While I think that a clean function interface is nice, I can live
without it...
So I could go with the patch, as Uwe provided it.
Mondrian
Luc Verhaegen wrote:
> On Sat, Apr 28, 2007 at 02:00:42AM +0200, Uwe Hermann wrote:
>> On Fri, Apr 27, 2007 at 10:33:53PM +0200, Luc Verhaegen wrote:
>>> On Fri, Apr 27, 2007 at 04:44:10PM +0200, Mondrian Nuessle wrote:
>>>> Actually you do not need to set the index reg twice. I just used the
>>>> code pattern from the Agami Aruma board, which does the same thing...
>>> In that case, we should introduce functions like:
>>>
>>> void w83xxx_ext_enter(void);
>>> void w83xxx_ext_leave(void);
>>>
>>> /* Enter extended functions before using these! */
>>> unsigned char w83xxx_ext_read(unsigned char index);
>>> void w83xxx_ext_write(unsigned char index, unsigned char data);
>>> void w83xxx_ext_mask(unsigned char index, unsigned char data,
>>> unsigned char mask);
>> Hm, not sure. Looks a bit like overkill in this case.
>>
>> Uwe.
>
> Why?
>
> Three out of four functions in there (when mondrians patch finally gets
> in) would make use of this already. That says something about what
> future board enables will look like too.
>
> This isn't overkill in any way, it's formalizing what's there.
>
> Without mondrians code:
> board_enable.c | 137 +++++++++++++++++++++++++++++++-------------------------- 1 file
> changed, 76 insertions(+), 61 deletions(-)
>
> So when mondrians patch is in there, this is made up for completely, and
> the board enables really are a lot friendlier.
>
> Luc Verhaegen.
>
--
Mondrian Nuessle University of Mannheim
Phone: +49 621 181 2717 Computer Architecture Group
Fax: +49 621 181 2713 http://ra.ti.uni-mannheim.de
mailto:nuessle at uni-mannheim.de
More information about the coreboot
mailing list