Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-10-07 Thread Marco Felsch
On 19-10-07 01:21, Anson Huang wrote: > Hi, Marco > > > On 19-09-30 08:32, Anson Huang wrote: > > > Hi, Marco > > > > > > > On 19-09-30 07:42, Anson Huang wrote: > > > > > Hi, Leonard > > > > > > > > > > > On 2019-09-27 4:20 AM, Anson Huang wrote: > > > > > > >> On 2019-09-26 1:06 PM, Marco Felsch

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-10-06 Thread Anson Huang
Hi, Marco > On 19-09-30 08:32, Anson Huang wrote: > > Hi, Marco > > > > > On 19-09-30 07:42, Anson Huang wrote: > > > > Hi, Leonard > > > > > > > > > On 2019-09-27 4:20 AM, Anson Huang wrote: > > > > > >> On 2019-09-26 1:06 PM, Marco Felsch wrote: > > > > > >>> On 19-09-26 08:03, Anson Huang wrote

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-30 Thread Marco Felsch
Hi Anson, On 19-09-30 08:32, Anson Huang wrote: > Hi, Marco > > > On 19-09-30 07:42, Anson Huang wrote: > > > Hi, Leonard > > > > > > > On 2019-09-27 4:20 AM, Anson Huang wrote: > > > > >> On 2019-09-26 1:06 PM, Marco Felsch wrote: > > > > >>> On 19-09-26 08:03, Anson Huang wrote: > > > > > O

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-30 Thread Anson Huang
Hi, Marco > On 19-09-30 07:42, Anson Huang wrote: > > Hi, Leonard > > > > > On 2019-09-27 4:20 AM, Anson Huang wrote: > > > >> On 2019-09-26 1:06 PM, Marco Felsch wrote: > > > >>> On 19-09-26 08:03, Anson Huang wrote: > > > > On 19-09-25 18:07, Anson Huang wrote: > > > >> The SCU firmware

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-30 Thread Marco Felsch
Hi Anson, Leonard, On 19-09-30 07:42, Anson Huang wrote: > Hi, Leonard > > > On 2019-09-27 4:20 AM, Anson Huang wrote: > > >> On 2019-09-26 1:06 PM, Marco Felsch wrote: > > >>> On 19-09-26 08:03, Anson Huang wrote: > > > On 19-09-25 18:07, Anson Huang wrote: > > >> The SCU firmware does N

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-30 Thread Anson Huang
Hi, Leonard > > On 2019-09-27 4:20 AM, Anson Huang wrote: > > >> On 2019-09-26 1:06 PM, Marco Felsch wrote: > > >>> On 19-09-26 08:03, Anson Huang wrote: > > > On 19-09-25 18:07, Anson Huang wrote: > > >> The SCU firmware does NOT always have return value stored in > > >> message heade

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-30 Thread Anson Huang
Hi, Leonard > On 2019-09-27 4:20 AM, Anson Huang wrote: > >> On 2019-09-26 1:06 PM, Marco Felsch wrote: > >>> On 19-09-26 08:03, Anson Huang wrote: > > On 19-09-25 18:07, Anson Huang wrote: > >> The SCU firmware does NOT always have return value stored in > >> message header's function

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-30 Thread Leonard Crestez
On 2019-09-27 4:20 AM, Anson Huang wrote: >> On 2019-09-26 1:06 PM, Marco Felsch wrote: >>> On 19-09-26 08:03, Anson Huang wrote: > On 19-09-25 18:07, Anson Huang wrote: >> The SCU firmware does NOT always have return value stored in >> message header's function element even the API has

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-28 Thread Anson Huang
Hi, Leonard/Marco I think we should get aligned first, my original thought is to let SCU API caller NOT aware of those special APIs, so have to do the special handling in imx_scu_call_rpc(). And the short loop check has to be used which would impact the performance a little bit I think.

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-27 Thread Leonard Crestez
On 27.09.2019 12:06, Marco Felsch wrote: > Hi Anson, Leonard, > > On 19-09-27 01:20, Anson Huang wrote: >> Hi, Leonard >> >>> On 2019-09-26 1:06 PM, Marco Felsch wrote: On 19-09-26 08:03, Anson Huang wrote: >> On 19-09-25 18:07, Anson Huang wrote: >>> The SCU firmware does NOT always

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-27 Thread Leonard Crestez
On 27.09.2019 04:20, Anson Huang wrote: >> On 2019-09-26 1:06 PM, Marco Felsch wrote: >>> On 19-09-26 08:03, Anson Huang wrote: > On 19-09-25 18:07, Anson Huang wrote: >> The SCU firmware does NOT always have return value stored in >> message header's function element even the API has r

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-27 Thread Anson Huang
Hi, Marco > On 19-09-27 01:20, Anson Huang wrote: > > Hi, Leonard > > > > > On 2019-09-26 1:06 PM, Marco Felsch wrote: > > > > On 19-09-26 08:03, Anson Huang wrote: > > > >>> On 19-09-25 18:07, Anson Huang wrote: > > > The SCU firmware does NOT always have return value stored in > > > me

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-27 Thread Marco Felsch
Hi Anson, Leonard, On 19-09-27 01:20, Anson Huang wrote: > Hi, Leonard > > > On 2019-09-26 1:06 PM, Marco Felsch wrote: > > > On 19-09-26 08:03, Anson Huang wrote: > > >>> On 19-09-25 18:07, Anson Huang wrote: > > The SCU firmware does NOT always have return value stored in > > message

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-26 Thread Anson Huang
Hi, Leonard > On 2019-09-26 1:06 PM, Marco Felsch wrote: > > On 19-09-26 08:03, Anson Huang wrote: > >>> On 19-09-25 18:07, Anson Huang wrote: > The SCU firmware does NOT always have return value stored in > message header's function element even the API has response data, > those s

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-26 Thread Leonard Crestez
On 2019-09-26 1:06 PM, Marco Felsch wrote: > On 19-09-26 08:03, Anson Huang wrote: >>> On 19-09-25 18:07, Anson Huang wrote: The SCU firmware does NOT always have return value stored in message header's function element even the API has response data, those special APIs are defined a

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-26 Thread Marco Felsch
Hi Anson, On 19-09-26 08:03, Anson Huang wrote: > Hi, Marco > > > On 19-09-25 18:07, Anson Huang wrote: > > > The SCU firmware does NOT always have return value stored in message > > > header's function element even the API has response data, those > > > special APIs are defined as void function

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-26 Thread Anson Huang
Hi, Marco > On 19-09-25 18:07, Anson Huang wrote: > > The SCU firmware does NOT always have return value stored in message > > header's function element even the API has response data, those > > special APIs are defined as void function in SCU firmware, so they > > should be treated as return succ

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-26 Thread Marco Felsch
Hi, On 19-09-25 18:07, Anson Huang wrote: > The SCU firmware does NOT always have return value stored in message > header's function element even the API has response data, those special > APIs are defined as void function in SCU firmware, so they should be > treated as return success always. > >

RE: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-25 Thread Anson Huang
Hi, Leonard > On 25.09.2019 13:09, Anson Huang wrote: > > The SCU firmware does NOT always have return value stored in message > > header's function element even the API has response data, those > > special APIs are defined as void function in SCU firmware, so they > > should be treated as return

Re: [PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-25 Thread Leonard Crestez
On 25.09.2019 13:09, Anson Huang wrote: > The SCU firmware does NOT always have return value stored in message > header's function element even the API has response data, those special > APIs are defined as void function in SCU firmware, so they should be > treated as return success always. > > Si

[PATCH] firmware: imx: Skip return value check for some special SCU firmware APIs

2019-09-25 Thread Anson Huang
The SCU firmware does NOT always have return value stored in message header's function element even the API has response data, those special APIs are defined as void function in SCU firmware, so they should be treated as return success always. Signed-off-by: Anson Huang --- - This patch i