On 06/08, Sascha Hauer wrote: > On Fri, Jun 05, 2015 at 05:59:12PM -0700, Stephen Boyd wrote: > > > > And similar things could be done for the reset driver. > > The problem I see with this approach is that we scatter the code for a > otherwise simple driver over a bunch of directories. We would have > > drivers/clk/mediatek/vencsys.c > drivers/reset/mediatek/vencsys.c > drivers/soc/mediatek/vencsys.c > > The same must be added for vdecsys, imgsys and vencltsys. That will make > 12 drivers and three maintainers for 12 registers. I think this will be > a pain to maintain, hence my suggestion to put the vencsys code into a > single file and not split this up into more subsystem specific files. > I probably don't have enough information here, but why is it a pain to maintain? It seems more like a pain to setup the first time and then little to no pain to maintain because we clearly split functionality based on subsystem. No merge conflicts, clear division of functionality, etc. But again, I don't think it matters much either way given that reset and clk drivers are combined sometimes and don't always reside in drivers/clk or drivers/reset either. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- 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