Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755481AbdHYRJw (ORCPT ); Fri, 25 Aug 2017 13:09:52 -0400 Received: from dougal.metanate.com ([90.155.101.14]:1862 "EHLO metanate.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755368AbdHYRJv (ORCPT ); Fri, 25 Aug 2017 13:09:51 -0400 Date: Fri, 25 Aug 2017 18:09:13 +0100 From: John Keeping To: Andy Shevchenko Cc: Tom Rini , linux-kernel@vger.kernel.org, Bard Liao , Oder Chiou , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , alsa-devel@alsa-project.org, Mark Brown , Linus Torvalds Subject: Re: [PATCH] ASoC: rt5677: Reintroduce I2C device IDs Message-ID: <20170825180913.4f95ce5d.john@metanate.com> In-Reply-To: <1503679371.25945.110.camel@linux.intel.com> References: <1503453106-5564-1-git-send-email-trini@konsulko.com> <1503669407.25945.102.camel@linux.intel.com> <20170825142426.GN2827@bill-the-cat> <20170825170504.7d547aa4.john@metanate.com> <1503679371.25945.110.camel@linux.intel.com> Organization: Metanate Ltd X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id v7PHA8RE011056 Content-Length: 2048 Lines: 57 On Fri, 25 Aug 2017 19:42:51 +0300, Andy Shevchenko wrote: > On Fri, 2017-08-25 at 17:05 +0100, John Keeping wrote: > > On Fri, 25 Aug 2017 10:24:26 -0400, Tom Rini wrote: > > > On Fri, Aug 25, 2017 at 04:56:47PM +0300, Andy Shevchenko wrote: > > > > > Apparently you are the one who tested the commit > > > > 89128534f925 ("ASoC: rt5677: Add ACPI support") > > > > year ago.   > > > > > > Yes. > > > > > > > The commit states that ACPI properties that are used in Chromebook > > > > Pixel > > > > 2015 is non-standard (not the same as for DT). > > > > > > > > However, DSDT shows the opposite!   > > > > > > Interesting.  I'm not an ACPI person, I just tested what John came > > > up > > > with. > > > > And the patch adding this was the first (and still only) time I've > > really looked at ACPI, so it's quite possible that I misunderstood > > something at the time. > > Maybe. > > > From memory, I think the particular problem I was referring to in the > > commit message was that certain GPIOs were only defined by index and > > not > > by property name (specifically "plug-det-gpios", "mic-present-gpios" > > and > > "headphone-enable-gpios"), and having dumped DSDT just now I do not > > see > > those strings appearing anywhere. > > Exactly, and this part of the patch I'm _not_ talking about (it's pretty > much good and working). > > What I'm talking about is a specific function called > > rt5677_read_acpi_properties() > > in the rt5677.c codec driver. Right, I don't see any reason why it shouldn't be possible to replace that with rt5677_read_device_properties() given the DSDT I have. I expect that exists because I was using the chromeos-3.14 tree as a reference (which was the supported kernel on this hardware at the time), but it looks like the unified device property API was added in 3.18 so of course was not used there, and I did not realise that the device property versions could be used here. I'll try to find time to test this change over the weekend, if Tom doesn't beat me to it!