Re: [OE-core] [OE-Core][PATCH 0/9] Add MIPS release 6 ISA support

2016-10-07 Thread Richard Purdie
On Fri, 2016-10-07 at 17:19 +0100, Richard Purdie wrote: > On Fri, 2016-10-07 at 17:07 +0100, Zubair Lutfullah Kakakhel wrote: > > > > On 10/07/2016 04:48 PM, Richard Purdie wrote: > > > > > > On Fri, 2016-10-07 at 12:53 +0100, Zubair Lutfullah Kakakhel > > > wrote: > > > What I will say though i

Re: [OE-core] [OE-Core][PATCH 0/9] Add MIPS release 6 ISA support

2016-10-07 Thread Richard Purdie
On Fri, 2016-10-07 at 17:07 +0100, Zubair Lutfullah Kakakhel wrote: > On 10/07/2016 04:48 PM, Richard Purdie wrote: > > On Fri, 2016-10-07 at 12:53 +0100, Zubair Lutfullah Kakakhel wrote: > > What I will say though is that we really need to think about an > > "allmips", or "mipsarch" override, rath

Re: [OE-core] [OE-Core][PATCH 0/9] Add MIPS release 6 ISA support

2016-10-07 Thread Zubair Lutfullah Kakakhel
Hi, On 10/07/2016 04:48 PM, Richard Purdie wrote: On Fri, 2016-10-07 at 12:53 +0100, Zubair Lutfullah Kakakhel wrote: On 10/06/2016 05:43 PM, Mark Hatle wrote: A few general comments. I don't see an n32 variant implemented. I'd suggest a follow on patch (vs a new version) to add n32. It lo

Re: [OE-core] [OE-Core][PATCH 0/9] Add MIPS release 6 ISA support

2016-10-07 Thread Richard Purdie
On Fri, 2016-10-07 at 12:53 +0100, Zubair Lutfullah Kakakhel wrote: > On 10/06/2016 05:43 PM, Mark Hatle wrote: > > > > A few general comments.  I don't see an n32 variant > > implemented.  I'd suggest a > > follow on patch (vs a new version) to add n32. > > > > It looks like it's missing in the

Re: [OE-core] [OE-Core][PATCH 0/9] Add MIPS release 6 ISA support

2016-10-07 Thread Zubair Lutfullah Kakakhel
Hi, On 10/06/2016 05:43 PM, Mark Hatle wrote: A few general comments. I don't see an n32 variant implemented. I'd suggest a follow on patch (vs a new version) to add n32. It looks like it's missing in the files modified by patches 6-9. Thanks for the review. Yes I haven't added the n32 vari

Re: [OE-core] [OE-Core][PATCH 0/9] Add MIPS release 6 ISA support

2016-10-06 Thread Mark Hatle
A few general comments. I don't see an n32 variant implemented. I'd suggest a follow on patch (vs a new version) to add n32. It looks like it's missing in the files modified by patches 6-9. Otherwise I don't see any issues with adding the support for this new ISA. --Mark On 10/4/16 6:31 AM, Z

[OE-core] [OE-Core][PATCH 0/9] Add MIPS release 6 ISA support

2016-10-04 Thread Zubair Lutfullah Kakakhel
Hi, This patch series adds support for MIPS Release 6 ISA. A few key points to note are: 1 - The tuples have changed for Multiarch. For details, check https://wiki.debian.org/Multiarch/Tuples 2 - Dynamic linker loader name has changed. ld.so.1 -> ld-linux-mipsn8.so.1 For details, c