Re: [PATCH v2 1/5] dt-bindings: remoteproc: sse710: Add the External Systems remote processors

2024-10-09 Thread Abdellatif El Khlifi
Hello folks, > On 22/08/2024 6:09 pm, Abdellatif El Khlifi wrote: > > Add devicetree binding schema for the External Systems remote processors > > > > The External Systems remote processors are provided on the Corstone-1000 > > IoT Reference Design Platfor

Re: [PATCH v2 1/5] dt-bindings: remoteproc: sse710: Add the External Systems remote processors

2024-09-23 Thread Abdellatif El Khlifi
Hi Krzysztof, > >>> + '#extsys-id': > >> > >> '#' is not correct for sure, that's not a cell specifier. > >> > >> But anyway, we do not accept in general instance IDs. > > > > I'm happy to replace the instance ID with another solution. > >>

Re: [PATCH v2 1/5] dt-bindings: remoteproc: sse710: Add the External Systems remote processors

2024-09-23 Thread Abdellatif El Khlifi
Hi Krzysztof, > > + '#extsys-id': > > '#' is not correct for sure, that's not a cell specifier. > > But anyway, we do not accept in general instance IDs. > >>> > >>> I'm happy to replace the instance ID with another solution. > >>> In our c

Re: [PATCH v2 1/5] dt-bindings: remoteproc: sse710: Add the External Systems remote processors

2024-09-20 Thread Abdellatif El Khlifi
Hi Krzysztof, > >>> + '#extsys-id': > >> > >> '#' is not correct for sure, that's not a cell specifier. > >> > >> But anyway, we do not accept in general instance IDs. > > > > I'm happy to replace the instance ID with another solution. > > In our case the remotepr

Re: [PATCH v2 1/5] dt-bindings: remoteproc: sse710: Add the External Systems remote processors

2024-09-20 Thread Abdellatif El Khlifi
Hi Krzysztof, > > + '#extsys-id': > > '#' is not correct for sure, that's not a cell specifier. > > But anyway, we do not accept in general instance IDs. > >>> > >>> I'm happy to replace the instance ID with another solution. > >>> In our case the remoteproc instance does

Re: [PATCH v2 1/5] dt-bindings: remoteproc: sse710: Add the External Systems remote processors

2024-09-19 Thread Abdellatif El Khlifi
Hi Krzysztof, > >>> + '#extsys-id': > >> > >> '#' is not correct for sure, that's not a cell specifier. > >> > >> But anyway, we do not accept in general instance IDs. > > > > I'm happy to replace the instance ID with another solution. > > In our case the remoteproc instance does not have a bas

Re: [PATCH v2 1/5] dt-bindings: remoteproc: sse710: Add the External Systems remote processors

2024-09-19 Thread Abdellatif El Khlifi
; > > [1]: > > https://developer.arm.com/documentation/102360//Overview-of-Corstone-1000/Corstone-SSE-710-subsystem-features > > > > Signed-off-by: Abdellatif El Khlifi > > --- > > .../remoteproc/arm,sse710-extsys.yaml | 90 +++ &

Re: [PATCH v2 5/5] remoteproc: arm64: corstone1000: Add the External Systems driver

2024-09-18 Thread Abdellatif El Khlifi
[2]: Documentation/devicetree/bindings/arm/arm,sse710-host-base-sysctrl.yaml > > Signed-off-by: Abdellatif El Khlifi > --- > drivers/remoteproc/Kconfig | 14 + > drivers/remoteproc/Makefile | 1 + > drivers/remoteproc/corstone1000_rproc.c | 350

[PATCH v2 2/5] dt-bindings: arm: sse710: Add Host Base System Control

2024-08-22 Thread Abdellatif El Khlifi
-System-Control-register-summary Signed-off-by: Abdellatif El Khlifi --- .../arm/arm,sse710-host-base-sysctrl.yaml | 56 +++ 1 file changed, 56 insertions(+) create mode 100644 Documentation/devicetree/bindings/arm/arm,sse710-host-base-sysctrl.yaml diff --git a/Documentation

[PATCH v2 5/5] remoteproc: arm64: corstone1000: Add the External Systems driver

2024-08-22 Thread Abdellatif El Khlifi
Signed-off-by: Abdellatif El Khlifi --- drivers/remoteproc/Kconfig | 14 + drivers/remoteproc/Makefile | 1 + drivers/remoteproc/corstone1000_rproc.c | 350 3 files changed, 365 insertions(+) create mode 100644 drivers/remoteproc

[PATCH v2 4/5] arm64: dts: corstone1000: Add External System support

2024-08-22 Thread Abdellatif El Khlifi
-descriptions/Host-Base-System-Control-register-summary [3]: Documentation/devicetree/bindings/arm/arm,sse710-host-base-sysctrl.yaml Signed-off-by: Abdellatif El Khlifi --- arch/arm64/boot/dts/arm/corstone1000.dtsi | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm64/boot

[PATCH v2 3/5] arm64: dts: corstone1000: Add MHU nodes used by the External System

2024-08-22 Thread Abdellatif El Khlifi
device tree through mhu0_hes0 and mhu0_es0h. Signed-off-by: Abdellatif El Khlifi --- arch/arm64/boot/dts/arm/corstone1000.dtsi | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/arm/corstone1000.dtsi b/arch/arm64/boot/dts/arm

[PATCH v2 1/5] dt-bindings: remoteproc: sse710: Add the External Systems remote processors

2024-08-22 Thread Abdellatif El Khlifi
[1]. [1]: https://developer.arm.com/documentation/102360//Overview-of-Corstone-1000/Corstone-SSE-710-subsystem-features Signed-off-by: Abdellatif El Khlifi --- .../remoteproc/arm,sse710-extsys.yaml | 90 +++ 1 file changed, 90 insertions(+) create mode 100644

[PATCH v2 0/5] remoteproc: arm64: Introduce remoteproc support for Corstone-1000 External Systems

2024-08-22 Thread Abdellatif El Khlifi
Abdellatif El Khlifi (5): dt-bindings: remoteproc: sse710: Add the External Systems remote processors dt-bindings: arm: sse710: Add Host Base System Control arm64: dts: corstone1000: Add MHU nodes used by the External System arm64: dts: corstone1000: Add External System support remoteproc

Re: [PATCH 1/3] remoteproc: Add Arm remoteproc driver

2024-03-26 Thread Abdellatif El Khlifi
Hi Mathieu, > > > > > > > > > > This is an initial patchset for allowing to turn on and off > > > > > > > > > > the remote processor. > > > > > > > > > > The FW is already loaded before the Corstone-1000 SoC is > > > > > > > > > > powered on and this > > > > > > > > > > is done through the FPGA

Re: [PATCH 1/3] remoteproc: Add Arm remoteproc driver

2024-03-25 Thread Abdellatif El Khlifi
Hi Mathieu, > > > > > > > > This is an initial patchset for allowing to turn on and off the > > > > > > > > remote processor. > > > > > > > > The FW is already loaded before the Corstone-1000 SoC is > > > > > > > > powered on and this > > > > > > > > is done through the FPGA board bootloader in

Re: [PATCH 3/3] dt-bindings: remoteproc: Add Arm remoteproc

2024-03-15 Thread Abdellatif El Khlifi
Hi Sudeep, On Thu, Mar 14, 2024 at 03:19:13PM +, Sudeep Holla wrote: > > The plan for the driver is as follows: > > > > Step 1: provide a foundation driver capable of turning the core on/off > > Step 2: provide mailbox support for comms > > Step 3: provide FW reload capability > >

Re: [PATCH 3/3] dt-bindings: remoteproc: Add Arm remoteproc

2024-03-14 Thread Abdellatif El Khlifi
Hi Krzysztof, On Thu, Mar 14, 2024 at 02:56:53PM +0100, Krzysztof Kozlowski wrote: > On 14/03/2024 14:49, Abdellatif El Khlifi wrote: > >> Frankly at the moment I'd be inclined to say it isn't even a remoteproc > >> binding (or driver) at all, it'

Re: [PATCH 1/3] remoteproc: Add Arm remoteproc driver

2024-03-14 Thread Abdellatif El Khlifi
Hi Sudeep, On Thu, Mar 14, 2024 at 02:59:20PM +, Sudeep Holla wrote: > On Thu, Mar 14, 2024 at 08:52:59AM -0600, Mathieu Poirier wrote: > > On Wed, Mar 13, 2024 at 05:17:56PM +, Abdellatif El Khlifi wrote: > > > Hi Mathieu, > > > > > > On Wed, Mar 1

Re: [PATCH 3/3] dt-bindings: remoteproc: Add Arm remoteproc

2024-03-14 Thread Abdellatif El Khlifi
Hi Robin, > > + firmware-name: > > +description: | > > + Default name of the firmware to load to the remote processor. > > So... is loading the firmware image achieved by somehow bitbanging it > through the one reset register, maybe? I find it hard to believe this is a > complete and fu

Re: [PATCH 1/3] remoteproc: Add Arm remoteproc driver

2024-03-13 Thread Abdellatif El Khlifi
Hi Mathieu, On Wed, Mar 13, 2024 at 10:25:32AM -0600, Mathieu Poirier wrote: > On Tue, Mar 12, 2024 at 05:32:52PM +0000, Abdellatif El Khlifi wrote: > > Hi Mathieu, > > > > On Tue, Mar 12, 2024 at 10:29:52AM -0600, Mathieu Poirier wrote: > > > > This is an init

Re: [PATCH 1/3] remoteproc: Add Arm remoteproc driver

2024-03-12 Thread Abdellatif El Khlifi
Hi Mathieu, On Tue, Mar 12, 2024 at 10:29:52AM -0600, Mathieu Poirier wrote: > > This is an initial patchset for allowing to turn on and off the remote > > processor. > > The FW is already loaded before the Corstone-1000 SoC is powered on and this > > is done through the FPGA board bootloader in

Re: [PATCH 1/3] remoteproc: Add Arm remoteproc driver

2024-03-11 Thread Abdellatif El Khlifi
Hi Mathieu, On Fri, Mar 08, 2024 at 09:44:26AM -0700, Mathieu Poirier wrote: > On Thu, 7 Mar 2024 at 12:40, Abdellatif El Khlifi > wrote: > > > > Hi Mathieu, > > > > > > + do { > > > > + state_reg = readl(priv->

Re: [PATCH 2/3] arm64: dts: Add corstone1000 external system device node

2024-03-08 Thread Abdellatif El Khlifi
Hi Sudeep, > > + extsys0: remoteproc@1a010310 { > > + compatible = "arm,corstone1000-extsys"; > > + reg = <0x1a010310 0x4>, > > + <0x1a010314 0X4>; > > > As per [1], this is just a few registers within the 64kB block. > Not

Re: [PATCH 3/3] dt-bindings: remoteproc: Add Arm remoteproc

2024-03-08 Thread Abdellatif El Khlifi
Hi Krzysztof, Sudeep, > > > diff --git a/Documentation/devicetree/bindings/remoteproc/arm,rproc.yaml > > > b/Documentation/devicetree/bindings/remoteproc/arm,rproc.yaml > > > new file mode 100644 > > > index ..322197158059 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindi

Re: [PATCH 1/3] remoteproc: Add Arm remoteproc driver

2024-03-07 Thread Abdellatif El Khlifi
Hi Mathieu, > > + do { > > + state_reg = readl(priv->reset_cfg.state_reg); > > + *rst_ack = EXTSYS_RST_ST_RST_ACK(state_reg); > > + > > + if (*rst_ack == EXTSYS_RST_ACK_RESERVED) { > > + dev_err(dev, "unexpected RST_ACK value: 0x%x\n", > > +