Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-21 Thread Bill Pringlemeir
> On 2014-11-20 20:14, Jeroen Hofstee wrote: >> Ok thanks for digging that up, that doesn't sound like a problem >> then. Stefan, can you check if you can actually fix the warnings >> instead of suppressing them? On 21 Nov 2014, ste...@agner.ch wrote: > Ok, I could apply the changes from your p

Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-21 Thread Albert ARIBAUD
Hello Jeroen, Adding Stefan as it seems he was dropped from the recipients list. Plus : On Thu, 20 Nov 2014 20:14:01 +0100, Jeroen Hofstee wrote: > Hi, > > On 20-11-14 19:21, Bill Pringlemeir wrote: > --- > arch/arm/lib/memset.S | 40 > 1

Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-21 Thread Stefan Agner
On 2014-11-20 20:14, Jeroen Hofstee wrote: > Hi, > > On 20-11-14 19:21, Bill Pringlemeir wrote: > --- > arch/arm/lib/memset.S | 40 > 1 file changed, 20 insertions(+), 20 deletions(-) > > diff --git a/arch/arm/lib/memset.S b/arch/arm/lib/

Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-20 Thread Jeroen Hofstee
Hi, On 20-11-14 19:21, Bill Pringlemeir wrote: --- arch/arm/lib/memset.S | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/arch/arm/lib/memset.S b/arch/arm/lib/memset.S index 0cdf895..4fe38f6 100644 --- a/arch/arm/lib/memset.S +++ b/arc

Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-20 Thread Bill Pringlemeir
>>> --- >>> arch/arm/lib/memset.S | 40 >>> 1 file changed, 20 insertions(+), 20 deletions(-) >>> >>> diff --git a/arch/arm/lib/memset.S b/arch/arm/lib/memset.S >>> index 0cdf895..4fe38f6 100644 >>> --- a/arch/arm/lib/memset.S >>> +++ b/arch/arm/lib/memset.S

Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-20 Thread Jeroen Hofstee
Hi, On 20-11-14 16:18, Bill Pringlemeir wrote: --- arch/arm/lib/memset.S | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/arch/arm/lib/memset.S b/arch/arm/lib/memset.S index 0cdf895..4fe38f6 100644 --- a/arch/arm/lib/memset.S +++ b/a

Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-20 Thread Bill Pringlemeir
> On 20-11-14 13:15, Stefan Agner wrote: >> No particular reason, I did not know how to fix this without digging >> into it. Hence, after I discovered this, I checked why those warnings >> do not happen for the kernel, then I applied just the AFLAGS the >> kernel is using. I guess fixing the under

Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-20 Thread Jeroen Hofstee
Hello Stefan, On 20-11-14 13:15, Stefan Agner wrote: Hi Jeroen, On 2014-11-20 10:21, Jeroen Hofstee wrote: Hello Stefan, On 19-11-14 15:16, Stefan Agner wrote: Resynchronize memcpy/memset with kernel and build them explicitly in Thumb2 mode (unified syntax). Those assembler files can be buil

Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-20 Thread Stefan Agner
Hi Jeroen, On 2014-11-20 10:21, Jeroen Hofstee wrote: > Hello Stefan, > > On 19-11-14 15:16, Stefan Agner wrote: >> Resynchronize memcpy/memset with kernel and build them explicitly >> in Thumb2 mode (unified syntax). Those assembler files can be >> built and linked in ARM mode too, however when

Re: [U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-20 Thread Jeroen Hofstee
Hello Stefan, On 19-11-14 15:16, Stefan Agner wrote: Resynchronize memcpy/memset with kernel and build them explicitly in Thumb2 mode (unified syntax). Those assembler files can be built and linked in ARM mode too, however when calling them from Thumb2 built code, the stack got corrupted and the

[U-Boot] [PATCH] arm: make arch memset/memcpy to work with Thumb2 builds

2014-11-19 Thread Stefan Agner
Resynchronize memcpy/memset with kernel and build them explicitly in Thumb2 mode (unified syntax). Those assembler files can be built and linked in ARM mode too, however when calling them from Thumb2 built code, the stack got corrupted and the copy did not succeed (the exact details have not been t