> > The real problem with this is that it resides in orion-time.c which > > is fine for Orion SoCs. Armada 370/XP use a different timer and > > therefore the _common_ watchdog driver cannot call this function. > > Right, I had missed this part. > > > Moreover, Dove (out of Orion) and Armada 370/XP will happily live in > > one V7 kernel with both time-orion and time-armada-370-xp compiled in. > > It still wouldn't be too hard to do: both clock drivers could export > functions with different identifiers, and the wdt driver can use > conditional compilation (#ifdef or if(IS_ENABLED())) to only call > those functions when the respective clocksource drivers are enabled. Hi Arnd. The plan is to make dove part of MULTI_V7. At that point, you have both dove and armada-370-xp in the same kernel. Conditional compilation does not work, the watchdog driver would need to know at runtime which timer is in use, and call the right timer function. I'm working on making DT Kirkwood part of MULTI_V5. The changes needed are mostly trivial. I expect making Dove part of MULTI_V7 is also trivial, so could happen during the next cycle. So this mix of Dove and Armada 370 XP is not a theoretical problem, it is something which needs to be solved ready for merging during the next window. Andrew -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html