Re: [PATCH RESEND] spi: spidev: Allow matching DT compatible strings from ACPI

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

 



On Wed, Jun 29, 2016 at 09:51:55PM +0300, Mika Westerberg wrote:
> On Wed, Jun 29, 2016 at 07:31:01PM +0100, Mark Brown wrote:

> > And we're *sure* that's going to be maintained?  People do use the
> > fallback matching that DT does, I don't trust the ACPI maintainers not
> > to do the same thing.

> That fallback matching does not even work in ACPI like I told you
> already. We have no plans to do anything like that either. That's the
> reason why we complain if there is PRP0001 without compatible string.

No, you're completely missing the point here.  The problem is someone
using something like linux,spidev as a compatible string and going into
fallback matching on SPI bus IDs rather than compatible strings.  I just
don't have confidence that someone isn't going to try to add that
fallback path given that it's used for DT.

> > Right, but rather than just define a translation from ACPI to DT and use
> > the DT code paths in their entirety for the relevant nodes what's
> > happening is that there's a shim layer between ACPI and DT which sort of
> > emulates bits of the DT interfaces but not quite and can lead to
> > surprises.

> Maybe but currently it works fine. And if problems are found they will
> be fixed as usual.

I'm worried that the reason it's working fine may be that there is very
little usage, and that there may be more gotchas like we're already
seeing lurking about.

Attachment: signature.asc
Description: PGP 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