Re: [PATCH v3 RESEND 2/3] power: supply: rt9467: Add Richtek RT9467 charger driver

[Rob Herring]

On Tue, Jan 3, 2023 at 1:30 AM ChiaEn Wu <peterwu.pub@xxxxxxxxx> wrote:
>
> From: ChiaEn Wu <chiaen_wu@xxxxxxxxxxx>
>
> RT9467 is a switch-mode single cell Li-Ion/Li-Polymer battery charger
> for portable applications.
>
> It integrates a synchronous PWM controller, power MOSFETs,
> input current sensing and regulation, high-accuracy voltage regulation,
> and charge termination. The charge current is regulated through
> integrated sensing resistors.
>
> The RT9467 also features USB On-The-Go (OTG) support. It also integrates
> D+/D- pin for USB host/charging port detection.
>
> Co-developed-by: ChiYuan Huang <cy_huang@xxxxxxxxxxx>
> Signed-off-by: ChiYuan Huang <cy_huang@xxxxxxxxxxx>
> Signed-off-by: ChiaEn Wu <chiaen_wu@xxxxxxxxxxx>
> ---
>  drivers/power/supply/Kconfig          |   19 +
>  drivers/power/supply/Makefile         |    1 +
>  drivers/power/supply/rt9467-charger.c | 1282 +++++++++++++++++++++++++
>  3 files changed, 1302 insertions(+)
>  create mode 100644 drivers/power/supply/rt9467-charger.c

[...]

> +static const struct of_device_id rt9467_charger_of_match_table[] = {
> +       { .compatible = "richtek,rt9467", },

This does not match the binding. Probably the binding should be fixed
if this is only a charger. In that case, '-charger' is redundant.

FYI, 'make dt_compatible_check' will detect this.

Rob