On Wed, Sep 14, 2022 at 3:00 PM Kent Gibson <warthog618@xxxxxxxxx> wrote: > The key distinction is whether you are describing a physical or logical > value/state. > High/low should be reserved for physical. > Active/inactive describe logical. > > (I personally dislike "deassert" as it is a manufactured word that feels > very awkward.) I would certainly trust anyone native Anglo-Saxon to say what is the best word here, so I will happily fold on this. > Changing gpiod_set_value() to gpiod_set_state(), while leaving > gpiod_set_raw_value() as is, does not reduce confusion - at least not for > me. Sloppy of me, certainly these would need to be renamed too. The _raw being high/low and the logic accessors active/inactive. Yours, Linus Walleij
