All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pratyush Yadav <pratyush@kernel.org>
To: "Kang.Kerry 康宇良 TAO" <Kang.Kerry@inventec.com>
Cc: Pratyush Yadav <pratyush@kernel.org>,
	 "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>
Subject: Re: mtd: spi-nor: Add Winbond w25q512jvdtr
Date: Tue, 16 Sep 2025 12:13:42 +0200	[thread overview]
Message-ID: <mafs0jz1yiu21.fsf@kernel.org> (raw)
In-Reply-To: <4645d0cbce50491c9a428b57d97e09e0@inventec.com>

Hi,

On Tue, Sep 16 2025, Kang.Kerry 康宇良 TAO wrote:

>>> I configured the DTS with 'compatible = "jedec,spi-nor";'
>> and enabled the kernel options:
>> CONFIG_MTD_PARTITIONED_MASTER=y
>> CONFIG_MTD_SPI_NOR=y
>>
>>> However, the dmesg output shows that the kernel is failing to recognize the flash's JEDEC ID. This suggests that the generic SFDP-based driver isn't working as expected.
>>>
>>> [  135.294624] spi-nor spi1.0: unrecognized JEDEC id bytes: ef 70 20 
>>> 00 00 00 [  135.301602] spi-nor: probe of spi1.0 failed with error -2
>>>
>>> I think we need to add this to table to make sure W25Q512JV-DTR is recognized.
>>
>>This means spi_nor_check_sfdp_signature() fails. Can you check why? The flash datasheet says it has a SFDP table so this should work.
>>
>>BTW, which kernel version are you testing this on?
>>
>>[...]
>>
>>--
>>Regards,
>>Pratyush Yadav
>
> I used kernel 5.15 for testing.
> It seems the kernel is failing to recognize JEDEC id of the Winbond flash, which is causing the error.
>
> When I test with Nuvoton kernel 6.6, it is able to recognize the flash as a generic device.
> [   50.804160] spi-nor spi1.0: spi-nor-generic (65536 Kbytes)

We only accept patches for the latest kernel. Since the latest seems to
be working for you, I think the onus is on you to backport the right set
of patches to your 5.15 kernel.

-- 
Regards,
Pratyush Yadav

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

      reply	other threads:[~2025-09-16 10:13 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-10  0:20 mtd: spi-nor: Add Winbond w25q512jvdtr Kang.Kerry 康宇良 TAO
2025-09-10 15:47 ` Pratyush Yadav
2025-09-15  9:29   ` Kang.Kerry 康宇良 TAO
2025-09-15 12:14     ` Pratyush Yadav
2025-09-16  8:14       ` Kang.Kerry 康宇良 TAO
2025-09-16 10:13         ` Pratyush Yadav [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=mafs0jz1yiu21.fsf@kernel.org \
    --to=pratyush@kernel.org \
    --cc=Kang.Kerry@inventec.com \
    --cc=linux-mtd@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.