Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1318726 > unrolled thread
| Started by | Lee Jones <lee.jones@linaro.org> |
|---|---|
| First post | 2016-01-27 08:40 +0100 |
| Last post | 2016-01-27 20:30 +0100 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v5 0/7] remoteproc: Add driver for STMicroelectronics platforms Lee Jones <lee.jones@linaro.org> - 2016-01-27 08:40 +0100
Re: [PATCH v5 0/7] remoteproc: Add driver for STMicroelectronics platforms Bjorn Andersson <bjorn@kryo.se> - 2016-01-27 20:30 +0100
| From | Lee Jones <lee.jones@linaro.org> |
|---|---|
| Date | 2016-01-27 08:40 +0100 |
| Subject | Re: [PATCH v5 0/7] remoteproc: Add driver for STMicroelectronics platforms |
| Message-ID | <qVsRQ-8v1-3@gated-at.bofh.it> |
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?
> 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 <s-anna@ti.com>
> - 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
[toc] | [next] | [standalone]
| From | Bjorn Andersson <bjorn@kryo.se> |
|---|---|
| Date | 2016-01-27 20:30 +0100 |
| Subject | Re: [PATCH v5 0/7] remoteproc: Add driver for STMicroelectronics platforms |
| Message-ID | <qVDWW-8oV-7@gated-at.bofh.it> |
| In reply to | #1318726 |
On Tue, Jan 26, 2016 at 11:31 PM, Lee Jones <lee.jones@linaro.org> 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!
Regards,
Bjorn
> > 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 <s-anna@ti.com>
> > - 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
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web