linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mika Westerberg <mika.westerberg@linux.intel.com>
To: Gabriele Paoloni <gabriele.paoloni@huawei.com>
Cc: "mark.rutland@arm.com" <mark.rutland@arm.com>,
	"minyard@acm.org" <minyard@acm.org>,
	Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"arnd@arndb.de" <arnd@arndb.de>,
	"Rafael J. Wysocki" <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>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"will.deacon@arm.com" <will.deacon@arm.com>,
	Linuxarm <linuxarm@huawei.com>, "olof@lixom.net" <olof@lixom.net>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"xuwei \(O\)" <xuwei5@hisilicon.com>,
	"benh@kernel.crashing.org" <benh@kernel.crashing.org>,
	"bhelgaas@google.com" <bhelgaas@google.com>,
	"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: Mon, 19 Jun 2017 13:02:09 +0300	[thread overview]
Message-ID: <20170619100209.GC629@lahna.fi.intel.com> (raw)
In-Reply-To: <EE11001F9E5DDD47B7634E2F8A612F2E40ACFB8D@FRAEML521-MBX.china.huawei.com>

On Mon, Jun 19, 2017 at 09:50:49AM +0000, Gabriele Paoloni wrote:
> Many thanks for your response and your help here.
> 
> I guess that as conclusion with respect to the current v9 patchset we can
> disregard the idea of MFD and modify the current v9 so that it doesn't
> touch directly ACPI resources.
> Instead as I proposed before we can have the scan handler to enumerate
> the children devices and translate its addresses filling dev->resources[] and
> at the same time we can modify acpi_default_enumeration to check
> acpi_device_enumerated() before continuing with device enumeration...?
> 
> Do you think it as a viable solution?

No, I think MFD + scan handler inside the MFD driver is the way to go.
We don't want to trash ACPI core with stuff that does not belong there
IMHO.

Also you don't need to modify acpi_default_enumeration() because you can
mark your device enumerated in the MFD driver. So all the dirty details
will be in the MFD driver and not in ACPI core.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2017-06-19 10:02 UTC|newest]

Thread overview: 43+ 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 ` [PATCH v9 1/7] LIB: Introduce a generic PIO mapping method Gabriele Paoloni
2017-05-26 20:57   ` Bjorn Helgaas
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-26 21:20   ` Bjorn Helgaas
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 ` [PATCH v9 4/7] LPC: Support the device-tree LPC host on Hip06/Hip07 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-26  0:03   ` kbuild test robot
2017-05-30 13:24   ` Lorenzo Pieralisi
2017-05-31 10:24     ` Gabriele Paoloni
2017-06-06  8:55       ` Lorenzo Pieralisi
2017-06-12 15:57         ` Lorenzo Pieralisi
2017-06-13  7:24           ` Gabriele Paoloni
2017-06-13  8:48           ` Mika Westerberg
2017-06-13 14:38             ` Gabriele Paoloni
2017-06-13 15:10               ` Mika Westerberg
2017-06-13 19:01                 ` Gabriele Paoloni
2017-06-13 20:03                   ` Mika Westerberg
2017-06-15 18:01                     ` Gabriele Paoloni
2017-06-16  8:33                       ` Mika Westerberg
2017-06-16 11:24                         ` Rafael J. Wysocki
2017-06-16 12:00                           ` Mika Westerberg
2017-06-16 12:22                             ` Rafael J. Wysocki
2017-06-19  9:50                               ` Gabriele Paoloni
2017-06-19 10:02                                 ` Mika Westerberg [this message]
2017-06-19 10:04                                   ` Gabriele Paoloni
2017-07-03 16:08                                     ` Gabriele Paoloni
2017-07-03 16:23                                       ` Gabriele Paoloni
2017-07-03 20:22                                       ` Andy Shevchenko
2017-07-04 15:14                                         ` Gabriele Paoloni
2017-07-04 15:46                                           ` Andy Shevchenko
2017-07-04 16:22                                             ` Gabriele Paoloni
2017-06-29 16:16                           ` John Garry
2017-06-30  9:05                             ` Mika Westerberg
2017-06-30  9:28                               ` John Garry
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-26  3:12   ` kbuild test robot
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

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=20170619100209.GC629@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=john.garry@huawei.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linuxarm@huawei.com \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=mark.rutland@arm.com \
    --cc=minyard@acm.org \
    --cc=olof@lixom.net \
    --cc=rafael@kernel.org \
    --cc=rjw@rjwysocki.net \
    --cc=robh+dt@kernel.org \
    --cc=will.deacon@arm.com \
    --cc=xuwei5@hisilicon.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 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).