From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darren Hart Subject: Re: [PATCH v6 1/3] clk: x86: Add Atom PMC platform clocks Date: Fri, 16 Dec 2016 11:19:51 -0800 Message-ID: <20161216191951.GC44199@f23x64.localdomain> References: <1481306510-7471-1-git-send-email-irina.tirdea@intel.com> <1481306510-7471-2-git-send-email-irina.tirdea@intel.com> <1a98dcda-a662-958e-307f-5fe3f281ed9f@linux.intel.com> <20161216183607.GB44199@f23x64.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: platform-driver-x86-owner@vger.kernel.org To: Andy Shevchenko Cc: Pierre-Louis Bossart , Irina Tirdea , linux-clk@vger.kernel.org, "x86@kernel.org" , platform-driver-x86@vger.kernel.org, Stephen Boyd , Thomas Gleixner , Michael Turquette , Ingo Molnar , "H. Peter Anvin" , ALSA Development Mailing List , Mark Brown , Takashi Iwai , Pierre-Louis Bossart , "Rafael J. Wysocki" , Len Brown , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" List-Id: linux-acpi@vger.kernel.org On Fri, Dec 16, 2016 at 08:49:13PM +0200, Andy Shevchenko wrote: > On Fri, Dec 16, 2016 at 8:36 PM, Darren Hart wrote: > > On Tue, Dec 13, 2016 at 02:26:21AM +0200, Andy Shevchenko wrote: > >> On Tue, Dec 13, 2016 at 2:15 AM, Pierre-Louis Bossart > >> wrote: > > >> For clock I would suggest include/linux/clk/ with x86_ prefix. > >> For the rest I have no strong opinion except trying to avoid > >> platform_data wording in the path as much as possible. > >> > >> As an example I could recall DMA engine subsystem where we have > >> > >> include/linux/platform_data/dma-*.h > >> > >> and > >> > >> include/linux/dma/*.h > >> > >> So, this sounds more to me as > >> > >> include/linux/x86/pmc_atom.h > > > > There should really be some Documentation about how to choose an include > > directory :-) > > So true! > > > My understanding is include/linux should be more generic, rather than platform > > specific headers. So while platform_data may refer specifically to the platform > > bus drivers, it's the closest thing we have to include/platform, which would be > > ideal. I would prefer to stick with include/platform_data because: > > > > 1) Semantically, it's the closest thing there is > > 2) include/linux should be for more generic headers related to the OS or > > subsystems Scratch #2 from the arguments since it's include/linux/platform_data that we're talking about here. > > 3) It doesn't make sense to create a separate include/platform directory for a > > single header. > > 4) We don't want to rename platform_data to platform now and change all the > > drivers, but it could be changed later. > > My understanding that part like P-Unit, PMIC, PMC, SCU, whatever we > have inside SoC is platform from hardware prospective, but from > software (driver) it doesn't use platform data since it's quite SoC > specific (like CPU model to differentiate). That's why something in > the middle between arch/x86/include/asm and > include/linux/platform_data. > > I assume I would be not good in naming schemes, though platform_data > for file which doesn't contain platform data for platform device > sounds a bit confusing to me. Like someone already noticed > include/platform_data is already messy. This might just add another > level of it. > > So, what is exactly confuses me is mixing data for *platform devices* > (as represented via *platform driver* -- struct platform_driver) and > for SoC devices (no struct platform_driver per se). > Maybe I misunderstood something... You're understanding is correct. We're just applying different values to the respective merits of each argument. The options are: a) include/linux/x86 b) include/linux/platform_data/x86 In my opinion, a) looks like architecture and would be difficult to distinguish from arch/x86/include. b) on the other hand clearly notes that it is for platform specific information. If it was platform instead of platform_data, that would be even better, but that could be a later change. But I think the confusion over x86 arch in a) is worse than the more subtle (in my opinion) distinction between "platform" and "platform_data". I would want x86 maintainer approval before adding a), while b) I'm happy to add ourselves - and we already have agreement from tglx on that. To move forward, let's go with b). The new x86 directory clearly separates out content which will make it trivial to move later if the need arises. -- Darren Hart Intel Open Source Technology Center