On 2013-12-06 10:57, Javier Martinez Canillas wrote:
>> + tfp410: encoder@0 {
>> + compatible = "ti,tfp410";
>> + gpios = <&gpio1 0 0>; /* 0, power-down */
>> +
>
> Please use the constants from include/dt-bindings/ instead of magic
> numbers, i.e:
>
> gpios = <&gpio1 0 GPIO_ACTIVE_HIGH>; /* 0, power-down */Thanks, fixed now (for all .dts files) However... The TFP410 gpio is "power-down". I think we should actually mark it as GPIO_ACTIVE_LOW, as setting it to 0 powers down the device. Tomi
signature.asc
Description: OpenPGP digital signature
