* [PATCH] arm: fix sparse flags for build on 64bit machines [not found] <20170726013020.GL2146@codeaurora.org> @ 2017-07-26 8:43 ` Luc Van Oostenryck 2017-07-26 18:26 ` Stephen Boyd 0 siblings, 1 reply; 5+ messages in thread From: Luc Van Oostenryck @ 2017-07-26 8:43 UTC (permalink / raw) To: linux-arm-kernel By default sparse uses the characteristics of the build machine to infer things like the wordsize. This is fine when doing native builds but for ARM it's, I suspect, very rarely the case and if the build are done on a 64bit machine we get a bunch of warnings like: 'cast truncates bits from constant value (... becomes ...)' Fix this by adding the -m32 flags for sparse. Reported-by: Stephen Boyd <sboyd@codeaurora.org> Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com> --- arch/arm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/Makefile b/arch/arm/Makefile index 65f4e2a4e..121fd65f8 100644 --- a/arch/arm/Makefile +++ b/arch/arm/Makefile @@ -131,7 +131,7 @@ endif KBUILD_CFLAGS +=$(CFLAGS_ABI) $(CFLAGS_ISA) $(arch-y) $(tune-y) $(call cc-option,-mshort-load-bytes,$(call cc-option,-malignment-traps,)) -msoft-float -Uarm KBUILD_AFLAGS +=$(CFLAGS_ABI) $(AFLAGS_ISA) $(arch-y) $(tune-y) -include asm/unified.h -msoft-float -CHECKFLAGS += -D__arm__ +CHECKFLAGS += -D__arm__ -m32 #Default value head-y := arch/arm/kernel/head$(MMUEXT).o -- 2.13.2 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] arm: fix sparse flags for build on 64bit machines 2017-07-26 8:43 ` [PATCH] arm: fix sparse flags for build on 64bit machines Luc Van Oostenryck @ 2017-07-26 18:26 ` Stephen Boyd 2017-08-07 20:56 ` Stephen Boyd 0 siblings, 1 reply; 5+ messages in thread From: Stephen Boyd @ 2017-07-26 18:26 UTC (permalink / raw) To: linux-arm-kernel On 07/26/2017 01:43 AM, Luc Van Oostenryck wrote: > By default sparse uses the characteristics of the build > machine to infer things like the wordsize. > This is fine when doing native builds but for ARM it's, > I suspect, very rarely the case and if the build are done > on a 64bit machine we get a bunch of warnings like: > 'cast truncates bits from constant value (... becomes ...)' > > Fix this by adding the -m32 flags for sparse. > > Reported-by: Stephen Boyd <sboyd@codeaurora.org> > Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com> Tested-by: Stephen Boyd <sboyd@codeaurora.org> Thanks. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] arm: fix sparse flags for build on 64bit machines 2017-07-26 18:26 ` Stephen Boyd @ 2017-08-07 20:56 ` Stephen Boyd 2017-09-29 18:37 ` Stephen Boyd 0 siblings, 1 reply; 5+ messages in thread From: Stephen Boyd @ 2017-08-07 20:56 UTC (permalink / raw) To: linux-arm-kernel On 07/26/2017 11:26 AM, Stephen Boyd wrote: > On 07/26/2017 01:43 AM, Luc Van Oostenryck wrote: >> By default sparse uses the characteristics of the build >> machine to infer things like the wordsize. >> This is fine when doing native builds but for ARM it's, >> I suspect, very rarely the case and if the build are done >> on a 64bit machine we get a bunch of warnings like: >> 'cast truncates bits from constant value (... becomes ...)' >> >> Fix this by adding the -m32 flags for sparse. >> >> Reported-by: Stephen Boyd <sboyd@codeaurora.org> >> Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com> > Tested-by: Stephen Boyd <sboyd@codeaurora.org> > > Russell, is this good to put in the patch tracker? Luc, can you submit this via Russell's patch tracking system (see http://www.arm.linux.org.uk/developer/patches/info.php)? -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] arm: fix sparse flags for build on 64bit machines 2017-08-07 20:56 ` Stephen Boyd @ 2017-09-29 18:37 ` Stephen Boyd 2017-10-02 19:36 ` Luc Van Oostenryck 0 siblings, 1 reply; 5+ messages in thread From: Stephen Boyd @ 2017-09-29 18:37 UTC (permalink / raw) To: linux-arm-kernel On 08/07, Stephen Boyd wrote: > On 07/26/2017 11:26 AM, Stephen Boyd wrote: > > On 07/26/2017 01:43 AM, Luc Van Oostenryck wrote: > >> By default sparse uses the characteristics of the build > >> machine to infer things like the wordsize. > >> This is fine when doing native builds but for ARM it's, > >> I suspect, very rarely the case and if the build are done > >> on a 64bit machine we get a bunch of warnings like: > >> 'cast truncates bits from constant value (... becomes ...)' > >> > >> Fix this by adding the -m32 flags for sparse. > >> > >> Reported-by: Stephen Boyd <sboyd@codeaurora.org> > >> Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com> > > Tested-by: Stephen Boyd <sboyd@codeaurora.org> > > > > > > Russell, is this good to put in the patch tracker? > > > Luc, can you submit this via Russell's patch tracking system (see > http://www.arm.linux.org.uk/developer/patches/info.php)? > It's been a while, and this isn't in linux-next or the patch tracker from what I can tell. I'll forward it along if nothing happens in the next couple days. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] arm: fix sparse flags for build on 64bit machines 2017-09-29 18:37 ` Stephen Boyd @ 2017-10-02 19:36 ` Luc Van Oostenryck 0 siblings, 0 replies; 5+ messages in thread From: Luc Van Oostenryck @ 2017-10-02 19:36 UTC (permalink / raw) To: linux-arm-kernel On Fri, Sep 29, 2017 at 8:37 PM, Stephen Boyd <sboyd@codeaurora.org> wrote: > On 08/07, Stephen Boyd wrote: >> On 07/26/2017 11:26 AM, Stephen Boyd wrote: >> > On 07/26/2017 01:43 AM, Luc Van Oostenryck wrote: >> >> By default sparse uses the characteristics of the build >> >> machine to infer things like the wordsize. >> >> This is fine when doing native builds but for ARM it's, >> >> I suspect, very rarely the case and if the build are done >> >> on a 64bit machine we get a bunch of warnings like: >> >> 'cast truncates bits from constant value (... becomes ...)' >> >> >> >> Fix this by adding the -m32 flags for sparse. >> >> >> >> Reported-by: Stephen Boyd <sboyd@codeaurora.org> >> >> Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com> >> > Tested-by: Stephen Boyd <sboyd@codeaurora.org> >> > >> > >> >> Russell, is this good to put in the patch tracker? >> >> >> Luc, can you submit this via Russell's patch tracking system (see >> http://www.arm.linux.org.uk/developer/patches/info.php)? >> > > It's been a while, and this isn't in linux-next or the patch > tracker from what I can tell. I'll forward it along if nothing > happens in the next couple days. Sorry, I had forgot about it. It's now added as patch 8701/1 in Russell's system. -- Luc ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-10-02 19:36 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20170726013020.GL2146@codeaurora.org>
2017-07-26 8:43 ` [PATCH] arm: fix sparse flags for build on 64bit machines Luc Van Oostenryck
2017-07-26 18:26 ` Stephen Boyd
2017-08-07 20:56 ` Stephen Boyd
2017-09-29 18:37 ` Stephen Boyd
2017-10-02 19:36 ` Luc Van Oostenryck
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).