Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1270867 > unrolled thread

[PATCH v5 0/2] simplefb: Add regulator handling support

Back to article view | Back to linux.kernel

Contents

  [PATCH v5 0/2] simplefb: Add regulator handling support Chen-Yu Tsai <wens@csie.org> - 2015-11-17 05:40 +0100
    Re: [PATCH v5 0/2] simplefb: Add regulator handling support Tomi Valkeinen <tomi.valkeinen@ti.com> - 2015-11-24 12:10 +0100

#1270867 — [PATCH v5 0/2] simplefb: Add regulator handling support

Hi everyone,

This is v5 of the simplefb regulator support series. This series adds
regulator claiming and enabling support for simplefb.

Hans, I dropped your Reviewed-by tag from patch 2 since v4.

Changes since v5:
  - Rebased onto v4.4-rc1
  - Dropped dts patches (merged)

Changes since v4:
  - Fixed inverted logic when testing the property name.
  - Fixed regulator supply name string copy length off by 1.
  - Added real world user, MSI Primo 81 dts patches.

Changes since v3:
  - Dropped extra "if" which is always true, leftover from v1.
  - Updated commit message of patch 1

Sometimes the simplefb display output path consits of external conversion
chips and/or LCD drivers and backlights. These devices normally have
GPIOs to turn them on and/or bring them out of reset, and regulators
supplying power to them.

While the kernel does not touch unclaimed GPIOs, the regulator core
happily disables unused regulators. Thus we need simplefb to claim
and enable the regulators used throughout the display pipeline.

The binding supports any named regulator supplies under its device
node. The driver will look through its properties, and claim any
regulators by matching "*-supply", as Mark suggested.

I've not done a generic helper in the regulator core yet, instead doing
the regulator property handling in the simplefb code for now.


Patch 1 adds the regulator properties to the DT binding.

Patch 2 adds code to the simplefb driver to claim and enable regulators.

Regards
ChenYu


Chen-Yu Tsai (2):
  dt-bindings: simplefb: Support regulator supply properties
  simplefb: Claim and enable regulators

 .../bindings/display/simple-framebuffer.txt        |  13 ++-
 drivers/video/fbdev/simplefb.c                     | 120 ++++++++++++++++++++-
 2 files changed, 128 insertions(+), 5 deletions(-)

-- 
2.6.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1276320

[Multipart message — attachments visible in raw view] — view raw

On 17/11/15 06:31, Chen-Yu Tsai wrote:
> Hi everyone,
> 
> This is v5 of the simplefb regulator support series. This series adds
> regulator claiming and enabling support for simplefb.
> 
> Hans, I dropped your Reviewed-by tag from patch 2 since v4.
> 
> Changes since v5:
>   - Rebased onto v4.4-rc1
>   - Dropped dts patches (merged)
> 
> Changes since v4:
>   - Fixed inverted logic when testing the property name.
>   - Fixed regulator supply name string copy length off by 1.
>   - Added real world user, MSI Primo 81 dts patches.
> 
> Changes since v3:
>   - Dropped extra "if" which is always true, leftover from v1.
>   - Updated commit message of patch 1
> 
> Sometimes the simplefb display output path consits of external conversion
> chips and/or LCD drivers and backlights. These devices normally have
> GPIOs to turn them on and/or bring them out of reset, and regulators
> supplying power to them.
> 
> While the kernel does not touch unclaimed GPIOs, the regulator core
> happily disables unused regulators. Thus we need simplefb to claim
> and enable the regulators used throughout the display pipeline.
> 
> The binding supports any named regulator supplies under its device
> node. The driver will look through its properties, and claim any
> regulators by matching "*-supply", as Mark suggested.
> 
> I've not done a generic helper in the regulator core yet, instead doing
> the regulator property handling in the simplefb code for now.
> 
> 
> Patch 1 adds the regulator properties to the DT binding.
> 
> Patch 2 adds code to the simplefb driver to claim and enable regulators.
> 
> Regards
> ChenYu
> 
> 
> Chen-Yu Tsai (2):
>   dt-bindings: simplefb: Support regulator supply properties
>   simplefb: Claim and enable regulators
> 
>  .../bindings/display/simple-framebuffer.txt        |  13 ++-
>  drivers/video/fbdev/simplefb.c                     | 120 ++++++++++++++++++++-
>  2 files changed, 128 insertions(+), 5 deletions(-)
> 

Thanks, queued for 4.5.

 Tomi

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web