All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: Andrew Chew <achew@nvidia.com>
Cc: thierry.reding@avionic-design.de, acourbot@nvidia.com,
	linux-omap@vger.kernel.org
Subject: Re: [PATCH V3 0/2] Add mandatory regulator for all users of pwm-backlight.
Date: Thu, 14 Mar 2013 09:44:51 +0100	[thread overview]
Message-ID: <51418E03.6020103@ti.com> (raw)
In-Reply-To: <1363214006-10662-1-git-send-email-achew@nvidia.com>

On 03/13/2013 11:33 PM, Andrew Chew wrote:
> Many backlights are enabled via GPIO.  We can generalize the GPIO to a
> fixed regulator.
> 
> The enable regulator needs to be mandatory because there was no good way
> to determine the difference between opting out of the regulator, and probe
> deferral.
> 
> This series of patches is intended to add a dummy regulator (or a GPIO
> regulator) for all users of the pwm-backlight.
> 
> The last patch in the series will always be the pwm-backlight change to add
> this mandatory regulator.  Patches following up to that patch add the
> mandatory regulator on a per mach family basis.  Once all users of
> pwm-backlight have been patched, this series can be applied in order to
> maintain bisectability.

I'm not really happy with the mandatory regulator since for example the
SDP4430 should not need one... Other than that the backlight works after the
series.

To both patch:

Tested-by: Peter Ujfalusi <peter.ujfalusi@ti.com>


      parent reply	other threads:[~2013-03-14  8:44 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-13 22:33 [PATCH V3 0/2] Add mandatory regulator for all users of pwm-backlight Andrew Chew
2013-03-13 22:33 ` [PATCH V3 1/2] ARM: OMAP: board-4430sdp: Provide regulator to pwm-backlight Andrew Chew
2013-04-08 21:46   ` Tony Lindgren
2013-04-08 21:56     ` Thierry Reding
2013-04-08 22:16       ` Tony Lindgren
2013-04-09  7:56         ` Thierry Reding
2013-04-09 16:40           ` Tony Lindgren
2013-04-09 19:40             ` Thierry Reding
2013-04-09 20:17               ` Tony Lindgren
2013-04-09 20:57                 ` Thierry Reding
2013-04-09 22:27                   ` Tony Lindgren
2013-03-13 22:33 ` [PATCH V3 2/2] pwm_bl: Add mandatory backlight enable regulator Andrew Chew
2013-03-14  8:44 ` Peter Ujfalusi [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51418E03.6020103@ti.com \
    --to=peter.ujfalusi@ti.com \
    --cc=achew@nvidia.com \
    --cc=acourbot@nvidia.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=thierry.reding@avionic-design.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.