linux-kernel - Re: [PATCH 0/2] mtd: spi-nor: macronix: reading SFDP by all Macronix chips

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CANeKEMPfCk1R1pc8s9MwNniLJUKGr_g99U5EtNmJKsLLty-9Qw@mail.gmail.com>
Date: Thu, 26 Sep 2024 13:02:38 +0200
From: Erez <erezgeva2@...il.com>
To: Esben Haabendal <esben@...nix.com>
Cc: Erez Geva <erezgeva@...ime.org>, linux-mtd@...ts.infradead.org, 
	Tudor Ambarus <tudor.ambarus@...aro.org>, Pratyush Yadav <pratyush@...nel.org>, 
	Michael Walle <mwalle@...nel.org>, linux-kernel@...r.kernel.org, 
	Miquel Raynal <miquel.raynal@...tlin.com>, Richard Weinberger <richard@....at>, 
	Vignesh Raghavendra <vigneshr@...com>
Subject: Re: [PATCH 0/2] mtd: spi-nor: macronix: reading SFDP by all Macronix chips

On Thu, 26 Sept 2024 at 09:22, Esben Haabendal <esben@...nix.com> wrote:
>
> Erez Geva <erezgeva@...ime.org> writes:
>
> > From: Erez Geva <ErezGeva2@...il.com>
> >
> > Add reading SFDP to all Macronix chips.
> > All chips in the Macronix competabilites table
> > have a replacement chip that support SFDP.
> > Macronix stop producing any chip without SFDP 15 years ago.
> > The benefit of reading SFDP is by supporting
> > fater op-codes for newer chips.
> >
> > This patch do not add an all flashes support.
> > This patch extend the already support of SFDP to all Macronix chips.
> > The new chips that reuse JEDEC IDs and chips that
> > are absend from the Macronix competabilites table.
> > The patch just remove the restriction!
>
> Hi Erez
>
> Did you see the response I got to the last revision of my patch series
> that tries to resolve the same problem that you are adressing?
>
> https://lore.kernel.org/all/D2NGXHZ2VTK0.M0AOB4CM7MHM@kernel.org/

As for .
I agree that doing SFDP first is a better approach than mine.
If Tudor Ambarus. Pratyush Yadav. Michael Walle agrees on that.
Than let's do that

>
> I think we should try go go in that direction. It sounds like it would
> be a simple solution that would improve the overall situation, not just
> for Macronix.

Again I agree. Doing SFDP first seems a better approach.

Tudor expressed concern on sending RDSFDP as no-op to chips that do
not support SFDP.
I did a search on the Macronix chips that appear on the table and all
of them have a new chip or a new version that uses the same JEDEC ID,
that does support SFDP.
If other Vendors reused all their JEDEC IDs with new chips, then sure.
I did not check other Vendors, so I can not state the same on them.

Erez

>
>
> /Esben