From: William Zhang <william.zhang@broadcom.com> To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>, Linux ARM List <linux-arm-kernel@lists.infradead.org> Cc: joel.peshkin@broadcom.com, kursad.oney@broadcom.com, f.fainelli@gmail.com, anand.gore@broadcom.com, Broadcom Kernel List <bcm-kernel-feedback-list@broadcom.com>, philippe.reynes@softathome.com, dan.beygelman@broadcom.com, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/9] ARM: dts: update dts files for bcmbca SoC BCM63138 Date: Wed, 6 Jul 2022 15:07:45 -0700 [thread overview] Message-ID: <27b3a1b7-a5ce-fc98-99d6-824ca6f67d9d@broadcom.com> (raw) In-Reply-To: <326247d4-28ae-87ca-6929-0833eafad3a0@linaro.org> [-- Attachment #1: Type: text/plain, Size: 2503 bytes --] On 7/6/22 00:54, Krzysztof Kozlowski wrote: > On 05/07/2022 19:26, William Zhang wrote: >> Update compatible string and UART node and clock name based on device tree >> binding documents. >> >> Signed-off-by: William Zhang <william.zhang@broadcom.com> >> --- >> >> arch/arm/boot/dts/bcm63138.dtsi | 18 +++++++++--------- >> arch/arm/boot/dts/bcm963138dvt.dts | 8 ++++---- >> 2 files changed, 13 insertions(+), 13 deletions(-) >> >> diff --git a/arch/arm/boot/dts/bcm63138.dtsi b/arch/arm/boot/dts/bcm63138.dtsi >> index cca49a2e2d62..bec13ed9f73b 100644 >> --- a/arch/arm/boot/dts/bcm63138.dtsi >> +++ b/arch/arm/boot/dts/bcm63138.dtsi >> @@ -9,13 +9,13 @@ >> / { >> #address-cells = <1>; >> #size-cells = <1>; >> - compatible = "brcm,bcm63138"; >> - model = "Broadcom BCM63138 DSL SoC"; >> + compatible = "brcm,bcm63138", "brcm,bcmbca"; > > Changing compatibles affects ABI, so you should not combine it with some > node renaming/cleanup patch. Split these. > >> + model = "Broadcom BCM963138 Reference Board"; >> interrupt-parent = <&gic>; >> >> aliases { >> - uart0 = &serial0; >> - uart1 = &serial1; >> + serial0 = &uart0; >> + serial1 = &uart1; >> }; >> >> cpus { >> @@ -174,7 +174,7 @@ sata_phy0: sata-phy@0 { >> }; >> >> /* Legacy UBUS base */ >> - ubus@fffe8000 { >> + bus@fffe8000 { >> compatible = "simple-bus"; >> #address-cells = <1>; >> #size-cells = <1>; >> @@ -185,21 +185,21 @@ timer: timer@80 { >> reg = <0x80 0x3c>; >> }; >> >> - serial0: serial@600 { >> + uart0: serial@600 { >> compatible = "brcm,bcm6345-uart"; >> reg = <0x600 0x1b>; >> interrupts = <GIC_SPI 32 IRQ_TYPE_LEVEL_HIGH>; >> clocks = <&periph_clk>; >> - clock-names = "periph"; >> + clock-names = "refclk"; >> status = "disabled"; >> }; >> >> - serial1: serial@620 { >> + uart1: serial@620 { >> compatible = "brcm,bcm6345-uart"; >> reg = <0x620 0x1b>; >> interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; >> clocks = <&periph_clk>; >> - clock-names = "periph"; >> + clock-names = "refclk"; > > This is even more confusing. You now rename clock-names. How is this > related to board compatible or to node names? Not related to board compatible string but it is the same board dts file that require some updates. Anyway I don't have to change anything else other than the compatible string for this patchset. So I will revert those changes. > > > > Best regards, > Krzysztof [-- Attachment #2: S/MIME Cryptographic Signature --] [-- Type: application/pkcs7-signature, Size: 4212 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: William Zhang <william.zhang@broadcom.com> To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>, Linux ARM List <linux-arm-kernel@lists.infradead.org> Cc: joel.peshkin@broadcom.com, kursad.oney@broadcom.com, f.fainelli@gmail.com, anand.gore@broadcom.com, Broadcom Kernel List <bcm-kernel-feedback-list@broadcom.com>, philippe.reynes@softathome.com, dan.beygelman@broadcom.com, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/9] ARM: dts: update dts files for bcmbca SoC BCM63138 Date: Wed, 6 Jul 2022 15:07:45 -0700 [thread overview] Message-ID: <27b3a1b7-a5ce-fc98-99d6-824ca6f67d9d@broadcom.com> (raw) In-Reply-To: <326247d4-28ae-87ca-6929-0833eafad3a0@linaro.org> [-- Attachment #1.1: Type: text/plain, Size: 2503 bytes --] On 7/6/22 00:54, Krzysztof Kozlowski wrote: > On 05/07/2022 19:26, William Zhang wrote: >> Update compatible string and UART node and clock name based on device tree >> binding documents. >> >> Signed-off-by: William Zhang <william.zhang@broadcom.com> >> --- >> >> arch/arm/boot/dts/bcm63138.dtsi | 18 +++++++++--------- >> arch/arm/boot/dts/bcm963138dvt.dts | 8 ++++---- >> 2 files changed, 13 insertions(+), 13 deletions(-) >> >> diff --git a/arch/arm/boot/dts/bcm63138.dtsi b/arch/arm/boot/dts/bcm63138.dtsi >> index cca49a2e2d62..bec13ed9f73b 100644 >> --- a/arch/arm/boot/dts/bcm63138.dtsi >> +++ b/arch/arm/boot/dts/bcm63138.dtsi >> @@ -9,13 +9,13 @@ >> / { >> #address-cells = <1>; >> #size-cells = <1>; >> - compatible = "brcm,bcm63138"; >> - model = "Broadcom BCM63138 DSL SoC"; >> + compatible = "brcm,bcm63138", "brcm,bcmbca"; > > Changing compatibles affects ABI, so you should not combine it with some > node renaming/cleanup patch. Split these. > >> + model = "Broadcom BCM963138 Reference Board"; >> interrupt-parent = <&gic>; >> >> aliases { >> - uart0 = &serial0; >> - uart1 = &serial1; >> + serial0 = &uart0; >> + serial1 = &uart1; >> }; >> >> cpus { >> @@ -174,7 +174,7 @@ sata_phy0: sata-phy@0 { >> }; >> >> /* Legacy UBUS base */ >> - ubus@fffe8000 { >> + bus@fffe8000 { >> compatible = "simple-bus"; >> #address-cells = <1>; >> #size-cells = <1>; >> @@ -185,21 +185,21 @@ timer: timer@80 { >> reg = <0x80 0x3c>; >> }; >> >> - serial0: serial@600 { >> + uart0: serial@600 { >> compatible = "brcm,bcm6345-uart"; >> reg = <0x600 0x1b>; >> interrupts = <GIC_SPI 32 IRQ_TYPE_LEVEL_HIGH>; >> clocks = <&periph_clk>; >> - clock-names = "periph"; >> + clock-names = "refclk"; >> status = "disabled"; >> }; >> >> - serial1: serial@620 { >> + uart1: serial@620 { >> compatible = "brcm,bcm6345-uart"; >> reg = <0x620 0x1b>; >> interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; >> clocks = <&periph_clk>; >> - clock-names = "periph"; >> + clock-names = "refclk"; > > This is even more confusing. You now rename clock-names. How is this > related to board compatible or to node names? Not related to board compatible string but it is the same board dts file that require some updates. Anyway I don't have to change anything else other than the compatible string for this patchset. So I will revert those changes. > > > > Best regards, > Krzysztof [-- Attachment #1.2: S/MIME Cryptographic Signature --] [-- Type: application/pkcs7-signature, Size: 4212 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-07-06 22:07 UTC|newest] Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-07-05 17:26 [PATCH 0/9] arm: bcmbca: Move BCM63138 SoC support under ARCH_BCMBCA William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-05 17:26 ` [PATCH 1/9] dt-bindings: arm: add BCM63138 SoC William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-06 7:53 ` Krzysztof Kozlowski 2022-07-06 7:53 ` Krzysztof Kozlowski 2022-07-05 17:26 ` [PATCH 2/9] ARM: dts: Move BCM963138DVT board dts to ARCH_BCMBCA William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-05 17:26 ` [PATCH 3/9] ARM: dts: update dts files for bcmbca SoC BCM63138 William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-06 7:54 ` Krzysztof Kozlowski 2022-07-06 7:54 ` Krzysztof Kozlowski 2022-07-06 22:07 ` William Zhang [this message] 2022-07-06 22:07 ` William Zhang 2022-07-05 17:26 ` [PATCH 4/9] ARM: dts: Add BCM63138 generic board dts William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-05 17:26 ` [PATCH 5/9] arm: bcmbca: Replace ARCH_BCM_63XX with ARCH_BCMBCA William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-05 19:10 ` Wolfram Sang 2022-07-05 19:10 ` Wolfram Sang 2022-07-06 3:45 ` Damien Le Moal 2022-07-06 3:45 ` Damien Le Moal 2022-07-06 16:52 ` Vinod Koul 2022-07-06 16:52 ` Vinod Koul 2022-07-05 17:26 ` [PATCH 6/9] arm: bcmbca: Move BCM63138 ARCH_BCM_63XX config to ARCH_BCMBCA William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-05 17:26 ` [PATCH 7/9] arm: bcmbca: Add BCMBCA sub platforms William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-05 17:26 ` [PATCH 8/9] MAINTAINERS: Move BCM63138 to bcmbca arch entry William Zhang 2022-07-05 17:26 ` William Zhang 2022-07-05 17:26 ` [PATCH 9/9] ARM: multi_v7_defconfig: Update configs for BCM63138 William Zhang 2022-07-05 17:26 ` William Zhang
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=27b3a1b7-a5ce-fc98-99d6-824ca6f67d9d@broadcom.com \ --to=william.zhang@broadcom.com \ --cc=anand.gore@broadcom.com \ --cc=bcm-kernel-feedback-list@broadcom.com \ --cc=dan.beygelman@broadcom.com \ --cc=devicetree@vger.kernel.org \ --cc=f.fainelli@gmail.com \ --cc=joel.peshkin@broadcom.com \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=krzysztof.kozlowski@linaro.org \ --cc=kursad.oney@broadcom.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=philippe.reynes@softathome.com \ --cc=robh+dt@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.