On Mon, Jun 30, 2014 at 10:01:19AM +0100, Thierry Reding wrote: > On Mon, Jun 30, 2014 at 09:20:30AM +0200, Arnd Bergmann wrote: > > On Saturday 28 June 2014 22:40:26 Thierry Reding wrote: > > > Also, the real win we get from moving code out into drivers/ is if we > > > can provide a common framework for them. I don't see how we can possibly > > > do that for this code since there simply isn't enough commonality > > > between SoCs. At the same time we now have a situation where both 32-bit > > > and 64-bit variants of some SoCs share some of the same hardware at the > > > very low level and since we don't have mach-* directories for 64-bit ARM > > > and shouldn't be duplicating code either, we need to find a new home for > > > this type of code. drivers/soc seemed to fit perfectly well. > > > > For the low-level stuff yes, but I agree with Santosh there needs to be > > some more work trying to split out individual high-level drivers. > > > > There are two common patterns for the interface between the low-level > > register access and the more high-level stuff: you can either use > > a "syscon" driver that just exports a struct regmap and that gets referenced > > from other drivers using a phandle in their device nodes, or you have > > a driver in drivers/soc that exports a somewhat higher-level interface > > and comes with its own header file that gets included by other drivers. > > At the moment, the syscon/regmap variant can only be used once device > > drivers are loaded, but there is some broad agreement that it should > > be changed to allow calling syscon_regmap_lookup_by_phandle() at > > early boot using just DT accessors. > > Note that we already have all these drivers and they do work for earlier > Tegra generations. My attempt to move these out of arch/arm/mach-tegra > is merely about being able to share them with upcoming 64-bit variants. > So it's not about adding new functionality but rather about keeping the > existing level of functionality on 64-bit. Only that for arm64 we try to do things slightly differently and not because we just like to but because we want better standardisation across SoCs. One example is the booting protocol. Another example, you don't get some early initcalls for your SoC (no machine descriptor). The hardware should be configured by firmware in such a way that it boots at least up to arch_initcall() level (ideally, we should move anything to device_initcall() but it's not always easy). > I'm not a fan of the syscon/regmap approach at all and the existing > drivers use a higher-level API already, so I think we're going to stick > with that. It's not about whether you are a fan of it or not but whether we can get more code sharing across several SoCs and not just between tegra arm32 and arm64 versions. -- Catalin -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html