Hello, On Wed, Apr 29, 2026 at 07:16:44PM +0200, Uwe Kleine-König (The Capable Hub) wrote: > ... and PCI device helpers. > > The struct pci_device_id array of supported device was initialized > by list expressions. This isn't easily readable if you're not into PCI. > > Use PCI_DEVICE* helper macros and named initializers which is more > explicit and thus easier to parse. Also skip explicit assignments of 0 > (which the compiler then takes care of). > > The secret plan is to make struct pci_device_id::driver_data an > anonymous union (similar to > https://lore.kernel.org/all/cover.1776579304.git.u.kleine-koenig@baylibre.com/) > and that requires named initializers. But it's also a nice cleanup on > its own. > > This change doesn't introduce changes to the compiled pci_device_id > array. Tested on x86 and arm64. > > Signed-off-by: Uwe Kleine-König (The Capable Hub) Gentil ping! I'd like to work on changing pci_device_id, but that requires that this patch hits mainline first ... Thanks for considering it Uwe