Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1667029

RE: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices before scanning

Show all headers | View raw

Hi Mika

> -----Original Message-----
> From: linux-pci-owner@vger.kernel.org [mailto:linux-pci-
> owner@vger.kernel.org] On Behalf Of Mika Westerberg
> Sent: 13 June 2017 21:04
> To: Gabriele Paoloni
> Cc: Lorenzo Pieralisi; rafael@kernel.org; Rafael J. Wysocki;
> catalin.marinas@arm.com; will.deacon@arm.com; robh+dt@kernel.org;
> frowand.list@gmail.com; bhelgaas@google.com; arnd@arndb.de; linux-arm-
> kernel@lists.infradead.org; mark.rutland@arm.com;
> brian.starkey@arm.com; olof@lixom.net; benh@kernel.crashing.org; linux-
> kernel@vger.kernel.org; linux-acpi@vger.kernel.org; Linuxarm; linux-
> pci@vger.kernel.org; minyard@acm.org; John Garry; xuwei (O)
> Subject: Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO
> devices before scanning
> 
> On Tue, Jun 13, 2017 at 07:01:38PM +0000, Gabriele Paoloni wrote:
> > I am not very familiar with Linux MFD however the main issue here is
> that
> > 1) for IPMI we want to re-use the standard IPMI driver without
> touching it:
> >    see
> >
> >    static const struct acpi_device_id acpi_ipmi_match[] = {
> >          { "IPI0001", 0 },
> >          { },
> >    };
> >
> >    in "drivers/char/ipmi/ipmi_si_intf.c" (and in general any standard
> driver
> >    of an LPC child)
> >
> > 2) We need a way to guarantee that all LPC children are not
> enumerated
> >    by acpi_default_enumeration() (so for example if an ipmi node is
> an LPC#
> >    child it should not be enumerated, otherwise it should be)
> >    Currently acpi_default_enumeration() skips spi/i2c slaves by
> checking:
> >    1) if the acpi resource type is a serial bus
> >    2) if the type of serial bus descriptor is I2C or SPI
> >
> >    For LPC we cannot leverage on any ACPI property to "recognize"
> that our
> >    devices are LPC children; hence before I proposed for
> acpi_default_enumeration()
> >    to skip devices that have already been enumerated (by calling
> >    acpi_device_enumerated() ).
> >
> > So in the current scenario, how do you think that MFD can help?
> 
> If you look at Documentation/acpi/enumeration.txt there is a chapter
> "MFD devices". I think it pretty much maches what you have here. An LPC
> device (MFD device) and bunch of child devices. The driver for your LPC
> device can specify _HID for each child device. Those are then mached by
> the MFD ACPI code to the corresponding ACPI nodes from which platform
> devices are created including "IPI0001".

So I guess here in the LPC driver I would have an MFD cell for IPMI. I.e.:

	static struct mfd_cell_acpi_match hisi_lpc_ipmi_acpi_match = {
		.pnpid = "IPI0001",
	};

correct?

> 
> It causes acpi_default_enumeration() to be called but it should be fine
> as we are dealing with platform device anyway.

I do not quite understand how declaring such MFD cell above would make sure
that the LPC probe is called before the IPMI device is enumerated...

Could you point me to the code that does this?

Many Thanks
Gab

> 
> Once the platform device is created your ipmi driver will be probed by
> the driver core.
> 
> Does that make sense?

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices  before scanning Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> - 2017-06-12 18:10 +0200
  RE: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO  devices before scanning Gabriele Paoloni <gabriele.paoloni@huawei.com> - 2017-06-13 09:30 +0200
  Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices  before scanning Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-06-13 11:00 +0200
    RE: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO  devices before scanning Gabriele Paoloni <gabriele.paoloni@huawei.com> - 2017-06-13 16:50 +0200
      Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices  before scanning Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-06-13 17:20 +0200
        RE: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO  devices before scanning Gabriele Paoloni <gabriele.paoloni@huawei.com> - 2017-06-13 21:10 +0200
          Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices  before scanning Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-06-13 22:10 +0200
            RE: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO  devices before scanning Gabriele Paoloni <gabriele.paoloni@huawei.com> - 2017-06-15 20:10 +0200
              Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices  before scanning Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-06-16 10:40 +0200
                Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices  before scanning "Rafael J. Wysocki" <rafael@kernel.org> - 2017-06-16 13:30 +0200
                Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices  before scanning Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-06-16 14:10 +0200
                Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices  before scanning "Rafael J. Wysocki" <rafael@kernel.org> - 2017-06-16 14:30 +0200
                RE: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO  devices before scanning Gabriele Paoloni <gabriele.paoloni@huawei.com> - 2017-06-19 12:00 +0200
                RE: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO  devices before scanning Gabriele Paoloni <gabriele.paoloni@huawei.com> - 2017-06-19 12:10 +0200
                Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices  before scanning Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-06-19 12:10 +0200

csiph-web