Re: [PATCH] spi: Force the registration of the spidev devices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 30, 2014 at 10:00:18PM +0200, Martin Sperl wrote:
> 
> On 30.04.2014, at 20:14, Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote:

> > I'd really be in favor of using the DT overlays for this. All this is
> > nice, but very fragile. You can't really handle any weird
> > configurations that you might encounter, like weird interrupt
> > controllers that might be requiring more informations, or you don't
> > set any interrupt parents, and this parameter list will basically be
> > an ever-growing list of parameters to deal with all kind of crazy
> > corner-cases, and I'm not sure it's something we want.

> Problem is getting DT overlays included in the kernel it seems...

We should fix that and solve this problem for all kinds of devices (and
some other similar use cases like the FPGAs).

> Following your argument we should not have this interface either
> but use DT for these GPIOs as well - similar argument for I2C...

> Still we have them and they are heavily used by those dev-boards,
> so why not something along the same lines also for SPI?

Those are legacy things, their presence doesn't mean they're a good
idea.  If there's problems with the userspace tooling for using overlays
when they get merged then that's the layer to fix things.

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux