linux-kernel - Re: [PATCH v2] mtd: core: Add nand_id sysfs attribute for NAND devices

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: <87sefaqf9s.fsf@bootlin.com>
Date: Thu, 23 Oct 2025 08:54:39 +0200
From: Miquel Raynal <miquel.raynal@...tlin.com>
To: Jay Xu <jayxu1990@...il.com>
Cc: Richard Weinberger <richard@....at>,  Vignesh Raghavendra
 <vigneshr@...com>,  linux-mtd@...ts.infradead.org,
  linux-kernel@...r.kernel.org,  avnerkhan@...xas.edu,
  rdlee.upstream@...il.com,  kernel test robot <lkp@...el.com>
Subject: Re: [PATCH v2] mtd: core: Add nand_id sysfs attribute for NAND devices

Hi Jay,

On 22/10/2025 at 11:53:41 -05, Jay Xu <jayxu1990@...il.com> wrote:

> Hi Miquèl:
> Thank you for the feedback and suggestions!
>
> You're right that the current implementation is raw NAND specific. I can look into extending this to support other NAND
> types by moving the functionality to the NAND core 
> layer and ensure the ID is populated at probe time.
>
> Let me know if you have any other suggestions/questions,

Not at the moment, I'd like to gather feedback from other MTD folks
before we go forward. Sysfs is part of the stable API, we must be
careful.

>  > Reported-by: kernel test robot <lkp@...el.com>
>  > Closes:
>  > https://lore.kernel.org/oe-kbuild-all/202510120356.STGKDkA5-lkp@intel.com/

Just as an FYI, these do not make any sense in a contribution like
yours. Include these tags if you are fixing something that is already in
the tree. This is a new submission and it is not a fix so "reported-by"
shall not be used.

Thanks,
Miquèl