Re: [PATCH] spi: spidev: Align ordering of spidev_spi_ids[] and spidev_dt_ids[]

[Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>]

Hi Uwe,

On Thu, Dec 19, 2024 at 9:02 AM Uwe Kleine-König
<u.kleine-koenig@xxxxxxxxxxxx> wrote:
> On Wed, Dec 18, 2024 at 08:10:57PM +0100, Geert Uytterhoeven wrote:
> > On Wed, Dec 18, 2024 at 8:02 PM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote:
> > > Further improvements could be:
> > >   - Generate spidev_spi_ids[] from spidev_dt_ids[] at runtime
> > >     during module_init() (consumes cycles :-(,
> > >   - Teach the subsystem matching code to strip the vendor prefix,
> > >     to get rid of spidev_spi_ids[].
> >
> > Oops, and modutils, as the tables are used by userspace :-(
>
> Then how about a build-time check comparing spi and of module info?

That's a good and non-disruptive idea, also for i2c!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds