Re: [PATCH v1 1/4] dt-bindings: watchdog: mediatek,mtk-wdt: add MT7988 syscon requirement

[Frank Wunderlich]

Am 13. Juli 2024 12:21:46 MESZ schrieb Krzysztof Kozlowski <krzk at kernel.org>:
>On 09/07/2024 12:13, Frank Wunderlich wrote:
>> From: Frank Wunderlich <frank-w at public-files.de>
>> 
>> This is needed by u-boot-driver when using OF_UPSTREAM.
>
>Thanks for providing context. What we miss here still, is the
>description of the hardware and for what this syscon is used. Why this
>become system controller and before was not? Or this is just for regmap
>- then it does not work that way... You cannot have whatever driver
>poking all around then claim - I have crappy driver poking to every
>driver without implementing proper driver model, so I add syscon to
>bindings.
>
>
>
>Best regards,
>Krzysztof
>

The nodes here are linked to the ethernet-node and syscon is only needed to get the regmap...the access to it is done by ethernet driver. It does work,but maybe not the right way...
regards Frank