From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Subject: Re: Re: Build regressions/improvements in v3.4-rc7 Date: Thu, 17 May 2012 13:42:44 -0500 Message-ID: <4FB546A4.7090502@freescale.com> References: <1337157034-22773-1-git-send-email-geert@linux-m68k.org> <1337212281.30558.17.camel@pasglop> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Cc: Geert Uytterhoeven , , Linuxppc-dev , the arch/x86 maintainers , Parisc List , Scott Wood , Kumar Gala To: Benjamin Herrenschmidt Return-path: In-Reply-To: <1337212281.30558.17.camel@pasglop> List-ID: List-Id: linux-parisc.vger.kernel.org Benjamin Herrenschmidt wrote: > + arch/powerpc/platforms/512x/mpc512x_shared.c: error: 'enum >> fsl_diu_monitor_port' declared inside parameter list [-Werror]: => >> 70:9, 84:9, 88:36 >> + arch/powerpc/platforms/512x/mpc512x_shared.c: error: 'return' with >> a value, in function returning void [-Werror]: => 189:2 >> + arch/powerpc/platforms/512x/mpc512x_shared.c: error: function >> declaration isn't a prototype [-Werror=strict-prototypes]: => 69:5, >> 88:6, 83:6, 187:1 >> + arch/powerpc/platforms/512x/mpc512x_shared.c: error: its scope is >> only this definition or declaration, which is probably not what you >> want [-Werror]: => 70:9 So I think I have a fix for these, but I still would like the actual .config so that I can be sure. >> + drivers/virt/fsl_hypervisor.c: error: 'MSR_GS' undeclared (first >> use in this function): => 799:80 I can't reproduce this one. MSR_GS is defined in reg_booke.h. fsl_hypervisor.c should not be compiled on a non-booke platform. I don't know if the fix is to add this to fsl_hypervisor.c: #include or if I should update the Kconfig: config FSL_HV_MANAGER tristate "Freescale hypervisor management driver" depends on FSL_SOC to depends on FSL_BOOKE or maybe depends on FSL_BOOKE && FSL_SOC -- Timur Tabi Linux kernel developer at Freescale