[coreboot] [patch] flashrom: resending my patch about SPI/LPC conflicts

Ward Vandewege ward at gnu.org
Fri Feb 20 19:03:49 CET 2009


On Fri, Feb 20, 2009 at 06:53:24PM +0100, Peter Stuge wrote:
> Ward Vandewege wrote:
> > > It's not a very generic fix. It's basically a compile time decision
> > > whether to use SPI or LPC on SB600 chipsets.
> > 
> > I understand. It works around the hang for me.
> 
> But it breaks all SB600 boards with SPI flash.

Ah, I didn't realize that.

> > > Is there no way we can detect what the system booted from?
> > 
> > Is that what should decide which rom to flash?
> 
> I think that is the best we can do at this point.
> 
> > Are there scenarios where that would cause problems?
> 
> I can't think of one. Do you have something in mind?

I was wondering if there would be a case where you boot from an SPI rom and
wanted to flash PLCC or something like that. But let's ignore that for now.

> > I suppose the 'easiest' way to fix this would be adding a command
> > line option that forces LPC/SPI?
> 
> It just makes me really sad and annoyed to require manual control
> like that when the SB600 for all other flash chip purposes is totally
> excellent - nice and simple! I just don't want to believe that noone
> thought of mirroring the boot straps in some register.

Oh, that would be very sad indeed.

> > Or, perhaps make the decision based on which board is detected?
> 
> Could do - would require every SB600 board to have an explicit
> board_enable entry..

Or maybe just the ones that we know have LPC? That's what I was thinking.

Thanks,
Ward.

-- 
Ward Vandewege <ward at fsf.org>
Free Software Foundation - Senior Systems Administrator




More information about the coreboot mailing list