On 04/03/2025 20:08, Wilson Ding wrote:
>
> I did consider shrinking the syscon's register address range to
> make the reset-controller node to be independent from the
> syscon node. However, I found the syscon node is also referred
> by some devices for miscellaneous configurations . The reset
> configuration register happens to be located in between these
> registers and clock/GPIO registers.
>
>> drop offset in your patch or unify everything into 'reg'.
>>
>
> This is exactly what I proposed in v3 patch. Do I misunderstand
> you?
>
> CP11X_LABEL(swrst): reset-controller@268 {
> compatible = "marvell,armada8k-reset";
> reg = <0x268 0x4>;
> #reset-cells = <1>;
> };
I don't see the other device being fixed here. How did you unify them?
Best regards,
Krzysztof
