On 09/11/2022 17:37, Krzysztof Kozlowski wrote: > The shutdown GPIO is active_low (sd_n), but this depends on actual board > layout. Linux drivers should only care about logical state, where high > (1) means shutdown and low (0) means do not shutdown. > > Invert the GPIO to match logical value. > > Fixes: 43b8c7dc85a1 ("ASoC: codecs: add wsa883x amplifier support") > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > --- I will also update the example in bindings. Not a blocker for this patch, but better to keep these together, so there will be a v2. Best regards, Krzysztof