Dear Maintainers,
Just a friendly reminder to ask if you could take a look at my patch. I'd
really appreciate your feedback and help.
Thank you! =)
-----Original Message-----
From: YuinYee Chew
Sent: Friday, September 29, 2023 9:35 PM
To: 'Heinrich Schuchardt' <heinrich.schucha...@canonical.com>
Cc: Sunil V L <suni...@ventanamicro.com>; Li Yong <yong...@intel.com>; Leif
Lindholm <quic_llind...@quicinc.com>; Ard Biesheuvel
<ardb+tianoc...@kernel.org>; Abner Chang <abner.ch...@amd.com>; Daniel Schaefer
<g...@danielschaefer.me>; devel@edk2.groups.io
Subject: RE: [PATCH v1] EmbeddedPkg/NorFlashInfoLib: Update norflash device list
Hi Heinrich,
Both gd25lq128d and gd25lq128e share the same ID "{ 0xc8, 0x60, 0x18}"
gd25lq128d :
https://www.gigadevice.com/product/flash/product-series/spi-nor-flash/gd25lq128d.html
gd25lq128e :
https://www.gigadevice.com/product/flash/product-series/spi-nor-flash/gd25lq128e
So, I'm thinking to put it as "gd25lq128" to represent both "d" and "e".
If sync with Linux, gd25lq128d will be better.
Do you have any suggestion on this?
Or anyone have any opinion?
Thanks again for the feedback! =)
-----Original Message-----
From: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
Sent: Friday, September 29, 2023 4:55 PM
To: YuinYee Chew <yuinyee.c...@starfivetech.com>
Cc: Sunil V L <suni...@ventanamicro.com>; Li Yong <yong...@intel.com>; Leif
Lindholm <quic_llind...@quicinc.com>; Ard Biesheuvel
<ardb+tianoc...@kernel.org>; Abner Chang <abner.ch...@amd.com>; Daniel Schaefer
<g...@danielschaefer.me>; devel@edk2.groups.io
Subject: Re: [PATCH v1] EmbeddedPkg/NorFlashInfoLib: Update norflash device list
On 9/29/23 04:02, John Chew wrote:
> Update Gigadevice norflash list:
> gd25q16, gd25q32, gd25q64, gd25lq64c, gd25q128, gd25lq128, gd25q256
>
> Add Silicon Kaiser norflash list:
> sk25lp128
>
> Cc: Sunil V L <suni...@ventanamicro.com>
> Cc: Li Yong <yong...@intel.com>
> Cc: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
> Cc: Leif Lindholm <quic_llind...@quicinc.com>
> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org>
> Cc: Abner Chang <abner.ch...@amd.com>
> Cc: Daniel Schaefer <g...@danielschaefer.me>
> Signed-off-by: John Chew <yuinyee.c...@starfivetech.com>
> ---
> EmbeddedPkg/Library/NorFlashInfoLib/NorFlashInfoLib.c | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/EmbeddedPkg/Library/NorFlashInfoLib/NorFlashInfoLib.c
> b/EmbeddedPkg/Library/NorFlashInfoLib/NorFlashInfoLib.c
> index e16c1c6a14..422fdac851 100644
> --- a/EmbeddedPkg/Library/NorFlashInfoLib/NorFlashInfoLib.c
> +++ b/EmbeddedPkg/Library/NorFlashInfoLib/NorFlashInfoLib.c
> @@ -1,6 +1,7 @@
> /** @file
> *
> * Copyright (c) 2017 Marvell International Ltd.
> +* Copyright (c) 2023 StarFive, Technology Co., Ltd. All rights
> +reserved.<BR>
> *
> * SPDX-License-Identifier: BSD-2-Clause-Patent
> *
> @@ -33,12 +34,20 @@ STATIC CONST NOR_FLASH_INFO NorFlashIds[] = {
> { L"en25q128b", { 0x1c, 0x30, 0x18 }, 3, 256, 64 * 1024, 256, 0
> },
> { L"en25s64", { 0x1c, 0x38, 0x17 }, 3, 256, 64 * 1024, 128, 0
> },
> /* GIGADEVICE */
> - { L"gd25q64b", { 0xc8, 0x40, 0x17 }, 3, 256, 64 * 1024, 128,
> NOR_FLASH_ERASE_4K },
> + { L"gd25q16", { 0xc8, 0x40, 0x15 }, 3, 256, 64 * 1024, 32,
> NOR_FLASH_ERASE_4K },
> + { L"gd25q32", { 0xc8, 0x40, 0x16 }, 3, 256, 64 * 1024, 64,
> NOR_FLASH_ERASE_4K },
> { L"gd25lq32", { 0xc8, 0x60, 0x16 }, 3, 256, 64 * 1024, 64,
> NOR_FLASH_ERASE_4K },
> + { L"gd25q64b", { 0xc8, 0x40, 0x17 }, 3, 256, 64 * 1024, 128,
> NOR_FLASH_ERASE_4K },
Linux calls this gd25q64 but probably we don't want to change the displayed
name.
> + { L"gd25lq64c", { 0xc8, 0x60, 0x17 }, 3, 256, 64 * 1024, 128,
> NOR_FLASH_ERASE_4K },
> + { L"gd25q128", { 0xc8, 0x40, 0x18 }, 3, 256, 64 * 1024, 256,
> NOR_FLASH_ERASE_4K },
> + { L"gd25lq128", { 0xc8, 0x60, 0x18 }, 3, 256, 64 * 1024, 256,
> NOR_FLASH_ERASE_4K },
Linux calls this gd25lq128d.
> + { L"gd25q256", { 0xc8, 0x40, 0x19 }, 3, 256, 64 * 1024, 512,
> NOR_FLASH_ERASE_4K },
Except for the two labels the changes match drivers/mtd/spi-nor/gigadevice.c in
Linux.
> /* ISSI */
> { L"is25lp032", { 0x9d, 0x60, 0x16 }, 3, 256, 64 * 1024, 64, 0
> },
> { L"is25lp064", { 0x9d, 0x60, 0x17 }, 3, 256, 64 * 1024, 128, 0
> },
> { L"is25lp128", { 0x9d, 0x60, 0x18 }, 3, 256, 64 * 1024, 256, 0
> },
> + /* XINKAI / SILICON KAISER */
> + { L"sk25lp128", { 0x27, 0x70, 0x18 }, 3, 256, 64 * 1024, 256,
> NOR_FLASH_ERASE_4K },
This matches
https://lore.kernel.org/all/20230726214401.929533-1-jo...@kwiboo.se/
Acked-by: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
> /* MACRONIX */
> { L"mx25l2006e", { 0xc2, 0x20, 0x12 }, 3, 256, 64 * 1024, 4, 0
> },
> { L"mx25l4005", { 0xc2, 0x20, 0x13 }, 3, 256, 64 * 1024, 8, 0
> },
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#109758): https://edk2.groups.io/g/devel/message/109758
Mute This Topic: https://groups.io/mt/101660590/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
