On Tue, Oct 01, 2024 at 03:41:37PM +0200, Nuno Sa wrote: > + > + switch (cfg) { > + case PIN_CONFIG_BIAS_PULL_UP: > + fallthrough; > + case PIN_CONFIG_BIAS_PULL_DOWN: > + fallthrough; > + case PIN_CONFIG_BIAS_DISABLE: > + return adp5589_gpio_set_bias(kpad, pin, cfg); I don't think you need to use "fallhrough" here, saying: case PIN_CONFIG_BIAS_PULL_UP: case PIN_CONFIG_BIAS_PULL_DOWN: case PIN_CONFIG_BIAS_DISABLE: return adp5589_gpio_set_bias(kpad, pin, cfg); should not result in any warnings. Thanks. -- Dmitry