Re: [PATCH v2] pinctrl: sunxi: set minimal debounce on input-debounce 0

[Maxime Ripard <maxime@xxxxxxxxxx>]

Hi,

On Sun, Feb 12, 2023 at 07:21:03PM +0100, Andreas Feldner wrote:
> sunxi-h3-h5 based boards have no support for switching
> off IRQ debouncing filter. This would be the expected
> behaviour of value 0 for the general pinctl parameter
> input-debounce.
> The current driver implementation ignores value 0
> for input-debounce, leaving the chip's default. This
> default, however, is not minimal, but equivalent to
> value 31 (microseconds).
> 
> This patch does not ignore value 0 but instead makes
> sure the corresponding IRQ debounce filter is set
> to the shortest time selectable, i. e. the fast
> oscillator with a divider of 1 == (2 ^ 0).
> 
> Fixes: 7c926492d38a ("pinctrl: sunxi: Add support for interrupt debouncing")
> 
> Signed-off-by: Andreas Feldner <pelzi@xxxxxxxxxxxxxxx>

Like I said in the other thread, I don't think that deviating from the
generic property semantics is a good thing.

Maxime