[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH for-2.12?] m25p80: Correct the at25
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH for-2.12?] m25p80: Correct the at25128a/at25256a eeproms size |
Date: |
Wed, 9 May 2018 00:50:42 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
Hi Peter, Kevin,
On 04/16/2018 05:52 PM, Alistair Francis wrote:
> On Mon, Apr 16, 2018 at 9:05 AM, mar.krzeminski
> <address@hidden> wrote:
>> W dniu 15.04.2018 o 22:31, Philippe Mathieu-Daudé pisze:
>>>
>>> >From the datasheet (3368J–SEEPR) description:
>>> The AT25128A/256A provides 131,072/262,144 bits of serial
>>> electrically-erasable programmable read only memory (EEPROM)
>>> organized as 16,384/32,768 words of 8 bits each.
>>>
>>> However QEMU models the flash size in bytes.
>>> Correct the at25128a/at25256a entries to reflect the datasheet size.
>>>
>>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>>
>> Reviewed-by: Marcin Krzeminski <address@hidden>
>
> Reviewed-by: Alistair Francis <address@hidden>
>
> Alistair
>
>>>
>>> ---
>>> This is not a regression, so can now wait for 2.13.
>>>
>>> hw/block/m25p80.c | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c
>>> index b49c8e9caa..d0b9fbfc50 100644
>>> --- a/hw/block/m25p80.c
>>> +++ b/hw/block/m25p80.c
>>> @@ -196,8 +196,8 @@ static const FlashPartInfo known_devices[] = {
>>> /* Atmel EEPROMS - it is assumed, that don't care bit in command
>>> * is set to 0. Block protection is not supported.
>>> */
>>> - { INFO("at25128a-nonjedec", 0x0, 0, 1, 131072, EEPROM) },
>>> - { INFO("at25256a-nonjedec", 0x0, 0, 1, 262144, EEPROM) },
>>> + { INFO("at25128a-nonjedec", 0x0, 0, 1, 16384, EEPROM) },
>>> + { INFO("at25256a-nonjedec", 0x0, 0, 1, 32768, EEPROM) },
>>
>> Thanks!
>> Marcin
>>
>>> /* EON -- en25xxx */
>>> { INFO("en25f32", 0x1c3116, 0, 64 << 10, 64, ER_4K) },
>>
>>
>>
Can this patch goes via the ARM or Block tree?
Thanks,
Phil.
signature.asc
Description: OpenPGP digital signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-block] [Qemu-devel] [PATCH for-2.12?] m25p80: Correct the at25128a/at25256a eeproms size,
Philippe Mathieu-Daudé <=