From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755152AbcCBPna (ORCPT ); Wed, 2 Mar 2016 10:43:30 -0500 Received: from mail-wm0-f54.google.com ([74.125.82.54]:37462 "EHLO mail-wm0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753225AbcCBPn1 (ORCPT ); Wed, 2 Mar 2016 10:43:27 -0500 Date: Wed, 2 Mar 2016 15:43:22 +0000 From: Lee Jones To: Bjorn Andersson Cc: "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , kernel@stlinux.com, Maxime Coquelin , Ohad Ben-Cohen , Nathan Lynch , Florian Fainelli , ludovic.barre@st.com, Suman Anna Subject: Re: [PATCH v5 0/7] remoteproc: Add driver for STMicroelectronics platforms Message-ID: <20160302154322.GD3444@x1> References: <1452602781-22424-1-git-send-email-lee.jones@linaro.org> <20160127073124.GI3368@x1> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 27 Jan 2016, Bjorn Andersson wrote: > On Tue, Jan 26, 2016 at 11:31 PM, Lee Jones wrote: > > > > Apologies for ping (I hate doing that), but I haven't heard anything > > from you. Do you want me to re-submit this set, or are you willing to > > take this and apply the relevant Acks? > > > > I had a chat with Ohad and offered my assistance with his work. So I > will pick up the role as co-maintainer of his subsystems. > > I will pick up patch 1-4 of your v5 as soon as I've sorted out the > practicalities around this, no need to resubmit them. > > Sorry for the delay! Any news on this? We're nearing the merge window. > > > ST's platforms often have multiple co-processors (usually ST40s or ST231s) > > > on-board. This provides the Linux-side infrastructure to flash and boot > > > them successfully. > > > > > > This set has been tested on an STiH410-B2120. > > > > > > v4 => v5: > > > - Check for invalid 'count' (command read length) in write fn()s > > > > > > v3 => v4: > > > Suggested-by: Suman Anna > > > - Move to using 'reserved-memory' API > > > - New 'reserved-memory' nodes > > > - Remove memory locations from RemoteProc's DT node's reg properties > > > - Remove C code obtaining/allocating DMA memory > > > - Re-order .start() and .stop() ops > > > - Add protection around Reset API in error path > > > - Explicitly set .has_iommu to false > > > > > > v2 => v3: > > > - Generify syscon property (st,syscfg-boot => st,syscfg) > > > - Rename IP in DT bindings doc (Remote Processor => Co-Processor) > > > - Remove superfluous 'clock-names' property > > > - Remove superfluous 'reg-names' property > > > - Populate MAINTAINERS > > > - Clean-up DTS formatting > > > - Use strings in debugfs to control procs ('1|0' => 'start|stop') > > > - Align copyright statement with MODULE() macros > > > - Rename driver data structure ('st_rproc' => 'ddata') > > > - Addition of a full error path in .start() > > > > > > v1 => v2: > > > - Remove Linux implementation specific comment from binding document > > > - Force debugfs '0' to shutdown co-processor - rather than !1 > > > - Supply more detailed commit message > > > - Propagate errors back from .stop() > > > - Review GPL wording > > > - Supply original author's SoBs > > > > > > Lee Jones (7): > > > remoteproc: debugfs: Check of invalid 'count' value > > > remoteproc: dt: Provide bindings for ST's Remote Processor Controller > > > driver > > > remoteproc: debugfs: Add ability to boot remote processor using > > > debugfs > > > remoteproc: Supply controller driver for ST's Remote Processors > > > MAINTAINERS: Add ST's Remote Processor Driver to ARM/STI ARCHITECTURE > > > ARM: STiH407: Add nodes for RemoteProc > > > ARM: STiH407: Move over to using the 'reserved-memory' API for > > > obtaining DMA memory > > > > > > .../devicetree/bindings/remoteproc/st-rproc.txt | 41 +++ > > > MAINTAINERS | 1 + > > > arch/arm/boot/dts/stih407-family.dtsi | 70 +++++ > > > drivers/remoteproc/Kconfig | 9 + > > > drivers/remoteproc/Makefile | 1 + > > > drivers/remoteproc/remoteproc_debugfs.c | 38 ++- > > > drivers/remoteproc/st_remoteproc.c | 297 +++++++++++++++++++++ > > > 7 files changed, 455 insertions(+), 2 deletions(-) > > > create mode 100644 Documentation/devicetree/bindings/remoteproc/st-rproc.txt > > > create mode 100644 drivers/remoteproc/st_remoteproc.c > > > > > -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: lee.jones@linaro.org (Lee Jones) Date: Wed, 2 Mar 2016 15:43:22 +0000 Subject: [PATCH v5 0/7] remoteproc: Add driver for STMicroelectronics platforms In-Reply-To: References: <1452602781-22424-1-git-send-email-lee.jones@linaro.org> <20160127073124.GI3368@x1> Message-ID: <20160302154322.GD3444@x1> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, 27 Jan 2016, Bjorn Andersson wrote: > On Tue, Jan 26, 2016 at 11:31 PM, Lee Jones wrote: > > > > Apologies for ping (I hate doing that), but I haven't heard anything > > from you. Do you want me to re-submit this set, or are you willing to > > take this and apply the relevant Acks? > > > > I had a chat with Ohad and offered my assistance with his work. So I > will pick up the role as co-maintainer of his subsystems. > > I will pick up patch 1-4 of your v5 as soon as I've sorted out the > practicalities around this, no need to resubmit them. > > Sorry for the delay! Any news on this? We're nearing the merge window. > > > ST's platforms often have multiple co-processors (usually ST40s or ST231s) > > > on-board. This provides the Linux-side infrastructure to flash and boot > > > them successfully. > > > > > > This set has been tested on an STiH410-B2120. > > > > > > v4 => v5: > > > - Check for invalid 'count' (command read length) in write fn()s > > > > > > v3 => v4: > > > Suggested-by: Suman Anna > > > - Move to using 'reserved-memory' API > > > - New 'reserved-memory' nodes > > > - Remove memory locations from RemoteProc's DT node's reg properties > > > - Remove C code obtaining/allocating DMA memory > > > - Re-order .start() and .stop() ops > > > - Add protection around Reset API in error path > > > - Explicitly set .has_iommu to false > > > > > > v2 => v3: > > > - Generify syscon property (st,syscfg-boot => st,syscfg) > > > - Rename IP in DT bindings doc (Remote Processor => Co-Processor) > > > - Remove superfluous 'clock-names' property > > > - Remove superfluous 'reg-names' property > > > - Populate MAINTAINERS > > > - Clean-up DTS formatting > > > - Use strings in debugfs to control procs ('1|0' => 'start|stop') > > > - Align copyright statement with MODULE() macros > > > - Rename driver data structure ('st_rproc' => 'ddata') > > > - Addition of a full error path in .start() > > > > > > v1 => v2: > > > - Remove Linux implementation specific comment from binding document > > > - Force debugfs '0' to shutdown co-processor - rather than !1 > > > - Supply more detailed commit message > > > - Propagate errors back from .stop() > > > - Review GPL wording > > > - Supply original author's SoBs > > > > > > Lee Jones (7): > > > remoteproc: debugfs: Check of invalid 'count' value > > > remoteproc: dt: Provide bindings for ST's Remote Processor Controller > > > driver > > > remoteproc: debugfs: Add ability to boot remote processor using > > > debugfs > > > remoteproc: Supply controller driver for ST's Remote Processors > > > MAINTAINERS: Add ST's Remote Processor Driver to ARM/STI ARCHITECTURE > > > ARM: STiH407: Add nodes for RemoteProc > > > ARM: STiH407: Move over to using the 'reserved-memory' API for > > > obtaining DMA memory > > > > > > .../devicetree/bindings/remoteproc/st-rproc.txt | 41 +++ > > > MAINTAINERS | 1 + > > > arch/arm/boot/dts/stih407-family.dtsi | 70 +++++ > > > drivers/remoteproc/Kconfig | 9 + > > > drivers/remoteproc/Makefile | 1 + > > > drivers/remoteproc/remoteproc_debugfs.c | 38 ++- > > > drivers/remoteproc/st_remoteproc.c | 297 +++++++++++++++++++++ > > > 7 files changed, 455 insertions(+), 2 deletions(-) > > > create mode 100644 Documentation/devicetree/bindings/remoteproc/st-rproc.txt > > > create mode 100644 drivers/remoteproc/st_remoteproc.c > > > > > -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog