Re: [PATCH 3/9] ARM: dts: update dts files for bcmbca SoC BCM63138

From: Krzysztof Kozlowski
Date: Wed Jul 06 2022 - 03:54:53 EST


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@xxxxxxxxxxxx>
> ---
>
> 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?



Best regards,
Krzysztof