linux-kernel - Re: [PATCH v6 6/6] samples: rust: add ACPI match table example to platform driver

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: <aFvA9LVmStikbBF7@cassiopeiae>
Date: Wed, 25 Jun 2025 11:27:16 +0200
From: Danilo Krummrich <dakr@...nel.org>
To: "Rafael J. Wysocki" <rafael@...nel.org>
Cc: Igor Korotin <igor.korotin.linux@...il.com>, ojeda@...nel.org,
	alex.gaynor@...il.com, gregkh@...uxfoundation.org,
	linux-kernel@...r.kernel.org, rust-for-linux@...r.kernel.org,
	linux-acpi@...r.kernel.org, boqun.feng@...il.com, gary@...yguo.net,
	bjorn3_gh@...tonmail.com, benno.lossin@...ton.me,
	a.hindborg@...nel.org, aliceryhl@...gle.com, tmgross@...ch.edu,
	lenb@...nel.org, wedsonaf@...il.com, viresh.kumar@...aro.org,
	alex.hung@....com, dingxiangfei2009@...il.com
Subject: Re: [PATCH v6 6/6] samples: rust: add ACPI match table example to
 platform driver

On Wed, Jun 25, 2025 at 11:17:33AM +0200, Rafael J. Wysocki wrote:
> On Tue, Jun 17, 2025 at 7:15 PM Danilo Krummrich <dakr@...nel.org> wrote:
> >
> > On Tue, Jun 17, 2025 at 05:39:07PM +0100, Igor Korotin wrote:
> > > I could suggest on of the following:
> > > DRV0001
> > > DRVR0001
> > > PDRV0001
> >
> > This one looks reasonable to me.
> >
> > > TEST0001
> > > TST0001
> 
> Can we please avoid using made up device IDs in the code, even if it
> is just sample code?
> 
> It is way better to use a real one that's been reserved already as
> "never use in real ACPI tables" for some reason.  I think I can find a
> few of these for you if need be.
> 
> Thanks!

Sure! Please let me know which one you think fits best and I'll use it instead
of "TST0001" when I apply the series.

- Danilo