Re: [PATCH] mfd: max8997: use regmap to access registers

2014-03-05 Thread Krzysztof Kozlowski
Hi, > > This patch modifies max8997 driver and each associated function driver, > to use regmap instead of operating directly on i2c bus. It will allow to > simplify IRQ handling using regmap-irq. > > Signed-off-by: Robert Baldyga (...) > diff --git a/drivers/mfd/max8997.c b/drivers/mfd/max89

Re: [PATCH 1/3] mfd: max8997: use regmap to access registers

2014-03-11 Thread Krzysztof Kozlowski
On Tue, 2014-03-11 at 14:58 +0100, Robert Baldyga wrote: > This patch modifies max8997 driver and each associated function driver, > to use regmap instead of operating directly on i2c bus. It will allow to > simplify IRQ handling using regmap-irq. > > Signed-off-by: Robert Baldyga > --- > driver

Re: [PATCH 2/3] mfd: max8997: handle IRQs using regmap

2014-03-11 Thread Krzysztof Kozlowski
On Tue, 2014-03-11 at 14:58 +0100, Robert Baldyga wrote: > This patch modifies mfd driver to use regmap for handling interrupts. > It allows to simplify irq handling process. This modifications needed > to make small changes in function drivers, which use interrupts. > > Signed-off-by: Robert Bald

Re: [PATCH 3/3] mfd: max8997: move regmap handling to function drivers

2014-03-11 Thread Krzysztof Kozlowski
On Tue, 2014-03-11 at 14:58 +0100, Robert Baldyga wrote: > This patch moves code creating new i2c clients and regmaps to function > drivers which uses them. It allow to avoid creating this instances when > intividual function drivers are not enabled. s/which uses/which use/ s/It allow/ It allows/

Re: [PATCH 1/3] mfd: max8997: use regmap to access registers

2014-03-11 Thread Krzysztof Kozlowski
On Tue, 2014-03-11 at 15:59 +0100, Robert Baldyga wrote: > >> @@ -180,12 +167,12 @@ static struct irq_chip max8997_irq_chip = { > >> static irqreturn_t max8997_irq_thread(int irq, void *data) { > >> struct max8997_dev *max8997 = data; - u8 > >> irq_reg[MAX8997_IRQ_GROUP_NR] = {}; - u8 ir

Re: [PATCH v2 1/3] mfd: max8997: use regmap to access registers

2014-03-12 Thread Krzysztof Kozlowski
inux/mfd/max8997-private.h | 17 ++--- > 10 files changed, 228 insertions(+), 249 deletions(-) Looks fine to me: Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majord..

Re: [PATCH v2 2/3] mfd: max8997: handle IRQs using regmap

2014-03-12 Thread Krzysztof Kozlowski
On Wed, 2014-03-12 at 14:37 +0100, Robert Baldyga wrote: > This patch modifies mfd driver to use regmap for handling interrupts. > It allows to simplify irq handling process. This modifications needed > to make small changes in function drivers, which use interrupts. > > Signed-off-by: Robert Ba

Re: [PATCH v2 3/3] mfd: max8997: move regmap handling to function drivers

2014-03-12 Thread Krzysztof Kozlowski
On Wed, 2014-03-12 at 14:37 +0100, Robert Baldyga wrote: > This patch moves code creating new i2c clients and regmaps to function > drivers which use them. It allows to avoid creating this instances when > individual function drivers are not enabled. > > Signed-off-by: Robert Baldyga > > Conflic

Re: [PATCH v3 2/4] mfd: max8997: handle IRQs using regmap

2014-03-13 Thread Krzysztof Kozlowski
s/mfd/max8997.c | 112 ++- > drivers/rtc/rtc-max8997.c |2 +- > include/linux/mfd/max8997-private.h | 63 +- > 7 files changed, 165 insertions(+), 392 deletions(-) > delete mode 100644 drivers/mfd/max8997-irq.c Reviewed-by: Krzysztof Kozlowski Best

Re: [PATCH v3 3/4] mfd: max8997: change irq names to upper case

2014-03-13 Thread Krzysztof Kozlowski
gt; --- > drivers/extcon/extcon-max8997.c | 32 > include/linux/mfd/max8997-private.h | 16 > 2 files changed, 24 insertions(+), 24 deletions(-) Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof Kozlowski -- To unsubscribe fr

Re: [PATCH v3 4/4] mfd: max8997: move regmap handling to function drivers

2014-03-13 Thread Krzysztof Kozlowski
78 +++- > include/linux/mfd/max8997-private.h | 14 +-- > 5 files changed, 284 insertions(+), 225 deletions(-) Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a me

Re: [PATCH v2 5/8] regulator: AXP20x: Add support for regulators subsystem

2014-03-17 Thread Krzysztof Kozlowski
> > AXP202 and AXP209 come with two synchronous step-down DC-DCs and five > LDOs. This patch introduces basic support for those regulators. > > Signed-off-by: Carlo Caione > --- > drivers/regulator/Kconfig| 7 + > drivers/regulator/Makefile | 1 + > drivers/regulator/

[PATCH v3 18/20] power_supply: bq2415x_charger: Decrement the power supply's device reference counter

2015-01-30 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter (increased by power_supply_get_by_name() or power_supply_get_by_phandle()). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Re

[PATCH v3 15/20] x86/olpc/xo1/sci: Use newly added power_supply_put API

2015-01-30 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo1-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v3 17/20] power_supply: 88pm860x_charger: Decrement the power supply's device reference counter

2015-01-30 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/88pm860x_charger.c | 11 +-- 1 file changed, 9 inser

[PATCH v3 16/20] x86/olpc/xo15/sci: Use newly added power_supply_put API

2015-01-30 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo15-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v3 12/20] power_supply: Add power_supply_put for decrementing device reference counter

2015-01-30 Thread Krzysztof Kozlowski
w API power_supply_put() for decrementing the reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/power_supply_core.c | 38 ++ include/linux/power_supply.h

[PATCH v3 08/20] power_supply: apm_power: Use power_supply_*() API for accessing function attrs

2015-01-30 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v3 14/20] power_supply: charger-manager: Decrement the power supply's device reference counter

2015-01-30 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/charger-manager.c

[PATCH v3 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-01-30 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/arm/mach-pxa/raumfeld.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --

[PATCH v3 13/20] power_supply: Increment power supply use counter when obtaining references

2015-01-30 Thread Krzysztof Kozlowski
power_supply_get_property-like functions even when driver unregisters this power supply. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz --- drivers/power/power_supply_core.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/drivers/power

[PATCH v3 19/20] mfd: ab8500: Decrement the power supply's device reference counter

2015-01-30 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Acked-by: Linus Walleij Acked-by: Lee Jones Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/mfd/ab8500-sysctrl.

[PATCH v3 09/20] power_supply: bq2415x_charger: Use power_supply_*() API for accessing function attrs

2015-01-30 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v3 10/20] power_supply: charger-manager: Use power_supply_*() API for accessing function attrs

2015-01-30 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v3 06/20] power_supply: ab8500: Use power_supply_*() API for accessing function attrs

2015-01-30 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v3 07/20] mfd: ab8500: Use power_supply_*() API for accessing function attrs

2015-01-30 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v3 05/20] power_supply: 88pm860x_charger: Use power_supply_*() API for accessing function attrs

2015-01-30 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property - set_property -> power_supply_set_property Signed-off-by: Krz

[PATCH v3 00/20] power_supply: Allow safe usage of power supply

2015-01-30 Thread Krzysztof Kozlowski
r. 4. Patch 3: Don't add wrapper for set_charged() because already exists one. [1] https://lkml.org/lkml/2014/11/4/527 [2] https://lkml.org/lkml/2014/10/16/89 [3] https://lkml.org/lkml/2015/1/21/471 Best regards, Krzysztof Krzysztof Kozlowski (20): power_supply: Add driver priva

[PATCH v3 04/20] power_supply: sysfs: Use power_supply_*() API for accessing function attrs

2015-01-30 Thread Krzysztof Kozlowski
teable -> power_supply_property_is_writeable Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/power_supply_sysfs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH v3 01/20] power_supply: Add driver private data

2015-01-30 Thread Krzysztof Kozlowski
re as a pointer, for example: struct some_driver_info { struct i2c_client *client; struct power_supply *power_supply; ... } In such case one cannot use container_of() and must store pointer to state container as private data. Signed-off-by: Krzysztof Kozlows

