On Thu, Aug 31, 2017 at 1:01 PM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > + cec-gpio = <&gpio 7 GPIO_OPEN_DRAIN>; Actually if you want to be 100% specific: cec-gpio = <&gpio 7 (GPIO_ACTIVE_HIGH|GPIO_OPEN_DRAIN)>; (Parens are needed.) But I'm not very picky about that, active high is implicit. Yours, Linus Walleij