[coreboot] [PATCH] flashrom: Fix probe_spi_res failover
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Fri Nov 28 00:01:08 CET 2008
On 27.11.2008 19:37, FENG Yu Ning wrote:
> On Fri, Nov 28, 2008 at 1:32 AM, Carl-Daniel Hailfinger
> <c-d.hailfinger.devel.2006 at gmx.net> wrote:
>
>> The existing check in probe_spi_res() was right for SPI controllers
>> which support all commands, but may not exist.
>> For controllers which support only a subset of commands, it will fail in
>> unexpected ways.
>>
>
> sometimes the controller supports that command, but the command is
> 1. not in the opmenu, and we cannot change it (spi conf locked down)
> 2. the slave spi flash chip does not support rdid.
> for example, my ich7 + sst25lf040a (both 1. and 2.)
>
> There is also BBAR issue in the probing process and more. spi_res only
> reads from address 0x0, which could be below BBA. And there is
> read-only opmenu index difference with our hardcoded opcodes table.
> They are other problems to attack against seperately, though.
>
> Obviousely ack from me is not decisive, but still
> Acked-by: FENG yu ning <fengyuning1984 at gmail.com>
>
Thanks, r3774.
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
More information about the coreboot
mailing list