Re: [U-Boot] [PATCH] PPC: fix Warning: FOO uses hard float, BAR uses soft float.
Hi Wolfgang, On Thu, 22 Dec 2011 15:29:41 +0100 Wolfgang Denk w...@denx.de wrote: It appears that with recent versions of GCC the explicit -mhard-float command line option takes precedence over the ``asm(.gnu_attribute 4, 2);'' in the source file, so this no longer helps to avoid the warnings we get when linking code that uses FP instructions with other code that was built using soft-float. We can remove the .gnu_attribute (which appears to carry no other information, at least so far) from the object files, but we also have to make sure we don't pull in the __gcc_qsub() and __gcc_qmul() functions from the standard libgcc, as these would again infect our linking. We copy this code from: gcc-4.2.2/gcc/config/rs6000/darwin-ldouble.c This old version was chosen because it was still available under a compatible license (GCC v2+). The file was stripped down to the needed parts, and reformatted so it passes checkpatch with only one warning (do not add new typedefs). Signed-off-by: Wolfgang Denk w...@denx.de Cc: Kumar Gala ga...@kernel.crashing.org Cc: Stefan Roese s...@denx.de Cc: Andy Fleming aflem...@gmail.com Cc: Kim Phillips kim.phill...@freescale.com Tested on TQM5200. Tested-by: Anatolij Gustschin ag...@denx.de Anatolij ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] PPC: fix Warning: FOO uses hard float, BAR uses soft float.
Dear Wolfgang Denk, In message 1324564181-8949-1-git-send-email...@denx.de you wrote: It appears that with recent versions of GCC the explicit -mhard-float command line option takes precedence over the ``asm(.gnu_attribute 4, 2);'' in the source file, so this no longer helps to avoid the warnings we get when linking code that uses FP instructions with other code that was built using soft-float. We can remove the .gnu_attribute (which appears to carry no other information, at least so far) from the object files, but we also have to make sure we don't pull in the __gcc_qsub() and __gcc_qmul() functions from the standard libgcc, as these would again infect our linking. We copy this code from: gcc-4.2.2/gcc/config/rs6000/darwin-ldouble.c This old version was chosen because it was still available under a compatible license (GCC v2+). The file was stripped down to the needed parts, and reformatted so it passes checkpatch with only one warning (do not add new typedefs). Signed-off-by: Wolfgang Denk w...@denx.de Cc: Kumar Gala ga...@kernel.crashing.org Cc: Stefan Roese s...@denx.de Cc: Andy Fleming aflem...@gmail.com Cc: Kim Phillips kim.phill...@freescale.com --- This patch silences build warnings for the following boards: aev BC3450 charon, cm5200 fo300 korat korat_perm lwmon5 MiniFAP PMC440 sacsng sequoia sequoia_nand sequoia_ramboot TB5200 TB5200_B TQM5200 TQM5200_B TQM5200_B_HIGHBOOT TQM5200S TQM5200S_HIGHBOOT TQM5200_STK100 post/lib_powerpc/fpu/Makefile | 17 - post/lib_powerpc/fpu/darwin-ldouble.c | 141 + 2 files changed, 156 insertions(+), 2 deletions(-) create mode 100644 post/lib_powerpc/fpu/darwin-ldouble.c Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de He who hesitates is not only lost, but miles from the next exit. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] PPC: fix Warning: FOO uses hard float, BAR uses soft float.
Hi Wolfgang, On Thursday 22 December 2011 15:29:41 Wolfgang Denk wrote: It appears that with recent versions of GCC the explicit -mhard-float command line option takes precedence over the ``asm(.gnu_attribute 4, 2);'' in the source file, so this no longer helps to avoid the warnings we get when linking code that uses FP instructions with other code that was built using soft-float. We can remove the .gnu_attribute (which appears to carry no other information, at least so far) from the object files, but we also have to make sure we don't pull in the __gcc_qsub() and __gcc_qmul() functions from the standard libgcc, as these would again infect our linking. We copy this code from: gcc-4.2.2/gcc/config/rs6000/darwin-ldouble.c This old version was chosen because it was still available under a compatible license (GCC v2+). The file was stripped down to the needed parts, and reformatted so it passes checkpatch with only one warning (do not add new typedefs). Signed-off-by: Wolfgang Denk w...@denx.de Cc: Kumar Gala ga...@kernel.crashing.org Cc: Stefan Roese s...@denx.de Cc: Andy Fleming aflem...@gmail.com Cc: Kim Phillips kim.phill...@freescale.com Tested successfully on sequoia, so: Tested-by: Stefan Roese s...@denx.de Best regards, Stefan -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: off...@denx.de ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot