From: Mika Westerberg <mika.westerberg@linux.intel.com>
To: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: Gabriele Paoloni <gabriele.paoloni@huawei.com>,
rafael@kernel.org, "Rafael J. Wysocki" <rjw@rjwysocki.net>,
"catalin.marinas@arm.com" <catalin.marinas@arm.com>,
"will.deacon@arm.com" <will.deacon@arm.com>,
"robh+dt@kernel.org" <robh+dt@kernel.org>,
"frowand.list@gmail.com" <frowand.list@gmail.com>,
"bhelgaas@google.com" <bhelgaas@google.com>,
"arnd@arndb.de" <arnd@arndb.de>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"mark.rutland@arm.com" <mark.rutland@arm.com>,
"brian.starkey@arm.com" <brian.starkey@arm.com>,
"olof@lixom.net" <olof@lixom.net>,
"benh@kernel.crashing.org" <benh@kernel.crashing.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
Li
Subject: Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices before scanning
Date: Tue, 13 Jun 2017 11:48:31 +0300 [thread overview]
Message-ID: <20170613084831.GP3187@lahna.fi.intel.com> (raw)
In-Reply-To: <20170612155700.GA31930@red-moon>
On Mon, Jun 12, 2017 at 04:57:00PM +0100, Lorenzo Pieralisi wrote:
> I had a more in-depth look at this series and from my understanding
> the problem are the following to manage the LPC bindings in ACPI.
>
> (1) Child devices of an LPC controller require special handling when
> filling their resources (ie they need to be translated - in DT
> that's guaranteed by the "isa" binding, in ACPI it has to be
> done by new code)
> (2) In DT systems, LPC child devices are created by the LPC bus
> controller driver through an of_platform_populate() call with
> the LPC controller node as the fwnode root. For ACPI to work
> the same way there must be a way to prevent LPC children to
> be enumerated in acpi_default_enumeration() something like
> I2C does (and then the LPC driver would enumerate its children as
> DT does)
>
> I am not sure how (1) and (2) can be managed with current ACPI bindings
> and kernel code - I suspect it may be done by mirroring what's done
> for I2C but I am not sure, that's why I CC'ed Mika (ie the LPC adapter
> is matched as a platform device and it takes care of enumerating its
> children - problem though is preventing enumeration from core ACPI code).
Is there an example ASL showing how these LPC devices are
currently presented in ACPI?
WARNING: multiple messages have this Message-ID (diff)
From: Mika Westerberg <mika.westerberg@linux.intel.com>
To: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: "mark.rutland@arm.com" <mark.rutland@arm.com>,
"Rafael J. Wysocki" <rjw@rjwysocki.net>,
"minyard@acm.org" <minyard@acm.org>,
"arnd@arndb.de" <arnd@arndb.de>,
Gabriele Paoloni <gabriele.paoloni@huawei.com>,
rafael@kernel.org,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
"catalin.marinas@arm.com" <catalin.marinas@arm.com>,
John Garry <john.garry@huawei.com>,
"will.deacon@arm.com" <will.deacon@arm.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"xuwei \(O\)" <xuwei5@hisilicon.com>,
Linuxarm <linuxarm@huawei.com>, "olof@lixom.net" <olof@lixom.net>,
"robh+dt@kernel.org" <robh+dt@kernel.org>,
"benh@kernel.crashing.org" <benh@kernel.crashing.org>,
"bhelgaas@google.com" <bhelgaas@google.com>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
"frowand.list@gmail.com" <frowand.list@gmail.com>,
"brian.starkey@arm.com" <brian.starkey@arm.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices before scanning
Date: Tue, 13 Jun 2017 11:48:31 +0300 [thread overview]
Message-ID: <20170613084831.GP3187@lahna.fi.intel.com> (raw)
In-Reply-To: <20170612155700.GA31930@red-moon>
On Mon, Jun 12, 2017 at 04:57:00PM +0100, Lorenzo Pieralisi wrote:
> I had a more in-depth look at this series and from my understanding
> the problem are the following to manage the LPC bindings in ACPI.
>
> (1) Child devices of an LPC controller require special handling when
> filling their resources (ie they need to be translated - in DT
> that's guaranteed by the "isa" binding, in ACPI it has to be
> done by new code)
> (2) In DT systems, LPC child devices are created by the LPC bus
> controller driver through an of_platform_populate() call with
> the LPC controller node as the fwnode root. For ACPI to work
> the same way there must be a way to prevent LPC children to
> be enumerated in acpi_default_enumeration() something like
> I2C does (and then the LPC driver would enumerate its children as
> DT does)
>
> I am not sure how (1) and (2) can be managed with current ACPI bindings
> and kernel code - I suspect it may be done by mirroring what's done
> for I2C but I am not sure, that's why I CC'ed Mika (ie the LPC adapter
> is matched as a platform device and it takes care of enumerating its
> children - problem though is preventing enumeration from core ACPI code).
Is there an example ASL showing how these LPC devices are
currently presented in ACPI?
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: mika.westerberg@linux.intel.com (Mika Westerberg)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices before scanning
Date: Tue, 13 Jun 2017 11:48:31 +0300 [thread overview]
Message-ID: <20170613084831.GP3187@lahna.fi.intel.com> (raw)
In-Reply-To: <20170612155700.GA31930@red-moon>
On Mon, Jun 12, 2017 at 04:57:00PM +0100, Lorenzo Pieralisi wrote:
> I had a more in-depth look at this series and from my understanding
> the problem are the following to manage the LPC bindings in ACPI.
>
> (1) Child devices of an LPC controller require special handling when
> filling their resources (ie they need to be translated - in DT
> that's guaranteed by the "isa" binding, in ACPI it has to be
> done by new code)
> (2) In DT systems, LPC child devices are created by the LPC bus
> controller driver through an of_platform_populate() call with
> the LPC controller node as the fwnode root. For ACPI to work
> the same way there must be a way to prevent LPC children to
> be enumerated in acpi_default_enumeration() something like
> I2C does (and then the LPC driver would enumerate its children as
> DT does)
>
> I am not sure how (1) and (2) can be managed with current ACPI bindings
> and kernel code - I suspect it may be done by mirroring what's done
> for I2C but I am not sure, that's why I CC'ed Mika (ie the LPC adapter
> is matched as a platform device and it takes care of enumerating its
> children - problem though is preventing enumeration from core ACPI code).
Is there an example ASL showing how these LPC devices are
currently presented in ACPI?
WARNING: multiple messages have this Message-ID (diff)
From: Mika Westerberg <mika.westerberg@linux.intel.com>
To: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: Gabriele Paoloni <gabriele.paoloni@huawei.com>,
rafael@kernel.org, "Rafael J. Wysocki" <rjw@rjwysocki.net>,
"catalin.marinas@arm.com" <catalin.marinas@arm.com>,
"will.deacon@arm.com" <will.deacon@arm.com>,
"robh+dt@kernel.org" <robh+dt@kernel.org>,
"frowand.list@gmail.com" <frowand.list@gmail.com>,
"bhelgaas@google.com" <bhelgaas@google.com>,
"arnd@arndb.de" <arnd@arndb.de>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"mark.rutland@arm.com" <mark.rutland@arm.com>,
"brian.starkey@arm.com" <brian.starkey@arm.com>,
"olof@lixom.net" <olof@lixom.net>,
"benh@kernel.crashing.org" <benh@kernel.crashing.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
Linuxarm <linuxarm@huawei.com>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
"minyard@acm.org" <minyard@acm.org>,
John Garry <john.garry@huawei.com>,
"xuwei (O)" <xuwei5@hisilicon.com>
Subject: Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices before scanning
Date: Tue, 13 Jun 2017 11:48:31 +0300 [thread overview]
Message-ID: <20170613084831.GP3187@lahna.fi.intel.com> (raw)
In-Reply-To: <20170612155700.GA31930@red-moon>
On Mon, Jun 12, 2017 at 04:57:00PM +0100, Lorenzo Pieralisi wrote:
> I had a more in-depth look at this series and from my understanding
> the problem are the following to manage the LPC bindings in ACPI.
>
> (1) Child devices of an LPC controller require special handling when
> filling their resources (ie they need to be translated - in DT
> that's guaranteed by the "isa" binding, in ACPI it has to be
> done by new code)
> (2) In DT systems, LPC child devices are created by the LPC bus
> controller driver through an of_platform_populate() call with
> the LPC controller node as the fwnode root. For ACPI to work
> the same way there must be a way to prevent LPC children to
> be enumerated in acpi_default_enumeration() something like
> I2C does (and then the LPC driver would enumerate its children as
> DT does)
>
> I am not sure how (1) and (2) can be managed with current ACPI bindings
> and kernel code - I suspect it may be done by mirroring what's done
> for I2C but I am not sure, that's why I CC'ed Mika (ie the LPC adapter
> is matched as a platform device and it takes care of enumerating its
> children - problem though is preventing enumeration from core ACPI code).
Is there an example ASL showing how these LPC devices are
currently presented in ACPI?
next prev parent reply other threads:[~2017-06-13 8:48 UTC|newest]
Thread overview: 160+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-25 11:37 [PATCH v9 0/7] LPC: legacy ISA I/O support Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` [PATCH v9 1/7] LIB: Introduce a generic PIO mapping method Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-26 20:57 ` Bjorn Helgaas
2017-05-26 20:57 ` Bjorn Helgaas
2017-05-26 20:57 ` Bjorn Helgaas
2017-05-26 20:57 ` Bjorn Helgaas
2017-05-30 15:09 ` Gabriele Paoloni
2017-05-30 15:09 ` Gabriele Paoloni
2017-05-30 15:09 ` Gabriele Paoloni
2017-05-30 15:09 ` Gabriele Paoloni
2017-05-25 11:37 ` [PATCH v9 2/7] PCI: Apply the new generic I/O management on PCI IO hosts Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-26 21:20 ` Bjorn Helgaas
2017-05-26 21:20 ` Bjorn Helgaas
2017-05-26 21:20 ` Bjorn Helgaas
2017-05-26 21:20 ` Bjorn Helgaas
2017-05-30 8:12 ` Gabriele Paoloni
2017-05-30 8:12 ` Gabriele Paoloni
2017-05-30 8:12 ` Gabriele Paoloni
2017-05-25 11:37 ` [PATCH v9 3/7] OF: Add missing I/O range exception for indirect-IO devices Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` [PATCH v9 4/7] LPC: Support the device-tree LPC host on Hip06/Hip07 Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices before scanning Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-26 0:03 ` kbuild test robot
2017-05-26 0:03 ` kbuild test robot
2017-05-26 0:03 ` kbuild test robot
2017-05-26 0:03 ` kbuild test robot
2017-05-30 13:24 ` Lorenzo Pieralisi
2017-05-30 13:24 ` Lorenzo Pieralisi
2017-05-30 13:24 ` Lorenzo Pieralisi
2017-05-31 10:24 ` Gabriele Paoloni
2017-05-31 10:24 ` Gabriele Paoloni
2017-05-31 10:24 ` Gabriele Paoloni
2017-05-31 10:24 ` Gabriele Paoloni
2017-06-06 8:55 ` Lorenzo Pieralisi
2017-06-06 8:55 ` Lorenzo Pieralisi
2017-06-06 8:55 ` Lorenzo Pieralisi
2017-06-12 15:57 ` Lorenzo Pieralisi
2017-06-12 15:57 ` Lorenzo Pieralisi
2017-06-12 15:57 ` Lorenzo Pieralisi
2017-06-12 15:57 ` Lorenzo Pieralisi
2017-06-13 7:24 ` Gabriele Paoloni
2017-06-13 7:24 ` Gabriele Paoloni
2017-06-13 7:24 ` Gabriele Paoloni
2017-06-13 8:48 ` Mika Westerberg [this message]
2017-06-13 8:48 ` Mika Westerberg
2017-06-13 8:48 ` Mika Westerberg
2017-06-13 8:48 ` Mika Westerberg
2017-06-13 14:38 ` Gabriele Paoloni
2017-06-13 14:38 ` Gabriele Paoloni
2017-06-13 14:38 ` Gabriele Paoloni
2017-06-13 14:38 ` Gabriele Paoloni
2017-06-13 15:10 ` Mika Westerberg
2017-06-13 15:10 ` Mika Westerberg
2017-06-13 15:10 ` Mika Westerberg
2017-06-13 19:01 ` Gabriele Paoloni
2017-06-13 19:01 ` Gabriele Paoloni
2017-06-13 19:01 ` Gabriele Paoloni
2017-06-13 20:03 ` Mika Westerberg
2017-06-13 20:03 ` Mika Westerberg
2017-06-13 20:03 ` Mika Westerberg
2017-06-15 18:01 ` Gabriele Paoloni
2017-06-15 18:01 ` Gabriele Paoloni
2017-06-15 18:01 ` Gabriele Paoloni
2017-06-15 18:01 ` Gabriele Paoloni
2017-06-16 8:33 ` Mika Westerberg
2017-06-16 8:33 ` Mika Westerberg
2017-06-16 8:33 ` Mika Westerberg
2017-06-16 11:24 ` Rafael J. Wysocki
2017-06-16 11:24 ` Rafael J. Wysocki
2017-06-16 11:24 ` Rafael J. Wysocki
2017-06-16 11:24 ` Rafael J. Wysocki
2017-06-16 12:00 ` Mika Westerberg
2017-06-16 12:00 ` Mika Westerberg
2017-06-16 12:00 ` Mika Westerberg
2017-06-16 12:22 ` Rafael J. Wysocki
2017-06-16 12:22 ` Rafael J. Wysocki
2017-06-16 12:22 ` Rafael J. Wysocki
2017-06-16 12:22 ` Rafael J. Wysocki
2017-06-19 9:50 ` Gabriele Paoloni
2017-06-19 9:50 ` Gabriele Paoloni
2017-06-19 9:50 ` Gabriele Paoloni
2017-06-19 9:50 ` Gabriele Paoloni
2017-06-19 10:02 ` Mika Westerberg
2017-06-19 10:02 ` Mika Westerberg
2017-06-19 10:02 ` Mika Westerberg
2017-06-19 10:02 ` Mika Westerberg
2017-06-19 10:04 ` Gabriele Paoloni
2017-06-19 10:04 ` Gabriele Paoloni
2017-06-19 10:04 ` Gabriele Paoloni
2017-07-03 16:08 ` Gabriele Paoloni
2017-07-03 16:08 ` Gabriele Paoloni
2017-07-03 16:08 ` Gabriele Paoloni
2017-07-03 16:08 ` Gabriele Paoloni
2017-07-03 16:23 ` Gabriele Paoloni
2017-07-03 16:23 ` Gabriele Paoloni
2017-07-03 16:23 ` Gabriele Paoloni
2017-07-03 16:23 ` Gabriele Paoloni
2017-07-03 20:22 ` Andy Shevchenko
2017-07-03 20:22 ` Andy Shevchenko
2017-07-03 20:22 ` Andy Shevchenko
2017-07-03 20:22 ` Andy Shevchenko
2017-07-04 15:14 ` Gabriele Paoloni
2017-07-04 15:14 ` Gabriele Paoloni
2017-07-04 15:14 ` Gabriele Paoloni
2017-07-04 15:14 ` Gabriele Paoloni
2017-07-04 15:46 ` Andy Shevchenko
2017-07-04 15:46 ` Andy Shevchenko
2017-07-04 15:46 ` Andy Shevchenko
2017-07-04 15:46 ` Andy Shevchenko
2017-07-04 16:22 ` Gabriele Paoloni
2017-07-04 16:22 ` Gabriele Paoloni
2017-07-04 16:22 ` Gabriele Paoloni
2017-07-04 16:22 ` Gabriele Paoloni
2017-06-29 16:16 ` John Garry
2017-06-29 16:16 ` John Garry
2017-06-29 16:16 ` John Garry
2017-06-30 9:05 ` Mika Westerberg
2017-06-30 9:05 ` Mika Westerberg
2017-06-30 9:05 ` Mika Westerberg
2017-06-30 9:28 ` John Garry
2017-06-30 9:28 ` John Garry
2017-06-30 9:28 ` John Garry
2017-06-30 9:28 ` John Garry
2017-06-30 12:56 ` Rafael J. Wysocki
2017-06-30 12:56 ` Rafael J. Wysocki
2017-06-30 12:56 ` Rafael J. Wysocki
2017-06-30 12:56 ` Rafael J. Wysocki
2017-05-25 11:37 ` [PATCH v9 6/7] LPC: Add the ACPI LPC support Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-26 3:12 ` kbuild test robot
2017-05-26 3:12 ` kbuild test robot
2017-05-26 3:12 ` kbuild test robot
2017-05-26 3:12 ` kbuild test robot
2017-05-26 10:12 ` Gabriele Paoloni
2017-05-26 10:12 ` Gabriele Paoloni
2017-05-26 10:12 ` Gabriele Paoloni
2017-05-26 10:12 ` Gabriele Paoloni
2017-05-25 11:37 ` [PATCH v9 7/7] MANTAINERS: Add maintainer for HiSilicon LPC driver Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
2017-05-25 11:37 ` Gabriele Paoloni
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=20170613084831.GP3187@lahna.fi.intel.com \
--to=mika.westerberg@linux.intel.com \
--cc=arnd@arndb.de \
--cc=benh@kernel.crashing.org \
--cc=bhelgaas@google.com \
--cc=brian.starkey@arm.com \
--cc=catalin.marinas@arm.com \
--cc=frowand.list@gmail.com \
--cc=gabriele.paoloni@huawei.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=mark.rutland@arm.com \
--cc=olof@lixom.net \
--cc=rafael@kernel.org \
--cc=rjw@rjwysocki.net \
--cc=robh+dt@kernel.org \
--cc=will.deacon@arm.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.