Hi Krzysztof,
> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx>
> Sent: Monday, October 11, 2021 2:34 AM
> To: Linus Walleij <linus.walleij@xxxxxxxxxx>; Chanho Park
> <chanho61.park@xxxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>
> Cc: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx>;
> devicetree@xxxxxxxxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx; Tomasz Figa
> <tomasz.figa@xxxxxxxxx>
> Subject: Re: (subset) [PATCH v3 2/3] arm64: dts: exynos: add initial
> support for exynosautov9 SoC
>
> On 10/10/2021 19:17, Krzysztof Kozlowski wrote:
> > On Sun, 10 Oct 2021 12:22:45 +0900, Chanho Park wrote:
> >> Add minimal support for ExynosAuto v9 SoC[1].
> >>
> >> - Enumarate all pinctrl nodes
> >> - UART with exynos850 compatible
> >> - UFS0 HCI + Phy
> >>
> >> Like exynos850, this also uses fixed-rate clock nodes until clock
> >> driver has been supported. The clock nodes are initialized on
> >> bootloader stage thus we don't need to control them so far.
> >>
> >> [...]
> >
> > Applied, thanks!
> >
> > [2/3] arm64: dts: exynos: add initial support for exynosautov9 SoC
> > commit: f695b3f4c45d0051416fa7315cf553bec70198d8
>
> ... and dropped as build system found dtc W=1 warnings:
>
> ../arch/arm64/boot/dts/exynos/exynosautov9.dtsi:78.20-83.5: Warning
> (unit_address_format): /cpus/cpu@000000: unit name should not have leading
> 0s
> ../arch/arm64/boot/dts/exynos/exynosautov9.dtsi:85.20-90.5: Warning
> (unit_address_format): /cpus/cpu@000100: unit name should not have leading
> 0s
> ../arch/arm64/boot/dts/exynos/exynosautov9.dtsi:92.20-97.5: Warning
> (unit_address_format): /cpus/cpu@000200: unit name should not have leading
> 0s
> ../arch/arm64/boot/dts/exynos/exynosautov9.dtsi:99.20-104.5: Warning
> (unit_address_format): /cpus/cpu@000300: unit name should not have leading
> 0s
> ../arch/arm64/boot/dts/exynos/exynosautov9.dtsi:187.38-198.5: Warning
> (simple_bus_reg): /soc@0/interrupt-controller@11001000: simple-bus unit
> address format error, expected "10101000"
> ../arch/arm64/boot/dts/exynos/exynosautov9.dtsi:255.28-264.5: Warning
> (simple_bus_reg): /soc@0/uart@103000000: simple-bus unit address format
> error, expected "10300000"
>
> Please fix them plus these fixes:
Sorry for your inconvenience. I'll fix above and below what you changed as well.
>
>
> --- a/arch/arm64/boot/dts/exynos/exynosautov9-pinctrl.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynosautov9-pinctrl.dtsi
> @@ -570,14 +570,14 @@ hsi2c1_bus: hsi2c1-bus-pins {
> };
>
> /* PERIC0 USI01 */
> - hsi2c2_bus_pins: hsi2c2-bus-pins {
> + hsi2c2_bus: hsi2c2-bus-pins {
> samsung,pins = "gpp0-4", "gpp0-5";
> samsung,pin-function = <EXYNOS_PIN_FUNC_2>;
> samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>;
> };
>
> /* PERIC0 USI01_I2C */
> - hsi2c3_bus_pins: hsi2c3-bus-pins {
> + hsi2c3_bus: hsi2c3-bus-pins {
> samsung,pins = "gpp0-6", "gpp0-7";
> samsung,pin-function = <EXYNOS_PIN_FUNC_3>;
> samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>; @@ -1066,7 +1066,7
> @@ spi9_cs_func: spi9-cs-func-pins {
> };
>
> /* PERIC1 USI10_SPI */
> - spi10_pins: spi10-pins {
> + spi10_bus: spi10-bus-pins {
> samsung,pins = "gpp5-2", "gpp5-1", "gpp5-0";
> samsung,pin-function = <EXYNOS_PIN_FUNC_2>;
> samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>; @@ -1088,7 +1088,7
> @@ spi10_cs_func: spi10-cs-func-pins {
> };
>
> /* PERIC1 USI11_SPI */
> - spi11_pins: spi11-pins {
> + spi11_bus: spi11-bus-pins {
> samsung,pins = "gpp3-6", "gpp3-5", "gpp3-4";
> samsung,pin-function = <EXYNOS_PIN_FUNC_2>;
> samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>; diff --git
> a/arch/arm64/boot/dts/exynos/exynosautov9.dtsi
> b/arch/arm64/boot/dts/exynos/exynosautov9.dtsi
> index d884ae421e12..1dd1d73a52b8 100644
> --- a/arch/arm64/boot/dts/exynos/exynosautov9.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynosautov9.dtsi
> @@ -171,7 +171,7 @@ uart_clock: uart-clock {
> * Keep the stub clock for ufs driver, until proper clock
> * driver is implemented.
> */
> - ufs_core_clock: uart-core-clock {
> + ufs_core_clock: ufs-core-clock {
>
Thanks.
Best Regards,
Chanho Park
