* [PATCH 3/7] leds-lm3530: add 'is_vin_always_on' in the lm3530_platform_data
@ 2012-01-20 1:48 ` Kim, Milo
2012-01-23 19:13 ` Linus Walleij
0 siblings, 1 reply; 4+ messages in thread
From: Kim, Milo @ 2012-01-20 1:48 UTC (permalink / raw)
To: Linus Walleij, shreshthakumar.sahu@stericsson.com
Cc: linux-kernel@vger.kernel.org, rpurdie@rpsys.net
The 'IN' pin(Input voltage connection) can be always turned on
in case it is connected with VBATT.
To support this case, 'is_vin_always_on' is added.
If VIN is always on, then we don't need to control the regulator for IN pin.
Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
diff --git a/drivers/leds/leds-lm3530.c b/drivers/leds/leds-lm3530.c
index 963a1db..ba2bb7e 100644
--- a/drivers/leds/leds-lm3530.c
+++ b/drivers/leds/leds-lm3530.c
@@ -200,7 +200,7 @@ static int lm3530_init_registers(struct lm3530_data *drvdata)
reg_val[13] = LM3530_DEF_ZT_3; /* LM3530_ALS_Z3T_REG */
reg_val[14] = LM3530_DEF_ZT_4; /* LM3530_ALS_Z4T_REG */
- if (!drvdata->enable) {
+ if (!drvdata->enable && !pltfm->is_vin_always_on) {
ret = regulator_enable(drvdata->regulator);
if (ret) {
dev_err(&drvdata->client->dev,
@@ -256,7 +256,7 @@ static void lm3530_brightness_set(struct led_classdev *led_cdev,
else
drvdata->brightness = brt_val / 2;
- if (brt_val == 0) {
+ if (brt_val == 0 && !pdata->is_vin_always_on) {
err = regulator_disable(drvdata->regulator);
if (err)
dev_err(&drvdata->client->dev,
@@ -340,18 +340,20 @@ static int __devinit lm3530_probe(struct i2c_client *client,
drvdata->client = client;
drvdata->pdata = pdata;
drvdata->brightness = LED_OFF;
- drvdata->enable = false;
+ drvdata->enable = pdata->is_vin_always_on;
drvdata->led_dev.name = pdata->name ? pdata->name : LM3530_LED_DEV;
drvdata->led_dev.brightness_set = lm3530_brightness_set;
i2c_set_clientdata(client, drvdata);
- drvdata->regulator = regulator_get(&client->dev, "vin");
- if (IS_ERR(drvdata->regulator)) {
- dev_err(&client->dev, "regulator get failed\n");
- err = PTR_ERR(drvdata->regulator);
- drvdata->regulator = NULL;
- goto err_regulator_get;
+ if (!drvdata->enable) {
+ drvdata->regulator = regulator_get(&client->dev, "vin");
+ if (IS_ERR(drvdata->regulator)) {
+ dev_err(&client->dev, "regulator get failed\n");
+ err = PTR_ERR(drvdata->regulator);
+ drvdata->regulator = NULL;
+ goto err_regulator_get;
+ }
}
if (drvdata->pdata->brt_val) {
@@ -394,10 +396,11 @@ err_out:
static int __devexit lm3530_remove(struct i2c_client *client)
{
struct lm3530_data *drvdata = i2c_get_clientdata(client);
+ struct lm3530_platform_data *pdata = drvdata->pdata;
device_remove_file(drvdata->led_dev.dev, &dev_attr_mode);
- if (drvdata->enable)
+ if (drvdata->enable && !pdata->is_vin_always_on)
regulator_disable(drvdata->regulator);
regulator_put(drvdata->regulator);
led_classdev_unregister(&drvdata->led_dev);
diff --git a/include/linux/led-lm3530.h b/include/linux/led-lm3530.h
index 513e9c3..0240204 100644
--- a/include/linux/led-lm3530.h
+++ b/include/linux/led-lm3530.h
@@ -92,6 +92,8 @@ struct lm3530_pwm_data {
* @als2_resistor_sel: internal resistance from ALS2 input to ground
* @brt_val: brightness value (0-255)
* @name: led device name
+ * @is_vin_always_on: set to true if VIN is always on.
+ * ex) VIN is connected with VBATT
* @pwm_data: PWM control functions. only valid when the mode is PWM.
*/
struct lm3530_platform_data {
@@ -112,6 +114,7 @@ struct lm3530_platform_data {
u8 brt_val;
const char *name;
+ bool is_vin_always_on;
struct lm3530_pwm_data pwm_data;
};
--
1.7.4.1
Best Regards,
Milo (Woogyom) Kim
Texas Instruments Incorporated
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 3/7] leds-lm3530: add 'is_vin_always_on' in the lm3530_platform_data
2012-01-20 1:48 ` [PATCH 3/7] leds-lm3530: add 'is_vin_always_on' in the lm3530_platform_data Kim, Milo
@ 2012-01-23 19:13 ` Linus Walleij
2012-01-23 19:31 ` Mark Brown
0 siblings, 1 reply; 4+ messages in thread
From: Linus Walleij @ 2012-01-23 19:13 UTC (permalink / raw)
To: Kim, Milo, Mark Brown
Cc: shreshthakumar.sahu@stericsson.com, linux-kernel@vger.kernel.org,
rpurdie@rpsys.net
On Fri, Jan 20, 2012 at 2:48 AM, Kim, Milo <Milo.Kim@ti.com> wrote:
> The 'IN' pin(Input voltage connection) can be always turned on
> in case it is connected with VBATT.
> To support this case, 'is_vin_always_on' is added.
> If VIN is always on, then we don't need to control the regulator for IN pin.
>
> Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
(...)
> + if (!drvdata->enable && !pltfm->is_vin_always_on) {
> ret = regulator_enable(drvdata->regulator);
> if (ret) {
> dev_err(&drvdata->client->dev,
> @@ -256,7 +256,7 @@ static void lm3530_brightness_set(struct led_classdev *led_cdev,
> else
> drvdata->brightness = brt_val / 2;
>
> - if (brt_val == 0) {
> + if (brt_val == 0 && !pdata->is_vin_always_on) {
> err = regulator_disable(drvdata->regulator);
NACK on this, and paging Mark.
As far as I know we want to get rid of different design patterns
avoiding the regulator interface. The way forward for this is
either:
(A) Provide a fixed voltage regulator in your platform for this
device (AFAIK the voltage does not even have to be correct)
(B) Enable dummy regulators in your platform
(A) is more elegant.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 3/7] leds-lm3530: add 'is_vin_always_on' in the lm3530_platform_data
2012-01-23 19:13 ` Linus Walleij
@ 2012-01-23 19:31 ` Mark Brown
2012-01-25 1:57 ` Kim, Milo
0 siblings, 1 reply; 4+ messages in thread
From: Mark Brown @ 2012-01-23 19:31 UTC (permalink / raw)
To: Linus Walleij
Cc: Kim, Milo, shreshthakumar.sahu@stericsson.com,
linux-kernel@vger.kernel.org, rpurdie@rpsys.net
On Mon, Jan 23, 2012 at 08:13:18PM +0100, Linus Walleij wrote:
> As far as I know we want to get rid of different design patterns
> avoiding the regulator interface. The way forward for this is
> either:
> (A) Provide a fixed voltage regulator in your platform for this
> device (AFAIK the voltage does not even have to be correct)
> (B) Enable dummy regulators in your platform
> (A) is more elegant.
Yeah, regulator usage should essentially never be conditional at a
driver level unless there's actual optional supplies in the hardware.
The situation has never been any different to that.
^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH 3/7] leds-lm3530: add 'is_vin_always_on' in the lm3530_platform_data
2012-01-23 19:31 ` Mark Brown
@ 2012-01-25 1:57 ` Kim, Milo
0 siblings, 0 replies; 4+ messages in thread
From: Kim, Milo @ 2012-01-25 1:57 UTC (permalink / raw)
To: Mark Brown, Linus Walleij
Cc: shreshthakumar.sahu@stericsson.com, linux-kernel@vger.kernel.org,
rpurdie@rpsys.net
OK, I see. I'll try to follow design patterns for the regulator.
Thanks a lot for your comment.
Best Regards,
Milo -
-----Original Message-----
From: Mark Brown [mailto:broonie@opensource.wolfsonmicro.com]
Sent: Tuesday, January 24, 2012 4:31 AM
To: Linus Walleij
Cc: Kim, Milo; shreshthakumar.sahu@stericsson.com; linux-kernel@vger.kernel.org; rpurdie@rpsys.net
Subject: Re: [PATCH 3/7] leds-lm3530: add 'is_vin_always_on' in the lm3530_platform_data
On Mon, Jan 23, 2012 at 08:13:18PM +0100, Linus Walleij wrote:
> As far as I know we want to get rid of different design patterns
> avoiding the regulator interface. The way forward for this is
> either:
> (A) Provide a fixed voltage regulator in your platform for this
> device (AFAIK the voltage does not even have to be correct)
> (B) Enable dummy regulators in your platform
> (A) is more elegant.
Yeah, regulator usage should essentially never be conditional at a
driver level unless there's actual optional supplies in the hardware.
The situation has never been any different to that.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-01-25 1:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <AczXFZjmtQHOiXDCSJi9Lr1dSJlaCQ==>
2012-01-20 1:48 ` [PATCH 3/7] leds-lm3530: add 'is_vin_always_on' in the lm3530_platform_data Kim, Milo
2012-01-23 19:13 ` Linus Walleij
2012-01-23 19:31 ` Mark Brown
2012-01-25 1:57 ` Kim, Milo
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.