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

From: Krzysztof Kozlowski

Date: Fri Jul 10 2026 - 06:29:36 EST


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.

Best regards,
Krzysztof