[PATCH] Add support for more XMC series

[PATCH] Add support for more XMC series

[SSunk]

Add XMC XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C
site: https://www.xmcwh.com/site/product

Signed-off-by: Kankan Sun <ssunkkan@gmail.com>
---
 configs/evb-ast2600_defconfig | 1 +
 drivers/mtd/spi/spi-nor-ids.c | 4 ++++
 2 files changed, 5 insertions(+)

diff --git a/configs/evb-ast2600_defconfig b/configs/evb-ast2600_defconfig
index 9244654c82..f06c0e1fe1 100644
--- a/configs/evb-ast2600_defconfig
+++ b/configs/evb-ast2600_defconfig
@@ -100,6 +100,7 @@ CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_SST=y
 CONFIG_SPI_FLASH_WINBOND=y
+CONFIG_SPI_FLASH_XMC=y
 # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_PHY_REALTEK=y
 CONFIG_PHY_NCSI=y
diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c
index 4587215984..80d7678293 100644
--- a/drivers/mtd/spi/spi-nor-ids.c
+++ b/drivers/mtd/spi/spi-nor-ids.c
@@ -531,6 +531,10 @@ const struct flash_info spi_nor_ids[] = {
 	{ INFO("XM25QH64A", 0x207017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
 	{ INFO("XM25QH64C", 0x204017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
 	{ INFO("XM25QH128A", 0x207018, 0, 64 * 1024, 256, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
+	{ INFO("XM25QH256C", 0x204019, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
+	{ INFO("XM25QU256C", 0x204119, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
+	{ INFO("XM25QH512C", 0x204020, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
+	{ INFO("XM25QU512C", 0x204120, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
 #endif
 #ifdef CONFIG_SPI_FLASH_XTX
 	/* XTX Technology Limited */
-- 
2.34.1

RE: [PATCH] Add support for more XMC series

[ChiaWei Wang]

> From: SSunk <ssunkkan@gmail.com>
> Sent: Saturday, August 12, 2023 11:08 AM
> 
> Add XMC
> XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C
> site: https://www.xmcwh.com/site/product
> 
> Signed-off-by: Kankan Sun <ssunkkan@gmail.com>

Reviewed-by: Chia-Wei Wang <chiawei_wang@aspeedtech.com>

Re: [PATCH] Add support for more XMC series

[Jagan Teki]

On Sat, Aug 12, 2023 at 8:37 AM SSunk <ssunkkan@gmail.com> wrote:
>
> Add XMC XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C
> site: https://www.xmcwh.com/site/product
>
> Signed-off-by: Kankan Sun <ssunkkan@gmail.com>
> ---
>  configs/evb-ast2600_defconfig | 1 +
>  drivers/mtd/spi/spi-nor-ids.c | 4 ++++
>  2 files changed, 5 insertions(+)
>
> diff --git a/configs/evb-ast2600_defconfig b/configs/evb-ast2600_defconfig
> index 9244654c82..f06c0e1fe1 100644
> --- a/configs/evb-ast2600_defconfig
> +++ b/configs/evb-ast2600_defconfig
> @@ -100,6 +100,7 @@ CONFIG_SPI_FLASH_SPANSION=y
>  CONFIG_SPI_FLASH_STMICRO=y
>  CONFIG_SPI_FLASH_SST=y
>  CONFIG_SPI_FLASH_WINBOND=y
> +CONFIG_SPI_FLASH_XMC=y
>  # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
>  CONFIG_PHY_REALTEK=y
>  CONFIG_PHY_NCSI=y
> diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c
> index 4587215984..80d7678293 100644
> --- a/drivers/mtd/spi/spi-nor-ids.c
> +++ b/drivers/mtd/spi/spi-nor-ids.c
> @@ -531,6 +531,10 @@ const struct flash_info spi_nor_ids[] = {
>         { INFO("XM25QH64A", 0x207017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>         { INFO("XM25QH64C", 0x204017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>         { INFO("XM25QH128A", 0x207018, 0, 64 * 1024, 256, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> +       { INFO("XM25QH256C", 0x204019, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> +       { INFO("XM25QU256C", 0x204119, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> +       { INFO("XM25QH512C", 0x204020, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> +       { INFO("XM25QU512C", 0x204120, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },

Defconfig and ID patches must be separate. Send v2 for the same.

Thanks,
Jagan.

[PATCH] Add XMC XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C site: https://www.xmcwh.com/site/product

[Ssunk]

Signed-off-by: Kankan Sun <ssunkkan@gmail.com>
---
V1 -> V2: separate the ID and Defconfig 
 drivers/mtd/spi/spi-nor-ids.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c
index 8db522fca0..32329704d5 100644
--- a/drivers/mtd/spi/spi-nor-ids.c
+++ b/drivers/mtd/spi/spi-nor-ids.c
@@ -554,6 +554,10 @@ const struct flash_info spi_nor_ids[] = {
 	{ INFO("XM25QH64C", 0x204017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
 	{ INFO("XM25QH128A", 0x207018, 0, 64 * 1024, 256, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
 	{ INFO("XM25QU128C", 0x204118, 0, 64 * 1024, 256, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
+	{ INFO("XM25QH256C", 0x204019, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
+	{ INFO("XM25QU256C", 0x204119, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
+	{ INFO("XM25QH512C", 0x204020, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
+	{ INFO("XM25QU512C", 0x204120, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
 #endif
 #ifdef CONFIG_SPI_FLASH_XTX
 	/* XTX Technology Limited */
-- 
2.34.1

Re: [PATCH] Add XMC XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C site: https://www.xmcwh.com/site/product

[Jagan Teki]

On Tue, Jan 16, 2024 at 11:09 AM Ssunk <ssunkkan@gmail.com> wrote:
>
> Signed-off-by: Kankan Sun <ssunkkan@gmail.com>
> ---

Applied to u-boot-spi/master

[PATCH] Add support for more XMC series

[SSunk]

Add XMC XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C
site: https://www.xmcwh.com/site/product

Signed-off-by: Kankan Sun <ssunkkan@gmail.com>
---
 configs/evb-ast2600_defconfig | 1 +
 drivers/mtd/spi/spi-nor-ids.c | 4 ++++
 2 files changed, 5 insertions(+)

diff --git a/configs/evb-ast2600_defconfig b/configs/evb-ast2600_defconfig
index 9244654c82..f06c0e1fe1 100644
--- a/configs/evb-ast2600_defconfig
+++ b/configs/evb-ast2600_defconfig
@@ -100,6 +100,7 @@ CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_SST=y
 CONFIG_SPI_FLASH_WINBOND=y
+CONFIG_SPI_FLASH_XMC=y
 # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_PHY_REALTEK=y
 CONFIG_PHY_NCSI=y
diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c
index 4587215984..80d7678293 100644
--- a/drivers/mtd/spi/spi-nor-ids.c
+++ b/drivers/mtd/spi/spi-nor-ids.c
@@ -531,6 +531,10 @@ const struct flash_info spi_nor_ids[] = {
 	{ INFO("XM25QH64A", 0x207017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
 	{ INFO("XM25QH64C", 0x204017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
 	{ INFO("XM25QH128A", 0x207018, 0, 64 * 1024, 256, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
+	{ INFO("XM25QH256C", 0x204019, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
+	{ INFO("XM25QU256C", 0x204119, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
+	{ INFO("XM25QH512C", 0x204020, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
+	{ INFO("XM25QU512C", 0x204120, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
 #endif
 #ifdef CONFIG_SPI_FLASH_XTX
 	/* XTX Technology Limited */
-- 
2.34.1

Re: [PATCH] Add support for more XMC series

[Simon Glass]

On Fri, 11 Aug 2023 at 21:19, SSunk <ssunkkan@gmail.com> wrote:
>
> Add XMC XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C
> site: https://www.xmcwh.com/site/product
>
> Signed-off-by: Kankan Sun <ssunkkan@gmail.com>
> ---
>  configs/evb-ast2600_defconfig | 1 +
>  drivers/mtd/spi/spi-nor-ids.c | 4 ++++
>  2 files changed, 5 insertions(+)

Reviewed-by: Simon Glass <sjg@chromium.org>

I think this is v2 so it should have that as well as a change list.
You can use 'patman' to help with this.

Regards,
Simon



>
> diff --git a/configs/evb-ast2600_defconfig b/configs/evb-ast2600_defconfig
> index 9244654c82..f06c0e1fe1 100644
> --- a/configs/evb-ast2600_defconfig
> +++ b/configs/evb-ast2600_defconfig
> @@ -100,6 +100,7 @@ CONFIG_SPI_FLASH_SPANSION=y
>  CONFIG_SPI_FLASH_STMICRO=y
>  CONFIG_SPI_FLASH_SST=y
>  CONFIG_SPI_FLASH_WINBOND=y
> +CONFIG_SPI_FLASH_XMC=y
>  # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
>  CONFIG_PHY_REALTEK=y
>  CONFIG_PHY_NCSI=y
> diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c
> index 4587215984..80d7678293 100644
> --- a/drivers/mtd/spi/spi-nor-ids.c
> +++ b/drivers/mtd/spi/spi-nor-ids.c
> @@ -531,6 +531,10 @@ const struct flash_info spi_nor_ids[] = {
>         { INFO("XM25QH64A", 0x207017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>         { INFO("XM25QH64C", 0x204017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>         { INFO("XM25QH128A", 0x207018, 0, 64 * 1024, 256, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> +       { INFO("XM25QH256C", 0x204019, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> +       { INFO("XM25QU256C", 0x204119, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> +       { INFO("XM25QH512C", 0x204020, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> +       { INFO("XM25QU512C", 0x204120, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
>  #endif
>  #ifdef CONFIG_SPI_FLASH_XTX
>         /* XTX Technology Limited */
> --
> 2.34.1
>

Re: [PATCH] Add support for more XMC series

[Clus Tom]

Hi Simon,
I'm not quite sure what you mean by v2, if it's the previous email, it only
removes the XM25QH128C part of the commit message compared to the previous
one.

Thanks,
SSunk

Simon Glass <sjg@google.com> 于2023年8月13日周日 21:36写道：

> On Fri, 11 Aug 2023 at 21:19, SSunk <ssunkkan@gmail.com> wrote:
> >
> > Add XMC XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C
> > site: https://www.xmcwh.com/site/product
> >
> > Signed-off-by: Kankan Sun <ssunkkan@gmail.com>
> > ---
> >  configs/evb-ast2600_defconfig | 1 +
> >  drivers/mtd/spi/spi-nor-ids.c | 4 ++++
> >  2 files changed, 5 insertions(+)
>
> Reviewed-by: Simon Glass <sjg@chromium.org>
>
> I think this is v2 so it should have that as well as a change list.
> You can use 'patman' to help with this.
>
> Regards,
> Simon
>
>
>
> >
> > diff --git a/configs/evb-ast2600_defconfig
> b/configs/evb-ast2600_defconfig
> > index 9244654c82..f06c0e1fe1 100644
> > --- a/configs/evb-ast2600_defconfig
> > +++ b/configs/evb-ast2600_defconfig
> > @@ -100,6 +100,7 @@ CONFIG_SPI_FLASH_SPANSION=y
> >  CONFIG_SPI_FLASH_STMICRO=y
> >  CONFIG_SPI_FLASH_SST=y
> >  CONFIG_SPI_FLASH_WINBOND=y
> > +CONFIG_SPI_FLASH_XMC=y
> >  # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
> >  CONFIG_PHY_REALTEK=y
> >  CONFIG_PHY_NCSI=y
> > diff --git a/drivers/mtd/spi/spi-nor-ids.c
> b/drivers/mtd/spi/spi-nor-ids.c
> > index 4587215984..80d7678293 100644
> > --- a/drivers/mtd/spi/spi-nor-ids.c
> > +++ b/drivers/mtd/spi/spi-nor-ids.c
> > @@ -531,6 +531,10 @@ const struct flash_info spi_nor_ids[] = {
> >         { INFO("XM25QH64A", 0x207017, 0, 64 * 1024, 128, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> >         { INFO("XM25QH64C", 0x204017, 0, 64 * 1024, 128, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> >         { INFO("XM25QH128A", 0x207018, 0, 64 * 1024, 256, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> > +       { INFO("XM25QH256C", 0x204019, 0, 64 * 1024, 512, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> > +       { INFO("XM25QU256C", 0x204119, 0, 64 * 1024, 512, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> > +       { INFO("XM25QH512C", 0x204020, 0, 64 * 1024, 1024, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> > +       { INFO("XM25QU512C", 0x204120, 0, 64 * 1024, 1024, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> >  #endif
> >  #ifdef CONFIG_SPI_FLASH_XTX
> >         /* XTX Technology Limited */
> > --
> > 2.34.1
> >
>

Re: [PATCH] Add support for more XMC series

[Simon Glass]

Hi,

On Sun, 13 Aug 2023 at 21:19, Clus Tom <ssunkkan@gmail.com> wrote:
>
> Hi Simon,
> I'm not quite sure what you mean by v2, if it's the previous email, it only removes the XM25QH128C part of the commit message compared to the previous one.
>

Right, so it is v2.

Your emails seem to have generated 3 patches:

https://patchwork.ozlabs.org/project/uboot/patch/20230811082000.4277-1-ssunkkan@gmail.com/

https://patchwork.ozlabs.org/project/uboot/patch/20230812031846.3958-1-ssunkkan@gmail.com/

https://patchwork.ozlabs.org/project/uboot/patch/20230812030731.3711-1-ssunkkan@gmail.com/

The version number helps the maintainer figure out which one to apply.

Regards,
Simon



- Simon


> Thanks,
> SSunk
>
> Simon Glass <sjg@google.com> 于2023年8月13日周日 21:36写道：
>>
>> On Fri, 11 Aug 2023 at 21:19, SSunk <ssunkkan@gmail.com> wrote:
>> >
>> > Add XMC XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C
>> > site: https://www.xmcwh.com/site/product
>> >
>> > Signed-off-by: Kankan Sun <ssunkkan@gmail.com>
>> > ---
>> >  configs/evb-ast2600_defconfig | 1 +
>> >  drivers/mtd/spi/spi-nor-ids.c | 4 ++++
>> >  2 files changed, 5 insertions(+)
>>
>> Reviewed-by: Simon Glass <sjg@chromium.org>
>>
>> I think this is v2 so it should have that as well as a change list.
>> You can use 'patman' to help with this.
>>
>> Regards,
>> Simon
>>
>>
>>
>> >
>> > diff --git a/configs/evb-ast2600_defconfig b/configs/evb-ast2600_defconfig
>> > index 9244654c82..f06c0e1fe1 100644
>> > --- a/configs/evb-ast2600_defconfig
>> > +++ b/configs/evb-ast2600_defconfig
>> > @@ -100,6 +100,7 @@ CONFIG_SPI_FLASH_SPANSION=y
>> >  CONFIG_SPI_FLASH_STMICRO=y
>> >  CONFIG_SPI_FLASH_SST=y
>> >  CONFIG_SPI_FLASH_WINBOND=y
>> > +CONFIG_SPI_FLASH_XMC=y
>> >  # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
>> >  CONFIG_PHY_REALTEK=y
>> >  CONFIG_PHY_NCSI=y
>> > diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c
>> > index 4587215984..80d7678293 100644
>> > --- a/drivers/mtd/spi/spi-nor-ids.c
>> > +++ b/drivers/mtd/spi/spi-nor-ids.c
>> > @@ -531,6 +531,10 @@ const struct flash_info spi_nor_ids[] = {
>> >         { INFO("XM25QH64A", 0x207017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>> >         { INFO("XM25QH64C", 0x204017, 0, 64 * 1024, 128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>> >         { INFO("XM25QH128A", 0x207018, 0, 64 * 1024, 256, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>> > +       { INFO("XM25QH256C", 0x204019, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
>> > +       { INFO("XM25QU256C", 0x204119, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
>> > +       { INFO("XM25QH512C", 0x204020, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
>> > +       { INFO("XM25QU512C", 0x204120, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
>> >  #endif
>> >  #ifdef CONFIG_SPI_FLASH_XTX
>> >         /* XTX Technology Limited */
>> > --
>> > 2.34.1
>> >

Re: [PATCH] Add support for more XMC series

[Clus Tom]

Hi Simon,
I just want to apply:
https://patchwork.ozlabs.org/project/uboot/patch/20230812031846.3958-1-ssunkkan@gmail.com

Thanks,
SSunk

Simon Glass <sjg@google.com> 于2023年8月15日周二 06:43写道：

> Hi,
>
> On Sun, 13 Aug 2023 at 21:19, Clus Tom <ssunkkan@gmail.com> wrote:
> >
> > Hi Simon,
> > I'm not quite sure what you mean by v2, if it's the previous email, it
> only removes the XM25QH128C part of the commit message compared to the
> previous one.
> >
>
> Right, so it is v2.
>
> Your emails seem to have generated 3 patches:
>
>
> https://patchwork.ozlabs.org/project/uboot/patch/20230811082000.4277-1-ssunkkan@gmail.com/
>
>
> https://patchwork.ozlabs.org/project/uboot/patch/20230812031846.3958-1-ssunkkan@gmail.com/
>
>
> https://patchwork.ozlabs.org/project/uboot/patch/20230812030731.3711-1-ssunkkan@gmail.com/
>
> The version number helps the maintainer figure out which one to apply.
>
> Regards,
> Simon
>
>
>
> - Simon
>
>
> > Thanks,
> > SSunk
> >
> > Simon Glass <sjg@google.com> 于2023年8月13日周日 21:36写道：
> >>
> >> On Fri, 11 Aug 2023 at 21:19, SSunk <ssunkkan@gmail.com> wrote:
> >> >
> >> > Add XMC XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C
> >> > site: https://www.xmcwh.com/site/product
> >> >
> >> > Signed-off-by: Kankan Sun <ssunkkan@gmail.com>
> >> > ---
> >> >  configs/evb-ast2600_defconfig | 1 +
> >> >  drivers/mtd/spi/spi-nor-ids.c | 4 ++++
> >> >  2 files changed, 5 insertions(+)
> >>
> >> Reviewed-by: Simon Glass <sjg@chromium.org>
> >>
> >> I think this is v2 so it should have that as well as a change list.
> >> You can use 'patman' to help with this.
> >>
> >> Regards,
> >> Simon
> >>
> >>
> >>
> >> >
> >> > diff --git a/configs/evb-ast2600_defconfig
> b/configs/evb-ast2600_defconfig
> >> > index 9244654c82..f06c0e1fe1 100644
> >> > --- a/configs/evb-ast2600_defconfig
> >> > +++ b/configs/evb-ast2600_defconfig
> >> > @@ -100,6 +100,7 @@ CONFIG_SPI_FLASH_SPANSION=y
> >> >  CONFIG_SPI_FLASH_STMICRO=y
> >> >  CONFIG_SPI_FLASH_SST=y
> >> >  CONFIG_SPI_FLASH_WINBOND=y
> >> > +CONFIG_SPI_FLASH_XMC=y
> >> >  # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
> >> >  CONFIG_PHY_REALTEK=y
> >> >  CONFIG_PHY_NCSI=y
> >> > diff --git a/drivers/mtd/spi/spi-nor-ids.c
> b/drivers/mtd/spi/spi-nor-ids.c
> >> > index 4587215984..80d7678293 100644
> >> > --- a/drivers/mtd/spi/spi-nor-ids.c
> >> > +++ b/drivers/mtd/spi/spi-nor-ids.c
> >> > @@ -531,6 +531,10 @@ const struct flash_info spi_nor_ids[] = {
> >> >         { INFO("XM25QH64A", 0x207017, 0, 64 * 1024, 128, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> >> >         { INFO("XM25QH64C", 0x204017, 0, 64 * 1024, 128, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> >> >         { INFO("XM25QH128A", 0x207018, 0, 64 * 1024, 256, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> >> > +       { INFO("XM25QH256C", 0x204019, 0, 64 * 1024, 512, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> >> > +       { INFO("XM25QU256C", 0x204119, 0, 64 * 1024, 512, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> >> > +       { INFO("XM25QH512C", 0x204020, 0, 64 * 1024, 1024, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> >> > +       { INFO("XM25QU512C", 0x204120, 0, 64 * 1024, 1024, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> >> >  #endif
> >> >  #ifdef CONFIG_SPI_FLASH_XTX
> >> >         /* XTX Technology Limited */
> >> > --
> >> > 2.34.1
> >> >
>

Re: [PATCH] Add support for more XMC series

[SSunk]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=y, Size: 2085 bytes --]

>Hi Michael,
>Since this is the first time I've submitted code to the Linux community, please forgive me for some of the missteps that I've made.
>I uploaded the incorrect patch file.
>In a previous e-mail you said：
>XMC parts seem to have SFDP tables and they should work out of the box
>without any patches with the generic spi nor driver [1]. Therefore,
>you don't need any entry at all.
>
>I think it is related to the PARSE_SFDP flag.
>The XMC flash part supports SFDP, so i don’t need to add NO_SFDP_FLAGS() macro.
> drivers/mtd/spi-nor/xmc.c | 17 +++++++++++++----
> 1 file changed, 13 insertions(+), 4 deletions(-)
>
>diff --git a/drivers/mtd/spi-nor/xmc.c b/drivers/mtd/spi-nor/xmc.c
>index 051411e86339..e78bf11792d9 100644
>--- a/drivers/mtd/spi-nor/xmc.c
>+++ b/drivers/mtd/spi-nor/xmc.c
>@@ -11,11 +11,20 @@
> static const struct flash_info xmc_nor_parts[] = {
> 	/* XMC (Wuhan Xinxin Semiconductor Manufacturing Corp.) */
> 	{ "XM25QH64A", INFO(0x207017, 0, 64 * 1024, 128)
>-		NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ |
>-			      SPI_NOR_QUAD_READ) },
>+		PARSE_SFDP },
> 	{ "XM25QH128A", INFO(0x207018, 0, 64 * 1024, 256)
>-		NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ |
>-			      SPI_NOR_QUAD_READ) },
>+		PARSE_SFDP },
>+	{ "XM25QH128C", INFO(0x204018, 0, 64 * 1024, 256)
>+		PARSE_SFDP },
>+	{ "XM25QH256C", INFO(0x204019, 0, 64 * 1024, 512)
>+		PARSE_SFDP },
>+	{ "XM25QU256C", INFO(0x204119, 0, 64 * 1024, 512)
>+		PARSE_SFDP },
>+	{ "XM25QH512C", INFO(0x204020, 0, 64 * 1024, 1024)
>+		PARSE_SFDP },
>+	{ "XM25QU512C", INFO(0x204120, 0, 64 * 1024, 1024)
>+		PARSE_SFDP },
>+
> };
> 
> const struct spi_nor_manufacturer spi_nor_xmc = {
>> Correct. If the flash is working with PARSE_SFDP, it has SFDP and
>> all the information of the flash_info table is pulled from the SFDP
>> tables, except for the part name. Therefore, you don't need any entry
>> at all. The flash should just work out of the box. Are you using the
>> latest kernel?
>> 
>> -michael
--
Hi Michael,

According to you, does this piece of my code still need to be modified?

Thanks,
Ssunk

Re: [PATCH] Add support for more XMC series

[SSunk]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=y, Size: 1075 bytes --]

>Hi Simon,
>I just want to apply:
>https://patchwork.ozlabs.org/project/uboot/patch/20230812031846.3958-1-ssunkkan@gmail.com
>
>Thanks,
>SSunk
>
>Simon Glass <sjg@google.com> 于2023年8月15日周二 06:43写道：
>
>> Hi,
>>
>> On Sun, 13 Aug 2023 at 21:19, Clus Tom <ssunkkan@gmail.com> wrote:
>>>
>>> Hi Simon,
>>> I'm not quite sure what you mean by v2, if it's the previous email, it
>> only removes the XM25QH128C part of the commit message compared to the
>> previous one.
>>>
>>
>> Right, so it is v2.
>>
>> Your emails seem to have generated 3 patches:
>>
>>
>> https://patchwork.ozlabs.org/project/uboot/patch/20230811082000.4277-1-ssunkkan@gmail.com/
>>
>>
>> https://patchwork.ozlabs.org/project/uboot/patch/20230812031846.3958-1-ssunkkan@gmail.com/
>>
>>
>> https://patchwork.ozlabs.org/project/uboot/patch/20230812030731.3711-1-ssunkkan@gmail.com/
>>
>> The version number helps the maintainer figure out which one to apply.
>>
>> Regards,
>> Simon
>>
>>
>>
>> - Simon
>>
>>
--
Hi Simon,

When will this patch be merged into the master branch?

Thanks,
SSunk