[PATCH v2 1/3] mtd: spi-nor: macronix: Drop the redundant flash info fields

[Maarten Zanders]

Hi all,

On Tue, Sep 30, 2025 at 2:19 PM Pratyush Yadav <pratyush at kernel.org> wrote:
> > I agree with reverting this patch. When I initially verified it, the
> > devices I had on hand all supported SFDP, so I did not catch this issue.
> > After checking again, I confirm that some older flashes without SFDP are
> > indeed affected.
>
> Do you know if these flashes are used in any devices that are actively
> used and maintained? If so, we should revert. If it is likely they
> aren't actively used, then maybe we just keep things as they are?
> Dunno...

The non-SFDP parts have been obsoleted in 2009-2010 according to
Macronix's PCN's. So they're pretty ancient.

If we choose to keep the patch in, I think we should make it more
consistent and drop support for the smaller flashes without SFDP as
well. The behavior is different in the spi-nor core for SFDP-parsed vs
non-SFDP-parsed cases.
In particular ID's 0xc22016 and 0xc22017 could be handled in the same
way I believe?

Best regards,
Maarten