linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Lee Jones <lee.jones@linaro.org>
Cc: linux-acpi@vger.kernel.org, linux-i2c@vger.kernel.org,
	linux-gpio@vger.kernel.org, Wolfram Sang <wsa@the-dreams.de>,
	Linus Walleij <linus.walleij@linaro.org>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Mika Westerberg <mika.westerberg@intel.com>,
	"Puustinen, Ismo" <ismo.puustinen@intel.com>,
	"Pandruvada, Srinivas" <srinivas.pandruvada@intel.com>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 2/8] mfd: core: redo ACPI matching of the children devices
Date: Wed, 23 Sep 2015 14:06:59 +0300	[thread overview]
Message-ID: <1443006419.8361.169.camel@linux.intel.com> (raw)
In-Reply-To: <20150922221515.GB9317@x1>

On Tue, 2015-09-22 at 23:15 +0100, Lee Jones wrote:
> On Tue, 22 Sep 2015, Andy Shevchenko wrote:
> 
> > There is at least one board on the market, i.e. Intel Galileo Gen2, 
> > that uses
> > _ADR to distinguish the devices under one actual device. Due to 
> > this we have to
> > improve the quirk in the MFD core to handle that board.
> 
> This will require an ACPI Ack.

Rafael is Cc'ed, so, I hope he will comment on this.

> 
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > ---
> >  Documentation/acpi/enumeration.txt | 11 +++++---
> >  drivers/mfd/mfd-core.c             | 52 ++++++++++++++++++++++++++
> > ------------
> >  include/linux/mfd/core.h           | 10 ++++++--
> >  3 files changed, 52 insertions(+), 21 deletions(-)
> > 
> > diff --git a/Documentation/acpi/enumeration.txt 
> > b/Documentation/acpi/enumeration.txt
> > index b731b29..a91ec5a 100644
> > --- a/Documentation/acpi/enumeration.txt
> > +++ b/Documentation/acpi/enumeration.txt
> > @@ -347,13 +347,18 @@ For the first case, the MFD drivers do not 
> > need to do anything. The
> >  resulting child platform device will have its ACPI_COMPANION() set 
> > to point
> >  to the parent device.
> >  
> > -If the ACPI namespace has a device that we can match using an ACPI 
> > id,
> > -the id should be set like:
> > +If the ACPI namespace has a device that we can match using an ACPI 
> > id or ACPI
> > +adr, the cell should be set like:
> > +
> > +	static struct mfd_cell_acpi_match 
> > my_subdevice_cell_acpi_match = {
> > +		.pnpid = "XYZ0001",
> > +		.adr = 0,
> > +	};
> >  
> >  	static struct mfd_cell my_subdevice_cell = {
> >  		.name = "my_subdevice",
> >  		/* set the resources relative to the parent */
> > -		.acpi_pnpid = "XYZ0001",
> > +		.acpi_match = &my_subdevice_cell_acpi_match,
> >  	};
> >  
> >  The ACPI id "XYZ0001" is then used to lookup an ACPI device 
> > directly under
> > diff --git a/drivers/mfd/mfd-core.c b/drivers/mfd/mfd-core.c
> > index c17635d..60b60dc 100644
> > --- a/drivers/mfd/mfd-core.c
> > +++ b/drivers/mfd/mfd-core.c
> > @@ -82,29 +82,49 @@ static int mfd_platform_add_cell(struct 
> > platform_device *pdev,
> >  static void mfd_acpi_add_device(const struct mfd_cell *cell,
> >  				struct platform_device *pdev)
> >  {
> > -	struct acpi_device *parent_adev;
> > +	const struct mfd_cell_acpi_match *match = cell
> > ->acpi_match;
> > +	struct acpi_device *parent, *child;
> >  	struct acpi_device *adev;
> >  
> > -	parent_adev = ACPI_COMPANION(pdev->dev.parent);
> > -	if (!parent_adev)
> > +	parent = ACPI_COMPANION(pdev->dev.parent);
> > +	if (!parent)
> >  		return;
> >  
> >  	/*
> > -	 * MFD child device gets its ACPI handle either from the 
> > ACPI
> > -	 * device directly under the parent that matches the 
> > acpi_pnpid or
> > -	 * it will use the parent handle if is no acpi_pnpid is 
> > given.
> > +	 * MFD child device gets its ACPI handle either from the 
> > ACPI device
> > +	 * directly under the parent that matches the either _HID 
> > or _CID, or
> > +	 * _ADR or it will use the parent handle if is no ID is 
> > given.
> > +	 *
> > +	 * Note that use of _ADR is a grey area in the ACPI 
> > specification,
> > +	 * though Intel Galileo Gen2 is using it to distinguish 
> > the children
> > +	 * devices.
> >  	 */
> > -	adev = parent_adev;
> > -	if (cell->acpi_pnpid) {
> > -		struct acpi_device_id ids[2] = {};
> > -		struct acpi_device *child_adev;
> > -
> > -		strlcpy(ids[0].id, cell->acpi_pnpid, 
> > sizeof(ids[0].id));
> > -		list_for_each_entry(child_adev, &parent_adev
> > ->children, node)
> > -			if (acpi_match_device_ids(child_adev, 
> > ids)) {
> > -				adev = child_adev;
> > -				break;
> > +	adev = parent;
> > +	if (match) {
> > +		if (match->pnpid) {
> > +			struct acpi_device_id ids[2] = {};
> > +
> > +			strlcpy(ids[0].id, match->pnpid, 
> > sizeof(ids[0].id));
> > +			list_for_each_entry(child, &parent
> > ->children, node) {
> > +				if (acpi_match_device_ids(child, 
> > ids)) {
> > +					adev = child;
> > +					break;
> > +				}
> > +			}
> > +		} else {
> > +			unsigned long long adr;
> > +			acpi_status status;
> > +
> > +			list_for_each_entry(child, &parent
> > ->children, node) {
> > +				status = 
> > acpi_evaluate_integer(child->handle,
> > +							      
> >  "_ADR", NULL,
> > +							      
> >  &adr);
> > +				if (ACPI_SUCCESS(status) && match
> > ->adr == adr) {
> > +					adev = child;
> > +					break;
> > +				}
> >  			}
> > +		}
> >  	}
> >  
> >  	ACPI_COMPANION_SET(&pdev->dev, adev);
> > diff --git a/include/linux/mfd/core.h b/include/linux/mfd/core.h
> > index a76bc10..27dac3f 100644
> > --- a/include/linux/mfd/core.h
> > +++ b/include/linux/mfd/core.h
> > @@ -18,6 +18,12 @@
> >  
> >  struct irq_domain;
> >  
> > +/* Matches ACPI PNP id, either _HID or _CID, or ACPI _ADR */
> > +struct mfd_cell_acpi_match {
> > +	const char			*pnpid;
> > +	const unsigned long long	adr;
> > +};
> > +
> >  /*
> >   * This struct describes the MFD part ("cell").
> >   * After registration the copy of this structure will become the 
> > platform data
> > @@ -44,8 +50,8 @@ struct mfd_cell {
> >  	 */
> >  	const char		*of_compatible;
> >  
> > -	/* Matches ACPI PNP id, either _HID or _CID */
> > -	const char		*acpi_pnpid;
> > +	/* Matches ACPI */
> > +	const struct mfd_cell_acpi_match	*acpi_match;
> >  
> >  	/*
> >  	 * These resources can be specified relative to the parent 
> > device.
> 

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

  reply	other threads:[~2015-09-23 11:07 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-22 10:10 [PATCH v1 0/8] enable I2C devices behind I2C bus on Gen2 Andy Shevchenko
2015-09-22 10:10 ` [PATCH v1 1/8] i2c / ACPI: Rework I2C device scanning Andy Shevchenko
2015-09-22 10:10 ` [PATCH v1 2/8] mfd: core: redo ACPI matching of the children devices Andy Shevchenko
2015-09-22 22:15   ` Lee Jones
2015-09-23 11:06     ` Andy Shevchenko [this message]
2015-09-22 10:10 ` [PATCH v1 3/8] mfd: intel_quark_i2c_gpio: load gpio driver first Andy Shevchenko
2015-09-22 22:23   ` Lee Jones
2015-09-23  7:53     ` Andy Shevchenko
2015-09-24 17:29   ` Lee Jones
2015-09-22 10:10 ` [PATCH v1 4/8] mfd: intel_quark_i2c_gpio: support devices behind i2c bus Andy Shevchenko
2015-09-22 22:25   ` Lee Jones
2015-09-22 10:10 ` [PATCH v1 5/8] gpio: pca953x: store driver_data for future use Andy Shevchenko
2015-10-02 10:53   ` Linus Walleij
2015-10-02 12:16     ` Andy Shevchenko
2015-09-22 10:10 ` [PATCH v1 6/8] gpio: pca953x: support ACPI devices found on Galileo Gen2 Andy Shevchenko
2015-10-02 10:54   ` Linus Walleij
2015-09-22 10:10 ` [PATCH v1 7/8] at24: enable ACPI device " Andy Shevchenko
2015-09-22 10:10 ` [PATCH v1 8/8] pwm-pca9685: " Andy Shevchenko
2015-09-22 14:37   ` Thierry Reding
2015-09-23  8:41     ` Andy Shevchenko
2015-09-23 12:48       ` Thierry Reding
2015-09-23 12:56         ` Andy Shevchenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1443006419.8361.169.camel@linux.intel.com \
    --to=andriy.shevchenko@linux.intel.com \
    --cc=ismo.puustinen@intel.com \
    --cc=lee.jones@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mika.westerberg@intel.com \
    --cc=rjw@rjwysocki.net \
    --cc=srinivas.pandruvada@intel.com \
    --cc=wsa@the-dreams.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).