[coreboot] [PATCH] v2: BCOM WinNETP680 improvements

Alex Mauer hawke at hawkesnest.net
Fri Oct 17 23:12:56 CEST 2008


Corey Osgood wrote:
> 2008/10/17 Uwe Hermann <uwe at hermann-uwe.de <mailto:uwe at hermann-uwe.de>>
> 
>     See patch.
> 
> 
>     Drop unneeded enable_mainboard_devices() function, the lines
> 
>           register "fn_ctrl_lo" = "0x80"
>           register "fn_ctrl_hi" = "0x1d"
> 
>     in the Config.lb file should do exactly the same (but please test that)!
> 
>     Also, drop some unused variables and (IMHO) unneeded debug messages.
> 
> 
> NACK. If the devices aren't enabled/disabled (and in the process 
> hidden/revealed) before PCI bringup, coreboot will get very confused 
> when they disappear and appear out of nowhere.

would it be feasible to have enable_mainboard_devices() read the 
structure setup by Config.lb so that those values can come from just one 
place?

-Alex Mauer "hawke"





More information about the coreboot mailing list