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

Groups > linux.kernel > #1252530

Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports

Show all headers | View raw

On Wed, Oct 21, 2015 at 01:12:01AM +0200, Rafael J. Wysocki wrote:
> Well, we already have that in the MFD case, but in principle it may be
> problematic for things like power management (say you want to put a
> child device into D3, so you use _PS3 on its ACPI companion and then
> the parent is powere down instead).

That case I understand and we should not allow that. However, here we
have an I2C adapter which is purely Linux abstraction that does not have
any representation either in DT nor ACPI. And we don't do any power
management for that either.

If I understand correctly, DT shares the same of_node for both the host
controller device and the adapter which then makes it possible to
enumerate devices behind by just looking adap->dev.of_node. In case of
ACPI we need to know that it's the parent device that we should look for
child devices which may not be true always (like what this patch is
trying to resolve).

> At least, devices in that setup should not be attached to the ACPI PM
> domain.

Agreed.

Currently acpi_dev_pm_attach() only attaches the first physical device
to the ACPI power domain so this should be taken care already.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

Thread

[PATCH v3 0/1] i2c: acpi: scan ACPI enumerated I2C mux channels Dustin Byford <dustin@cumulusnetworks.com> - 2015-10-20 00:30 +0200
  [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Dustin Byford <dustin@cumulusnetworks.com> - 2015-10-20 00:30 +0200
    Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2015-10-20 11:20 +0200
    Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Mika Westerberg <mika.westerberg@linux.intel.com> - 2015-10-20 15:00 +0200
      Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Dustin Byford <dustin@cumulusnetworks.com> - 2015-10-20 20:00 +0200
        Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2015-10-21 00:50 +0200
        Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Mika Westerberg <mika.westerberg@linux.intel.com> - 2015-10-21 10:20 +0200
          Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Dustin Byford <dustin@cumulusnetworks.com> - 2015-10-21 10:30 +0200
            Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Mika Westerberg <mika.westerberg@linux.intel.com> - 2015-10-21 10:40 +0200
              Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Dustin Byford <dustin@cumulusnetworks.com> - 2015-10-21 11:00 +0200
                Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Mika Westerberg <mika.westerberg@linux.intel.com> - 2015-10-21 11:20 +0200
                Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Dustin Byford <dustin@cumulusnetworks.com> - 2015-10-21 11:30 +0200
                Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports "Rafael J. Wysocki" <rafael@kernel.org> - 2015-10-22 00:40 +0200
                Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Dustin Byford <dustin@cumulusnetworks.com> - 2015-10-22 11:30 +0200
      Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2015-10-21 00:50 +0200
        Re: [PATCH v3 1/1] i2c: add ACPI support for I2C mux ports Mika Westerberg <mika.westerberg@linux.intel.com> - 2015-10-21 10:10 +0200

csiph-web