On Fri, 11 Oct 2013 10:37:38 -0700 Brian Norris <computersforpe...@gmail.com> wrote:
> Several functions are only ever referenced locally, so make them static. > Of those functions, many of them are protected by an #if. However, the > code which can compile fine in either case. > > Now that (1) the unneeded code is marked 'static' and (2) the code is > only used under a C 'if (IS_ENABLED(CONFIG_FB_FSL_DIU))', the compiler > can automatically remove the unneeded code, and we don't need the #if or > the empty stub functions. > > Signed-off-by: Brian Norris <computersforpe...@gmail.com> > --- > v2: left out a diff I was holding locally (to remove an #if/#endif > completely). Sorry for the noise. > > Based off of Gerhard Sittig's patch: > powerpc/mpc512x: silence build warning upon disabled DIU > > Compile-tested with CONFIG_FB_FSL_DIU=n > > arch/powerpc/platforms/512x/mpc512x_shared.c | 21 +++++++-------------- > arch/powerpc/sysdev/fsl_soc.h | 3 --- > 2 files changed, 7 insertions(+), 17 deletions(-) Applied, thanks! Anatolij _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev