Re: [PATCH 20/22] pstore: %pF is only for function pointers

2015-06-16 Thread Anton Vorontsov
On Tue, Jun 16, 2015 at 03:02:50PM -0700, Kees Cook wrote: > On Wed, Mar 11, 2015 at 8:13 PM, Scott Wood wrote: > > Use %pS for actual addresses, otherwise you'll get bad output > > on arches like ppc64 where %pF expects a function descriptor. > > > > Signed-off-by: Scott Wood ... > > -

Re: [PATCH 20/22] pstore: %pF is only for function pointers

2015-06-16 Thread Anton Vorontsov
On Tue, Jun 16, 2015 at 03:02:50PM -0700, Kees Cook wrote: On Wed, Mar 11, 2015 at 8:13 PM, Scott Wood scottw...@freescale.com wrote: Use %pS for actual addresses, otherwise you'll get bad output on arches like ppc64 where %pF expects a function descriptor. Signed-off-by: Scott Wood

CNS3xxx maintainer (was Re: master build: 0 failures 28 warnings (v3.17-rc4-158-ge874a5f))

2014-09-10 Thread Anton Vorontsov
On Wed, Sep 10, 2014 at 10:01:43AM +0200, Arnd Bergmann wrote: [...] > > --- > > arm-allmodconfig : PASS, 0 errors, 18 warnings, 0 section mismatches > > > > Warnings: > > ../arch/arm/mach-cns3xxx/pcie.c:311:1:

CNS3xxx maintainer (was Re: master build: 0 failures 28 warnings (v3.17-rc4-158-ge874a5f))

2014-09-10 Thread Anton Vorontsov
On Wed, Sep 10, 2014 at 10:01:43AM +0200, Arnd Bergmann wrote: [...] --- arm-allmodconfig : PASS, 0 errors, 18 warnings, 0 section mismatches Warnings: ../arch/arm/mach-cns3xxx/pcie.c:311:1: warning: the

Re: [RESEND PATCH v3 2/5] charger: tps65090: Allow charger module to be used when no irq

2014-05-05 Thread Anton Vorontsov
wer supply maintainership due to the lack of time. By agreement with him and David, pick up power supply tree. Signed-off-by: Dmitry Eremin-Solenikov Acked-by: Anton Vorontsov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message

Re: [RESEND PATCH v3 2/5] charger: tps65090: Allow charger module to be used when no irq

2014-05-05 Thread Anton Vorontsov
to the lack of time. By agreement with him and David, pick up power supply tree. Signed-off-by: Dmitry Eremin-Solenikov dbarysh...@gmail.com Acked-by: Anton Vorontsov an...@enomsg.org -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message

Re: [PATCH v3 4/6] timerfd: Factor out timer-type unspecific timerfd_expire()

2014-02-20 Thread Anton Vorontsov
On Thu, Feb 20, 2014 at 11:52:03AM +0100, Thomas Gleixner wrote: > > From: Anton Vorontsov > > > > There is nothing hrtimer-specific inside the timerfd_tmrproc(), except > > the function prototype. We're about to add other timer types, so factor > > out generi

Re: [PATCH v3 5/6] timerfd: Add support for deferrable timers

2014-02-20 Thread Anton Vorontsov
On Thu, Feb 20, 2014 at 12:09:43PM +0100, Thomas Gleixner wrote: > On Thu, 20 Feb 2014, Alexey Perevalov wrote: > > From: Anton Vorontsov > > > > This patch implements a userland-side API for generic deferrable timers, > > per linux/timer.h: > > > > * A d

Re: [PATCH v3 5/6] timerfd: Add support for deferrable timers

2014-02-20 Thread Anton Vorontsov
On Thu, Feb 20, 2014 at 12:09:43PM +0100, Thomas Gleixner wrote: On Thu, 20 Feb 2014, Alexey Perevalov wrote: From: Anton Vorontsov an...@enomsg.org This patch implements a userland-side API for generic deferrable timers, per linux/timer.h: * A deferrable timer will work normally

Re: [PATCH v3 4/6] timerfd: Factor out timer-type unspecific timerfd_expire()

2014-02-20 Thread Anton Vorontsov
On Thu, Feb 20, 2014 at 11:52:03AM +0100, Thomas Gleixner wrote: From: Anton Vorontsov an...@enomsg.org There is nothing hrtimer-specific inside the timerfd_tmrproc(), except the function prototype. We're about to add other timer types, so factor out generic timerfd_expire() helper from

Re: [PATCH v2 1/3] power_supply: Add power_supply notifier

2014-01-03 Thread Anton Vorontsov
On Fri, Jan 03, 2014 at 11:09:49AM +, Tc, Jenny wrote: > Anton, > > I don't see this patch in Linux tree. Any update on this would be helpful http://git.infradead.org/battery-2.6.git/commit/d36240d26025bec95f3499e2401a56db98d9f01c ?.. -- To unsubscribe from this list: send the line

[GIT PULL] battery-2.6.git

2014-01-03 Thread Anton Vorontsov
Hello Linus, Please pull battery-2.6 git tree to receive two fixes prepared for the v3.13 release: - Fix build error caused by max17042_battery conversion to the regmap API. - Fix kernel oops when booting with wakeup_source_activate enabled. Thank you! Anton The following changes since

[GIT PULL] battery-2.6.git

2014-01-03 Thread Anton Vorontsov
Hello Linus, Please pull battery-2.6 git tree to receive two fixes prepared for the v3.13 release: - Fix build error caused by max17042_battery conversion to the regmap API. - Fix kernel oops when booting with wakeup_source_activate enabled. Thank you! Anton The following changes since

Re: [PATCH v2 1/3] power_supply: Add power_supply notifier

2014-01-03 Thread Anton Vorontsov
On Fri, Jan 03, 2014 at 11:09:49AM +, Tc, Jenny wrote: Anton, I don't see this patch in Linux tree. Any update on this would be helpful http://git.infradead.org/battery-2.6.git/commit/d36240d26025bec95f3499e2401a56db98d9f01c ?.. -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH] gpio-charger: support wakeup events

2013-12-23 Thread Anton Vorontsov
On Wed, Dec 11, 2013 at 02:47:16AM +0400, Dmitry Eremin-Solenikov wrote: > Add support for using gpio-charger IRQ as a wakeup event. > > Signed-off-by: Dmitry Eremin-Solenikov > --- Applied, thanks! Anton -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of

Re: [PATCH v7 1/3] charger: max14577: Add charger support for Maxim 14577

2013-12-23 Thread Anton Vorontsov
On Fri, Dec 06, 2013 at 12:32:12PM +0100, Krzysztof Kozlowski wrote: > MAX14577 chip is a multi-function device which includes MUIC, charger > and voltage regulator. The driver is located in drivers/mfd. > > This patch supports battery charging control of MAX14577 chip and > provides power supply

Re: [PATCHv2 0/2] DT support for isp1704-charger

2013-12-23 Thread Anton Vorontsov
On Sun, Dec 01, 2013 at 02:00:09AM +0100, Sebastian Reichel wrote: > This is the second iteration of the isp1704 DT patches. > > Changes since v1: > * reword the binding documentation slightly according >to the suggestions of Mark Rutland > * keep supporting the set_power callback and leave

Re: [PATCH 1/3] power_supply: add power_supply_get_by_phandle

2013-12-23 Thread Anton Vorontsov
On Sun, Nov 24, 2013 at 05:49:29PM +0100, Sebastian Reichel wrote: > Add method to get power supply by device tree phandle. > > Signed-off-by: Sebastian Reichel > --- > drivers/power/power_supply_core.c | 24 > include/linux/power_supply.h | 2 ++ > 2 files

Re: [PATCH 0/3] DT support for bq2415x

2013-12-23 Thread Anton Vorontsov
On Sun, Nov 24, 2013 at 05:49:28PM +0100, Sebastian Reichel wrote: > This patchset adds DT support for the bq2415x charger, which is used in the > Nokia N900. The changes depend on Pali Rohár's "[PATCH v2 0/3] Add support for > charging battery in Nokia RX-51" patchset [0]. Patches 1/3 and 2/3

Re: [PATCH v2 2/3] bq2415x_charger: Use power_supply notifier for automode

2013-12-23 Thread Anton Vorontsov
On Tue, Nov 19, 2013 at 02:24:16PM +0100, Pavel Machek wrote: > On Tue 2013-11-19 11:18:04, Pali Rohár wrote: > > This patch removing set_mode_hook function from board data and replacing it > > with > > new string variable of notifier power supply device. After this change it is > > possible to

Re: [PATCH V2 2/2] power: reset: as3722: add power-off driver

2013-12-23 Thread Anton Vorontsov
On Fri, Dec 20, 2013 at 06:54:00PM +0530, Laxman Dewangan wrote: > ams AS3722 supports the power off functionality to turn off > system. > > Add power off driver for ams AS3722. > > Signed-off-by: Laxman Dewangan > Tested-by: Stephen Warren > --- > Changes from V1: > - Just added tested by.

Re: [PATCH V2 2/2] power: reset: as3722: add power-off driver

2013-12-23 Thread Anton Vorontsov
On Fri, Dec 20, 2013 at 06:54:00PM +0530, Laxman Dewangan wrote: ams AS3722 supports the power off functionality to turn off system. Add power off driver for ams AS3722. Signed-off-by: Laxman Dewangan ldewan...@nvidia.com Tested-by: Stephen Warren swar...@nvidia.com --- Changes from V1:

Re: [PATCH v2 2/3] bq2415x_charger: Use power_supply notifier for automode

2013-12-23 Thread Anton Vorontsov
On Tue, Nov 19, 2013 at 02:24:16PM +0100, Pavel Machek wrote: On Tue 2013-11-19 11:18:04, Pali Rohár wrote: This patch removing set_mode_hook function from board data and replacing it with new string variable of notifier power supply device. After this change it is possible to add DT

Re: [PATCH 1/3] power_supply: add power_supply_get_by_phandle

2013-12-23 Thread Anton Vorontsov
On Sun, Nov 24, 2013 at 05:49:29PM +0100, Sebastian Reichel wrote: Add method to get power supply by device tree phandle. Signed-off-by: Sebastian Reichel s...@debian.org --- drivers/power/power_supply_core.c | 24 include/linux/power_supply.h | 2 ++ 2

Re: [PATCH 0/3] DT support for bq2415x

2013-12-23 Thread Anton Vorontsov
On Sun, Nov 24, 2013 at 05:49:28PM +0100, Sebastian Reichel wrote: This patchset adds DT support for the bq2415x charger, which is used in the Nokia N900. The changes depend on Pali Rohár's [PATCH v2 0/3] Add support for charging battery in Nokia RX-51 patchset [0]. Patches 1/3 and 2/3

Re: [PATCHv2 0/2] DT support for isp1704-charger

2013-12-23 Thread Anton Vorontsov
On Sun, Dec 01, 2013 at 02:00:09AM +0100, Sebastian Reichel wrote: This is the second iteration of the isp1704 DT patches. Changes since v1: * reword the binding documentation slightly according to the suggestions of Mark Rutland * keep supporting the set_power callback and leave the

Re: [PATCH v7 1/3] charger: max14577: Add charger support for Maxim 14577

2013-12-23 Thread Anton Vorontsov
On Fri, Dec 06, 2013 at 12:32:12PM +0100, Krzysztof Kozlowski wrote: MAX14577 chip is a multi-function device which includes MUIC, charger and voltage regulator. The driver is located in drivers/mfd. This patch supports battery charging control of MAX14577 chip and provides power supply

Re: [PATCH] gpio-charger: support wakeup events

2013-12-23 Thread Anton Vorontsov
On Wed, Dec 11, 2013 at 02:47:16AM +0400, Dmitry Eremin-Solenikov wrote: Add support for using gpio-charger IRQ as a wakeup event. Signed-off-by: Dmitry Eremin-Solenikov dbarysh...@gmail.com --- Applied, thanks! Anton -- To unsubscribe from this list: send the line unsubscribe linux-kernel

Re: [PATCH v2 2/3] bq2415x_charger: Use power_supply notifier for automode

2013-12-01 Thread Anton Vorontsov
On Mon, Dec 02, 2013 at 01:02:40AM +0100, Michael Trimarchi wrote: > On Sun, Dec 1, 2013 at 11:37 PM, Anton Vorontsov wrote: > > On Mon, Nov 25, 2013 at 08:16:34PM +0100, Michael Trimarchi wrote: > > ... > >> >> So you can read this value witho

Re: [PATCH v2 2/3] bq2415x_charger: Use power_supply notifier for automode

2013-12-01 Thread Anton Vorontsov
On Mon, Nov 25, 2013 at 08:16:34PM +0100, Michael Trimarchi wrote: ... > >> So you can read this value without any type of synchronization > >> with the power_supply_core > >> and sysfs implementation? ... > https://lists.ubuntu.com/archives/kernel-team/2013-January/025206.html > > I found and

Re: [PATCH] max17042: Fix build errors caused by missing REGMAP_I2C config

2013-12-01 Thread Anton Vorontsov
On Mon, Nov 25, 2013 at 09:40:04AM +0900, jonghwa3@samsung.com wrote: > > max17042 now uses regmap interface but does not enable config option. This > > patch fixes the following build errors: > > > > drivers/power/max17042_battery.c:661:15: error: variable > > ‘max17042_regmap_config’ has

Re: [PATCH v2 1/3] power_supply: Add power_supply notifier

2013-12-01 Thread Anton Vorontsov
On Tue, Nov 19, 2013 at 11:18:03AM +0100, Pali Rohár wrote: > This patch adds a notifier chain to the power_supply. > This notifier helps drivers in other subsystem to listen to > changes in power supply subsystem. This would help to take some > actions in those drivers on changing the power

Re: [PATCH v2 1/3] power_supply: Add power_supply notifier

2013-12-01 Thread Anton Vorontsov
On Wed, Nov 27, 2013 at 05:23:34PM +, Tc, Jenny wrote: > > > Subject: [PATCH v2 1/3] power_supply: Add power_supply notifier > > > > This patch adds a notifier chain to the power_supply. > > This notifier helps drivers in other subsystem to listen to changes in > > power supply > >

Re: [PATCH v3 1/2] power_supply: Fix Oops from NULL pointer dereference from wakeup_source_activate

2013-12-01 Thread Anton Vorontsov
On Fri, Nov 22, 2013 at 10:54:28AM -0700, Shuah Khan wrote: > power_supply_register() calls device_init_wakeup() to register a wakeup > source before initializing dev_name. As a result, device_wakeup_enable() > end up registering wakeup source with a null name when > wakeup_source_register() >

Re: [PATCH v3 1/2] power_supply: Fix Oops from NULL pointer dereference from wakeup_source_activate

2013-12-01 Thread Anton Vorontsov
On Fri, Nov 22, 2013 at 10:54:28AM -0700, Shuah Khan wrote: power_supply_register() calls device_init_wakeup() to register a wakeup source before initializing dev_name. As a result, device_wakeup_enable() end up registering wakeup source with a null name when wakeup_source_register() gets

Re: [PATCH v2 1/3] power_supply: Add power_supply notifier

2013-12-01 Thread Anton Vorontsov
On Wed, Nov 27, 2013 at 05:23:34PM +, Tc, Jenny wrote: Subject: [PATCH v2 1/3] power_supply: Add power_supply notifier This patch adds a notifier chain to the power_supply. This notifier helps drivers in other subsystem to listen to changes in power supply subsystem. This would

Re: [PATCH v2 1/3] power_supply: Add power_supply notifier

2013-12-01 Thread Anton Vorontsov
On Tue, Nov 19, 2013 at 11:18:03AM +0100, Pali Rohár wrote: This patch adds a notifier chain to the power_supply. This notifier helps drivers in other subsystem to listen to changes in power supply subsystem. This would help to take some actions in those drivers on changing the power supply

Re: [PATCH] max17042: Fix build errors caused by missing REGMAP_I2C config

2013-12-01 Thread Anton Vorontsov
On Mon, Nov 25, 2013 at 09:40:04AM +0900, jonghwa3@samsung.com wrote: max17042 now uses regmap interface but does not enable config option. This patch fixes the following build errors: drivers/power/max17042_battery.c:661:15: error: variable ‘max17042_regmap_config’ has

Re: [PATCH v2 2/3] bq2415x_charger: Use power_supply notifier for automode

2013-12-01 Thread Anton Vorontsov
On Mon, Nov 25, 2013 at 08:16:34PM +0100, Michael Trimarchi wrote: ... So you can read this value without any type of synchronization with the power_supply_core and sysfs implementation? ... https://lists.ubuntu.com/archives/kernel-team/2013-January/025206.html I found and equivalent

Re: [PATCH v2 2/3] bq2415x_charger: Use power_supply notifier for automode

2013-12-01 Thread Anton Vorontsov
On Mon, Dec 02, 2013 at 01:02:40AM +0100, Michael Trimarchi wrote: On Sun, Dec 1, 2013 at 11:37 PM, Anton Vorontsov an...@enomsg.org wrote: On Mon, Nov 25, 2013 at 08:16:34PM +0100, Michael Trimarchi wrote: ... So you can read this value without any type of synchronization

[GIT PULL] battery-2.6.git

2013-11-17 Thread Anton Vorontsov
unneeded NULL checks (2013-11-12 22:36:34 -0800) Alexandre Belloni (1): bq2415x_charger: Fix max battery regulation voltage Anton Vorontsov (1): power_supply: Fix documentation for TEMP_*ALERT* properties Dan Carpenter (1

[GIT PULL] battery-2.6.git

2013-11-17 Thread Anton Vorontsov
unneeded NULL checks (2013-11-12 22:36:34 -0800) Alexandre Belloni (1): bq2415x_charger: Fix max battery regulation voltage Anton Vorontsov (1): power_supply: Fix documentation for TEMP_*ALERT* properties Dan Carpenter (1

Re: [patch] pm2301-charger: remove unneeded NULL checks

2013-11-12 Thread Anton Vorontsov
On Thu, Nov 07, 2013 at 11:06:17AM +0300, Dan Carpenter wrote: > If "pm2" were NULL we would oops printing the error message. > Fortunately, that's not possible so I have removed the NULL checks. > > Signed-off-by: Dan Carpenter Applied, thanks! -- To unsubscribe from this list: send the line

Re: [patch] pm2301-charger: remove unneeded NULL checks

2013-11-12 Thread Anton Vorontsov
On Thu, Nov 07, 2013 at 11:06:17AM +0300, Dan Carpenter wrote: If pm2 were NULL we would oops printing the error message. Fortunately, that's not possible so I have removed the NULL checks. Signed-off-by: Dan Carpenter dan.carpen...@oracle.com Applied, thanks! -- To unsubscribe from this

Re: [PATCH] power:power_supply_syfs : Treat PROP_TYPE as a regular attribute first

2013-10-28 Thread Anton Vorontsov
On Sat, Oct 26, 2013 at 02:01:22AM +0300, Philippe De Swert wrote: ... > I think I did not make myself very clear. Well there are actually no > different chargers here. It is one USB charger input, however its > properties are different depending on the actual connected "charger > device/type". It

Re: N900 DT

2013-10-28 Thread Anton Vorontsov
On Mon, Oct 28, 2013 at 03:01:35AM +, Tc, Jenny wrote: > > On Saturday 26 October 2013 02:25:02 Sebastian Reichel wrote: > > > On Fri, Oct 25, 2013 at 08:39:40PM +0200, Pali Rohár wrote: > > > > Now I found this patch and it looks like it will be in mainline > > > > kernel. And after that it

Re: [PATCH 7/7] power_supply: Introduce PSE compliant algorithm

2013-10-28 Thread Anton Vorontsov
On Mon, Sep 23, 2013 at 11:34:05PM +0530, Jenny TC wrote: [...] > +#define BATTID_STR_LEN 8 > +#define BATT_TEMP_NR_RNG 6 > +/* Charging Profile */ > +struct psy_ps_pse_mod_prof { > + /* battery id */ > + char batt_id[BATTID_STR_LEN]; > + /* type of battery */ > +

Re: [PATCH 1/7] power_supply: Add charger control properties

2013-10-28 Thread Anton Vorontsov
On Mon, Oct 28, 2013 at 03:36:36AM +, Tc, Jenny wrote: > > But do we really want to control the chargers through the power_supply's > > user-visible > > interface? It makes the whole power supply thing so complicated that I'm > > already losing > > track of it. Right now I think I would

Re: [PATCH 1/7] power_supply: Add charger control properties

2013-10-28 Thread Anton Vorontsov
On Mon, Oct 28, 2013 at 03:36:36AM +, Tc, Jenny wrote: But do we really want to control the chargers through the power_supply's user-visible interface? It makes the whole power supply thing so complicated that I'm already losing track of it. Right now I think I would prefer to move

Re: [PATCH 7/7] power_supply: Introduce PSE compliant algorithm

2013-10-28 Thread Anton Vorontsov
On Mon, Sep 23, 2013 at 11:34:05PM +0530, Jenny TC wrote: [...] +#define BATTID_STR_LEN 8 +#define BATT_TEMP_NR_RNG 6 +/* Charging Profile */ +struct psy_ps_pse_mod_prof { + /* battery id */ + char batt_id[BATTID_STR_LEN]; + /* type of battery */ + u16

Re: N900 DT

2013-10-28 Thread Anton Vorontsov
On Mon, Oct 28, 2013 at 03:01:35AM +, Tc, Jenny wrote: On Saturday 26 October 2013 02:25:02 Sebastian Reichel wrote: On Fri, Oct 25, 2013 at 08:39:40PM +0200, Pali Rohár wrote: Now I found this patch and it looks like it will be in mainline kernel. And after that it could be

Re: [PATCH] power:power_supply_syfs : Treat PROP_TYPE as a regular attribute first

2013-10-28 Thread Anton Vorontsov
On Sat, Oct 26, 2013 at 02:01:22AM +0300, Philippe De Swert wrote: ... I think I did not make myself very clear. Well there are actually no different chargers here. It is one USB charger input, however its properties are different depending on the actual connected charger device/type. It is

Re: [PATCH 1/7] power_supply: Add charger control properties

2013-10-27 Thread Anton Vorontsov
Hello Jenny, Thanks a lot for your work on this! On Mon, Sep 23, 2013 at 11:33:59PM +0530, Jenny TC wrote: > The battery charger needs to have control path along > with the reporting charger properties. In existing solutions > this is implemented using regulator framework. A regulator >

Re: [PATCH 1/7] power_supply: Add charger control properties

2013-10-27 Thread Anton Vorontsov
Hello Jenny, Thanks a lot for your work on this! On Mon, Sep 23, 2013 at 11:33:59PM +0530, Jenny TC wrote: The battery charger needs to have control path along with the reporting charger properties. In existing solutions this is implemented using regulator framework. A regulator framework

Re: [PATCH] Fuel Guague: MAX17040: Use regmap to interface with internal registers

2013-10-25 Thread Anton Vorontsov
On Mon, Oct 21, 2013 at 05:48:35PM +0900, �� wrote: > We use regmap to interface with internal registers in fuel guague MAX17040. > > Signed-off-by: Nam KwanWoo > Signed-off-by: Kyungmin Park > --- $ git am -s patch fatal: cannot convert from ks_c_5601-1987 to UTF-8 I fixed this manually,

Re: [PATCH] max17042_battery: use SIMPLE_DEV_PM_OPS

2013-10-25 Thread Anton Vorontsov
On Mon, Oct 14, 2013 at 10:56:51AM +0530, Manish Badarkhe wrote: > Use the SIMPLE_DEV_PM_OPS macro to declare the driver's > pm_ops. > > Signed-off-by: Manish Badarkhe Applied, thanks! Anton -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH] bq2415x_charger: fix max battery regulation voltage

2013-10-25 Thread Anton Vorontsov
On Wed, Oct 16, 2013 at 04:09:40PM +0200, Alexandre Belloni wrote: > I forgot to add this is a v2 and I just added a comment. > > On 16/10/2013 16:08, Alexandre Belloni wrote: > > As per the datasheets, maximum battery regulation voltage is 4440mV. > > > > The formula is (voltage - offset) /

Re: [PATCH] tps65090-charger: Use "IS_ENABLED(CONFIG_OF)" for DT code.

2013-10-25 Thread Anton Vorontsov
On Mon, Sep 30, 2013 at 12:03:40PM -0400, Rhyland Klein wrote: > On 9/28/2013 12:32 AM, Manish Badarkhe wrote: > > Instead of "#if defined(CONFIG_OF)" use "IS_ENABLED(CONFIG_OF)" option > > for DT code to avoid if-deffery in code. > > Also, arranged header files in alphabetically. > > > >

Re: [Patch V5] drivers: power: Add support for bq24735 charger

2013-10-25 Thread Anton Vorontsov
On Fri, Oct 25, 2013 at 03:48:41PM -0700, Anton Vorontsov wrote: > On Mon, Oct 14, 2013 at 12:40:26PM -0600, Stephen Warren wrote: > > >>> diff --git > > >>> a/Documentation/devicetree/bindings/power_supply/ti,bq24735.txt > > >>> b/Documentation

Re: [Patch V5] drivers: power: Add support for bq24735 charger

2013-10-25 Thread Anton Vorontsov
On Mon, Oct 14, 2013 at 12:40:26PM -0600, Stephen Warren wrote: > >>> diff --git > >>> a/Documentation/devicetree/bindings/power_supply/ti,bq24735.txt > >>> b/Documentation/devicetree/bindings/power_supply/ti,bq24735.txt > >> > >>> +Optional properties : > >> > >>> + - ti,ac-detect-gpios : This

Re: [PATCH v3 1/4] ab8500-charger: Check return value of regulator_enable

2013-10-25 Thread Anton Vorontsov
On Fri, Sep 06, 2013 at 10:38:00AM +0100, Lee Jones wrote: > On Fri, 06 Sep 2013, Sachin Kamat wrote: > > > Check the return value of regulator_enable to silence the following > > type of warnings: > > drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value > > of

Re: [PATCH] power:power_supply_syfs : Treat PROP_TYPE as a regular attribute first

2013-10-25 Thread Anton Vorontsov
On Thu, Aug 15, 2013 at 02:09:58AM +0300, Philippe De Swert wrote: > These days we often have USB powered devices. This means that often the > type is variable. Common examples are smartphones which can be charged > through a normal USB port on a PC/laptop, a dedicated charger, etc... > Often

Re: [PATCH] power:power_supply_syfs : Treat PROP_TYPE as a regular attribute first

2013-10-25 Thread Anton Vorontsov
On Thu, Aug 15, 2013 at 02:09:58AM +0300, Philippe De Swert wrote: These days we often have USB powered devices. This means that often the type is variable. Common examples are smartphones which can be charged through a normal USB port on a PC/laptop, a dedicated charger, etc... Often those

Re: [PATCH v3 1/4] ab8500-charger: Check return value of regulator_enable

2013-10-25 Thread Anton Vorontsov
On Fri, Sep 06, 2013 at 10:38:00AM +0100, Lee Jones wrote: On Fri, 06 Sep 2013, Sachin Kamat wrote: Check the return value of regulator_enable to silence the following type of warnings: drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value of ‘regulator_enable’,

Re: [Patch V5] drivers: power: Add support for bq24735 charger

2013-10-25 Thread Anton Vorontsov
On Mon, Oct 14, 2013 at 12:40:26PM -0600, Stephen Warren wrote: diff --git a/Documentation/devicetree/bindings/power_supply/ti,bq24735.txt b/Documentation/devicetree/bindings/power_supply/ti,bq24735.txt +Optional properties : + - ti,ac-detect-gpios : This GPIO is optionally used to

Re: [Patch V5] drivers: power: Add support for bq24735 charger

2013-10-25 Thread Anton Vorontsov
On Fri, Oct 25, 2013 at 03:48:41PM -0700, Anton Vorontsov wrote: On Mon, Oct 14, 2013 at 12:40:26PM -0600, Stephen Warren wrote: diff --git a/Documentation/devicetree/bindings/power_supply/ti,bq24735.txt b/Documentation/devicetree/bindings/power_supply/ti,bq24735.txt +Optional

Re: [PATCH] tps65090-charger: Use IS_ENABLED(CONFIG_OF) for DT code.

2013-10-25 Thread Anton Vorontsov
On Mon, Sep 30, 2013 at 12:03:40PM -0400, Rhyland Klein wrote: On 9/28/2013 12:32 AM, Manish Badarkhe wrote: Instead of #if defined(CONFIG_OF) use IS_ENABLED(CONFIG_OF) option for DT code to avoid if-deffery in code. Also, arranged header files in alphabetically. Signed-off-by: Manish

Re: [PATCH] bq2415x_charger: fix max battery regulation voltage

2013-10-25 Thread Anton Vorontsov
On Wed, Oct 16, 2013 at 04:09:40PM +0200, Alexandre Belloni wrote: I forgot to add this is a v2 and I just added a comment. On 16/10/2013 16:08, Alexandre Belloni wrote: As per the datasheets, maximum battery regulation voltage is 4440mV. The formula is (voltage - offset) / step, so the

Re: [PATCH] max17042_battery: use SIMPLE_DEV_PM_OPS

2013-10-25 Thread Anton Vorontsov
On Mon, Oct 14, 2013 at 10:56:51AM +0530, Manish Badarkhe wrote: Use the SIMPLE_DEV_PM_OPS macro to declare the driver's pm_ops. Signed-off-by: Manish Badarkhe badarkhe.man...@gmail.com Applied, thanks! Anton -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the

Re: [PATCH] Fuel Guague: MAX17040: Use regmap to interface with internal registers

2013-10-25 Thread Anton Vorontsov
On Mon, Oct 21, 2013 at 05:48:35PM +0900, �� wrote: We use regmap to interface with internal registers in fuel guague MAX17040. Signed-off-by: Nam KwanWoo kw46@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- $ git am -s patch fatal: cannot convert from

Re: [PATCH 2/4] power: isp1704_charger: Fix driver to work with changes introduced in v3.5

2013-10-22 Thread Anton Vorontsov
On Sun, Sep 08, 2013 at 10:50:37AM +0200, Pali Rohár wrote: > * omap musb driver does not report USB_EVENT_ENUMERATED event anymore > * omap musb driver reporting USB_EVENT_VBUS when charger is connected > * read last event from phy->last_event (instead from ulpi register) > * do not call wall

Re: [PATCH 2/4] power: isp1704_charger: Fix driver to work with changes introduced in v3.5

2013-10-22 Thread Anton Vorontsov
On Sun, Sep 08, 2013 at 10:50:37AM +0200, Pali Rohár wrote: * omap musb driver does not report USB_EVENT_ENUMERATED event anymore * omap musb driver reporting USB_EVENT_VBUS when charger is connected * read last event from phy-last_event (instead from ulpi register) * do not call wall charger

Re: [patch] mm, vmpressure: add high level

2013-10-16 Thread Anton Vorontsov
Hello David, On Wed, Oct 16, 2013 at 05:43:55PM -0700, David Rientjes wrote: > Vmpressure has two important levels: medium and critical. Medium is > defined at 60% and critical is defined at 95%. > > We have a customer who needs a notification at a higher level than medium, > which is slight

Re: [patch] mm, vmpressure: add high level

2013-10-16 Thread Anton Vorontsov
Hello David, On Wed, Oct 16, 2013 at 05:43:55PM -0700, David Rientjes wrote: Vmpressure has two important levels: medium and critical. Medium is defined at 60% and critical is defined at 95%. We have a customer who needs a notification at a higher level than medium, which is slight to

Re: [PATCH] vmpressure: fix divide-by-0 in vmpressure_work_fn

2013-09-11 Thread Anton Vorontsov
On Wed, Sep 11, 2013 at 06:03:57PM +0200, Michal Hocko wrote: > The patch below. I find it little bit nicer than Hugh's original one > because having the two checks sounds more confusing. > What do you think Hugh, Anton? Acked-by: Anton Vorontsov Thanks! >

Re: [PATCH] vmpressure: fix divide-by-0 in vmpressure_work_fn

2013-09-11 Thread Anton Vorontsov
On Mon, Sep 09, 2013 at 01:08:47PM +0200, Michal Hocko wrote: > On Fri 06-09-13 22:59:16, Hugh Dickins wrote: > > Hit divide-by-0 in vmpressure_work_fn(): checking vmpr->scanned before > > taking the lock is not enough, we must check scanned afterwards too. > > As vmpressure_work_fn seems the be

Re: [PATCH] vmpressure: fix divide-by-0 in vmpressure_work_fn

2013-09-11 Thread Anton Vorontsov
On Mon, Sep 09, 2013 at 01:08:47PM +0200, Michal Hocko wrote: On Fri 06-09-13 22:59:16, Hugh Dickins wrote: Hit divide-by-0 in vmpressure_work_fn(): checking vmpr-scanned before taking the lock is not enough, we must check scanned afterwards too. As vmpressure_work_fn seems the be the only

Re: [PATCH] vmpressure: fix divide-by-0 in vmpressure_work_fn

2013-09-11 Thread Anton Vorontsov
On Wed, Sep 11, 2013 at 06:03:57PM +0200, Michal Hocko wrote: The patch below. I find it little bit nicer than Hugh's original one because having the two checks sounds more confusing. What do you think Hugh, Anton? Acked-by: Anton Vorontsov an...@enomsg.org Thanks! --- From

Re: [PATCH] vmpressure: fix divide-by-0 in vmpressure_work_fn

2013-09-10 Thread Anton Vorontsov
On Fri, Sep 06, 2013 at 10:59:16PM -0700, Hugh Dickins wrote: > Hit divide-by-0 in vmpressure_work_fn(): checking vmpr->scanned before > taking the lock is not enough, we must check scanned afterwards too. > > Signed-off-by: Hugh Dickins > Cc: sta...@vger.kernel.org Hm... Just trying to

[GIT PULL] battery-2.6.git

2013-09-10 Thread Anton Vorontsov
) Abhimanyu Kapur (1): power: reset: Add msm restart support Andrea Adami (2): power supply: collie_battery: Convert to use dev_pm_ops power_supply: tosa_battery: Get rid of irq_to_gpio usage Anton Vorontsov (1): bq24190_charger

[GIT PULL] battery-2.6.git

2013-09-10 Thread Anton Vorontsov
) Abhimanyu Kapur (1): power: reset: Add msm restart support Andrea Adami (2): power supply: collie_battery: Convert to use dev_pm_ops power_supply: tosa_battery: Get rid of irq_to_gpio usage Anton Vorontsov (1): bq24190_charger

Re: [PATCH] vmpressure: fix divide-by-0 in vmpressure_work_fn

2013-09-10 Thread Anton Vorontsov
On Fri, Sep 06, 2013 at 10:59:16PM -0700, Hugh Dickins wrote: Hit divide-by-0 in vmpressure_work_fn(): checking vmpr-scanned before taking the lock is not enough, we must check scanned afterwards too. Signed-off-by: Hugh Dickins hu...@google.com Cc: sta...@vger.kernel.org Hm... Just trying

Re: [PATCH] power: rx51_battery: Replace hardcoded channels values.

2013-08-30 Thread Anton Vorontsov
> This issue was introduced in commit: > power: rx51_battery: Replace hardcoded channels values. > > Original code use channel as argument which was shifted by one in function. > After mentioned commit argument is already shifted so we need to get index > back. > > Signed-off-by: Marek Belisko

Re: [PATCH] power: rx51_battery: Replace hardcoded channels values.

2013-08-30 Thread Anton Vorontsov
This issue was introduced in commit: power: rx51_battery: Replace hardcoded channels values. Original code use channel as argument which was shifted by one in function. After mentioned commit argument is already shifted so we need to get index back. Signed-off-by: Marek Belisko

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Anton Vorontsov
On Wed, Aug 28, 2013 at 07:07:14PM -0700, 'Greg Kroah-Hartman' wrote: > On Wed, Aug 28, 2013 at 06:18:49PM -0700, Anton Vorontsov wrote: > > On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: > > > He doesn't want to take the patch. He's the maintainer so it'

