From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Date: Wed, 20 Apr 2016 15:56:00 -0600 Subject: [U-Boot] [PATCH 17/60] ARM: tegra: move apb_misc.h In-Reply-To: References: <1461099580-3866-1-git-send-email-swarren@wwwdotorg.org> <1461099580-3866-18-git-send-email-swarren@wwwdotorg.org> Message-ID: <5717FAF0.3010305@wwwdotorg.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 04/20/2016 01:26 PM, Simon Glass wrote: > Hi Stephen, > > On 19 April 2016 at 14:58, Stephen Warren wrote: >> From: Stephen Warren >> >> This header is only needed by code local to mach-tegra/, so move it there >> to avoid polluting the global include path. >> diff --git a/arch/arm/mach-tegra/tegra20/warmboot_avp.c b/arch/arm/mach-tegra/tegra20/warmboot_avp.c >> index 616358b657be..0ae401c569b6 100644 >> --- a/arch/arm/mach-tegra/tegra20/warmboot_avp.c >> +++ b/arch/arm/mach-tegra/tegra20/warmboot_avp.c >> @@ -11,10 +11,10 @@ >> #include >> #include >> #include >> -#include >> #include >> #include >> #include >> +#include "../apb_misc.h" > > Is this really an improvement? Yes:-P > What's the rationale? As mentioned in the commit description, the content of the header is only used by code in arch/arm/mach-tegra/. By moving the header file out of a directory that's part of the include path, we ensure that code (such as drivers and core code) can't access the header without explicitly doing something unusual, which should ring alarm bells. This will help avoid future additions of code that touches Tegra internals rather than accessing functionality through standard/generic (or even custom but explicitly "exported") APIs.