[LinuxBIOS] [PATCH] improved SPI flash support (restructured)
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Mon Oct 1 12:28:46 CEST 2007
On 29.09.2007 15:59, Stefan Reinauer wrote:
> * Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net> [070929 04:08]:
>> +#define ITE_SUPERIO_PORT1 0x2e
>> +#define ITE_SUPERIO_PORT2 0x4e
>
> This has nothing to do with the IDE. All PC SuperIOs are on 2e or 4e.
>
>> +int probe_spi(struct flashchip *flash)
>> +{
>> + unsigned char readarr[3];
>
> This should be a struct imho
>
> typedef struct spi_id {
> unsigned char vendor_id;
> unsigned short device_id;
> } spi_id_t;
Yes, but the underlying generic SPI function uses an array for commands
and results. Once I add write/erase support, it will become obvious why
I used unsigned char arrays.
> The driver does only probe so far, right?
Yes.
> what about the other SPI commands?
Will be implemented once this patch has been tested.
Carl-Daniel
More information about the coreboot
mailing list