Hi Anshul,

Just a gentle follow-up on this v2 patch, which is still marked as
"New" in Patchwork.

The v2 addresses the previous review comments by adding the
Signed-off-by tag and providing the XMC Product Change Notification
(PCN) as evidence of the replacement of XM25QH512C with XM25QH512D.

Could you please let me know if any further changes or information are needed?

Thanks for your time and review.

Best regards,
Ssunk

On Wed,  6 May 2026 18:30:53 +0800, Ssunk <[email protected]> wrote:
> The XM25QH512C has been officially replaced by the XM25QH512D according
> to the XMC Product Change Notification (PCN). Both parts share the same
> JEDEC ID (0x204020) and have identical parameters.
>
> Update the flash name string to reflect the current active part in
> production.
>
> Reference PCN document for the replacement:
> https://drive.google.com/file/d/1e-FfQFAD9qqlOO3iWe6PXY7EeGyN1e-l/view?usp=sharing
>
> Signed-off-by: Ssunk <[email protected]>
> ---
> Changes in v2:
> - Added Signed-off-by tag.
> - Added PCN document link in the commit message as proof of the part 
> replacement to address reviewer's concern.
> - Updated commit message to explicitly mention the PCN.
> ---
> drivers/mtd/spi/spi-nor-ids.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c
> index c0fa98424aa..3b8e309e0f2 100644
> --- a/drivers/mtd/spi/spi-nor-ids.c
> +++ b/drivers/mtd/spi/spi-nor-ids.c
> @@ -672,7 +672,7 @@ const struct flash_info spi_nor_ids[] = {
> { 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("XM25QH512D", 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) },
> { INFO("XM25QH01D", 0x204021, 0, 64 * 1024, 2048, SECT_4K | SPI_NOR_DUAL_READ 
> | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> #endif
> --
> 2.53.0.windows.2

Reply via email to