* [PATCH] Add support for more XMC series
@ 2023-08-12 3:07 SSunk
2023-08-14 0:04 ` ChiaWei Wang
2023-12-06 12:33 ` Jagan Teki
0 siblings, 2 replies; 12+ messages in thread
From: SSunk @ 2023-08-12 3:07 UTC (permalink / raw)
To: chiawei_wang, jagan, vigneshr; +Cc: u-boot, 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
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH] Add support for more XMC series
@ 2023-08-12 3:18 SSunk
2023-08-13 13:36 ` Simon Glass
0 siblings, 1 reply; 12+ messages in thread
From: SSunk @ 2023-08-12 3:18 UTC (permalink / raw)
To: chiawei_wang, jagan, vigneshr; +Cc: u-boot, 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
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH] Add support for more XMC series
2023-08-12 3:18 [PATCH] Add support for more XMC series SSunk
@ 2023-08-13 13:36 ` Simon Glass
2023-08-14 3:18 ` Clus Tom
0 siblings, 1 reply; 12+ messages in thread
From: Simon Glass @ 2023-08-13 13:36 UTC (permalink / raw)
To: SSunk; +Cc: chiawei_wang, jagan, vigneshr, u-boot
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
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH] Add support for more XMC series
2023-08-12 3:07 [PATCH] Add support for more XMC series SSunk
@ 2023-08-14 0:04 ` ChiaWei Wang
2023-12-06 12:33 ` Jagan Teki
1 sibling, 0 replies; 12+ messages in thread
From: ChiaWei Wang @ 2023-08-14 0:04 UTC (permalink / raw)
To: SSunk, jagan@amarulasolutions.com, vigneshr@ti.com; +Cc: u-boot@lists.denx.de
> 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>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] Add support for more XMC series
2023-08-13 13:36 ` Simon Glass
@ 2023-08-14 3:18 ` Clus Tom
2023-08-14 22:42 ` Simon Glass
0 siblings, 1 reply; 12+ messages in thread
From: Clus Tom @ 2023-08-14 3:18 UTC (permalink / raw)
To: Simon Glass; +Cc: chiawei_wang, jagan, vigneshr, u-boot
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
> >
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] Add support for more XMC series
2023-08-14 3:18 ` Clus Tom
@ 2023-08-14 22:42 ` Simon Glass
2023-08-15 2:46 ` Clus Tom
0 siblings, 1 reply; 12+ messages in thread
From: Simon Glass @ 2023-08-14 22:42 UTC (permalink / raw)
To: Clus Tom; +Cc: chiawei_wang, jagan, vigneshr, u-boot
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
>> >
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] Add support for more XMC series
2023-08-14 22:42 ` Simon Glass
@ 2023-08-15 2:46 ` Clus Tom
2023-08-31 4:07 ` SSunk
2023-08-31 5:48 ` SSunk
0 siblings, 2 replies; 12+ messages in thread
From: Clus Tom @ 2023-08-15 2:46 UTC (permalink / raw)
To: Simon Glass; +Cc: chiawei_wang, jagan, vigneshr, u-boot
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
> >> >
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] Add support for more XMC series
2023-08-15 2:46 ` Clus Tom
@ 2023-08-31 4:07 ` SSunk
2023-08-31 5:48 ` SSunk
1 sibling, 0 replies; 12+ messages in thread
From: SSunk @ 2023-08-31 4:07 UTC (permalink / raw)
To: ssunkkan; +Cc: chiawei_wang, jagan, sjg, u-boot, vigneshr
[-- 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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] Add support for more XMC series
2023-08-15 2:46 ` Clus Tom
2023-08-31 4:07 ` SSunk
@ 2023-08-31 5:48 ` SSunk
1 sibling, 0 replies; 12+ messages in thread
From: SSunk @ 2023-08-31 5:48 UTC (permalink / raw)
To: ssunkkan; +Cc: chiawei_wang, jagan, sjg, u-boot, vigneshr
[-- 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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] Add support for more XMC series
2023-08-12 3:07 [PATCH] Add support for more XMC series SSunk
2023-08-14 0:04 ` ChiaWei Wang
@ 2023-12-06 12:33 ` Jagan Teki
2024-01-16 5:38 ` [PATCH] Add XMC XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C site: https://www.xmcwh.com/site/product Ssunk
1 sibling, 1 reply; 12+ messages in thread
From: Jagan Teki @ 2023-12-06 12:33 UTC (permalink / raw)
To: SSunk; +Cc: chiawei_wang, vigneshr, u-boot
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.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] Add XMC XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C site: https://www.xmcwh.com/site/product
2023-12-06 12:33 ` Jagan Teki
@ 2024-01-16 5:38 ` Ssunk
2024-01-29 12:07 ` Jagan Teki
0 siblings, 1 reply; 12+ messages in thread
From: Ssunk @ 2024-01-16 5:38 UTC (permalink / raw)
To: jagan; +Cc: chiawei_wang, ssunkkan, u-boot, vigneshr
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
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH] Add XMC XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C site: https://www.xmcwh.com/site/product
2024-01-16 5:38 ` [PATCH] Add XMC XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C site: https://www.xmcwh.com/site/product Ssunk
@ 2024-01-29 12:07 ` Jagan Teki
0 siblings, 0 replies; 12+ messages in thread
From: Jagan Teki @ 2024-01-29 12:07 UTC (permalink / raw)
To: Ssunk; +Cc: chiawei_wang, u-boot, vigneshr
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
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2024-01-29 12:07 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-12 3:07 [PATCH] Add support for more XMC series SSunk
2023-08-14 0:04 ` ChiaWei Wang
2023-12-06 12:33 ` Jagan Teki
2024-01-16 5:38 ` [PATCH] Add XMC XM25QH128C/XM25QH256C/XM25QU256C/XM25QH512C/XM25QU512C site: https://www.xmcwh.com/site/product Ssunk
2024-01-29 12:07 ` Jagan Teki
-- strict thread matches above, loose matches on Subject: below --
2023-08-12 3:18 [PATCH] Add support for more XMC series SSunk
2023-08-13 13:36 ` Simon Glass
2023-08-14 3:18 ` Clus Tom
2023-08-14 22:42 ` Simon Glass
2023-08-15 2:46 ` Clus Tom
2023-08-31 4:07 ` SSunk
2023-08-31 5:48 ` SSunk
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox