Re: [PATCH v3] power_supply: Add driver for TWL4030/TPS65950 BCI charger

2010-09-27 Thread Arun Murthy
On Mon, Sep 27, 2010 at 1:05 AM, Grazvydas Ignotas nota...@gmail.com wrote: TWL4030/TPS65950 is a multi-function device with integrated charger, which allows charging from AC or USB. This driver enables the charger and provides several monitoring functions. Tested on OMAP3 Pandora board.

Re: [PATCH v3] power_supply: Add driver for TWL4030/TPS65950 BCI charger

2010-09-27 Thread Arun Murthy
On Mon, Sep 27, 2010 at 4:24 PM, Grazvydas Ignotas nota...@gmail.com wrote: On Mon, Sep 27, 2010 at 10:21 AM, Arun Murthy arunrmurthy...@gmail.com wrote: On Mon, Sep 27, 2010 at 1:05 AM, Grazvydas Ignotas nota...@gmail.com wrote: --- Only AC and USB monitoring is achieved by registering

Re: [PATCH v3] power_supply: Add driver for TWL4030/TPS65950 BCI charger

2010-09-27 Thread Arun Murthy
On Mon, Sep 27, 2010 at 6:38 PM, Grazvydas Ignotas nota...@gmail.com wrote: On Mon, Sep 27, 2010 at 3:35 PM, Arun Murthy arunrmurthy...@gmail.com wrote: On Mon, Sep 27, 2010 at 4:24 PM, Grazvydas Ignotas nota...@gmail.com wrote: On Mon, Sep 27, 2010 at 10:21 AM, Arun Murthy arunrmurthy

[PATCH 0/7] PWM core driver for pwm based led and backlight driver

2010-09-28 Thread Arun Murthy
The series of patch add a new pwm core driver. Align the existing pwm drivers to make use of the pwm core driver. Arun Murthy (7): pwm: Add pwm core driver backlight:pwm: add an element 'name' to platform data leds: pwm: add a new element 'name' to platform data pwm: Align existing pwm

[PATCH 3/7] leds: pwm: add a new element 'name' to platform data

2010-09-28 Thread Arun Murthy
A new element 'name' is added to pwm led platform data structure. This is required to identify the pwm device. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/leds/leds-pwm.c |4 +++- include/linux/leds_pwm.h |1 + 2

[PATCH 2/7] backlight:pwm: add an element 'name' to platform data

2010-09-28 Thread Arun Murthy
A new element 'name' is added to pwm backlight platform data structure. This is required to identify the pwm device. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/video/backlight/pwm_bl.c |4 +++- include/linux

[PATCH 4/7] pwm: Align existing pwm drivers with pwm-core driver

2010-09-28 Thread Arun Murthy
driver with pwm core driver mips-jz4740: pwm: Align with new pwm core driver PWM core driver has been added and has been enabled only for ARM platform. The same can be utilised for mips also. Please align with the pwm core driver(drivers/pwm-core.c). Signed-off-by: Arun Murthy arun.mur

[PATCH 6/7] pwm: move existing pwm driver to drivers/pwm

2010-09-28 Thread Arun Murthy
As of now only ab8500 and twl6030 are moved. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/mfd/Kconfig |9 - drivers/mfd/Makefile |1 - drivers/misc

[PATCH 7/7] pwm: Modify backlight and led Kconfig aligning to pwm core

2010-09-28 Thread Arun Murthy
PWM based backlight and led driver will not be calling the pwm drivers through the pwm core driver and hence adding dependancy on the same. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/leds/Kconfig|2

[PATCH 5/7] platform: Update the pwm based led and backlight platform data

2010-09-28 Thread Arun Murthy
mxc-pwm: Update the platform data with pwm name for backlight s3c24xx-pwm: update platform data for backlight with pwm name Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- arch/arm/mach-pxa/cm-x300.c

RE: [PATCH 3/7] leds: pwm: add a new element 'name' to platform data

2010-09-28 Thread Arun MURTHY
@@ -16,6 +16,7 @@ struct led_pwm { struct led_pwm_platform_data { int num_leds; struct led_pwm *leds; + char *name; }; Shouldn't '*name'be aligned, at least with '*leds'? Sure, will take care of this in the v2 patch. Thanks and Regards, Arun R

RE: [PATCH 1/7] pwm: Add pwm core driver

2010-09-28 Thread Arun MURTHY
Let me be more specific, consider an environment where there is an APE and Power Management subsystem(separate IC but on same board/platform) APE has a pwm module and Power Management SubSystem also has pwm module. Both are part of the platform. Not there exists two drivers in a single

RE: [PATCH 1/7] pwm: Add pwm core driver

2010-09-28 Thread Arun MURTHY
Arun MURTHY wrote: Shouldn't PWM_DEVICES select HAVE_PWM? No not required, the entire concept is to remove HAVE_PWM and use PWM_CORE. Well in patch 4 you say that PWM_CORE is currently limited to ARM. Furthermore you change the pwm-backlight and pwm-led Kconfig entries to depend

[PATCHv2 2/7] backlight:pwm: add an element 'name' to platform data

2010-10-05 Thread Arun Murthy
A new element 'name' is added to pwm backlight platform data structure. This is required to identify the pwm device. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/video/backlight/pwm_bl.c |4 +++- include/linux

[PATCHv2 0/7] PWM core driver for pwm based led and backlight driver

2010-10-05 Thread Arun Murthy
PWM core driver for pwm based led and backlight driver. The intention of the pwm core driver is not to break the build if two or more pwm drivers are enabled. Align the existing pwm drivers to make use of the pwm core driver Arun Murthy (7): pwm: Add pwm core driver backlight:pwm: add

[PATCHv2 4/7] pwm: Align existing pwm drivers with pwm-core driver

2010-10-05 Thread Arun Murthy
driver with pwm core driver mips-jz4740: pwm: Align with new pwm core driver PWM core driver has been added and has been enabled only for ARM platform. The same can be utilised for mips also. Please align with the pwm core driver(drivers/pwm-core.c). Signed-off-by: Arun Murthy arun.mur

[PATCHv2 1/7] pwm: Add pwm core driver

2010-10-05 Thread Arun Murthy
, pwm_enable, pwm_disable). The clients of pwm device will have to call pwm_request, wherein they will get the pointer to struct pwm_ops. This structure include function pointers for pwm_config, pwm_enable and pwm_disable. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij

[PATCHv2 7/7] pwm: Modify backlight and led Kconfig aligning to pwm core

2010-10-05 Thread Arun Murthy
PWM based backlight and led driver will not be calling the pwm drivers through the pwm core driver and hence adding dependancy on the same. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/leds/Kconfig|2

[PATCHv2 3/7] leds: pwm: add a new element 'name' to platform data

2010-10-05 Thread Arun Murthy
A new element 'name' is added to pwm led platform data structure. This is required to identify the pwm device. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/leds/leds-pwm.c |4 +++- include/linux/leds_pwm.h |3

[PATCHv2 6/7] pwm: move existing pwm driver to drivers/pwm

2010-10-05 Thread Arun Murthy
As of now only ab8500 and twl6030 are moved. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/mfd/Kconfig |9 -- drivers/mfd/Makefile |1 - drivers/mfd/twl6030-pwm.c | 196

[PATCHv2 5/7] platform: Update the pwm based led and backlight platform data

2010-10-05 Thread Arun Murthy
mxc-pwm: Update the platform data with pwm name for backlight s3c24xx-pwm: update platform data for backlight with pwm name Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- arch/arm/mach-pxa/cm-x300.c

RE: [PATCHv2 0/7] PWM core driver for pwm based led and backlight driver

2010-10-05 Thread Arun MURTHY
Arun Murthy arun.mur...@stericsson.com writes: PWM core driver for pwm based led and backlight driver. The intention of the pwm core driver is not to break the build if two or more pwm drivers are enabled. Align the existing pwm drivers to make use of the pwm core driver Hi Arun

RE: [PATCHv2 1/7] pwm: Add pwm core driver

2010-10-05 Thread Arun MURTHY
On Tue, 5 Oct 2010 17:29:56 +0530 Arun Murthy arun.mur...@stericsson.com wrote: The existing pwm based led and backlight driver makes use of the pwm(include/linux/pwm.h). So all the board specific pwm drivers will be exposing the same set of function name as in include/linux/pwm.h

[PATCHv3 1/7] pwm: Add pwm core driver

2010-10-06 Thread Arun Murthy
, pwm_enable, pwm_disable). The clients of pwm device will have to call pwm_request, wherein they will get the pointer to struct pwm_ops. This structure include function pointers for pwm_config, pwm_enable and pwm_disable. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij

[PATCHv3 3/7] leds: pwm: add a new element 'name' to platform data

2010-10-06 Thread Arun Murthy
A new element 'name' is added to pwm led platform data structure. This is required to identify the pwm device. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/leds/leds-pwm.c |4 +++- include/linux/leds_pwm.h |3

[PATCHv3 4/7] pwm: Align existing pwm drivers with pwm-core driver

2010-10-06 Thread Arun Murthy
with pwm core driver mips-jz4740: pwm: Align with new pwm core driver PWM core driver has been added and has been enabled only for ARM platform. The same can be utilised for mips also. Please align with the pwm core driver(drivers/pwm-core.c). Signed-off-by: Arun Murthy arun.mur

[PATCHv3 2/7] backlight:pwm: add an element 'name' to platform data

2010-10-06 Thread Arun Murthy
A new element 'name' is added to pwm backlight platform data structure. This is required to identify the pwm device. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/video/backlight/pwm_bl.c |4 +++- include/linux

[PATCHv3 6/7] pwm: move existing pwm driver to drivers/pwm

2010-10-06 Thread Arun Murthy
As of now only ab8500 and twl6030 are moved. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/mfd/Kconfig |9 -- drivers/mfd/Makefile |1 - drivers/mfd/twl6030-pwm.c | 195

[PATCHv3 7/7] pwm: Modify backlight and led Kconfig aligning to pwm core

2010-10-06 Thread Arun Murthy
PWM based backlight and led driver will not be calling the pwm drivers through the pwm core driver and hence adding dependancy on the same. Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- drivers/leds/Kconfig|2

[PATCHv3 0/7] PWM core driver for pwm based led and backlight driver

2010-10-06 Thread Arun Murthy
(ref: pwm based led and backlight driver). I have provided a patch(backlight: add low threshold to pwm backlight) for handling this in the pwm backlight (one such client) which is now in Andrew's mm tree. TODO: Align Atmel pwm driver with my pwm core driver patch set. Arun

[PATCHv3 5/7] platform: Update the pwm based led and backlight platform data

2010-10-06 Thread Arun Murthy
mxc-pwm: Update the platform data with pwm name for backlight s3c24xx-pwm: update platform data for backlight with pwm name Signed-off-by: Arun Murthy arun.mur...@stericsson.com Acked-by: Linus Walleij linus.wall...@stericsson.com --- arch/arm/mach-pxa/cm-x300.c