On Sun, Aug 4, 2013 at 11:21 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote: > * Christian Daudt <csd_b@xxxxxxxxx> [130802 16:13]: >> On Fri, Aug 2, 2013 at 1:33 AM, Tony Lindgren <tony@xxxxxxxxxxx> wrote: >> > * Jason Cooper <jason@xxxxxxxxxxxxxx> [130731 07:25]: >> >> So, I'd like to propose we discuss some lessons learned and maybe arrive >> >> at some best practices. eg, should we just go with mach-$COMPANY/? How >> >> best to handle config symbols for efficient building? Deprecation path >> >> for legacy (unconverted) boards? >> > >> > A lot of that problem goes away by initializing everything as late >> > as possible, and making things to live under drivers. >> One category of items that we haven't found a good place for in this >> new multiplatform world is where does dt-driven non-driver code reside >> ? e.g. we have a secure monitor access function that only kicks in if >> the appropriate dt entry is available . It currently resides in >> mach-bcm/bcm_kona_smc.c as it seems like the only location for it at >> the moment, but that doesn't seem like the best place because (a) >> mach-bcm might end up littered with one-of cases like this and (b) >> anything in mach-bcm is not visible to arm64 SoCs, and some of those >> in the future will need to share with their arm32 cousins. >> But putting in drivers (e.g. drivers/smc) seems like the wrong thing >> to do also because this is not a driver. >> We have a couple of other smallish pieces of IP that just need a bit >> of generic init code to keep them happy, which we were discussing >> internally where to best land them. At present they are also headed to >> mach-bcm. >> Ultimately the question is 'what is allowed to reside in mach-<misc> >> ?' And by extension: 'is there a good home for everything else ?'' > > Well I guess the question is how early do you need it? > > We only need the following things early on that might be SoC specific: > > 1. Timers for clockevents > > 2. Interrupt controller > > If you need the SoC specific code for the two items above, then > you probably want to set it up in the SoC specific init_early. > > Everything else should be possible to do as device drivers with > initramfs. If the code has an interrupt handler, chances are it > can be a driver :) > I agree that most anything can be made into a driver. I'm asking if that's what we want to do for all arm code that doesn't need to be done at the earliest stages. If there's code that doesn't need interrupts, doesn't register for fileops, doesn't register for any framework, etc... Do we still want to make a driver out of it ? thanks, csd -- 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
