2023-02-17 09:56:16

by ChiaEn Wu

[permalink] [raw]
Subject: [PATCH v4] dt-bindings: power: supply: Revise Richtek RT9467 compatible name

Revise RT9467 compatible name from "richtek,rt9467-charger" to
"richtek,rt9467" because it has to match the "compatible name" in
the source code.

Fixes: e1b4620fb503 ("dt-bindings: power: supply: Add Richtek RT9467 battery charger")
Reported-by: Rob Herring <[email protected]>
Signed-off-by: ChiaEn Wu <[email protected]>
---
I apologize for misunderstanding your meaning and using the wrong command
to pack the patch, which caused trouble. Sorry......

v4:
- Change to use the correct git format-patch command (-M)

v3:
- Roll back the file name.

v2:
- Add more description about this change in the commit message.
- Rename "richtek,rt9467-charger.yaml" to "richtek,rt9467.yaml".
- Rename "$id" as above.
---
.../supply/{richtek,rt9467-charger.yaml => richtek,rt9467.yaml} | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
rename Documentation/devicetree/bindings/power/supply/{richtek,rt9467-charger.yaml => richtek,rt9467.yaml} (92%)

diff --git a/Documentation/devicetree/bindings/power/supply/richtek,rt9467-charger.yaml b/Documentation/devicetree/bindings/power/supply/richtek,rt9467.yaml
similarity index 92%
rename from Documentation/devicetree/bindings/power/supply/richtek,rt9467-charger.yaml
rename to Documentation/devicetree/bindings/power/supply/richtek,rt9467.yaml
index 92c5706..3723717 100644
--- a/Documentation/devicetree/bindings/power/supply/richtek,rt9467-charger.yaml
+++ b/Documentation/devicetree/bindings/power/supply/richtek,rt9467.yaml
@@ -1,7 +1,7 @@
# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
%YAML 1.2
---
-$id: http://devicetree.org/schemas/power/supply/richtek,rt9467-charger.yaml#
+$id: http://devicetree.org/schemas/power/supply/richtek,rt9467.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

title: Richtek RT9467 Switching Battery Charger with Power Path Management
@@ -25,7 +25,7 @@ description: |

properties:
compatible:
- const: richtek,rt9467-charger
+ const: richtek,rt9467

reg:
maxItems: 1
@@ -65,7 +65,7 @@ examples:
#size-cells = <0>;

charger@5b {
- compatible = "richtek,rt9467-charger";
+ compatible = "richtek,rt9467";
reg = <0x5b>;
wakeup-source;
interrupts-extended = <&gpio_intc 32 IRQ_TYPE_LEVEL_LOW>;
--
2.7.4



2023-02-17 10:11:57

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v4] dt-bindings: power: supply: Revise Richtek RT9467 compatible name

On 17/02/2023 10:55, ChiaEn Wu wrote:
> Revise RT9467 compatible name from "richtek,rt9467-charger" to
> "richtek,rt9467" because it has to match the "compatible name" in
> the source code.
>
> Fixes: e1b4620fb503 ("dt-bindings: power: supply: Add Richtek RT9467 battery charger")
> Reported-by: Rob Herring <[email protected]>
> Signed-off-by: ChiaEn Wu <[email protected]>
> ---


Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof


2023-02-25 00:19:26

by Sebastian Reichel

[permalink] [raw]
Subject: Re: [PATCH v4] dt-bindings: power: supply: Revise Richtek RT9467 compatible name

Hi,

On Fri, Feb 17, 2023 at 11:11:47AM +0100, Krzysztof Kozlowski wrote:
> On 17/02/2023 10:55, ChiaEn Wu wrote:
> > Revise RT9467 compatible name from "richtek,rt9467-charger" to
> > "richtek,rt9467" because it has to match the "compatible name" in
> > the source code.
> >
> > Fixes: e1b4620fb503 ("dt-bindings: power: supply: Add Richtek RT9467 battery charger")
> > Reported-by: Rob Herring <[email protected]>
> > Signed-off-by: ChiaEn Wu <[email protected]>
> > ---
>
>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>

Thanks, queued for v6.3.

-- Sebastian


Attachments:
(No filename) (590.00 B)
signature.asc (833.00 B)
Download all attachments