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

[Krzysztof Kozlowski <krzk@kernel.org>]

On 10/07/2026 12:20, Alim Akhtar wrote:
> 
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzk@kernel.org>
>> Sent: Friday, July 10, 2026 3:30 PM
>> To: Alim Akhtar <alim.akhtar@samsung.com>; 'Peter Griffin'
>> <peter.griffin@linaro.org>
>> Cc: robh@kernel.org; conor+dt@kernel.org; linusw@kernel.org; linux-samsung-
>> soc@vger.kernel.org; linux-kernel@vger.kernel.org;
>> devicetree@vger.kernel.org; linux-gpio@vger.kernel.org;
>> hajun.sung@samsung.com
>> 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