[U-Boot] [PATCH] spi: add support SuperH SPI module
Yoshihiro Shimoda
yoshihiro.shimoda.uh at renesas.com
Wed Jan 19 09:16:33 CET 2011
Dear Wolfgang Denk,
Thank you very much for your review.
2011/01/18 18:28, Wolfgang Denk wrote:
>> @@ -37,6 +37,7 @@ COBJS-$(CONFIG_MPC8XXX_SPI) += mpc8xxx_spi.o
>> COBJS-$(CONFIG_MXC_SPI) += mxc_spi.o
>> COBJS-$(CONFIG_OMAP3_SPI) += omap3_spi.o
>> COBJS-$(CONFIG_SOFT_SPI) += soft_spi.o
>> +COBJS-$(CONFIG_SH_SPI) += sh_spi.o
>
> Please keep list sorted.
I will fix it.
>> +#define SPI_TBR 0x00
>> +#define SPI_RBR 0x00
>> +#define SPI_CR1 0x08
>> +#define SPI_CR2 0x10
>> +#define SPI_CR3 0x18
>> +#define SPI_CR4 0x20
>
> Please declare a poper C struct instead.
I will modify it.
>> +/* CR4 */
>> +#define SPI_TBEI 0x80
>> +#define SPI_TBFI 0x40
>> +#define SPI_RBEI 0x20
>> +#define SPI_RBFI 0x10
>> +#define SPI_WPABRT 0x04
>> +#define SPI_SSS 0x01
>> +
>> +#define SPI_FIFO_SIZE 32
>
> All this stuff should better go into a separate header file.
OK, I will create the sh_spi.h file.
>> +static void sh_spi_write(unsigned long data, unsigned long offset)
>> +{
>> + writel(data, CONFIG_SH_SPI_BASE + offset);
>
> NAK. We do not allow the base + offset notation. Pleaseuse a proper
> C struct instead.
>
> Please fix globally.
Yes, I will fix the driver.
Best regards,
Yoshihiro Shimoda
More information about the U-Boot
mailing list