On 07/04/2016 02:24 PM, Peter Maydell wrote:
> On 4 July 2016 at 13:18, Cédric Le Goater <c...@kaod.org> wrote:
>> This test uses the palmetto platform and the AST2400 SPI controller to
>> test the m25p80 flash module device model. The flash model is defined
>> by the platform (n25q256a) and it would be nice to find way to control
>> it, using a property probably.
>
>
>> +static inline void flash_writel(uint32_t data)
>> +{
>> + data = cpu_to_be32(data);
>> + memwrite(AST2400_FLASH_BASE, &data, 4);
>> +}
>> +
>> +static inline uint32_t flash_readl(void)
>> +{
>> + uint32_t data;
>> +
>> + memread(AST2400_FLASH_BASE, &data, 4);
>> + return be32_to_cpu(data);
>> +}
>
> These are weird. As per my question in the other thread, if you
> were writing this as a piece of test code that ran natively
> in the guest, how would you write it?
Here is the uboot flash code :
https://github.com/openbmc/u-boot/blob/v2016.05-aspeed-openbmc/arch/arm/mach-aspeed/flash.c#L420
So these are doing byte per byte, but there is no reason not to do
long.
C.
