[coreboot] Add support for SPI-Chips on ICH9 to flash rom / please delete the last mail
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Thu May 15 14:28:50 CEST 2008
Hi Dominik,
On 15.05.2008 09:43, Dominik Geyer wrote:
> On Wednesday 14 May 2008 17:11, Carl-Daniel Hailfinger wrote:
>
>> On 14.05.2008 16:17, Carl-Daniel Hailfinger wrote:
>>
>>> On 14.05.2008 08:23, Claus Gindhart wrote:
>>>
>>>> Hi Carl-Daniel,
>>>>
>>>> my colleague Dominik will investigate into this; we will provide a new
>>>> patch within the next few days.
>>>>
>>> Thanks. By the way, the ICH8 datasheet says your code will not work with
>>> ICH8 (wrong SPIBAR address), so you might want to remove ICH8 bits from
>>> the patch. Hardcoding the RCRB address is a problem as well.
>>>
>>> I have sent another patch a few minutes ago which will allow you to
>>> replace is_supported_chipset() with ich9_detected and
>>> curflash->virtual_registers with *ich_spibar. That patch also calculates
>>> the correct RCRB address and the correct SPIBAR address. The subject of
>>> the mail was "[coreboot] [PATCH] flashrom: Infrastructure for ICH9
>>> merge".
>>>
>> That patch has been merged in r3314. You can also drop
>> munmap_ich_registers() and map_ich_registers() now.
>>
>> If you can create a wrapper for run_opcode() which can be called from
>> spi_command(), you should be able to drop ~80% of the code in ichspi.c.
>> Such a wrapper would look up the opcode from spi_command() in the list
>> of programmed opcodes and convert the arguments. AFAICS this should be
>> easy.
>>
>
> Sounds good. Your infrastructure/restructure patches are a nice approach. I
> will adjust our code and provide you with the patches.
>
Thanks! I'm looking forward to these patches.
Regards,
Carl-Daniel
More information about the coreboot
mailing list