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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux