path: root/drivers/video
diff options
authorMichal Vokáč <michal.vokac@ysoft.com>2018-10-08 10:57:37 (GMT)
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>2018-10-08 10:57:37 (GMT)
commit9827f26374fb85e1811f2adbcc25c8a3992dbe7f (patch)
tree0434aaeff7a455f14b00780432b6627ff8332172 /drivers/video
parent7d1b8d7a858024bd6a1dca206646fbcab5785220 (diff)
video: ssd1307fb: Do not hard code active-low reset sequence
The SSD130x OLED display reset signal is active low. Now the reset sequence is implemented in such a way that users are forced to define reset-gpios as GPIO_ACTIVE_HIGH in DT to make the reset work. Do not hard code the active-low sequence into the driver but instead allow the user to specify the gpio as GPIO_ACTIVE_LOW to reflect the real world. Signed-off-by: Michal Vokáč <michal.vokac@ysoft.com> Cc: Shawn Guo <shawnguo@kernel.org> Cc: Rob Herring <robh+dt@kernel.org> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Diffstat (limited to 'drivers/video')
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/video/fbdev/ssd1307fb.c b/drivers/video/fbdev/ssd1307fb.c
index 4061a20..3b361bc 100644
--- a/drivers/video/fbdev/ssd1307fb.c
+++ b/drivers/video/fbdev/ssd1307fb.c
@@ -667,10 +667,10 @@ static int ssd1307fb_probe(struct i2c_client *client,
if (par->reset) {
/* Reset the screen */
- gpiod_set_value_cansleep(par->reset, 0);
- udelay(4);
gpiod_set_value_cansleep(par->reset, 1);
+ gpiod_set_value_cansleep(par->reset, 0);
+ udelay(4);
if (par->vbat_reg) {

Privacy Policy