Re: [RFC PATCH v3 4/5] input: add onkey driver for Marvell 88PM886 PMIC

[Dmitry Torokhov <dmitry.torokhov@gmail.com>]

Hi Karel,

On Sun, Mar 03, 2024 at 11:04:25AM +0100, Karel Balej wrote:
> From: Karel Balej <balejk@matfyz.cz>
> 
> Marvell 88PM886 PMIC provides onkey among other things. Add client
> driver to handle it. The driver currently only provides a basic support
> omitting additional functions found in the vendor version, such as long
> onkey and GPIO integration.
> 
> Signed-off-by: Karel Balej <balejk@matfyz.cz>
> ---
> 
> Notes:
>     RFC v3:
>     - Drop wakeup-source.
>     RFC v2:
>     - Address Dmitry's feedback:
>       - Sort includes alphabetically.
>       - Drop onkey->irq.
>       - ret -> err in irq_handler and no initialization.
>       - Break long lines and other formatting.
>       - Do not clobber platform_get_irq error.
>       - Do not set device parent manually.
>       - Use input_set_capability.
>       - Use the wakeup-source DT property.
>       - Drop of_match_table.

I only said that you should not be using of_match_ptr(), but you still
need to have of_match_table set and have MODULE_DEVICE_TABLE() for the
proper module loading support.

With that fixed:

Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>

Thanks.

-- 
Dmitry