From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752014AbdECPaN (ORCPT ); Wed, 3 May 2017 11:30:13 -0400 Received: from mx08-00178001.pphosted.com ([91.207.212.93]:44674 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751601AbdECPaD (ORCPT ); Wed, 3 May 2017 11:30:03 -0400 Subject: Re: [PATCH v4 0/2] remoteproc: st: add virtio_rpmsg support To: , , References: <1487189929-24196-1-git-send-email-loic.pallardy@st.com> CC: , , , , From: Loic Pallardy Message-ID: <40a7f950-980a-2bc5-8a0b-2b457c00d07d@st.com> Date: Wed, 3 May 2017 17:29:45 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <1487189929-24196-1-git-send-email-loic.pallardy@st.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.75.127.45] X-ClientProxiedBy: SFHDAG1NODE3.st.com (10.75.127.3) To SFHDAG7NODE2.st.com (10.75.127.20) X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-05-03_12:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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(-) >