Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-31 Thread Pavel Machek
Hi! > >You can do without ret variable... Also more detailed changelog would > >be nice... like what exact problem this works around. > > > > Sure i can, but I don't see a reason to ignore SM's return value. Changelog > of what? > > I guess if you read the thread over the ML you'll have

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-31 Thread Pavel Machek
Hi! You can do without ret variable... Also more detailed changelog would be nice... like what exact problem this works around. Sure i can, but I don't see a reason to ignore SM's return value. Changelog of what? I guess if you read the thread over the ML you'll have your questions

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-30 Thread Ивайло Димитров
Hi, > Оригинално писмо >От: Pavel Machek >Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround >До: Pali Rohár >Изпратено на: Събота, 2013, Март 30 20:36:54 EET > > >Hi! > >> +u32 rx51_secure_dispatcher(u32 idx, u

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-30 Thread Pavel Machek
Hi! > +u32 rx51_secure_dispatcher(u32 idx, u32 flag, u32 nargs, u32 arg1, u32 arg2, > +u32 arg3, u32 arg4) > +{ > + u32 ret; > + u32 param[5]; > + > + param[0] = nargs+1; > + param[1] = arg1; > + param[2] = arg2; > + param[3] = arg3; > +

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-30 Thread Pavel Machek
Hi! +u32 rx51_secure_dispatcher(u32 idx, u32 flag, u32 nargs, u32 arg1, u32 arg2, +u32 arg3, u32 arg4) +{ + u32 ret; + u32 param[5]; + + param[0] = nargs+1; + param[1] = arg1; + param[2] = arg2; + param[3] = arg3; + param[4] = arg4;

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-30 Thread Ивайло Димитров
Hi, Оригинално писмо От: Pavel Machek Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround До: Pali Rohár Изпратено на: Събота, 2013, Март 30 20:36:54 EET Hi! +u32 rx51_secure_dispatcher(u32 idx, u32 flag, u32 nargs, u32 arg1, u32 arg2

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-28 Thread Tony Lindgren
* Ивайло Димитров [130327 22:34]: > Tony, > > Who do you expect to make that code merge? Do you expect us to mechanically > merge RX51 PPA API patch with the existing generic OMAP PPA API code putting > #ifdefs all over the place? Not that it is impossible, but the only real > piece of HW I

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-28 Thread Tony Lindgren
* Santosh Shilimkar [130328 03:10]: > On Thursday 28 March 2013 03:20 PM, Russell King - ARM Linux wrote: > > On Wed, Mar 27, 2013 at 01:56:07PM -0700, Tony Lindgren wrote: > >> * Pali Rohár [130324 07:31]: > >>> it is possible to upstream errata 430973 workaround for RX-51? > >> > >> I think we

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-28 Thread Santosh Shilimkar
On Thursday 28 March 2013 03:20 PM, Russell King - ARM Linux wrote: > On Wed, Mar 27, 2013 at 01:56:07PM -0700, Tony Lindgren wrote: >> * Pali Rohár [130324 07:31]: >>> it is possible to upstream errata 430973 workaround for RX-51? >> >> I think we should make the SMC handling a generic function

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-28 Thread Russell King - ARM Linux
On Wed, Mar 27, 2013 at 01:56:07PM -0700, Tony Lindgren wrote: > * Pali Rohár [130324 07:31]: > > it is possible to upstream errata 430973 workaround for RX-51? > > I think we should make the SMC handling a generic function for ARM. > > AFAIK just the SMC call numbering is different for various

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-28 Thread Russell King - ARM Linux
On Wed, Mar 27, 2013 at 01:56:07PM -0700, Tony Lindgren wrote: * Pali Rohár pali.ro...@gmail.com [130324 07:31]: it is possible to upstream errata 430973 workaround for RX-51? I think we should make the SMC handling a generic function for ARM. AFAIK just the SMC call numbering is

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-28 Thread Santosh Shilimkar
On Thursday 28 March 2013 03:20 PM, Russell King - ARM Linux wrote: On Wed, Mar 27, 2013 at 01:56:07PM -0700, Tony Lindgren wrote: * Pali Rohár pali.ro...@gmail.com [130324 07:31]: it is possible to upstream errata 430973 workaround for RX-51? I think we should make the SMC handling a generic

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-28 Thread Tony Lindgren
* Santosh Shilimkar santosh.shilim...@ti.com [130328 03:10]: On Thursday 28 March 2013 03:20 PM, Russell King - ARM Linux wrote: On Wed, Mar 27, 2013 at 01:56:07PM -0700, Tony Lindgren wrote: * Pali Rohár pali.ro...@gmail.com [130324 07:31]: it is possible to upstream errata 430973

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-28 Thread Tony Lindgren
* Ивайло Димитров freemangor...@abv.bg [130327 22:34]: Tony, Who do you expect to make that code merge? Do you expect us to mechanically merge RX51 PPA API patch with the existing generic OMAP PPA API code putting #ifdefs all over the place? Not that it is impossible, but the only real

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-27 Thread Ивайло Димитров
the code will still work on the other platforms besides RX51, once the code modified. Please, advice on how to proceed. Regards, Ivo > Оригинално писмо >От: Tony Lindgren >Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround >До: Pali Rohár &

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-27 Thread Tony Lindgren
* Pali Rohár [130324 07:31]: > it is possible to upstream errata 430973 workaround for RX-51? I think we should make the SMC handling a generic function for ARM. AFAIK just the SMC call numbering is different for various implementations. So the handler and passing of the parameters seems like

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-27 Thread Tony Lindgren
* Pali Rohár [130327 14:09]: > On Wednesday 27 March 2013 21:56:07 Tony Lindgren wrote: > > * Pali Rohár [130324 07:31]: > > > it is possible to upstream errata 430973 workaround for > > > RX-51? > > > > I think we should make the SMC handling a generic function for > > ARM. > > > > AFAIK just

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-27 Thread Pali Rohár
On Wednesday 27 March 2013 21:56:07 Tony Lindgren wrote: > * Pali Rohár [130324 07:31]: > > it is possible to upstream errata 430973 workaround for > > RX-51? > > I think we should make the SMC handling a generic function for > ARM. > > AFAIK just the SMC call numbering is different for various

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-27 Thread Pali Rohár
On Wednesday 27 March 2013 21:56:07 Tony Lindgren wrote: * Pali Rohár pali.ro...@gmail.com [130324 07:31]: it is possible to upstream errata 430973 workaround for RX-51? I think we should make the SMC handling a generic function for ARM. AFAIK just the SMC call numbering is different

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-27 Thread Tony Lindgren
* Pali Rohár pali.ro...@gmail.com [130327 14:09]: On Wednesday 27 March 2013 21:56:07 Tony Lindgren wrote: * Pali Rohár pali.ro...@gmail.com [130324 07:31]: it is possible to upstream errata 430973 workaround for RX-51? I think we should make the SMC handling a generic function for

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-27 Thread Tony Lindgren
* Pali Rohár pali.ro...@gmail.com [130324 07:31]: it is possible to upstream errata 430973 workaround for RX-51? I think we should make the SMC handling a generic function for ARM. AFAIK just the SMC call numbering is different for various implementations. So the handler and passing of the

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-27 Thread Ивайло Димитров
the code will still work on the other platforms besides RX51, once the code modified. Please, advice on how to proceed. Regards, Ivo Оригинално писмо От: Tony Lindgren Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround До: Pali Rohár Изпратено на: Сряда

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-24 Thread Pali Rohár
On Wednesday 06 March 2013 20:13:15 Pali Rohár wrote: > On Wednesday 06 March 2013 18:51:21 Tony Lindgren wrote: > > * Pali Rohár [130306 06:13]: > > > On Monday 04 March 2013 19:58:06 Tony Lindgren wrote: > > > > * Nishanth Menon [130301 06:42]: > > > > > On Fri, Mar 1, 2013 at 1:47 AM, Ивайло

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-24 Thread Pali Rohár
On Wednesday 06 March 2013 20:13:15 Pali Rohár wrote: On Wednesday 06 March 2013 18:51:21 Tony Lindgren wrote: * Pali Rohár pali.ro...@gmail.com [130306 06:13]: On Monday 04 March 2013 19:58:06 Tony Lindgren wrote: * Nishanth Menon n...@ti.com [130301 06:42]: On Fri, Mar 1, 2013 at

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-06 Thread Pali Rohár
On Wednesday 06 March 2013 18:51:21 Tony Lindgren wrote: > * Pali Rohár [130306 06:13]: > > On Monday 04 March 2013 19:58:06 Tony Lindgren wrote: > > > * Nishanth Menon [130301 06:42]: > > > > On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров > > > > wrote: > > > > > They look similar, but they

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-06 Thread Tony Lindgren
* Pali Rohár [130306 06:13]: > On Monday 04 March 2013 19:58:06 Tony Lindgren wrote: > > * Nishanth Menon [130301 06:42]: > > > On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров > wrote: > > > > They look similar, but they are not equivalent :). The > > > > first major difference is here (code

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-06 Thread Pali Rohár
On Monday 04 March 2013 19:58:06 Tony Lindgren wrote: > * Nishanth Menon [130301 06:42]: > > On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров wrote: > > > They look similar, but they are not equivalent :). The > > > first major difference is here (code taken from > > > omap-smc.S) > > > > > >>

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-06 Thread Ивайло Димитров
> Оригинално писмо >От: Aaro Koskinen >Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround >До: Ивайло Димитров >Изпратено на: Събота, 2013, Март 2 01:51:39 EET > > >On Fri, Mar 01, 2013 at 12:09:12PM +0200, Ивайло Димитров

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-06 Thread Ивайло Димитров
Оригинално писмо От: Aaro Koskinen Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround До: Ивайло Димитров Изпратено на: Събота, 2013, Март 2 01:51:39 EET On Fri, Mar 01, 2013 at 12:09:12PM +0200, Ивайло Димитров wrote: Unfortunately

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-06 Thread Pali Rohár
On Monday 04 March 2013 19:58:06 Tony Lindgren wrote: * Nishanth Menon n...@ti.com [130301 06:42]: On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров freemangor...@abv.bg wrote: They look similar, but they are not equivalent :). The first major difference is here (code taken from

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-06 Thread Tony Lindgren
* Pali Rohár pali.ro...@gmail.com [130306 06:13]: On Monday 04 March 2013 19:58:06 Tony Lindgren wrote: * Nishanth Menon n...@ti.com [130301 06:42]: On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров freemangor...@abv.bg wrote: They look similar, but they are not equivalent :). The

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-06 Thread Pali Rohár
On Wednesday 06 March 2013 18:51:21 Tony Lindgren wrote: * Pali Rohár pali.ro...@gmail.com [130306 06:13]: On Monday 04 March 2013 19:58:06 Tony Lindgren wrote: * Nishanth Menon n...@ti.com [130301 06:42]: On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров freemangor...@abv.bg wrote:

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-04 Thread Tony Lindgren
* Nishanth Menon [130301 06:42]: > On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров wrote: > > > > They look similar, but they are not equivalent :). The first major > > difference is here (code taken from omap-smc.S) > > > >> ENTRY(omap_smc2) > >> stmfd sp!, {r4-r12, lr} > >>

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-04 Thread Tony Lindgren
* Nishanth Menon n...@ti.com [130301 06:42]: On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров freemangor...@abv.bg wrote: They look similar, but they are not equivalent :). The first major difference is here (code taken from omap-smc.S) ENTRY(omap_smc2) stmfd sp!, {r4-r12,

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-01 Thread Aaro Koskinen
On Fri, Mar 01, 2013 at 12:09:12PM +0200, Ивайло Димитров wrote: > Unfortunately it is necessary, on RX-51 PPA/NOLO leaves IBE bit unset. You sure? I think you need to explain this more - the commit message in the original patch is empty/missing... A. -- To unsubscribe from this list: send the

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-01 Thread Nishanth Menon
On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров wrote: > > They look similar, but they are not equivalent :). The first major difference > is here (code taken from omap-smc.S) > >> ENTRY(omap_smc2) >> stmfd sp!, {r4-r12, lr} >> mov r3, r2 >> mov r2, r1 >>

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-01 Thread Ивайло Димитров
Unfortunately it is necessary, on RX-51 PPA/NOLO leaves IBE bit unset. > Оригинално писмо >От: Peter De Schrijver >Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround >До: Pali Rohár >Изпратено на: Петък, 2013, Март 1 11:43:14 EET >

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-01 Thread Peter De Schrijver
On Thu, Feb 28, 2013 at 10:42:28AM +0100, Pali Rohár wrote: > Signed-off-by: Ivaylo Dimitrov > Signed-off-by: Pali Rohár > --- > arch/arm/mach-omap2/Makefile|1 + > arch/arm/mach-omap2/board-rx51-secure.c | 66 > +++ >

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-01 Thread Peter De Schrijver
On Thu, Feb 28, 2013 at 10:42:28AM +0100, Pali Rohár wrote: Signed-off-by: Ivaylo Dimitrov freemangor...@abv.bg Signed-off-by: Pali Rohár pali.ro...@gmail.com --- arch/arm/mach-omap2/Makefile|1 + arch/arm/mach-omap2/board-rx51-secure.c | 66

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-01 Thread Ивайло Димитров
Unfortunately it is necessary, on RX-51 PPA/NOLO leaves IBE bit unset. Оригинално писмо От: Peter De Schrijver Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround До: Pali Rohár Изпратено на: Петък, 2013, Март 1 11:43:14 EET On Thu, Feb 28, 2013

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-01 Thread Nishanth Menon
On Fri, Mar 1, 2013 at 1:47 AM, Ивайло Димитров freemangor...@abv.bg wrote: They look similar, but they are not equivalent :). The first major difference is here (code taken from omap-smc.S) ENTRY(omap_smc2) stmfd sp!, {r4-r12, lr} mov r3, r2 mov r2,

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-03-01 Thread Aaro Koskinen
On Fri, Mar 01, 2013 at 12:09:12PM +0200, Ивайло Димитров wrote: Unfortunately it is necessary, on RX-51 PPA/NOLO leaves IBE bit unset. You sure? I think you need to explain this more - the commit message in the original patch is empty/missing... A. -- To unsubscribe from this list: send the

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-02-28 Thread Ивайло Димитров
nder does it worth. Not to say that I don't have BB around to test if the code still works if I make changes to omap2-secure.c/omap-smc.S :) > Оригинално писмо -------- >От: Nishanth Menon >Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround >До: Pali Roh

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-02-28 Thread Nishanth Menon
On 10:42-20130228, Pali Rohár wrote: > Signed-off-by: Ivaylo Dimitrov > Signed-off-by: Pali Rohár > --- > arch/arm/mach-omap2/Makefile|1 + > arch/arm/mach-omap2/board-rx51-secure.c | 66 > +++ > arch/arm/mach-omap2/board-rx51-secure.h | 36

[PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-02-28 Thread Pali Rohár
Signed-off-by: Ivaylo Dimitrov Signed-off-by: Pali Rohár --- arch/arm/mach-omap2/Makefile|1 + arch/arm/mach-omap2/board-rx51-secure.c | 66 +++ arch/arm/mach-omap2/board-rx51-secure.h | 36 + arch/arm/mach-omap2/board-rx51-smc.S

[PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-02-28 Thread Pali Rohár
Signed-off-by: Ivaylo Dimitrov freemangor...@abv.bg Signed-off-by: Pali Rohár pali.ro...@gmail.com --- arch/arm/mach-omap2/Makefile|1 + arch/arm/mach-omap2/board-rx51-secure.c | 66 +++ arch/arm/mach-omap2/board-rx51-secure.h | 36 +

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-02-28 Thread Nishanth Menon
On 10:42-20130228, Pali Rohár wrote: Signed-off-by: Ivaylo Dimitrov freemangor...@abv.bg Signed-off-by: Pali Rohár pali.ro...@gmail.com --- arch/arm/mach-omap2/Makefile|1 + arch/arm/mach-omap2/board-rx51-secure.c | 66 +++

Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround

2013-02-28 Thread Ивайло Димитров
to test if the code still works if I make changes to omap2-secure.c/omap-smc.S :) Оригинално писмо От: Nishanth Menon Относно: Re: [PATCH] arm: omap: RX-51: ARM errata 430973 workaround До: Pali Rohár Изпратено на: Четвъртък, 2013, Февруари 28 16:40:05 EET On 10:42