* [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight
@ 2013-03-12 1:54 Andrew Chew
2013-03-12 7:01 ` Thierry Reding
2013-03-12 9:24 ` Peter Ujfalusi
0 siblings, 2 replies; 6+ messages in thread
From: Andrew Chew @ 2013-03-12 1:54 UTC (permalink / raw)
To: peter.ujfalusi; +Cc: thierry.reding, acourbot, achew, linux-omap
The pwm-backlight driver now takes a mandatory regulator that is gotten
during driver probe. Initialize a dummy regulator to satisfy this
requirement.
Signed-off-by: Andrew Chew <achew@nvidia.com>
---
This patch, along with many more soon to follow, attempts to satisfy the new
mandatory regulator that pwm-backlight will grab during probe. The only
board in mach-omap2 to use the pwm-backlight appears to be the 4430sdp.
I thought I'd start small and use this board as an example.
I tested similar code in my Tegra board, so it should be okay. Of course,
I don't have a 4430sdp to test with.
arch/arm/mach-omap2/board-4430sdp.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/arm/mach-omap2/board-4430sdp.c b/arch/arm/mach-omap2/board-4430sdp.c
index 35f3ad0..62022c0 100644
--- a/arch/arm/mach-omap2/board-4430sdp.c
+++ b/arch/arm/mach-omap2/board-4430sdp.c
@@ -291,6 +291,10 @@ static struct platform_device sdp4430_leds_pwm = {
},
};
+/* Dummy regulator for pwm-backlight driver */
+static struct regulator_consumer_supply backlight_supply =
+ REGULATOR_SUPPLY("enable", NULL);
+
static struct platform_pwm_backlight_data sdp4430_backlight_data = {
.max_brightness = 127,
.dft_brightness = 127,
@@ -718,6 +722,7 @@ static void __init omap_4430sdp_init(void)
omap4_i2c_init();
omap_sfh7741prox_init();
+ regulator_register_always_on(-1, "bl-enable", &backlight_supply, 1, 0);
platform_add_devices(sdp4430_devices, ARRAY_SIZE(sdp4430_devices));
omap_serial_init();
omap_sdrc_init(NULL, NULL);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight
2013-03-12 1:54 [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight Andrew Chew
@ 2013-03-12 7:01 ` Thierry Reding
2013-03-12 19:11 ` Andrew Chew
2013-03-12 9:24 ` Peter Ujfalusi
1 sibling, 1 reply; 6+ messages in thread
From: Thierry Reding @ 2013-03-12 7:01 UTC (permalink / raw)
To: Andrew Chew; +Cc: peter.ujfalusi, acourbot, linux-omap
[-- Attachment #1: Type: text/plain, Size: 2262 bytes --]
On Mon, Mar 11, 2013 at 06:54:26PM -0700, Andrew Chew wrote:
> The pwm-backlight driver now takes a mandatory regulator that is gotten
> during driver probe. Initialize a dummy regulator to satisfy this
> requirement.
>
> Signed-off-by: Andrew Chew <achew@nvidia.com>
> ---
> This patch, along with many more soon to follow, attempts to satisfy the new
> mandatory regulator that pwm-backlight will grab during probe. The only
> board in mach-omap2 to use the pwm-backlight appears to be the 4430sdp.
>
> I thought I'd start small and use this board as an example.
>
> I tested similar code in my Tegra board, so it should be okay. Of course,
> I don't have a 4430sdp to test with.
Hi Andrew,
This looks good, one minor comment below. I think it might make sense to
post this patch as part of a series that includes the change to the
pwm-backlight driver. This will make things easier on potential testers.
You can later extend that series to include all the other users and
eventually the whole series can be merged.
> arch/arm/mach-omap2/board-4430sdp.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/mach-omap2/board-4430sdp.c b/arch/arm/mach-omap2/board-4430sdp.c
> index 35f3ad0..62022c0 100644
> --- a/arch/arm/mach-omap2/board-4430sdp.c
> +++ b/arch/arm/mach-omap2/board-4430sdp.c
> @@ -291,6 +291,10 @@ static struct platform_device sdp4430_leds_pwm = {
> },
> };
>
> +/* Dummy regulator for pwm-backlight driver */
> +static struct regulator_consumer_supply backlight_supply =
> + REGULATOR_SUPPLY("enable", NULL);
> +
> static struct platform_pwm_backlight_data sdp4430_backlight_data = {
> .max_brightness = 127,
> .dft_brightness = 127,
> @@ -718,6 +722,7 @@ static void __init omap_4430sdp_init(void)
>
> omap4_i2c_init();
> omap_sfh7741prox_init();
> + regulator_register_always_on(-1, "bl-enable", &backlight_supply, 1, 0);
I think I'd prefer to spell out "backlight-enable". I know it's rather
long but I find it is much easier on the eye to look at the various
outputs where the string is used and see "backlight" instead of "bl".
But it's really up to the OMAP crowd, if everybody else is fine with
"bl-enable" I'll shut up.
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread* RE: [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight
2013-03-12 7:01 ` Thierry Reding
@ 2013-03-12 19:11 ` Andrew Chew
2013-03-13 7:09 ` Thierry Reding
0 siblings, 1 reply; 6+ messages in thread
From: Andrew Chew @ 2013-03-12 19:11 UTC (permalink / raw)
To: Thierry Reding
Cc: peter.ujfalusi@ti.com, Alex Courbot, linux-omap@vger.kernel.org
> From: Thierry Reding [mailto:thierry.reding@avionic-design.de]
> Sent: Tuesday, March 12, 2013 12:01 AM
> To: Andrew Chew
> Cc: peter.ujfalusi@ti.com; Alex Courbot; linux-omap@vger.kernel.org
> Subject: Re: [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to
> pwm-backlight
>
> * PGP Signed by an unknown key
>
> On Mon, Mar 11, 2013 at 06:54:26PM -0700, Andrew Chew wrote:
> > The pwm-backlight driver now takes a mandatory regulator that is
> > gotten during driver probe. Initialize a dummy regulator to satisfy
> > this requirement.
> >
> > Signed-off-by: Andrew Chew <achew@nvidia.com>
> > ---
> > This patch, along with many more soon to follow, attempts to satisfy
> > the new mandatory regulator that pwm-backlight will grab during probe.
> > The only board in mach-omap2 to use the pwm-backlight appears to be the
> 4430sdp.
> >
> > I thought I'd start small and use this board as an example.
> >
> > I tested similar code in my Tegra board, so it should be okay. Of
> > course, I don't have a 4430sdp to test with.
>
> Hi Andrew,
>
> This looks good, one minor comment below. I think it might make sense to
> post this patch as part of a series that includes the change to the pwm-
> backlight driver. This will make things easier on potential testers.
> You can later extend that series to include all the other users and eventually
> the whole series can be merged.
Do you mean I should do a single patch series, accumulating other boards
as I go?
And by the way, do I have your reviewed-by for the pwm-backlight regulator
patch?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight
2013-03-12 19:11 ` Andrew Chew
@ 2013-03-13 7:09 ` Thierry Reding
0 siblings, 0 replies; 6+ messages in thread
From: Thierry Reding @ 2013-03-13 7:09 UTC (permalink / raw)
To: Andrew Chew
Cc: peter.ujfalusi@ti.com, Alex Courbot, linux-omap@vger.kernel.org
[-- Attachment #1: Type: text/plain, Size: 2140 bytes --]
On Tue, Mar 12, 2013 at 12:11:54PM -0700, Andrew Chew wrote:
> > From: Thierry Reding [mailto:thierry.reding@avionic-design.de]
> > Sent: Tuesday, March 12, 2013 12:01 AM
> > To: Andrew Chew
> > Cc: peter.ujfalusi@ti.com; Alex Courbot; linux-omap@vger.kernel.org
> > Subject: Re: [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to
> > pwm-backlight
> >
> > * PGP Signed by an unknown key
> >
> > On Mon, Mar 11, 2013 at 06:54:26PM -0700, Andrew Chew wrote:
> > > The pwm-backlight driver now takes a mandatory regulator that is
> > > gotten during driver probe. Initialize a dummy regulator to satisfy
> > > this requirement.
> > >
> > > Signed-off-by: Andrew Chew <achew@nvidia.com>
> > > ---
> > > This patch, along with many more soon to follow, attempts to satisfy
> > > the new mandatory regulator that pwm-backlight will grab during probe.
> > > The only board in mach-omap2 to use the pwm-backlight appears to be the
> > 4430sdp.
> > >
> > > I thought I'd start small and use this board as an example.
> > >
> > > I tested similar code in my Tegra board, so it should be okay. Of
> > > course, I don't have a 4430sdp to test with.
> >
> > Hi Andrew,
> >
> > This looks good, one minor comment below. I think it might make sense to
> > post this patch as part of a series that includes the change to the pwm-
> > backlight driver. This will make things easier on potential testers.
> > You can later extend that series to include all the other users and eventually
> > the whole series can be merged.
>
> Do you mean I should do a single patch series, accumulating other boards
> as I go?
Yes, that way everything needed will be in one patch series and people
can easily apply that to their trees. It will also make things easier if
we decide to merge all of them through one tree in the end.
> And by the way, do I have your reviewed-by for the pwm-backlight regulator
> patch?
I think it looks good, but I'll go over it again in your repost of the
whole series. I don't think you need my Reviewed-by, though, since I
maintain the pwm-backlight driver. =)
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight
2013-03-12 1:54 [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight Andrew Chew
2013-03-12 7:01 ` Thierry Reding
@ 2013-03-12 9:24 ` Peter Ujfalusi
2013-03-12 22:21 ` Andrew Chew
1 sibling, 1 reply; 6+ messages in thread
From: Peter Ujfalusi @ 2013-03-12 9:24 UTC (permalink / raw)
To: Andrew Chew; +Cc: thierry.reding, acourbot, linux-omap
Hi,
On 03/12/2013 02:54 AM, Andrew Chew wrote:
> The pwm-backlight driver now takes a mandatory regulator that is gotten
> during driver probe. Initialize a dummy regulator to satisfy this
> requirement.
Can you point me to the commit which makes a regulator mandatory for
pwm-backlight?
Why the regulator is mandatory?
The 4430SDP (or Blaze) and Zoom2 is just happy alone with the PWM why should
we have dummy regulators?
>
> Signed-off-by: Andrew Chew <achew@nvidia.com>
> ---
> This patch, along with many more soon to follow, attempts to satisfy the new
> mandatory regulator that pwm-backlight will grab during probe. The only
> board in mach-omap2 to use the pwm-backlight appears to be the 4430sdp.
>
> I thought I'd start small and use this board as an example.
>
> I tested similar code in my Tegra board, so it should be okay. Of course,
> I don't have a 4430sdp to test with.
>
> arch/arm/mach-omap2/board-4430sdp.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/mach-omap2/board-4430sdp.c b/arch/arm/mach-omap2/board-4430sdp.c
> index 35f3ad0..62022c0 100644
> --- a/arch/arm/mach-omap2/board-4430sdp.c
> +++ b/arch/arm/mach-omap2/board-4430sdp.c
> @@ -291,6 +291,10 @@ static struct platform_device sdp4430_leds_pwm = {
> },
> };
>
> +/* Dummy regulator for pwm-backlight driver */
> +static struct regulator_consumer_supply backlight_supply =
> + REGULATOR_SUPPLY("enable", NULL);
> +
> static struct platform_pwm_backlight_data sdp4430_backlight_data = {
> .max_brightness = 127,
> .dft_brightness = 127,
> @@ -718,6 +722,7 @@ static void __init omap_4430sdp_init(void)
>
> omap4_i2c_init();
> omap_sfh7741prox_init();
> + regulator_register_always_on(-1, "bl-enable", &backlight_supply, 1, 0);
> platform_add_devices(sdp4430_devices, ARRAY_SIZE(sdp4430_devices));
> omap_serial_init();
> omap_sdrc_init(NULL, NULL);
>
--
Péter
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread* RE: [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight
2013-03-12 9:24 ` Peter Ujfalusi
@ 2013-03-12 22:21 ` Andrew Chew
0 siblings, 0 replies; 6+ messages in thread
From: Andrew Chew @ 2013-03-12 22:21 UTC (permalink / raw)
To: Peter Ujfalusi
Cc: thierry.reding@avionic-design.de, Alex Courbot,
linux-omap@vger.kernel.org
> On 03/12/2013 02:54 AM, Andrew Chew wrote:
> > The pwm-backlight driver now takes a mandatory regulator that is
> > gotten during driver probe. Initialize a dummy regulator to satisfy
> > this requirement.
>
> Can you point me to the commit which makes a regulator mandatory for
> pwm-backlight?
> Why the regulator is mandatory?
> The 4430SDP (or Blaze) and Zoom2 is just happy alone with the PWM why
> should we have dummy regulators?
I'm resubmitting the patch as a patch series that includes the pwm-backlight
change. See the cover letter for an explanation of why this is needed.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-03-13 7:09 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-12 1:54 [PATCH 1/1] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight Andrew Chew
2013-03-12 7:01 ` Thierry Reding
2013-03-12 19:11 ` Andrew Chew
2013-03-13 7:09 ` Thierry Reding
2013-03-12 9:24 ` Peter Ujfalusi
2013-03-12 22:21 ` Andrew Chew
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox