mbox series

[v4,0/2] remoteproc: st: add virtio_rpmsg support

Message ID 1487189929-24196-1-git-send-email-loic.pallardy@st.com
Headers show
Series remoteproc: st: add virtio_rpmsg support | expand

Message

Loic Pallardy Feb. 15, 2017, 8:18 p.m. UTC
Goal of this series is:
- to add vring based communication link (virtio_rpmsg)
- to add rproc_da_to_va translation function to allow firmware loading in
  pre-reserved carveout memory region

V2:
Only changes in patch 1 to:
- fix typos
- fix mailbox allocation per coprocessor

V3:
- Integrate Bjorn's comments about mailbox support simplification
- Add missing Signed-of-by
- Move st_rproc_probe error management correction in a dedicated patch
- Fix warnings reported by checkpatch --strict

V4:
- Rebase on maintainer rproc-next branch on which patches 1 & 2 are present
- Remove check on device address (da) from patch 4
- Add Acked-by and Tested-by

Loic Pallardy (2):
  remoteproc: st: add da to va support
  remoteproc: core: don't allocate carveout if pa is defined

 drivers/remoteproc/remoteproc_core.c |  5 +++++
 drivers/remoteproc/st_remoteproc.c   | 43 +++++++++++++++++++++++++++++++-----
 2 files changed, 42 insertions(+), 6 deletions(-)

-- 
1.9.1

Comments

Loic Pallardy May 3, 2017, 3:29 p.m. UTC | #1
On 02/15/2017 09:18 PM, Loic Pallardy wrote:
> Goal of this series is:

> - to add vring based communication link (virtio_rpmsg)

> - to add rproc_da_to_va translation function to allow firmware loading in

>   pre-reserved carveout memory region

>

> V2:

> Only changes in patch 1 to:

> - fix typos

> - fix mailbox allocation per coprocessor

>

> V3:

> - Integrate Bjorn's comments about mailbox support simplification

> - Add missing Signed-of-by

> - Move st_rproc_probe error management correction in a dedicated patch

> - Fix warnings reported by checkpatch --strict

>

> V4:

> - Rebase on maintainer rproc-next branch on which patches 1 & 2 are present

> - Remove check on device address (da) from patch 4

> - Add Acked-by and Tested-by

>

Hi Bjorn,

As discussed during Linaro connect, could you review/accept these 2 
patches providing da to pa translation mechanism for firmware loading?
I agree a central solution will better, but today all drivers are based 
on this mechanism.
Could we move on this solution to complete remoteproc support for B2260 
96board?

Regards,
Loic

> Loic Pallardy (2):

>   remoteproc: st: add da to va support

>   remoteproc: core: don't allocate carveout if pa is defined

>

>  drivers/remoteproc/remoteproc_core.c |  5 +++++

>  drivers/remoteproc/st_remoteproc.c   | 43 +++++++++++++++++++++++++++++++-----

>  2 files changed, 42 insertions(+), 6 deletions(-)

>