[PATCH v3 02/20] power_supply: Move run-time configuration to separate structure

2015-01-30 Thread Krzysztof Kozlowski
es of_node or supplies then it should allocate config on stack and initialize it with proper values. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz --- drivers/acpi/ac.c | 2 +- drivers/acpi/battery.c| 3 ++- drivers/acpi

[PATCH v3 03/20] power_supply: Add API for safe access of power supply function attrs

2015-01-30 Thread Krzysztof Kozlowski
ership of power supply class is still needed to fully fix race conditions in accessing freed power supply. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/power_supply_core.c | 47

Re: [PATCH v3 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-06 Thread Krzysztof Kozlowski
On pią, 2015-02-06 at 14:49 +0100, Pavel Machek wrote: > On Fri 2015-01-30 15:47:58, Krzysztof Kozlowski wrote: > > Use power_supply_put() to decrement the power supply's device reference > > counter. > > > > Signed-off-by: Krzysztof Kozlowski > >

Re: [PATCH v3 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-09 Thread Krzysztof Kozlowski
On pią, 2015-02-06 at 15:59 +0100, Pavel Machek wrote: > On Fri 2015-02-06 15:43:08, Krzysztof Kozlowski wrote: > > On pią, 2015-02-06 at 14:49 +0100, Pavel Machek wrote: > > > On Fri 2015-01-30 15:47:58, Krzysztof Kozlowski wrote: > > > > Use power_supply_put() t

Re: [PATCH v3 04/20] power_supply: sysfs: Use power_supply_*() API for accessing function attrs

2015-02-09 Thread Krzysztof Kozlowski
On pon, 2015-02-09 at 20:02 +0100, Stefan Wahren wrote: > Hi Krzysztof, > > > Krzysztof Kozlowski hat am 30. Januar 2015 um > > 15:47 > > geschrieben: > > > > > > Replace direct calls to power supply function attributes with wrappers. > > Wrapp

[PATCH v4 00/20] power_supply: Allow safe usage of power supply

2015-02-23 Thread Krzysztof Kozlowski
ps://lkml.org/lkml/2015/2/20/170 Best regards, Krzysztof Krzysztof Kozlowski (20): power_supply: Add driver private data power_supply: Move run-time configuration to separate structure power_supply: Add API for safe access of power supply function attrs power_supply: sysfs: Use power_supp

[PATCH v4 10/20] power_supply: charger-manager: Use power_supply_*() API for accessing function attrs

2015-02-23 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v4 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-23 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel Acked-by: Robert Jarzmik --- arch/arm/mach-pxa/raumfeld.c | 4 +++- 1 file changed, 3 insertions(

[PATCH v4 17/20] power_supply: 88pm860x_charger: Decrement the power supply's device reference counter

2015-02-23 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/88pm860x_charger.c | 11 +-- 1 file changed, 9 inser

[PATCH v4 19/20] mfd: ab8500: Decrement the power supply's device reference counter

2015-02-23 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Acked-by: Linus Walleij Acked-by: Lee Jones Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/mfd/ab8500-sysctrl.

[PATCH v4 16/20] x86/olpc/xo15/sci: Use newly added power_supply_put API

2015-02-23 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo15-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v4 18/20] power_supply: bq2415x_charger: Decrement the power supply's device reference counter

2015-02-23 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter (increased by power_supply_get_by_name() or power_supply_get_by_phandle()). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Re

[PATCH v4 13/20] power_supply: Increment power supply use counter when obtaining references

2015-02-23 Thread Krzysztof Kozlowski
power_supply_get_property-like functions even when driver unregisters this power supply. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz --- drivers/power/power_supply_core.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/drivers/power

[PATCH v4 15/20] x86/olpc/xo1/sci: Use newly added power_supply_put API

2015-02-23 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo1-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v4 14/20] power_supply: charger-manager: Decrement the power supply's device reference counter

2015-02-23 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/charger-manager.c

[PATCH v4 12/20] power_supply: Add power_supply_put for decrementing device reference counter

2015-02-23 Thread Krzysztof Kozlowski
w API power_supply_put() for decrementing the reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/power_supply_core.c | 38 ++ include/linux/power_supply.h

[PATCH v4 03/20] power_supply: Add API for safe access of power supply function attrs

2015-02-23 Thread Krzysztof Kozlowski
ership of power supply class is still needed to fully fix race conditions in accessing freed power supply. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel Acked-by: Pavel Machek --- drivers/power/po

[PATCH v4 08/20] power_supply: apm_power: Use power_supply_*() API for accessing function attrs

2015-02-23 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v4 09/20] power_supply: bq2415x_charger: Use power_supply_*() API for accessing function attrs

2015-02-23 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v4 07/20] mfd: ab8500: Use power_supply_*() API for accessing function attrs

2015-02-23 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v4 06/20] power_supply: ab8500: Use power_supply_*() API for accessing function attrs

2015-02-23 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v4 05/20] power_supply: 88pm860x_charger: Use power_supply_*() API for accessing function attrs

2015-02-23 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property - set_property -> power_supply_set_property Signed-off-by: Krz

[PATCH v4 01/20] power_supply: Add driver private data

2015-02-23 Thread Krzysztof Kozlowski
re as a pointer, for example: struct some_driver_info { struct i2c_client *client; struct power_supply *power_supply; ... } In such case one cannot use container_of() and must store pointer to state container as private data. Signed-off-by: Krzysztof Kozlows

[PATCH v4 04/20] power_supply: sysfs: Use power_supply_*() API for accessing function attrs

2015-02-23 Thread Krzysztof Kozlowski
teable -> power_supply_property_is_writeable Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/power_supply_sysfs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH v4 02/20] power_supply: Move run-time configuration to separate structure

2015-02-23 Thread Krzysztof Kozlowski
es of_node or supplies then it should allocate config on stack and initialize it with proper values. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Acked-by: Pavel Machek [for the nvec part] Reviewed-by: Marc Dietrich [for drivers/platform/x86/compal-lapt

Re: [PATCH v4 11/20] power_supply: Change ownership from driver to core

2015-02-26 Thread Krzysztof Kozlowski
On czw, 2015-02-26 at 01:45 +0100, Sebastian Reichel wrote: > Hi, > > On Mon, Feb 23, 2015 at 12:47:32PM +0100, Krzysztof Kozlowski wrote: > > Change the ownership of power_supply structure from each driver > > implementing the class to the power supply core. > >

[PATCH v5 00/20] power_supply: Allow safe usage of power supply

2015-02-27 Thread Krzysztof Kozlowski
/471 [2] https://lkml.org/lkml/2015/2/20/150 Best regards, Krzysztof Krzysztof Kozlowski (20): power_supply: Add driver private data power_supply: Move run-time configuration to separate structure power_supply: Add API for safe access of power supply function attrs power_supply: sysfs:

[PATCH v5 16/20] x86/olpc/xo15/sci: Use newly added power_supply_put API

2015-02-27 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo15-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v5 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel Acked-by: Robert Jarzmik --- arch/arm/mach-pxa/raumfeld.c | 4 +++- 1 file changed, 3 insertions(

[PATCH v5 19/20] mfd: ab8500: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Acked-by: Linus Walleij Acked-by: Lee Jones Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/mfd/ab8500-sysctrl.

[PATCH v5 17/20] power_supply: 88pm860x_charger: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/88pm860x_charger.c | 11 +-- 1 file changed, 9 inser

[PATCH v5 14/20] power_supply: charger-manager: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/charger-manager.c

[PATCH v5 18/20] power_supply: bq2415x_charger: Decrement the power supply's device reference counter

2015-02-27 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter (increased by power_supply_get_by_name() or power_supply_get_by_phandle()). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Re

[PATCH v5 15/20] x86/olpc/xo1/sci: Use newly added power_supply_put API

2015-02-27 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo1-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v5 12/20] power_supply: Add power_supply_put for decrementing device reference counter

2015-02-27 Thread Krzysztof Kozlowski
w API power_supply_put() for decrementing the reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/power_supply_core.c | 38 ++ include/linux/power_supply.h

[PATCH v5 13/20] power_supply: Increment power supply use counter when obtaining references

2015-02-27 Thread Krzysztof Kozlowski
power_supply_get_property-like functions even when driver unregisters this power supply. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz --- drivers/power/power_supply_core.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/drivers/power

[PATCH v5 05/20] power_supply: 88pm860x_charger: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property - set_property -> power_supply_set_property Signed-off-by: Krz

[PATCH v5 10/20] power_supply: charger-manager: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v5 09/20] power_supply: bq2415x_charger: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v5 04/20] power_supply: sysfs: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
teable -> power_supply_property_is_writeable Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/power_supply_sysfs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH v5 01/20] power_supply: Add driver private data

2015-02-27 Thread Krzysztof Kozlowski
re as a pointer, for example: struct some_driver_info { struct i2c_client *client; struct power_supply *power_supply; ... } In such case one cannot use container_of() and must store pointer to state container as private data. Signed-off-by: Krzysztof Kozlows

[PATCH v5 08/20] power_supply: apm_power: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v5 07/20] mfd: ab8500: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v5 06/20] power_supply: ab8500: Use power_supply_*() API for accessing function attrs

2015-02-27 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v5 02/20] power_supply: Move run-time configuration to separate structure

2015-02-27 Thread Krzysztof Kozlowski
es of_node or supplies then it should allocate config on stack and initialize it with proper values. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Acked-by: Pavel Machek [for the nvec part] Reviewed-by: Marc Dietrich [for drivers/platform/x86/compal-lapt

[PATCH v5 03/20] power_supply: Add API for safe access of power supply function attrs

2015-02-27 Thread Krzysztof Kozlowski
ership of power supply class is still needed to fully fix race conditions in accessing freed power supply. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel Acked-by: Pavel Machek --- drivers/power/po

Re: [PATCH v7 3/5] power: max77843_battery: Add Max77843 fuel gauge device driver

2015-03-09 Thread Krzysztof Kozlowski
2015-03-09 1:36 GMT+01:00 Beomho Seo : > On 03/08/2015 05:14 AM, Sebastian Reichel wrote: >> Hi, >> >> On Mon, Mar 02, 2015 at 07:10:36PM +0900, Jaewon Kim wrote: >>> From: Beomho Seo >>> >>> This patch adds device driver of max77843 fuel gauge. >>> The driver support for battery fuel gauge in Max

Re: [PATCH v7 2/5] power: max77843_charger: Add Max77843 charger device driver

2015-03-09 Thread Krzysztof Kozlowski
2015-03-09 1:35 GMT+01:00 Beomho Seo : > On 03/08/2015 05:13 AM, Sebastian Reichel wrote: >> On Mon, Mar 02, 2015 at 07:10:35PM +0900, Jaewon Kim wrote: >>> From: Beomho Seo >>> >>> This patch adds device driver of max77843 charger. This driver provide >>> initialize each charging mode(e.g. fast c

Re: [PATCH v7 2/5] power: max77843_charger: Add Max77843 charger device driver

2015-03-09 Thread Krzysztof Kozlowski
On pon, 2015-03-09 at 20:46 +0900, Beomho Seo wrote: > On 03/09/2015 08:02 PM, Krzysztof Kozlowski wrote: > > 2015-03-09 1:35 GMT+01:00 Beomho Seo : > >> On 03/08/2015 05:13 AM, Sebastian Reichel wrote: > >>> On Mon, Mar 02, 2015 at 07:10:35PM +0900, Jaewon Kim

[PATCH v6 01/22] compal-laptop: Fix leaking hwmon device

2015-03-10 Thread Krzysztof Kozlowski
l or probe failure. Signed-off-by: Krzysztof Kozlowski Fixes: c2be45f09bb0 ("compal-laptop: Use devm_hwmon_device_register_with_groups") Cc: Acked-by: Guenter Roeck Acked-by: Darren Hart --- Changes since previous submission: 1. Add Darren's ack. http://lkml.iu.edu/hypermail/

[PATCH v6 00/22] power_supply: Allow safe usage of power supply

2015-03-10 Thread Krzysztof Kozlowski
ed() because already exists one. [1] https://lkml.org/lkml/2015/1/21/471 [2] https://lkml.org/lkml/2015/2/20/150 Best regards, Krzysztof Krzysztof Kozlowski (22): compal-laptop: Fix leaking hwmon device compal-laptop: Check return value of power_supply_register power_supply: Add dri

[PATCH v6 15/22] power_supply: Increment power supply use counter when obtaining references

2015-03-10 Thread Krzysztof Kozlowski
power_supply_get_property-like functions even when driver unregisters this power supply. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz --- drivers/power/power_supply_core.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/drivers/power

[PATCH v6 21/22] mfd: ab8500: Decrement the power supply's device reference counter

2015-03-10 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Acked-by: Linus Walleij Acked-by: Lee Jones Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/mfd/ab8500-sysctrl.

[PATCH v6 19/22] power_supply: 88pm860x_charger: Decrement the power supply's device reference counter

2015-03-10 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/88pm860x_charger.c | 11 +-- 1 file changed, 9 inser

[PATCH v6 18/22] x86/olpc/xo15/sci: Use newly added power_supply_put API

2015-03-10 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo15-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v6 20/22] power_supply: bq2415x_charger: Decrement the power supply's device reference counter

2015-03-10 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter (increased by power_supply_get_by_name() or power_supply_get_by_phandle()). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Re

[PATCH v6 12/22] power_supply: charger-manager: Use power_supply_*() API for accessing function attrs

2015-03-10 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v6 17/22] x86/olpc/xo1/sci: Use newly added power_supply_put API

