[Resend with linux-arm-kernel included and added acks from Rafael]
Hi all,
With the advent of ACPI 5.0 we are starting to see I2C client devices
described in ACPI namespace that support power management by the means of
standard ACPI _PSx-methods. For example Intel Haswell based platforms might
ha
On Mon, Sep 09, 2013 at 04:40:28PM +0100, Mark Brown wrote:
> On Mon, Sep 09, 2013 at 04:34:38PM +0300, Mika Westerberg wrote:
>
> > + /*
> > +* Enable runtime PM for the client device. If the client wants to
> > +* participate on runtime PM it should call
On Tue, Sep 10, 2013 at 01:27:54PM +0100, Mark Brown wrote:
> On Tue, Sep 10, 2013 at 10:51:00AM +0300, Mika Westerberg wrote:
> > On Mon, Sep 09, 2013 at 04:40:28PM +0100, Mark Brown wrote:
>
> > > How is this going to interact with client devices which are already
> >
On Wed, Sep 11, 2013 at 10:55:52AM +0100, Mark Brown wrote:
> On Wed, Sep 11, 2013 at 09:01:16AM +0800, Aaron Lu wrote:
>
> > Looks like, it all boils down to how many I2C devices should be allowed
> > for runtime PM by default and how many I2C devices should be forbidden.
> > , and then we allow/
On Wed, Sep 11, 2013 at 12:14:09PM +0100, Mark Brown wrote:
> On Wed, Sep 11, 2013 at 02:05:43PM +0300, Mika Westerberg wrote:
>
> > I'll also look into converting the existing I2C client drivers to use this
> > method. One question, though, is it better to have the con
as it should be the
same size than the native pointer type.
Signed-off-by: Mika Westerberg
---
drivers/mfd/wm8994-core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mfd/wm8994-core.c b/drivers/mfd/wm8994-core.c
index bd53879..9928bb1 100644
--- a/drivers/mfd/wm8
to pm_runtime_set_autosuspend_delay() in
mpu3050 driver because the driver doesn't seem to use autosuspend anyway.
Signed-off-by: Mika Westerberg
---
drivers/input/misc/bma150.c | 4 ++--
drivers/input/misc/mpu3050.c | 16
2 files changed, 6 insertions(+), 14 deletions(-)
diff --git a/drivers/in
that's the case, attach it to the ACPI power domain. In
addition we make sure that the device is fully powered when its ->probe()
function gets called.
For non-ACPI devices this patch is a no-op.
Signed-off-by: Lv Zheng
Signed-off-by: Mika Westerberg
Acked-by: Rafael J. Wysocki
---
drivers
ACPI power domain if the device has an ACPI handle. This
makes sure that the device is powered on when its ->probe() is called.
For non-ACPI devices this patch is a no-op.
Signed-off-by: Mika Westerberg
---
drivers/spi/spi.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/driv
The I2C core now prepares runtime PM on behalf of the I2C client device, so
only thing the driver needs to do is to call pm_runtime_put() at the end of
its ->probe().
This patch converts wm8994 driver to use this model.
Signed-off-by: Mika Westerberg
---
drivers/mfd/wm8994-core.c | 5 ++---
The I2C core now prepares runtime PM on behalf of the I2C client device, so
only thing the driver needs to do is to call pm_runtime_put() at the end of
its ->probe().
This patch converts I2C client drivers under drivers/misc to use this
model.
Signed-off-by: Mika Westerberg
---
drivers/m
afael ACKed the previous version but I've changed the patches a bit, so I
only kept his ACK in the second patch (it is unchanged). ]
Aaron Lu (1):
i2c: prepare runtime PM support for I2C client devices
Lv Zheng (1):
i2c: attach/detach I2C client device to the ACPI power domain
Mika Wes
stion is
regarded as being runtime PM active and powered on.
This patch adds also runtime PM support for the SPI master device because
it is needed to be able to runtime power manage the SPI controller device.
The SPI master device is handled along with the SPI controller device.
Signed-off-
The I2C core now prepares runtime PM on behalf of the I2C client device, so
only thing the driver needs to do is to call pm_runtime_put() at the end of
its ->probe().
This patch converts ASoC codec drivers to use this model.
Signed-off-by: Mika Westerberg
---
sound/soc/codecs/wm2200.c |
evice is handled along with the I2C controller device (it uses
pm_runtime_no_callbacks()).
Signed-off-by: Aaron Lu
Signed-off-by: Mika Westerberg
---
drivers/i2c/i2c-core.c | 44 +++-
1 file changed, 43 insertions(+), 1 deletion(-)
diff --git a/drivers
The I2C core now prepares runtime PM on behalf of the I2C client device, so
only thing the driver needs to do is to call pm_runtime_put() at the end of
its ->probe().
This patch converts s5p-tv driver to use this model.
Signed-off-by: Mika Westerberg
---
drivers/media/platform/s5p
On Wed, Sep 11, 2013 at 06:12:43PM +0200, Samuel Ortiz wrote:
> Hi Mika,
>
> On Wed, Sep 11, 2013 at 06:32:37PM +0300, Mika Westerberg wrote:
> > The I2C core now prepares runtime PM on behalf of the I2C client device, so
> > only thing the driver needs to do is to call pm
On Wed, Sep 11, 2013 at 04:51:20PM +0100, Mark Brown wrote:
> On Wed, Sep 11, 2013 at 06:32:39PM +0300, Mika Westerberg wrote:
> > This patch adds runtime PM support for the SPI bus analogous to what has
> > been done for the I2C bus. This means that the SPI core prepares runti
On Thu, Sep 12, 2013 at 10:31:45AM +0100, Mark Brown wrote:
> On Thu, Sep 12, 2013 at 12:27:43PM +0300, Mika Westerberg wrote:
> > On Wed, Sep 11, 2013 at 04:51:20PM +0100, Mark Brown wrote:
>
> > > I would be able to have this and the other patch in the SPI tree in case
&
On Thu, Sep 12, 2013 at 01:04:28PM +0200, Wolfram Sang wrote:
> On Thu, Sep 12, 2013 at 01:04:55PM +0200, Rafael J. Wysocki wrote:
> > On Thursday, September 12, 2013 12:43:02 PM Mika Westerberg wrote:
> > > On Thu, Sep 12, 2013 at 10:31:45AM +0100, Mark Brown wrote:
> >
On Thu, Sep 12, 2013 at 01:04:55PM +0200, Rafael J. Wysocki wrote:
> On Thursday, September 12, 2013 12:43:02 PM Mika Westerberg wrote:
> > On Thu, Sep 12, 2013 at 10:31:45AM +0100, Mark Brown wrote:
> > > On Thu, Sep 12, 2013 at 12:27:43PM +0300, Mika Westerberg wrote:
>
On Thu, Sep 12, 2013 at 02:34:21PM -0700, Kevin Hilman wrote:
> > diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c
> > index f32ca29..44374b4 100644
> > --- a/drivers/i2c/i2c-core.c
> > +++ b/drivers/i2c/i2c-core.c
> > @@ -248,11 +248,30 @@ static int i2c_device_probe(struct device *dev
On Fri, Sep 13, 2013 at 10:59:50AM +0100, Mark Brown wrote:
> On Fri, Sep 13, 2013 at 09:54:34AM +0300, Mika Westerberg wrote:
> > On Thu, Sep 12, 2013 at 02:34:21PM -0700, Kevin Hilman wrote:
>
> > > For hardware that is disabled/powered-off on startup, there will now be
>
On Fri, Sep 13, 2013 at 11:31:52AM +0100, Mark Brown wrote:
> On Fri, Sep 13, 2013 at 01:16:11PM +0300, Mika Westerberg wrote:
> > On Fri, Sep 13, 2013 at 10:59:50AM +0100, Mark Brown wrote:
>
> > > Accessing the bus isn't an issue for I2C outside of ACPI, the power
>
On Fri, Sep 13, 2013 at 07:30:55AM -0700, Kevin Hilman wrote:
> Mika Westerberg writes:
>
> > On Thu, Sep 12, 2013 at 02:34:21PM -0700, Kevin Hilman wrote:
> >> > diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c
> >> > index f32ca29..44374b4 100644
structure.
Signed-off-by: Mika Westerberg
---
drivers/gpio/gpiolib-acpi.c | 131 ++--
1 file changed, 126 insertions(+), 5 deletions(-)
diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index e12a08e..f52536a 100644
--- a/drivers/gp
nterface to be private to gpiolib-acpi and remove call
to the API from the one existing user (pinctrl-baytrail.c).
Signed-off-by: Mika Westerberg
---
drivers/gpio/gpiolib-acpi.c| 18 ++
drivers/gpio/gpiolib.c | 3 +++
drivers/pinctrl/pinctrl-baytrai
On Fri, Sep 13, 2013 at 05:14:56PM +0200, Sylwester Nawrocki wrote:
> On 09/13/2013 08:54 AM, Mika Westerberg wrote:
> > On Thu, Sep 12, 2013 at 02:34:21PM -0700, Kevin Hilman wrote:
> >>> > > diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c
> >>
On Fri, Sep 13, 2013 at 06:55:11PM +0300, Andy Shevchenko wrote:
> On Fri, Sep 13, 2013 at 6:14 PM, Mika Westerberg
> wrote:
> > GPIO operation regions is a new feature introduced in ACPI 5.0
> > specification. In practise it means that now ASL code can toggle GPIOs with
>
On Fri, Sep 13, 2013 at 05:50:22PM +0300, Mika Westerberg wrote:
> On Fri, Sep 13, 2013 at 07:30:55AM -0700, Kevin Hilman wrote:
> > Mika Westerberg writes:
> >
> > > On Thu, Sep 12, 2013 at 02:34:21PM -0700, Kevin Hilman wrote:
> > >> > diff --git a/d
On Mon, Oct 07, 2013 at 09:54:13PM -0700, Alexandre Courbot wrote:
> > +struct gpio_desc *acpi_get_gpiod(char *path, int pin)
> > {
> > struct gpio_chip *chip;
> > acpi_handle handle;
> > @@ -48,18 +49,15 @@ int acpi_get_gpio(char *path, int pin)
> >
> > status = acpi_get_h
On Tue, Oct 08, 2013 at 11:45:08AM +0300, Mika Westerberg wrote:
> > I also wonder whether you could also avoid exporting acpi_get_gpiod*
> > (which allow GPIO consumers to shortcut gpiolib) by implementing
> > acpi_get_gpio* into the C file (maybe even using gpiod_get()). I see
On Tue, Oct 08, 2013 at 05:00:54PM +0200, Romain Baeriswyl wrote:
> static void __i2c_dw_enable(struct dw_i2c_dev *dev, bool enable)
> @@ -286,6 +287,7 @@ int i2c_dw_init(struct dw_i2c_dev *dev)
> u32 input_clock_khz;
> u32 hcnt, lcnt;
> u32 reg;
> + u32 sda_falling_time, scl
arameter is not set, the fast mode is selected.
> Only when the parameter is set at 10, the standard mode is selected.
>
> Signed-off-by: Romain Baeriswyl
> Reviewed-by: Christian Ruppert
Reviewed-by: Mika Westerberg
--
To unsubscribe from this list: send the line "unsubscrib
On Tue, Oct 08, 2013 at 09:24:50AM -0700, Alexandre Courbot wrote:
> On Tue, Oct 8, 2013 at 1:45 AM, Mika Westerberg
> wrote:
> > On Mon, Oct 07, 2013 at 09:54:13PM -0700, Alexandre Courbot wrote:
> >> > +struct gpio_desc *acpi_get_gpiod(char *path, int pin)
> &g
On Tue, Oct 08, 2013 at 09:26:14AM -0700, Alexandre Courbot wrote:
> On Tue, Oct 8, 2013 at 3:36 AM, Mika Westerberg
> wrote:
> > On Tue, Oct 08, 2013 at 11:45:08AM +0300, Mika Westerberg wrote:
> >> > I also wonder whether you could also avoid exporting acpi_get_gpiod*
that's the case, attach it to the ACPI power domain. In
addition we make sure that the device is fully powered when its ->probe()
function gets called.
For non-ACPI devices this patch is a no-op.
Signed-off-by: Lv Zheng
Signed-off-by: Mika Westerberg
---
drivers/i2c/i2c-core.c | 10 +
when the device is powered on/off.
Signed-off-by: Mika Westerberg
---
drivers/acpi/device_pm.c | 8 +---
include/acpi/acpi_bus.h | 3 ++-
2 files changed, 7 insertions(+), 4 deletions(-)
diff --git a/drivers/acpi/device_pm.c b/drivers/acpi/device_pm.c
index 59d3202..ca723a4 100644
---
):
i2c: attach/detach I2C client device to the ACPI power domain
Mika Westerberg (2):
ACPI / PM: allow child devices to ignore parent power state
spi: attach/detach SPI device to the ACPI power domain
drivers/acpi/device_pm.c | 8 +---
drivers/i2c/i2c-core.c | 10 ++
drivers/spi
ACPI power domain if the device has an ACPI handle. This
makes sure that the device is powered on when its ->probe() is called.
For non-ACPI devices this patch is a no-op.
Signed-off-by: Mika Westerberg
---
drivers/spi/spi.c | 20 ++--
1 file changed, 18 insertions(+)
On Wed, Oct 09, 2013 at 06:55:28PM +0100, Mark Brown wrote:
> On Wed, Oct 09, 2013 at 05:04:21PM +0300, Mika Westerberg wrote:
> > If the SPI device is enumerated from ACPI namespace (it has an ACPI handle)
> > it might have ACPI methods that needs to be called in order to tra
integer.
We also drop acpi_get_gpio() as it is not used anywhere outside
gpiolib-acpi and even there we use acpi_get_gpiod() instead.
Signed-off-by: Mika Westerberg
Acked-by: Rafael J. Wysocki
Reviewed-by: Alexandre Courbot
---
drivers/gpio/gpiolib-acpi.c | 51
The ACPI GpioInt resources contain polarity field that is used to specify
whether the interrupt is active high or low. Since gpiolib supports
GPIOF_ACTIVE_LOW we can pass this information in the flags field in
acpi_find_gpio(), analogous to the DeviceTree version.
Signed-off-by: Mika Westerberg
In addition to the existing ACPI specific GPIO interface, document the new
descriptor based GPIO interface in Documentation/acpi/enumeration.txt, so
it is clear that this new interface is preferred over the ACPI specific
version.
Signed-off-by: Mika Westerberg
Acked-by: Rafael J. Wysocki
and
tested on Intel Haswell machine.
Mika Westerberg (5):
gpiolib / ACPI: move acpi_gpiochip_free_interrupts next to the request
function
gpiolib / ACPI: convert to gpiod interfaces
gpiolib / ACPI: add ACPI support for gpiod_get_index()
gpiolib / ACPI: allow passing GPIOF_ACTIVE_LOW for
gpiod_get_index() and gpiod_get() are now the new preferred way to request
GPIOs. Add support for finding the corresponding GPIO descriptor from ACPI
namespace.
Signed-off-by: Mika Westerberg
Acked-by: Rafael J. Wysocki
Reviewed-by: Alexandre Courbot
---
drivers/gpio/gpiolib.c | 10
It makes more sense to have these functions close to each other. No
functional changes.
Signed-off-by: Mika Westerberg
Acked-by: Rafael J. Wysocki
---
drivers/gpio/gpiolib-acpi.c | 76 ++---
1 file changed, 38 insertions(+), 38 deletions(-)
diff --git a
On Thu, Oct 10, 2013 at 10:09:19AM +0200, Wolfram Sang wrote:
> On Wed, Oct 09, 2013 at 05:04:20PM +0300, Mika Westerberg wrote:
> > From: Lv Zheng
> >
> > If the I2C client device is enumerated from ACPI namespace it might have
> > ACPI methods that needs to be called
ACPI power domain if the device has an ACPI handle. This
makes sure that the device is powered on when its ->probe() is called.
For non-ACPI devices this patch is a no-op.
Signed-off-by: Mika Westerberg
Acked-by: Mark Brown
---
drivers/spi/spi.c | 18 --
1 file changed,
when the device is powered on/off.
Signed-off-by: Mika Westerberg
---
drivers/acpi/device_pm.c | 8 +---
include/acpi/acpi_bus.h | 3 ++-
2 files changed, 7 insertions(+), 4 deletions(-)
diff --git a/drivers/acpi/device_pm.c b/drivers/acpi/device_pm.c
index 59d3202..ca723a4 100644
---
/detach I2C client device to the ACPI power domain
Mika Westerberg (2):
ACPI / PM: allow child devices to ignore parent power state
spi: attach/detach SPI device to the ACPI power domain
drivers/acpi/device_pm.c | 8 +---
drivers/i2c/i2c-core.c | 5 +
drivers/spi/spi.c| 18
that's the case, attach it to the ACPI power domain. In
addition we make sure that the device is fully powered when its ->probe()
function gets called.
For non-ACPI devices this patch is a no-op.
Signed-off-by: Lv Zheng
Signed-off-by: Mika Westerberg
Acked-by: Wolfram Sang
---
drivers
v_desc.
Reported-and-tested-by: Imre Kaloz
Signed-off-by: Mika Westerberg
Cc: sta...@vger.kernel.org # 3.10+
---
drivers/acpi/acpi_lpss.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
index 6a38218..fb78bb9 100644
--- a/driver
On Wed, Aug 28, 2013 at 11:38:58AM +0200, Wolfram Sang wrote:
> On Tue, Aug 20, 2013 at 05:03:35PM +0300, Mika Westerberg wrote:
> > From: Aaron Lu
> >
> > This patch adds runtime PM support for the I2C bus in a similar way that
> > has been done for PCI bus already.
On Fri, Sep 27, 2013 at 04:43:13AM +0200, Wolfram Sang wrote:
> On Fri, Sep 27, 2013 at 12:24:03AM +0200, Vincent Stehlé wrote:
> > Make sure i2c_dw_pci_runtime_idle() is defined only when actually used, when
> > CONFIG_PM_RUNTIME is defined.
> >
> > This fixes the following compilation warning:
>
| 4 +
> 7 files changed, 324 insertions(+), 205 deletions(-)
Just tested this series on my HP revolve 810 and this fixes the backlight
issue it had :)
Feel free to add my,
Tested-by: Mika Westerberg
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel"
are OK with this.
I didn't include changes to the SPI bus (the $subject is changed again to
reflect that) because we don't need to change the existing drivers anymore.
If this approach is accepted we can do the same for the SPI bus as well.
Mika Westerberg (1):
i2c: enable runti
addition to that we attach the I2C client devices to the ACPI power domain
and make sure that they are powered on when the driver ->probe() is called.
Non-ACPI devices are not affected by this change.
This patch is based on the work by Aaron Lu and Lv Zheng.
Signed-off-by: Mika Westerb
On Mon, Sep 30, 2013 at 07:20:59PM +0200, Rafael J. Wysocki wrote:
> > +static void acpi_i2c_device_pm_get(struct i2c_client *client)
> > +{
> > + struct i2c_adapter *adap = client->adapter;
> > +
> > + /* Make sure the adapter is active */
> > + if (ACPI_HANDLE(adap->dev.parent))
> > +
On Mon, Sep 30, 2013 at 05:43:48PM +0300, Mika Westerberg wrote:
> +static void acpi_i2c_device_pm_get(struct i2c_client *client)
> +{
> + struct i2c_adapter *adap = client->adapter;
> +
> + /* Make sure the adapter is active */
> + if (ACPI_HA
On Fri, Sep 20, 2013 at 08:40:48PM +0200, Linus Walleij wrote:
> On Wed, Sep 4, 2013 at 9:56 PM, Stephen Warren wrote:
> > On 09/04/2013 05:29 AM, Alexandre Courbot wrote:
> >> Add gpiod_get() and gpiod_put() functions that provide safer handling of
> >> GPIOs.
> >>
> >> These functions put the GP
On Fri, Sep 20, 2013 at 08:06:00PM +0200, Linus Walleij wrote:
> On Wed, Sep 4, 2013 at 1:29 PM, Alexandre Courbot wrote:
>
> > Here is a first RFC for the new GPIO interface.
>
> I'm quite happy with this, and given that all DT-implementations will start
> to use it from day 1 I'll happily merg
On Fri, Sep 20, 2013 at 09:21:37PM +0200, Linus Walleij wrote:
> On Fri, Sep 13, 2013 at 5:14 PM, Mika Westerberg
> wrote:
>
> > diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
> (...)
> > +st
On Fri, Sep 20, 2013 at 09:08:57PM +0200, Linus Walleij wrote:
> On Fri, Sep 13, 2013 at 5:14 PM, Mika Westerberg
> wrote:
>
> > Instead of asking each driver to register to the ACPI events we can just
> > call acpi_gpiochip_register_interrupts() for each chip that has ACPI
On Tue, Sep 24, 2013 at 12:47:56AM +, Zheng, Lv wrote:
> > From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com]
> > Sent: Tuesday, September 17, 2013 4:37 PM
> >
> > On Mon, Sep 16, 2013 at 11:35:56PM +, Zheng, Lv wrote:
> > > > From:
On Tue, Sep 17, 2013 at 01:07:37PM +0200, Sylwester Nawrocki wrote:
> On 09/16/2013 10:47 AM, Mika Westerberg wrote:
> > I'm actually thinking that it is probably better now if we don't touch the
> > client runtime PM at all in the I2C core.
> >
> > I propo
;
as this same check is already done at the beginning of
acpi_bus_register_driver().
I think these should all go via ACPI tree because they all depend on the
first patch which touches the ACPI subsystem.
Mika Westerberg (12):
ACPI: introduce module_acpi_driver() helper macro
ACPI/button
Add a helper macro module_acpi_driver() which reduces the boilerplate code
for ACPI drivers. This is similar what is done for other busses (PCI, SPI,
I2C etc).
Signed-off-by: Mika Westerberg
---
include/acpi/acpi_bus.h | 13 +
1 file changed, 13 insertions(+)
diff --git a/include
Signed-off-by: Mika Westerberg
---
drivers/acpi/fan.c | 22 +-
1 file changed, 1 insertion(+), 21 deletions(-)
diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c
index bc36a47..3bd6a54 100644
--- a/drivers/acpi/fan.c
+++ b/drivers/acpi/fan.c
@@ -212,24 +212,4 @@ static
Signed-off-by: Mika Westerberg
---
drivers/acpi/hed.c | 20 +---
1 file changed, 1 insertion(+), 19 deletions(-)
diff --git a/drivers/acpi/hed.c b/drivers/acpi/hed.c
index d0c1967..20a0f2c 100644
--- a/drivers/acpi/hed.c
+++ b/drivers/acpi/hed.c
@@ -86,25 +86,7 @@ static
Signed-off-by: Mika Westerberg
Acked-by: Ike Panhc
---
drivers/platform/x86/ideapad-laptop.c | 14 +-
1 file changed, 1 insertion(+), 13 deletions(-)
diff --git a/drivers/platform/x86/ideapad-laptop.c
b/drivers/platform/x86/ideapad-laptop.c
index dae7abe..5ff4f2e 100644
--- a
Signed-off-by: Mika Westerberg
---
drivers/acpi/button.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/acpi/button.c b/drivers/acpi/button.c
index 314a3b8..f0d936b 100644
--- a/drivers/acpi/button.c
+++ b/drivers/acpi/button.c
@@ -450,15 +450,4
Signed-off-by: Mika Westerberg
---
drivers/acpi/sbshc.c | 18 +-
1 file changed, 1 insertion(+), 17 deletions(-)
diff --git a/drivers/acpi/sbshc.c b/drivers/acpi/sbshc.c
index f8d2a47..cf6129a 100644
--- a/drivers/acpi/sbshc.c
+++ b/drivers/acpi/sbshc.c
@@ -310,23 +310,7
Signed-off-by: Mika Westerberg
Acked-by: Herton R. Krzesinski
---
drivers/platform/x86/topstar-laptop.c | 22 +-
1 file changed, 1 insertion(+), 21 deletions(-)
diff --git a/drivers/platform/x86/topstar-laptop.c
b/drivers/platform/x86/topstar-laptop.c
index d528daa
Signed-off-by: Mika Westerberg
Reviewed-by: Éric Piel
---
drivers/platform/x86/hp_accel.c | 25 +
1 file changed, 1 insertion(+), 24 deletions(-)
diff --git a/drivers/platform/x86/hp_accel.c b/drivers/platform/x86/hp_accel.c
index 6b9af98..18d74f2 100644
--- a/drivers
Signed-off-by: Mika Westerberg
---
drivers/platform/x86/toshiba_bluetooth.c | 22 +-
1 file changed, 1 insertion(+), 21 deletions(-)
diff --git a/drivers/platform/x86/toshiba_bluetooth.c
b/drivers/platform/x86/toshiba_bluetooth.c
index 5e5d631..e95be0b 100644
--- a
Signed-off-by: Mika Westerberg
---
drivers/platform/x86/xo15-ebook.c | 14 +-
1 file changed, 1 insertion(+), 13 deletions(-)
diff --git a/drivers/platform/x86/xo15-ebook.c
b/drivers/platform/x86/xo15-ebook.c
index 38ba39d..16d340c 100644
--- a/drivers/platform/x86/xo15-ebook.c
Signed-off-by: Mika Westerberg
Acked-by: Jean Delvare
---
drivers/i2c/busses/i2c-scmi.c | 14 +-
1 file changed, 1 insertion(+), 13 deletions(-)
diff --git a/drivers/i2c/busses/i2c-scmi.c b/drivers/i2c/busses/i2c-scmi.c
index 388cbdc..6aafa3d 100644
--- a/drivers/i2c/busses/i2c
Signed-off-by: Mika Westerberg
Acked-by: Dmitry Torokhov
---
drivers/input/misc/atlas_btns.c | 17 +
1 file changed, 1 insertion(+), 16 deletions(-)
diff --git a/drivers/input/misc/atlas_btns.c b/drivers/input/misc/atlas_btns.c
index 601f737..26f1313 100644
--- a/drivers
On Wed, Dec 05, 2012 at 12:08:12PM +0100, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki
>
> The header of acpi_dev_pm_detach() in include/linux/acpi.h has an
> incorrect return type, which should be void. Fix that.
>
> Signed-off-by: Rafael J. Wysocki
Reviewed-b
On Tue, Oct 16, 2012 at 09:23:23PM +0200, Linus Walleij wrote:
> Switch from creating the IRQ domain mapping to finding it. In this
> case we know very well that the driver has created the apropriate
> mapping, we just need to locate it, no need to create any
> on-the-fly mappings.
I may be missin
Add a document that describes how to take advantage of ACPI enumeration for
buses like platform, I2C and SPI. In addition to that we document how to
translate ACPI GpioIo and GpioInt resources to be useful in Linux device
drivers.
Signed-off-by: Mika Westerberg
---
Documentation/acpi
All devices behind Haswell LPSS (Low Power Subsystem) should be represented
as platform devices so add them to the acpi_platform_device_ids list.
Signed-off-by: Mika Westerberg
---
drivers/acpi/scan.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/drivers/acpi/scan.c b
On Fri, Dec 07, 2012 at 11:01:26AM +, Alan Cox wrote:
> On Fri, 7 Dec 2012 12:08:02 +0200
> Mika Westerberg wrote:
>
> > All devices behind Haswell LPSS (Low Power Subsystem) should be represented
> > as platform devices
>
> What about the bus heirarchy ?
On Mon, Nov 12, 2012 at 01:00:26PM +0100, Rafael J. Wysocki wrote:
> +/**
> + * acpi_dev_irq_flags - Determine IRQ resource flags.
> + * @triggering: Triggering type as provided by ACPI.
> + * @polarity: Interrupt polarity as provided by ACPI.
> + * @shareable: Whether or not the interrupt is share
On Mon, Nov 12, 2012 at 01:02:11PM +0100, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki
>
> Currently, whoever wants to use ACPI device resources has to call
> acpi_walk_resources() to browse the buffer returned by the _CRS
> method for the given device and create filters passed to that
> ro
On Mon, Nov 12, 2012 at 10:03:56PM +0100, Rafael J. Wysocki wrote:
> > > +static acpi_status acpi_bus_add_resource(struct acpi_resource *res,
> > > + void *context)
> > > +{
> > > + struct list_head *list = context;
> > > + struct acpi_resource_list_entry *entry;
>
On Tue, Nov 13, 2012 at 01:06:59PM +0100, Rafael J. Wysocki wrote:
> > I don't know any better option.
>
> Well, a better option would be to convince ACPICA to provide us different
> interfaces. :-)
Heh, indeed.
> We might actually try to do that in the future, but for now we still need
> _CRS t
On Tue, Nov 13, 2012 at 04:15:51PM +0100, Rafael J. Wysocki wrote:
> > Could we change this so that you can pass NULL list as well (provided that
> > the preproc is given)?
> >
> > This is useful in case when we try to find the SPI/I2C device handle based
> > on the ACPI serial bus resource addres
On Tue, Nov 13, 2012 at 04:28:50PM +0100, Rafael J. Wysocki wrote:
> On Tuesday, November 13, 2012 05:18:13 PM Mika Westerberg wrote:
> > On Tue, Nov 13, 2012 at 04:15:51PM +0100, Rafael J. Wysocki wrote:
> > > > Could we change this so that you can pass NULL list as well (p
itations I didn't realize before.
I've now tested this series on my ACPI 5 enabled test machine and
everything works fine - I can see the platform devices and that they have
the ACPI handle attached.
For the whole series,
Reviewed-by: Mika Westerberg
Tested-by: Mika Westerberg
(test
re is a dependency to linux-pm.git/linux-next so I think it would be
better if these will be merged via that tree.
Mathias Nyman (1):
gpio / ACPI: add ACPI support
Mika Westerberg (2):
spi / ACPI: add ACPI enumeration support
i2c / ACPI: add ACPI enumeration support
drivers/acpi/Kco
order to get its slave devices enumerated, created and bound to the
corresponding ACPI handle.
Signed-off-by: Mika Westerberg
---
drivers/acpi/Kconfig |6 ++
drivers/acpi/Makefile|1 +
drivers/acpi/acpi_i2c.c | 212 ++
drivers/i2c/i2c
for further configuration.
Signed-off-by: Mika Westerberg
---
drivers/spi/spi.c | 201 -
1 file changed, 200 insertions(+), 1 deletion(-)
diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c
index 84c2861..e39a484 100644
--- a/drivers/spi/spi.c
d-off-by: Mika Westerberg
---
drivers/gpio/Kconfig|4
drivers/gpio/Makefile |1 +
drivers/gpio/gpiolib-acpi.c | 56 +++
include/linux/acpi_gpio.h | 19 +++
4 files changed, 80 insertions(+)
create mode 100644 dr
On Mon, Nov 05, 2012 at 03:40:14PM +0100, Linus Walleij wrote:
> > I forgot to mention that we want to hook up _existing_ drivers to those
> > things,
> > and they already use the global GPIO numbers, don't they?
>
> Yes they do, usually this is either passed from the platform using platform
> da
evice name instead of
trusting that the BIOS sets the _UIDs correctly.
Signed-off-by: Mika Westerberg
Cc: Bjorn Helgaas
---
This was suggested by Bjorn Helgaas in another thread. The patch applies on
top of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git
linux-next.
drivers
On Tue, Nov 06, 2012 at 05:21:03PM +0100, Linus Walleij wrote:
> +static inline int
> +gpiochip_add_pin_range(struct gpio_chip *chip, const char *pinctl_name,
> +unsigned int pin_base, unsigned int npins)
> +{
> +}
Should you return some value above? Like -ENOSYS?
--
To unsubsc
On Tue, Nov 06, 2012 at 11:15:21AM +0100, Linus Walleij wrote:
> On Tue, Nov 6, 2012 at 10:39 AM, Mika Westerberg
> wrote:
> > On Mon, Nov 05, 2012 at 03:40:14PM +0100, Linus Walleij wrote:
> >> > I forgot to mention that we want to hook up _existing_ drivers to those
&
On Tue, Nov 06, 2012 at 11:18:11PM +0100, Rafael J. Wysocki wrote:
> > How is the SPI controller different than this? Is there some logical
> > difference that requires a different framework? Or are you proposing
> > that we get rid of acpi_bus_register_driver() and migrate everything
> > to this
101 - 200 of 2936 matches
Mail list logo
