On Tue, Mar 25, 2025 at 12:26 PM Peng Fan <peng.fan@xxxxxxx> wrote: > > On 24/03/2025 05:21, Peng Fan wrote: > > > On Tue, Mar 18, 2025 at 02:48:05PM +0100, Krzysztof Kozlowski > > > To keep DTS unchanged, we need update polarity in gpiolib to force > > > GPIO_ACTIVE_LOW. > > > > How are you going to achieve it if one DTS has LOW and other has > > HIGH? > > With this gpiolib-of change, to fix polarity as HIGH. Yep those quirks is what we have done to handle legacy cases, mostly from old devicetrees using bindings where proper polarity wasn't enforced or properly used because the drivers would override it anyway. Ideally the bindings schema should enforce consumers to use GPIO_ACTIVE_LOW but I don't know if that is possible, but maybe Krzysztof has ideas! Yours, Linus Walleij