diff mbox series

[1/2] dt-bindings: arm: qcom: rename vendor of apq8026-lenok

Message ID 20210928203815.77175-1-luca@z3ntu.xyz
State Not Applicable, archived
Headers show
Series [1/2] dt-bindings: arm: qcom: rename vendor of apq8026-lenok | expand

Checks

Context Check Description
robh/checkpatch warning total: 0 errors, 1 warnings, 8 lines checked

Commit Message

Luca Weiss Sept. 28, 2021, 8:38 p.m. UTC
In order to avoid having prefixes for multiple internal divisions of LG
use the "lg" prefix instead of "lge".

Fixes: 21f3cbf693b0 ("dt-bindings: arm: qcom: Document APQ8026 SoC binding")
Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Rob Herring (Arm) Oct. 4, 2021, 6:39 p.m. UTC | #1
On Tue, 28 Sep 2021 22:38:14 +0200, Luca Weiss wrote:
> In order to avoid having prefixes for multiple internal divisions of LG
> use the "lg" prefix instead of "lge".
> 
> Fixes: 21f3cbf693b0 ("dt-bindings: arm: qcom: Document APQ8026 SoC binding")
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>
David Heidelberg Oct. 5, 2021, 11:08 p.m. UTC | #2
Acked-by: David Heidelberg <david@ixit.cz>
Best regards
David Heidelberg

On Mon, Oct 4 2021 at 13:39:31 -0500, Rob Herring <robh@kernel.org> 
wrote:
> On Tue, 28 Sep 2021 22:38:14 +0200, Luca Weiss wrote:
>>  In order to avoid having prefixes for multiple internal divisions 
>> of LG
>>  use the "lg" prefix instead of "lge".
>> 
>>  Fixes: 21f3cbf693b0 ("dt-bindings: arm: qcom: Document APQ8026 SoC 
>> binding")
>>  Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
>>  ---
>>   Documentation/devicetree/bindings/arm/qcom.yaml | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>> 
> 
> Acked-by: Rob Herring <robh@kernel.org>
Geert Uytterhoeven Nov. 2, 2021, 1:07 p.m. UTC | #3
Hi Luca,

On Tue, Sep 28, 2021 at 10:39 PM Luca Weiss <luca@z3ntu.xyz> wrote:
> In order to avoid having prefixes for multiple internal divisions of LG
> use the "lg" prefix instead of "lge".
>
> Fixes: ad3f04b7bef6 ("ARM: dts: qcom: Add support for LG G Watch R")
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
>  arch/arm/boot/dts/Makefile                                      | 2 +-
>  .../{qcom-apq8026-lge-lenok.dts => qcom-apq8026-lg-lenok.dts}   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>  rename arch/arm/boot/dts/{qcom-apq8026-lge-lenok.dts => qcom-apq8026-lg-lenok.dts} (99%)

What about the second board DTS using "lge"?
arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index a2e679112737..06da9ab5c0b6 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -96,7 +96,7 @@  properties:
 
       - items:
           - enum:
-              - lge,lenok
+              - lg,lenok
           - const: qcom,apq8026
 
       - items: