On Sun, Feb 11, 2024 at 11:14 AM Kent Gibson <warthog618@xxxxxxxxx> wrote: > The documentation for default_values mentions high/low which can be > confusing, particularly when the ACTIVE_LOW flag is set. > > Replace high/low with active/inactive to clarify that the values are > logical not physical. > > Similarly, clarify the interpretation of values in struct gpiohandle_data. > > Signed-off-by: Kent Gibson <warthog618@xxxxxxxxx> After the fixes pointed out by others: Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Yours, Linus Walleij