From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mika Westerberg Subject: Re: [PATCH] spi: spidev: Add ACPI probing support Date: Mon, 4 Jul 2016 16:27:36 +0300 Message-ID: <20160704132736.GD23527@lahna.fi.intel.com> References: <1467628683-5783-1-git-send-email-mika.westerberg@linux.intel.com> <20160704125600.GB6247@sirena.org.uk> <20160704130806.GB23527@lahna.fi.intel.com> <20160704132048.GC6247@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-spi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Jarkko Nikula To: Mark Brown Return-path: Content-Disposition: inline In-Reply-To: <20160704132048.GC6247-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> Sender: linux-spi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On Mon, Jul 04, 2016 at 03:20:48PM +0200, Mark Brown wrote: > On Mon, Jul 04, 2016 at 04:08:06PM +0300, Mika Westerberg wrote: > > On Mon, Jul 04, 2016 at 02:56:00PM +0200, Mark Brown wrote: > > > > This is fine for now since all the ACPI devices are dummy ones but as > > > soon as someone comes up with an actual ACPI ID for something that > > > should be managed via spidev it'll need changing... > > > Sure. That's why the comment above also mentions SPT000* there. > > I'm trying to suggest that it'd be nice to write the code to check the > IDs and only warns when appropriate. :) Ah right, got it now. I'll update the patch then to check the IDs first. -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html