Re: [PATCH 2/6] mips: dts: ralink: rt2880: update system controller node and its consumers

[Krzysztof Kozlowski <krzk@kernel.org>]

On 15/01/2025 16:30, Sergio Paracuellos wrote:
> Current RT2880 device tree file is out of date and must be merged with real
> device tree file used in openWRT project [0]. As a first iteration for this

No, that's not the reason. Whatever forks have, is their problem. The
reason is that you are improving or adding here missing bits but each
bit must be justified. Again: whatever is in OpenWRT is not
justification. Justification is that device is like this, SoC is like
that or DTS is written not according to bindings.

> changes, align the current file with the needed changes for system controller

Please wrap commit message according to Linux coding style / submission
process (neither too early nor over the limit):
https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst#L597

> from '6f3b15586eef ("clk: ralink: add clock and reset driver for MTMIPS SoCs")'.

Use proper commit syntax. '' are wrong, missing commit word.

> 
> [0]: https://github.com/openwrt/openwrt/blob/main/target/linux/ramips/dts/rt2880.dtsi
> 
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> ---
>  arch/mips/boot/dts/ralink/rt2880.dtsi | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/mips/boot/dts/ralink/rt2880.dtsi b/arch/mips/boot/dts/ralink/rt2880.dtsi
> index 8fc1987d9063..1f2ea3434324 100644
> --- a/arch/mips/boot/dts/ralink/rt2880.dtsi
> +++ b/arch/mips/boot/dts/ralink/rt2880.dtsi
> @@ -1,4 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0
> +#include <dt-bindings/clock/mediatek,mtmips-sysc.h>
> +
>  / {
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> @@ -25,9 +27,11 @@ palmbus@300000 {
>  		#address-cells = <1>;
>  		#size-cells = <1>;
>  
> -		sysc@0 {
> -			compatible = "ralink,rt2880-sysc";
> +		sysc: syscon@0 {
> +			compatible = "ralink,rt2880-sysc", "syscon";
>  			reg = <0x0 0x100>;
> +			#clock-cells = <1>;
> +			#reset-cells = <1>;

Nothing from here is explained in commit msg. Why this is suddenly clock
and reset controller?

>  		};
>  


Best regards,
Krzysztof