Hi Bjorn, On Tue, Apr 13, 2021 at 10:59 PM Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> wrote: [...] > Describing these kinds blocks in DT is indeed tricky, I've had > both cases where a block maps to multiple "functions" or where they > contain misc registers to be used in relation to some other block. > > The prior typically lends itself to be modelled as a "simple-mfd" and > the latter as a "syscon". I think here the former description matches better each set of registers has one very specific purpose (pinctrl, GPIO, I2C, RTC, IR receiver, ...). there's only one exception inside the whole AO region called "PMU" (which mostly contains power management registers and a few clock control bits) > So perhaps you could do a simple-mfd that spans the entire block and > then describe the remoteproc, watchdog?, pinctrl pieces as children > under that? I can send patches for the simple-mfd conversion (syscon won't be involved) so Rob can also give his feedback. in my opinion this would not change the dt-bindings for the AO ARC remote-processor. Please let me know if there would be any dt-bindings changes so I can also include this when updating this series. Best regards, Martin
