On 10/11/2023 21:57, Rob Herring wrote:
On Thu, Nov 09, 2023 at 10:00:05AM +0100, Neil Armstrong wrote:
Now this bindings is referred from amlogic,meson-gx-hhi-sysctrl.yaml and is
documented as a subnode of a simple-mfd, drop the invalid reg property.
Why is it invalid? It's preferred to have 'reg' in MFDs even if Linux
doesn't use them. If there's a chunk of registers you can define, then
do so. If it's all register bit soup, then fine, omit it.
I still don't understand why this particular MFD subnode needs a reg and not
the other ones, using reg would need adding #address-cells/#size-cells on top
node and change all examples/DT for nothing.
Like the other meson-gx-hhi-sysctrl subnodes, it's a register bit soup and this
one is no exception.
Neil
Rob