| Message ID | fc99babab77bbf9fcbe34fc82d7e2a18f9a1e464.1465303439.git.jsarha@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Tue, Jun 07, 2016 at 04:51:13PM +0300, Jyri Sarha wrote: > Remove reset-active-low from the devicetree binding document. The actual > implementation has never been there in the driver code and there is no > reason to add it because the gpiod API supports gpio flags, including > GPIO_ACTIVE_LOW, directly trough its own devicetree binding. > > Signed-off-by: Jyri Sarha <jsarha@ti.com> > --- > Documentation/devicetree/bindings/display/ssd1307fb.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/ssd1307fb.txt b/Documentation/devicetree/bindings/display/ssd1307fb.txt > index eb31ed4..4aee67f 100644 > --- a/Documentation/devicetree/bindings/display/ssd1307fb.txt > +++ b/Documentation/devicetree/bindings/display/ssd1307fb.txt > @@ -8,14 +8,14 @@ Required properties: > 0x3c or 0x3d > - pwm: Should contain the pwm to use according to the OF device tree PWM > specification [0]. Only required for the ssd1307. > - - reset-gpios: Should contain the GPIO used to reset the OLED display > + - reset-gpios: Should contain the GPIO used to reset the OLED display. See > + Documentation/devicetree/bindings/gpio/gpio.txt for details. You should state here it is active low. > - solomon,height: Height in pixel of the screen driven by the controller > - solomon,width: Width in pixel of the screen driven by the controller > - solomon,page-offset: Offset of pages (band of 8 pixels) that the screen is > mapped to. > > Optional properties: > - - reset-active-low: Is the reset gpio is active on physical low? > - solomon,segment-no-remap: Display needs normal (non-inverted) data column > to segment mapping > - solomon,com-seq: Display uses sequential COM pin configuration > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/display/ssd1307fb.txt b/Documentation/devicetree/bindings/display/ssd1307fb.txt index eb31ed4..4aee67f 100644 --- a/Documentation/devicetree/bindings/display/ssd1307fb.txt +++ b/Documentation/devicetree/bindings/display/ssd1307fb.txt @@ -8,14 +8,14 @@ Required properties: 0x3c or 0x3d - pwm: Should contain the pwm to use according to the OF device tree PWM specification [0]. Only required for the ssd1307. - - reset-gpios: Should contain the GPIO used to reset the OLED display + - reset-gpios: Should contain the GPIO used to reset the OLED display. See + Documentation/devicetree/bindings/gpio/gpio.txt for details. - solomon,height: Height in pixel of the screen driven by the controller - solomon,width: Width in pixel of the screen driven by the controller - solomon,page-offset: Offset of pages (band of 8 pixels) that the screen is mapped to. Optional properties: - - reset-active-low: Is the reset gpio is active on physical low? - solomon,segment-no-remap: Display needs normal (non-inverted) data column to segment mapping - solomon,com-seq: Display uses sequential COM pin configuration
Remove reset-active-low from the devicetree binding document. The actual implementation has never been there in the driver code and there is no reason to add it because the gpiod API supports gpio flags, including GPIO_ACTIVE_LOW, directly trough its own devicetree binding. Signed-off-by: Jyri Sarha <jsarha@ti.com> --- Documentation/devicetree/bindings/display/ssd1307fb.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)