[PATCH] of/irq: Provide struct device_node forward-declaration for !OF case

2013-08-28 Thread Anton Vorontsov
: its scope is only this definition or declaration, which is probably not what you want [enabled by default] include/linux/of_irq.h:87:118: warning: ‘struct device_node’ declared inside parameter list [enabled by default] Signed-off-by: Anton Vorontsov --- include/linux/of_irq.h | 3 +++ 1

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Anton Vorontsov
On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: > He doesn't want to take the patch. He's the maintainer so it's his > choice. That's the end of the story. Just to clarify: I don't want to take the patch for a reason, not just because of my mood today. Once the patch comes in

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Anton Vorontsov
On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end of the story. Just to clarify: I don't want to take the patch for a reason, not just because of my mood today. Once the patch comes in

[PATCH] of/irq: Provide struct device_node forward-declaration for !OF case

2013-08-28 Thread Anton Vorontsov
: its scope is only this definition or declaration, which is probably not what you want [enabled by default] include/linux/of_irq.h:87:118: warning: ‘struct device_node’ declared inside parameter list [enabled by default] Signed-off-by: Anton Vorontsov an...@enomsg.org --- include/linux

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Anton Vorontsov
On Wed, Aug 28, 2013 at 07:07:14PM -0700, 'Greg Kroah-Hartman' wrote: On Wed, Aug 28, 2013 at 06:18:49PM -0700, Anton Vorontsov wrote: On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-27 Thread Anton Vorontsov
ch? > > > > CC'ed Joe Perches, Dan Carpenter > > > > Hi Anton Vorontsov, > > > > Usually, using the wrapper function makes the code simpler. > > Also, it make the code more readable. > > Since people are asking my opinion, then yes using > dev_get_

Re: [RFC PATCH] pm: prevent suspend until power supply events are processed

2013-08-27 Thread Anton Vorontsov
s guarantees that the screen lights up and displays > the battery charging state. The implementation uses the power > supply wakeup_source object. > > Cc: Anton Vorontsov > Cc: David Woodhouse > Cc: Arve Hjonnevag > Cc: Todd Poynor > Cc: John Stu

Re: [PATCH] power/reset: Make vexpress driver depend on required infrastructure

2013-08-27 Thread Anton Vorontsov
On Thu, Aug 15, 2013 at 04:35:52PM +0100, Pawel Moll wrote: > ARM Versatile Express reset driver requires platform-specific > config infrastructure to be present in the kernel. When > VEXPRESS_CONFIG is not selected, the build will fail like this: > > drivers/built-in.o: In function

Re: [PATCH] power: twl4030-charger: Fix compiler warning with regulator_enable()

2013-08-27 Thread Anton Vorontsov
On Wed, Aug 21, 2013 at 11:31:37AM +0300, Peter Ujfalusi wrote: > The return value of regulator_enable need to be checked. This patch fixes > the following warning: > drivers/power/twl4030_charger.c: In function ‘twl4030_charger_enable_usb’: > drivers/power/twl4030_charger.c:192:20: warning:

Re: [PATCH] power: rx51_battery: Replace hardcoded channels values.

2013-08-27 Thread Anton Vorontsov
On Thu, Aug 22, 2013 at 12:45:10AM +0200, Marek Belisko wrote: > In twl4030_madc header exist defines for fixed channels > + add rx51 specific channels and replace all hardcoded channels > values. > > Signed-off-by: Marek Belisko Applied, thanks! Anton -- To unsubscribe from this list: send

Re: [PATCH v2] bq24190_charger: Add support for TI BQ24190 Battery Charger

2013-08-27 Thread Anton Vorontsov
On Fri, Aug 23, 2013 at 07:21:03PM -0700, Mark A. Greer wrote: > Add driver support for the Texas Instruments BQ24190 > battery charger. Some of the information provided by > the device is about the charger and other information > is about the battery so create two power_supply objects > (one for

Re: [PATCH] power: max8925: fix missing of_node_put

2013-08-27 Thread Anton Vorontsov
On Mon, Aug 26, 2013 at 03:06:36PM +0800, Libo Chen wrote: > > decrease np device_node refcount after task completion > > Signed-off-by: Libo Chen Applied, thanks! Anton -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH] power: max8925: fix missing of_node_put

2013-08-27 Thread Anton Vorontsov
On Mon, Aug 26, 2013 at 03:06:36PM +0800, Libo Chen wrote: decrease np device_node refcount after task completion Signed-off-by: Libo Chen libo.c...@huawei.com Applied, thanks! Anton -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: [PATCH v2] bq24190_charger: Add support for TI BQ24190 Battery Charger

2013-08-27 Thread Anton Vorontsov
On Fri, Aug 23, 2013 at 07:21:03PM -0700, Mark A. Greer wrote: Add driver support for the Texas Instruments BQ24190 battery charger. Some of the information provided by the device is about the charger and other information is about the battery so create two power_supply objects (one for

Re: [PATCH] power: rx51_battery: Replace hardcoded channels values.

2013-08-27 Thread Anton Vorontsov
On Thu, Aug 22, 2013 at 12:45:10AM +0200, Marek Belisko wrote: In twl4030_madc header exist defines for fixed channels + add rx51 specific channels and replace all hardcoded channels values. Signed-off-by: Marek Belisko marek.beli...@open-nandra.com Applied, thanks! Anton -- To unsubscribe

  1   2   3   4   5   6   7   8   9   10   >