Re: [PATCH v2 4/5] fbdev/ssd1307fb: add support to enable VBAT

2 messages, 2 authors, 2017-01-19 · open the first message on its own page

Re: [PATCH v2 4/5] fbdev/ssd1307fb: add support to enable VBAT

From: Rob Herring <hidden>
Date: 2017-01-18 22:33:10

On Fri, Jan 13, 2017 at 12:35:48PM +0200, Jyri Sarha wrote:
quoted hunk
From: Tomi Valkeinen <redacted>

SSD1306 needs VBAT when it is wired in charge pump configuration. This
patch adds support to the driver to enable VBAT regulator at init time.

Signed-off-by: Tomi Valkeinen <redacted>
Reviewed-by: Roger Quadros <redacted>
Signed-off-by: Jyri Sarha <redacted>
---
 .../devicetree/bindings/display/ssd1307fb.txt        |  1 +
 drivers/video/fbdev/ssd1307fb.c                      | 20 +++++++++++++++++++-
 2 files changed, 20 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/ssd1307fb.txt b/Documentation/devicetree/bindings/display/ssd1307fb.txt
index 6617df6..209d931 100644
--- a/Documentation/devicetree/bindings/display/ssd1307fb.txt
+++ b/Documentation/devicetree/bindings/display/ssd1307fb.txt
@@ -16,6 +16,7 @@ Required properties:
 Optional properties:
   - reset-gpios: The GPIO used to reset the OLED display, if available. See
                  Documentation/devicetree/bindings/gpio/gpio.txt for details.
+  - vbat-supply: The supply for VBAT
According to the datasheet, SSD1307 has 2 supplies: Vdd and Vcc

I don't see any mention of a charge pump, so that must be an external 
component.
   - solomon,segment-no-remap: Display needs normal (non-inverted) data column
                               to segment mapping
   - solomon,com-seq: Display uses sequential COM pin configuration

Re: [PATCH v2 4/5] fbdev/ssd1307fb: add support to enable VBAT

From: Tomi Valkeinen <hidden>
Date: 2017-01-19 13:15:12

On 19/01/17 00:33, Rob Herring wrote:
On Fri, Jan 13, 2017 at 12:35:48PM +0200, Jyri Sarha wrote:
quoted
From: Tomi Valkeinen <redacted>

SSD1306 needs VBAT when it is wired in charge pump configuration. This
patch adds support to the driver to enable VBAT regulator at init time.

Signed-off-by: Tomi Valkeinen <redacted>
Reviewed-by: Roger Quadros <redacted>
Signed-off-by: Jyri Sarha <redacted>
---
 .../devicetree/bindings/display/ssd1307fb.txt        |  1 +
 drivers/video/fbdev/ssd1307fb.c                      | 20 +++++++++++++++++++-
 2 files changed, 20 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/ssd1307fb.txt b/Documentation/devicetree/bindings/display/ssd1307fb.txt
index 6617df6..209d931 100644
--- a/Documentation/devicetree/bindings/display/ssd1307fb.txt
+++ b/Documentation/devicetree/bindings/display/ssd1307fb.txt
@@ -16,6 +16,7 @@ Required properties:
 Optional properties:
   - reset-gpios: The GPIO used to reset the OLED display, if available. See
                  Documentation/devicetree/bindings/gpio/gpio.txt for details.
+  - vbat-supply: The supply for VBAT
According to the datasheet, SSD1307 has 2 supplies: Vdd and Vcc

I don't see any mention of a charge pump, so that must be an external 
component.
VBAT is for SSD1306. Perhaps the DT doc should mention it.

 Tomi
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help