Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1216713

Re: [STLinux Kernel] [PATCH v2 2/4] remoteproc: dt: Provide bindings for ST's Remote Processor Controller driver

Show all headers | View raw

On Tue, 01 Sep 2015, Peter Griffin wrote:

> Hi Lee,
> 
> On Fri, 28 Aug 2015, Lee Jones wrote:
> 
> > Signed-off-by: Ludovic Barre <ludovic.barre@st.com>
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > ---
> >  .../devicetree/bindings/remoteproc/st-rproc.txt    | 35 ++++++++++++++++++++++
> >  1 file changed, 35 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/remoteproc/st-rproc.txt
> 
> The patch documening the DT bindings should be ordered before the patch which adds
> the DT node to aid reviewing.
> > 
> > diff --git a/Documentation/devicetree/bindings/remoteproc/st-rproc.txt b/Documentation/devicetree/bindings/remoteproc/st-rproc.txt
> > new file mode 100644
> > index 0000000..fbd7d78
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/remoteproc/st-rproc.txt
> > @@ -0,0 +1,35 @@
> > +STMicroelectronics Remote Processor
> > +-----------------------------------
> > +
> > +This binding provides support for adjunct processors found on ST SoCs.
> > +
> > +The remote processors can be controlled from the bootloader or the primary OS.
> > +If the bootloader starts a remote processor processor the primary OS must detect
> > +its state and act accordingly.
> > +
> > +Required properties:
> > +- compatible		Should be one of:
> > +				"st,st231-rproc"
> > +				"st,st40-rproc"
> 
> st40-proc isn't used anywhere. The stih407 doesn't have a ST40 copro, and
> looking in the vendor tree remoteproc support isn't present for stih415/6 which
> are the the only upstream SoC's to have a ST40 co-pro.
> 
> So I think st40-rproc support can be removed.
> 
> > +- reg			Size and length of reserved co-processor memory
> > +- resets		Reset lines (See: ../reset/reset.txt)
> > +- reset-names		Must be "sw_reset" and "pwr_reset"
> 
> pwr_reset isn't used by any of the st231 co-processors. It seems to
> be related to ST40 support which I don't think is required upstream.
> Removing it would make the driver a fair bit smaller.
> 
> > +- clocks		Clock for co-processor (See: ../clock/clock-bindings.txt)
> > +- clock-names		Must be "rproc_clk"
> 
> I can't see any co-pro which uses more than one clock, so clock-names looks
> superflous.
> 
> > +- clock-frequency	Clock frequency to set co-processor at if the bootloader
> > +			hasn't already done so
> > +- st,syscfg-boot	The register that holds the boot vector for the co-processor
> 
> I would prefer to see this binding match how most other sti drivers reference syscfg
> registers which is: -
> 
> st,syscfg       = <&syscfg_core 0xf4>;
> 
> Description: phandle of sysconfig bank plus integer array containing register offsets.
> 
> It also means it is easily extendable if more than one syscfg register is
> required in the future to boot a co-pro.

Ack.  Good points, will fix.

-- 
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 linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/4] remoteproc: Add driver for STMicroelectronics platforms Lee Jones <lee.jones@linaro.org> - 2015-08-28 12:40 +0200
  [PATCH v2 2/4] remoteproc: dt: Provide bindings for ST's Remote Processor Controller driver Lee Jones <lee.jones@linaro.org> - 2015-08-28 12:40 +0200
    Re: [PATCH v2 2/4] remoteproc: dt: Provide bindings for ST's Remote  Processor Controller driver Rob Herring <robherring2@gmail.com> - 2015-08-31 17:30 +0200
      Re: [PATCH v2 2/4] remoteproc: dt: Provide bindings for ST's Remote  Processor Controller driver Lee Jones <lee.jones@linaro.org> - 2015-09-01 12:50 +0200
        Re: [PATCH v2 2/4] remoteproc: dt: Provide bindings for ST's Remote  Processor Controller driver Rob Herring <robherring2@gmail.com> - 2015-09-01 14:00 +0200
    Re: [STLinux Kernel] [PATCH v2 2/4] remoteproc: dt: Provide bindings  for ST's Remote Processor Controller driver Peter Griffin <peter.griffin@linaro.org> - 2015-09-01 11:00 +0200
      Re: [STLinux Kernel] [PATCH v2 2/4] remoteproc: dt: Provide bindings  for ST's Remote Processor Controller driver Lee Jones <lee.jones@linaro.org> - 2015-09-01 11:20 +0200
      Re: [STLinux Kernel] [PATCH v2 2/4] remoteproc: dt: Provide bindings  for ST's Remote Processor Controller driver Lee Jones <lee.jones@linaro.org> - 2015-09-01 15:00 +0200
  [PATCH v2 3/4] remoteproc: Supply controller driver for ST's Remote Processors Lee Jones <lee.jones@linaro.org> - 2015-08-28 12:40 +0200
    Re: [PATCH v2 3/4] remoteproc: Supply controller driver for ST's  Remote Processors Nathan Lynch <Nathan_Lynch@mentor.com> - 2015-08-28 18:30 +0200
      Re: [PATCH v2 3/4] remoteproc: Supply controller driver for ST's  Remote Processors Lee Jones <lee.jones@linaro.org> - 2015-09-01 10:00 +0200
        Re: [STLinux Kernel] [PATCH v2 3/4] remoteproc: Supply controller  driver for ST's Remote Processors Peter Griffin <peter.griffin@linaro.org> - 2015-09-01 10:20 +0200
          Re: [STLinux Kernel] [PATCH v2 3/4] remoteproc: Supply controller  driver for ST's Remote Processors Lee Jones <lee.jones@linaro.org> - 2015-09-01 11:20 +0200
  [PATCH v2 4/4] remoteproc: debugfs: Add ability to boot remote processor using debugfs Lee Jones <lee.jones@linaro.org> - 2015-08-28 12:40 +0200
    Re: [PATCH v2 4/4] remoteproc: debugfs: Add ability to boot remote  processor using debugfs Nathan Lynch <Nathan_Lynch@mentor.com> - 2015-08-28 17:30 +0200
      Re: [PATCH v2 4/4] remoteproc: debugfs: Add ability to boot remote  processor using debugfs Lee Jones <lee.jones@linaro.org> - 2015-09-01 09:50 +0200
    Re: [PATCH v2 4/4] remoteproc: debugfs: Add ability to boot remote  processor using debugfs Florian Fainelli <f.fainelli@gmail.com> - 2015-08-28 19:30 +0200
      Re: [PATCH v2 4/4] remoteproc: debugfs: Add ability to boot remote  processor using debugfs Lee Jones <lee.jones@linaro.org> - 2015-09-01 09:50 +0200
  [PATCH v2 1/4] ARM: STiH407: Add nodes for RemoteProc Lee Jones <lee.jones@linaro.org> - 2015-08-28 12:40 +0200
    Re: [STLinux Kernel] [PATCH v2 1/4] ARM: STiH407: Add nodes for  RemoteProc Peter Griffin <peter.griffin@linaro.org> - 2015-09-01 10:30 +0200
      Re: [STLinux Kernel] [PATCH v2 1/4] ARM: STiH407: Add nodes for  RemoteProc Lee Jones <lee.jones@linaro.org> - 2015-09-01 11:20 +0200
        Re: [STLinux Kernel] [PATCH v2 1/4] ARM: STiH407: Add nodes for  RemoteProc Peter Griffin <peter.griffin@linaro.org> - 2015-09-01 11:20 +0200

csiph-web