[PATCH v1] dt-bindings: serial: 8250_omap: add rs485-rts-active-high

Francesco Dolcini posted 1 patch 2 years, 8 months ago
There is a newer version of this series
Documentation/devicetree/bindings/serial/8250_omap.yaml | 1 +
1 file changed, 1 insertion(+)
[PATCH v1] dt-bindings: serial: 8250_omap: add rs485-rts-active-high
Posted by Francesco Dolcini 2 years, 8 months ago
From: Francesco Dolcini <francesco.dolcini@toradex.com>

Add rs485-rts-active-high property, this was removed by mistake.
In general we just use rs485-rts-active-low property, however the OMAP
UART for legacy reason uses the -high one.

Fixes: 767d3467eb60 ("dt-bindings: serial: 8250_omap: drop rs485 properties")
Reported-by: Francesco Dolcini <francesco.dolcini@toradex.com>
Closes: https://lore.kernel.org/all/ZGefR4mTHHo1iQ7H@francesco-nb.int.toradex.com/
Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
---
 Documentation/devicetree/bindings/serial/8250_omap.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/serial/8250_omap.yaml b/Documentation/devicetree/bindings/serial/8250_omap.yaml
index eb3488d8f9ee..6a7be42da523 100644
--- a/Documentation/devicetree/bindings/serial/8250_omap.yaml
+++ b/Documentation/devicetree/bindings/serial/8250_omap.yaml
@@ -70,6 +70,7 @@ properties:
   dsr-gpios: true
   rng-gpios: true
   dcd-gpios: true
+  rs485-rts-active-high: true
   rts-gpio: true
   power-domains: true
   clock-frequency: true
-- 
2.25.1
Re: [PATCH v1] dt-bindings: serial: 8250_omap: add rs485-rts-active-high
Posted by Krzysztof Kozlowski 2 years, 8 months ago
On 31/05/2023 08:40, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@toradex.com>
> 
> Add rs485-rts-active-high property, this was removed by mistake.
> In general we just use rs485-rts-active-low property, however the OMAP
> UART for legacy reason uses the -high one.
> 
> Fixes: 767d3467eb60 ("dt-bindings: serial: 8250_omap: drop rs485 properties")
> Reported-by: Francesco Dolcini <francesco.dolcini@toradex.com>

We do not add Reported-by tags for own reports. Otherwise all patches
would carry them. Reported-by is to credit 3rd party reported who did
not write the patch.


Best regards,
Krzysztof