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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAL_JsqJ4gueN3UFfea5JC6xfPjaH6a+7+dioTw=s0W-gBf6Anw@mail.gmail.com>
Date:   Tue, 7 Feb 2023 16:36:59 -0600
From:   Rob Herring <robh+dt@...nel.org>
To:     ChiaEn Wu <peterwu.pub@...il.com>, sre@...nel.org
Cc:     krzysztof.kozlowski+dt@...aro.org, chiaen_wu@...htek.com,
        cy_huang@...htek.com, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-pm@...r.kernel.org
Subject: Re: [PATCH v3 RESEND 2/3] power: supply: rt9467: Add Richtek RT9467
 charger driver

On Tue, Jan 3, 2023 at 1:30 AM ChiaEn Wu <peterwu.pub@...il.com> wrote:
>
> From: ChiaEn Wu <chiaen_wu@...htek.com>
>
> 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@...htek.com>
> Signed-off-by: ChiYuan Huang <cy_huang@...htek.com>
> Signed-off-by: ChiaEn Wu <chiaen_wu@...htek.com>
> ---
>  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