RE: [PATCH v3 5/6] arm64: dts: exynos: add initial support for Samsung Exynos8855 smdk

[Alim Akhtar]

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Sent: Friday, July 10, 2026 3:55 PM
> To: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>; 'Peter Griffin'
> <peter.griffin@xxxxxxxxxx>
> Cc: robh@xxxxxxxxxx; conor+dt@xxxxxxxxxx; linusw@xxxxxxxxxx; linux-samsung-
> soc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-gpio@xxxxxxxxxxxxxxx;
> hajun.sung@xxxxxxxxxxx
> Subject: Re: [PATCH v3 5/6] arm64: dts: exynos: add initial support for Samsung
> Exynos8855 smdk
> 
> On 10/07/2026 12:20, Alim Akhtar wrote:
> >
> >
> >> -----Original Message-----
> >> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> >> Sent: Friday, July 10, 2026 3:30 PM
> >> To: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>; 'Peter Griffin'
> >> <peter.griffin@xxxxxxxxxx>
> >> Cc: robh@xxxxxxxxxx; conor+dt@xxxxxxxxxx; linusw@xxxxxxxxxx;
> >> linux-samsung- soc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> >> devicetree@xxxxxxxxxxxxxxx; linux-gpio@xxxxxxxxxxxxxxx;
> >> hajun.sung@xxxxxxxxxxx
> >> Subject: Re: [PATCH v3 5/6] arm64: dts: exynos: add initial support
> >> for Samsung
> >> Exynos8855 smdk
> >>
> >> On 10/07/2026 11:55, Alim Akhtar wrote:
> >>>>> ---
> >>>>>  arch/arm64/boot/dts/exynos/Makefile           |   1 +
> >>>>>  .../boot/dts/exynos/exynos8855-pinctrl.dtsi   | 574
> ++++++++++++++++++
> >>>>>  .../arm64/boot/dts/exynos/exynos8855-smdk.dts |  32 +
> >>>>>  arch/arm64/boot/dts/exynos/exynos8855.dtsi    | 204 +++++++
> >>>>>  4 files changed, 811 insertions(+)  create mode 100644
> >>>>> arch/arm64/boot/dts/exynos/exynos8855-pinctrl.dtsi
> >>>>>  create mode 100644 arch/arm64/boot/dts/exynos/exynos8855-smdk.dts
> >>>>>  create mode 100644 arch/arm64/boot/dts/exynos/exynos8855.dtsi
> >>>>> +       };
> >>> [snip]
> >>>>> +
> >>>>> +       oscclk: clock-oscclk {
> >>>>> +               compatible = "fixed-clock";
> >>>>> +               clock-output-names = "oscclk";
> >>>>> +               #clock-cells = <0>;
> >>>>> +       };
> >>>>
> >>>> Small nit, but I believe oscclk node should be ordered
> >>>> alpha-numerically by the node name. See
> >>>> https://docs.kernel.org/devicetree/bindings/dts-coding-
> >>>> style.html
> >>>>
> >>> Thanks, will update in v4
> >>
> >>
> >> The node feels ordered, no? clock before cpu?
> >>
> > Right, " ordered alpha-numerically by the node name"
> >
> > Krzysztof, do you want to send v4 with all the reviewed-by tag collected?
> 
> No need, there are no changes planned, right? I just give it a day or two if Peter
> or someone wants to review this one in such case.
> 
Sure, no changes for now. 
Thanks!

> Best regards,
> Krzysztof