Re: [PATCH net 1/4] auxiliary: Allow empty id
From: Sean Anderson <sean.anderson@linux.dev>
Date: 2025-06-20 15:37:46
Also in:
lkml, netdev
From: Sean Anderson <sean.anderson@linux.dev>
Date: 2025-06-20 15:37:46
Also in:
lkml, netdev
On 6/20/25 01:13, Greg Kroah-Hartman wrote:
On Thu, Jun 19, 2025 at 04:05:34PM -0400, Sean Anderson wrote:quoted
Support creating auxiliary devices with the id included as part of the name. This allows for non-decimal ids, which may be more appropriate for auxiliary devices created as children of memory-mapped devices. For example, a name like "xilinx_emac.mac.802c0000" could be achieved by setting .name to "mac.802c0000" and .id to AUXILIARY_DEVID_NONE.I don't see the justification for this, sorry. An id is just an id, it doesn't matter what is is and nothing should be relying on it to be the same across reboots or anywhere else. The only requirement is that it be unique at this point in time in the system.
It identifies the device in log messages. Without this you have to read sysfs to determine what device is (for example) producing an error. This may be inconvenient to do if the error prevents the system from booting. This series converts a platform device with a legible ID like "802c0000.ethernet" to an auxiliary device, and I believe descriptive device names produce a better developer experience. This is also shorter and simpler than auto-generated IDs. --Sean