On 15/04/2014 10:00, Ludovic Desroches wrote: > On Tue, Apr 15, 2014 at 09:39:59AM +0200, Richard Genoud wrote: >> On 14/04/2014 17:22, Ludovic Desroches wrote: >>> Hi Richard, >>> >>> On Mon, Apr 14, 2014 at 04:58:02PM +0200, Richard Genoud wrote: >>>> RTS pin is an active low pin. >>>> For now, this doesn't change anything as the ACTIVE_LOW flag is not >>>> handled in atmel_serial, but it will be in 3.16. >>>> >>>> Signed-off-by: Richard Genoud <richard.genoud@xxxxxxxxx> >>>> --- >>>> Documentation/devicetree/bindings/serial/atmel-usart.txt | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/Documentation/devicetree/bindings/serial/atmel-usart.txt b/Documentation/devicetree/bindings/serial/atmel-usart.txt >>>> index 17c1042b2df8..2f7aad71b3c9 100644 >>>> --- a/Documentation/devicetree/bindings/serial/atmel-usart.txt >>>> +++ b/Documentation/devicetree/bindings/serial/atmel-usart.txt >>>> @@ -35,7 +35,7 @@ Example: >>>> clock-names = "usart"; >>>> atmel,use-dma-rx; >>>> atmel,use-dma-tx; >>>> - rts-gpios = <&pioD 15 0>; >>>> + rts-gpios = <&pioD 15 GPIO_ACTIVE_LOW>; >>> >>> I was told we should not use macros here. If it is still the truth: >>> >>> rts-gpios = <&pioD 15 1>; >> >> But I'd really like to know if macros are forbidden in DT documentation and why... >> Because, IMHO, <&pioD 15 1> talks less that <&pioD 15 GPIO_ACTIVE_LOW>. >> Which is a shame in a documentation. > > I don't remember who told me that but the reason was that device tree > should not depend on Linux and macros are relative to Linux. > Well, that makes sense, but I thought that include/dt-bindings/ was more DT relative. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html