Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-07-04 Thread Oleksij Rempel
Hi Bjorn, thank you for review, On 25.06.2017 22:41, Bjorn Andersson wrote: On Wed 14 Jun 13:48 PDT 2017, Oleksij Rempel wrote: From: Oleksij Rempel this driver was tested on NXP imx7d but should work on imx6sx as well. It will upload firmware to OCRAM, which shared

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-07-04 Thread Oleksij Rempel
Hi Bjorn, thank you for review, On 25.06.2017 22:41, Bjorn Andersson wrote: On Wed 14 Jun 13:48 PDT 2017, Oleksij Rempel wrote: From: Oleksij Rempel this driver was tested on NXP imx7d but should work on imx6sx as well. It will upload firmware to OCRAM, which shared memory between Cortex

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-25 Thread Bjorn Andersson
On Wed 14 Jun 13:48 PDT 2017, Oleksij Rempel wrote: > From: Oleksij Rempel > > this driver was tested on NXP imx7d but should work on > imx6sx as well. > It will upload firmware to OCRAM, which shared memory between > Cortex A7 and Cortex M4, then turn M4 on. > This

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-25 Thread Bjorn Andersson
On Wed 14 Jun 13:48 PDT 2017, Oleksij Rempel wrote: > From: Oleksij Rempel > > this driver was tested on NXP imx7d but should work on > imx6sx as well. > It will upload firmware to OCRAM, which shared memory between > Cortex A7 and Cortex M4, then turn M4 on. > This looks very generic, are

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-23 Thread Suman Anna
On 06/23/2017 09:35 AM, Oleksij Rempel wrote: > On Wed, Jun 21, 2017 at 04:09:26PM -0500, Suman Anna wrote: >> Hi Oleksij, >> >> On 06/19/2017 02:43 AM, Oleksij Rempel wrote: >>> Hi, >>> >>> Thank you for your review! >>> >>> On 15.06.2017 21:01, Suman Anna wrote: Hi Oleksij, On

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-23 Thread Suman Anna
On 06/23/2017 09:35 AM, Oleksij Rempel wrote: > On Wed, Jun 21, 2017 at 04:09:26PM -0500, Suman Anna wrote: >> Hi Oleksij, >> >> On 06/19/2017 02:43 AM, Oleksij Rempel wrote: >>> Hi, >>> >>> Thank you for your review! >>> >>> On 15.06.2017 21:01, Suman Anna wrote: Hi Oleksij, On

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-23 Thread Oleksij Rempel
On Wed, Jun 21, 2017 at 04:09:26PM -0500, Suman Anna wrote: > Hi Oleksij, > > On 06/19/2017 02:43 AM, Oleksij Rempel wrote: > > Hi, > > > > Thank you for your review! > > > > On 15.06.2017 21:01, Suman Anna wrote: > >> Hi Oleksij, > >> > >> On 06/14/2017 03:48 PM, Oleksij Rempel wrote: > >>>

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-23 Thread Oleksij Rempel
On Wed, Jun 21, 2017 at 04:09:26PM -0500, Suman Anna wrote: > Hi Oleksij, > > On 06/19/2017 02:43 AM, Oleksij Rempel wrote: > > Hi, > > > > Thank you for your review! > > > > On 15.06.2017 21:01, Suman Anna wrote: > >> Hi Oleksij, > >> > >> On 06/14/2017 03:48 PM, Oleksij Rempel wrote: > >>>

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-21 Thread Suman Anna
Hi Oleksij, On 06/19/2017 02:43 AM, Oleksij Rempel wrote: > Hi, > > Thank you for your review! > > On 15.06.2017 21:01, Suman Anna wrote: >> Hi Oleksij, >> >> On 06/14/2017 03:48 PM, Oleksij Rempel wrote: >>> From: Oleksij Rempel >>> >>> this driver was tested on NXP

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-21 Thread Suman Anna
Hi Oleksij, On 06/19/2017 02:43 AM, Oleksij Rempel wrote: > Hi, > > Thank you for your review! > > On 15.06.2017 21:01, Suman Anna wrote: >> Hi Oleksij, >> >> On 06/14/2017 03:48 PM, Oleksij Rempel wrote: >>> From: Oleksij Rempel >>> >>> this driver was tested on NXP imx7d but should work on

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-19 Thread Oleksij Rempel
Hi, Thank you for your review! On 15.06.2017 21:01, Suman Anna wrote: Hi Oleksij, On 06/14/2017 03:48 PM, Oleksij Rempel wrote: From: Oleksij Rempel this driver was tested on NXP imx7d but should work on imx6sx as well. It will upload firmware to OCRAM, which shared

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-19 Thread Oleksij Rempel
Hi, Thank you for your review! On 15.06.2017 21:01, Suman Anna wrote: Hi Oleksij, On 06/14/2017 03:48 PM, Oleksij Rempel wrote: From: Oleksij Rempel this driver was tested on NXP imx7d but should work on imx6sx as well. It will upload firmware to OCRAM, which shared memory between Cortex

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-16 Thread Suman Anna
On 06/15/2017 02:01 PM, Suman Anna wrote: > Hi Oleksij, > > On 06/14/2017 03:48 PM, Oleksij Rempel wrote: >> From: Oleksij Rempel >> >> this driver was tested on NXP imx7d but should work on >> imx6sx as well. >> It will upload firmware to OCRAM, which shared memory

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-16 Thread Suman Anna
On 06/15/2017 02:01 PM, Suman Anna wrote: > Hi Oleksij, > > On 06/14/2017 03:48 PM, Oleksij Rempel wrote: >> From: Oleksij Rempel >> >> this driver was tested on NXP imx7d but should work on >> imx6sx as well. >> It will upload firmware to OCRAM, which shared memory between >> Cortex A7 and

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-15 Thread Suman Anna
Hi Oleksij, On 06/14/2017 03:48 PM, Oleksij Rempel wrote: From: Oleksij Rempel this driver was tested on NXP imx7d but should work on imx6sx as well. It will upload firmware to OCRAM, which shared memory between Cortex A7 and Cortex M4, then turn M4 on. Mostly looks

Re: [RFC PATCH 2/3] remoteproc: imx_rproc: add a NXP/Freescale imx rproc driver

2017-06-15 Thread Suman Anna
Hi Oleksij, On 06/14/2017 03:48 PM, Oleksij Rempel wrote: From: Oleksij Rempel this driver was tested on NXP imx7d but should work on imx6sx as well. It will upload firmware to OCRAM, which shared memory between Cortex A7 and Cortex M4, then turn M4 on. Mostly looks fine, need to address