* [Buildroot] [PATCH] arch/mips: Restrict n32 for Sourcery CodeBench toolchains. @ 2013-08-12 12:43 Markos Chandras 2013-08-12 17:42 ` Thomas Petazzoni 0 siblings, 1 reply; 3+ messages in thread From: Markos Chandras @ 2013-08-12 12:43 UTC (permalink / raw) To: buildroot n32 is not supported in Sourcery CodeBench toolchains. Signed-off-by: Markos Chandras <markos.chandras@imgtec.com> --- See https://sourcery.mentor.com/GNUToolchain/release2477?@template=datasheet --- arch/Config.in.mips | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/arch/Config.in.mips b/arch/Config.in.mips index 6242bcc..a01d6bb 100644 --- a/arch/Config.in.mips +++ b/arch/Config.in.mips @@ -46,7 +46,10 @@ config BR2_MIPS_OABI32 bool "o32" config BR2_MIPS_NABI32 bool "n32" - depends on BR2_ARCH_IS_64 + depends on BR2_ARCH_IS_64 && \ + !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS201305 && \ + !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS201209 && \ + !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS201203 config BR2_MIPS_NABI64 bool "n64" depends on BR2_ARCH_IS_64 -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] arch/mips: Restrict n32 for Sourcery CodeBench toolchains. 2013-08-12 12:43 [Buildroot] [PATCH] arch/mips: Restrict n32 for Sourcery CodeBench toolchains Markos Chandras @ 2013-08-12 17:42 ` Thomas Petazzoni 2013-08-12 18:02 ` Markos Chandras 0 siblings, 1 reply; 3+ messages in thread From: Thomas Petazzoni @ 2013-08-12 17:42 UTC (permalink / raw) To: buildroot Dear Markos Chandras, On Mon, 12 Aug 2013 13:43:14 +0100, Markos Chandras wrote: > n32 is not supported in Sourcery CodeBench toolchains. > > Signed-off-by: Markos Chandras <markos.chandras@imgtec.com> > --- > See > https://sourcery.mentor.com/GNUToolchain/release2477?@template=datasheet > --- arch/Config.in.mips | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/arch/Config.in.mips b/arch/Config.in.mips > index 6242bcc..a01d6bb 100644 > --- a/arch/Config.in.mips > +++ b/arch/Config.in.mips > @@ -46,7 +46,10 @@ config BR2_MIPS_OABI32 > bool "o32" > config BR2_MIPS_NABI32 > bool "n32" > - depends on BR2_ARCH_IS_64 > + depends on BR2_ARCH_IS_64 && \ > + !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS201305 && \ > + !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS201209 && \ > + !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS201203 > config BR2_MIPS_NABI64 > bool "n64" > depends on BR2_ARCH_IS_64 I am not sure it's the right way of writing this: in the menuconfig, the ABI selection comes first, and then we have the toolchain selection. So it is quite likely that the user will chose the ABI first, and the enter the toolchain menu to make its toolchain configuration. With your proposal, if the user chooses n32, then goes in the toolchain menu and selects one of the CodeSourcery toolchain, the result would be a change in the ABI selection. I think I would prefer to add a depends on !BR2_MIPS_NABI32 on the affected toolchains (to prevent them from being selected), and then add something like: comment "CodeSourcery toolchains are only available for MIPS o32 and n64" depends on BR2_MIPS_NABI32 (of course adapt the comment to the reality) Would you mind reworking the patch in this direction? Thanks! Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] arch/mips: Restrict n32 for Sourcery CodeBench toolchains. 2013-08-12 17:42 ` Thomas Petazzoni @ 2013-08-12 18:02 ` Markos Chandras 0 siblings, 0 replies; 3+ messages in thread From: Markos Chandras @ 2013-08-12 18:02 UTC (permalink / raw) To: buildroot On 12 August 2013 18:42, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote: > Dear Markos Chandras, > > On Mon, 12 Aug 2013 13:43:14 +0100, Markos Chandras wrote: >> n32 is not supported in Sourcery CodeBench toolchains. >> >> Signed-off-by: Markos Chandras <markos.chandras@imgtec.com> >> --- >> See >> https://sourcery.mentor.com/GNUToolchain/release2477?@template=datasheet >> --- arch/Config.in.mips | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/arch/Config.in.mips b/arch/Config.in.mips >> index 6242bcc..a01d6bb 100644 >> --- a/arch/Config.in.mips >> +++ b/arch/Config.in.mips >> @@ -46,7 +46,10 @@ config BR2_MIPS_OABI32 >> bool "o32" >> config BR2_MIPS_NABI32 >> bool "n32" >> - depends on BR2_ARCH_IS_64 >> + depends on BR2_ARCH_IS_64 && \ >> + !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS201305 && \ >> + !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS201209 && \ >> + !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_MIPS201203 >> config BR2_MIPS_NABI64 >> bool "n64" >> depends on BR2_ARCH_IS_64 > > I am not sure it's the right way of writing this: in the menuconfig, > the ABI selection comes first, and then we have the toolchain > selection. So it is quite likely that the user will chose the ABI > first, and the enter the toolchain menu to make its toolchain > configuration. > > With your proposal, if the user chooses n32, then goes in the toolchain > menu and selects one of the CodeSourcery toolchain, the result would be > a change in the ABI selection. > > I think I would prefer to add a depends on !BR2_MIPS_NABI32 on the > affected toolchains (to prevent them from being selected), and then add > something like: > > comment "CodeSourcery toolchains are only available for MIPS o32 and n64" > depends on BR2_MIPS_NABI32 > > (of course adapt the comment to the reality) > > Would you mind reworking the patch in this direction? > > Thanks! > > Thomas > -- > Thomas Petazzoni, Free Electrons > Kernel, drivers, real-time and embedded Linux > development, consulting, training and support. > http://free-electrons.com > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot Hi Thomas, No problem. I will submit a new patch -- Regards, Markos Chandras ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-08-12 18:02 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-08-12 12:43 [Buildroot] [PATCH] arch/mips: Restrict n32 for Sourcery CodeBench toolchains Markos Chandras 2013-08-12 17:42 ` Thomas Petazzoni 2013-08-12 18:02 ` Markos Chandras
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox