Date:   Mon, 13 Feb 2023 21:53:21 +0100
From:   Sebastian Reichel <sebastian.reichel@collabora.com>
To:     ChiaEn Wu <chiaen_wu@richtek.com>
Cc:     robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        cy_huang@richtek.com, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org,
        peterwu.pub@gmail.com
Subject: Re: [Patch][next] dt-bindings: power: supply: Revise Richtek RT9467
 compatible name
Message-ID: <20230213205321.xrhvrdqy5ksiagbv@mercury.elektranox.org>
References: <dc8873c3125f7aa6f84dc7b33a44bf00907e0814.1675853673.git.chiaen_wu@richtek.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="r3cq4aakiv6m7xrb"
Content-Disposition: inline
In-Reply-To: <dc8873c3125f7aa6f84dc7b33a44bf00907e0814.1675853673.git.chiaen_wu@richtek.com>
Precedence: bulk


--r3cq4aakiv6m7xrb
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi,

On Wed, Feb 08, 2023 at 11:14:24AM +0800, ChiaEn Wu wrote:
> Revise RT9467 compatible name from "richtek,rt9467-charger" to
> "richtek,rt9467"
>=20
> Fixes: e1b4620fb503 ("dt-bindings: power: supply: Add Richtek RT9467 batt=
ery charger")
> Signed-off-by: ChiaEn Wu <chiaen_wu@richtek.com>
> ---

Is there a new version of this fixing the issues pointed out by Krzysztof?
Also I think the filename and $id should be changed to richtek,rt9467.yaml
in addition to the compatible change.

Thanks,

-- Sebastian

>  .../devicetree/bindings/power/supply/richtek,rt9467-charger.yaml      | =
4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>=20
> diff --git a/Documentation/devicetree/bindings/power/supply/richtek,rt946=
7-charger.yaml b/Documentation/devicetree/bindings/power/supply/richtek,rt9=
467-charger.yaml
> index 92c5706..9176808 100644
> --- a/Documentation/devicetree/bindings/power/supply/richtek,rt9467-charg=
er.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/richtek,rt9467-charg=
er.yaml
> @@ -25,7 +25,7 @@ description: |
> =20
>  properties:
>    compatible:
> -    const: richtek,rt9467-charger
> +    const: richtek,rt9467
> =20
>    reg:
>      maxItems: 1
> @@ -65,7 +65,7 @@ examples:
>        #size-cells =3D <0>;
> =20
>        charger@5b {
> -        compatible =3D "richtek,rt9467-charger";
> +        compatible =3D "richtek,rt9467";
>          reg =3D <0x5b>;
>          wakeup-source;
>          interrupts-extended =3D <&gpio_intc 32 IRQ_TYPE_LEVEL_LOW>;
> --=20
> 2.7.4
>=20

--r3cq4aakiv6m7xrb
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmPqo0EACgkQ2O7X88g7
+pqQSQ/+OkcfFPZuVrVvDR2ftl8gF8q3d0f9bhdHTcWG7rc3umiDAVn956mOKbMq
Uitnia+BeovzD1oiGhfF5F7sriXqMGo4Ul7Jr3xXmkXzDZw4l5dlRAwtv8R0tlAN
/ELRpIMViEFCWOBlAMEQYkhKECNKtSCe/U69iS6ouwoPkVnxWhpG46zCsMR+9pFX
VSgDyXNf/HPDaTvOhX0p7NMQG8Ov69TzSuwrO21krRCWEcrvdXaXfQ7VrFsYb/2z
XYntcAqeOawPKCn8Rm0QpDGuczujvIWvrqYJMJf5ItAcfH6ga5UwzHRs2dZmOm9q
UlhSxpc3/ZtiS0RzY5+cVT7iFlJ2UKxtVVS9YVhg+NXhwebAFdDIKwdsvGf9Qvzz
hB+FJ8GXVPcUQZr5E7002FDwa/OXum2434AFM0pBrlkxwLXpVthqAUAj3v5DTASW
Q0+qaDID8J84jDAr+0QWulGKp1HbuhOutJg9f34/5oE8Am73IQLBSYh8gq55s6S6
p4tfMnvY4n0ZyV9Elvg8cwlT1x869hLIq96jADp8rxaNMl8h+sXNRzsDq6ZYyLBe
Xs1LLukQSTHV8m8DHJ0BpEjfr+n3K+geisZL1feQSE248scNexm0nSbT8f9oGm+4
TnPKWjKtblKrkh7TGIr35abxhb5Gzq7lkPp2CtxYppW9WuMusfY=
=RCkA
-----END PGP SIGNATURE-----

--r3cq4aakiv6m7xrb--