[PATCH] ARM: dts: Protect pinctrl headers against multiple inclusions

Cousson, Benoit b-cousson at ti.com
Thu Jun 13 02:18:47 EST 2013


Hi Florian,

On 6/12/2013 8:42 AM, Florian Vaussard wrote:
> Hello Grant,
>
> On 06/11/2013 11:57 PM, Grant Likely wrote:
>> On Tue, 11 Jun 2013 16:50:50 +0200, Florian Vaussard
>> <florian.vaussard at epfl.ch> wrote:
>>> Pinctrl headers were not protected with #ifndef.
>>>
>>> Signed-off-by: Florian Vaussard <florian.vaussard at epfl.ch>
>>
>> Obviously this needs to go in via whatever tree added the modified
>> header files.
>>
>
> I authored these files, sorry for this stupid omission. Benoit, can you
> take this patch?

Yes, sure, I'll take it with Grant's ack.

Benoit

>
> Regards,
>
> Florian
>
>> Acked-by: Grant Likely <grant.likely at secretlab.ca>
>>
>>> ---
>>>   include/dt-bindings/pinctrl/am33xx.h |    5 +++++
>>>   include/dt-bindings/pinctrl/omap.h   |    5 +++++
>>>   2 files changed, 10 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/include/dt-bindings/pinctrl/am33xx.h
>>> b/include/dt-bindings/pinctrl/am33xx.h
>>> index a3fddd4..469e032 100644
>>> --- a/include/dt-bindings/pinctrl/am33xx.h
>>> +++ b/include/dt-bindings/pinctrl/am33xx.h
>>> @@ -2,6 +2,9 @@
>>>    * This header provides constants specific to AM33XX pinctrl bindings.
>>>    */
>>>
>>> +#ifndef _DT_BINDINGS_PINCTRL_AM33XX_H
>>> +#define _DT_BINDINGS_PINCTRL_AM33XX_H
>>> +
>>>   #include <include/dt-bindings/pinctrl/omap.h>
>>>
>>>   /* am33xx specific mux bit defines */
>>> @@ -35,3 +38,5 @@
>>>   #undef PIN_OFF_INPUT_PULLDOWN
>>>   #undef PIN_OFF_WAKEUPENABLE
>>>
>>> +#endif
>>> +
>>> diff --git a/include/dt-bindings/pinctrl/omap.h
>>> b/include/dt-bindings/pinctrl/omap.h
>>> index 370df3f..edbd250 100644
>>> --- a/include/dt-bindings/pinctrl/omap.h
>>> +++ b/include/dt-bindings/pinctrl/omap.h
>>> @@ -5,6 +5,9 @@
>>>    * Copyright (C) 2009-2010 Texas Instruments
>>>    */
>>>
>>> +#ifndef _DT_BINDINGS_PINCTRL_OMAP_H
>>> +#define _DT_BINDINGS_PINCTRL_OMAP_H
>>> +
>>>   /* 34xx mux mode options for each pin. See TRM for options */
>>>   #define MUX_MODE0    0
>>>   #define MUX_MODE1    1
>>> @@ -48,3 +51,5 @@
>>>   #define PIN_OFF_INPUT_PULLDOWN    (OFF_EN | OFF_PULL_EN)
>>>   #define PIN_OFF_WAKEUPENABLE    WAKEUP_EN
>>>
>>> +#endif
>>> +
>>> --
>>> 1.7.5.4
>>>
>>> _______________________________________________
>>> devicetree-discuss mailing list
>>> devicetree-discuss at lists.ozlabs.org
>>> https://lists.ozlabs.org/listinfo/devicetree-discuss
>>
>



More information about the devicetree-discuss mailing list