2015-03-10 Thread Krzysztof Kozlowski
Replace direct usage of put_device() with new API: power_supply_put(). Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- arch/x86/platform/olpc/olpc-xo1-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v6 16/22] power_supply: charger-manager: Decrement the power supply's device reference counter

2015-03-10 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/charger-manager.c

[PATCH v6 22/22] arm: mach-pxa: Decrement the power supply's device reference counter

2015-03-10 Thread Krzysztof Kozlowski
Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel Acked-by: Robert Jarzmik Acked-by: Pavel Machek --- arch/arm/mach-pxa/raumfeld.c | 4 +++- 1 file ch

[PATCH v6 06/22] power_supply: sysfs: Use power_supply_*() API for accessing function attrs

2015-03-10 Thread Krzysztof Kozlowski
teable -> power_supply_property_is_writeable Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/power_supply_sysfs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH v6 09/22] mfd: ab8500: Use power_supply_*() API for accessing function attrs

2015-03-10 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v6 14/22] power_supply: Add power_supply_put for decrementing device reference counter

2015-03-10 Thread Krzysztof Kozlowski
w API power_supply_put() for decrementing the reference counter. Signed-off-by: Krzysztof Kozlowski Acked-by: Pavel Machek Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel --- drivers/power/power_supply_core.c | 38 ++ include/linux/power_supply.h

[PATCH v6 03/22] power_supply: Add driver private data

2015-03-10 Thread Krzysztof Kozlowski
re as a pointer, for example: struct some_driver_info { struct i2c_client *client; struct power_supply *power_supply; ... } In such case one cannot use container_of() and must store pointer to state container as private data. Signed-off-by: Krzysztof Kozlows

[PATCH v6 10/22] power_supply: apm_power: Use power_supply_*() API for accessing function attrs

2015-03-10 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v6 02/22] compal-laptop: Check return value of power_supply_register

2015-03-10 Thread Krzysztof Kozlowski
power_supply_unregister() unconditionally cleans up given power supply. Fix this by checking return status of power_supply_register() call. In case of failure, clean up sysfs entries and fail the probe. Signed-off-by: Krzysztof Kozlowski Fixes: 9be0fcb5ed46 ("compal-laptop: add JHL90, battery & hwmon inter

[PATCH v6 07/22] power_supply: 88pm860x_charger: Use power_supply_*() API for accessing function attrs

2015-03-10 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property - set_property -> power_supply_set_property Signed-off-by: Krz

[PATCH v6 11/22] power_supply: bq2415x_charger: Use power_supply_*() API for accessing function attrs

2015-03-10 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v6 08/22] power_supply: ab8500: Use power_supply_*() API for accessing function attrs

2015-03-10 Thread Krzysztof Kozlowski
Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: - get_property -> power_supply_get_property Signed-off-by: Krzysztof Kozlowski Acked-by: Jonghwa Lee Acked

[PATCH v6 05/22] power_supply: Add API for safe access of power supply function attrs

2015-03-10 Thread Krzysztof Kozlowski
ership of power supply class is still needed to fully fix race conditions in accessing freed power supply. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Reviewed-by: Sebastian Reichel Acked-by: Pavel Machek --- drivers/power/po

  1   2   3   >