diff mbox series

arm-bsp/linux-yocto: corstone1000: Update the Upstream-Status of the remoteproc patches

Message ID 20241212101238.53250-1-abdellatif.elkhlifi@arm.com
State New
Headers show
Series arm-bsp/linux-yocto: corstone1000: Update the Upstream-Status of the remoteproc patches | expand

Commit Message

Abdellatif El Khlifi Dec. 12, 2024, 10:12 a.m. UTC 
From: Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>

Set the Upstream-Status to Denied because the community suggests a different design

The external system implementation in Corstone-1000 is user-defined.
In the implementation provided by he FPGA board and by the FVP, the
Cortex-A35 (Linux) can not access the memory of the external system (Cortex-M3).
So, Linux can not load the external system firmware and can not communicate
with the external system using Rpmsg over remoteproc subsystem. The reason is Rpmsg
needs vrings memory buffers to be shared between both cores.
The community prefers that the HW is updated with memory sharing before they
consider merging the remoteproc driver.

We reached the agreement that we will split the work in two parts:

Part 1: Writing an SSE-710 reset controller driver
Part 2: Corstone-1000 remoteproc driver

Part 1 is doable and we will be working on it.
Part 2 is waiting for the FPGA upgrade with the memory sharing feature.

For more details [1].

[1]: https://lore.kernel.org/all/20241009094635.GA14639@e130802.arm.com/

Signed-off-by: Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
---
 .../0001-remoteproc-Add-Arm-remoteproc-driver.patch             | 2 +-
 ...2-arm64-dts-Add-corstone1000-external-system-device-no.patch | 2 +-
 .../0003-dt-bindings-remoteproc-Add-Arm-remoteproc.patch        | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

Comments

Jon Mason Dec. 17, 2024, 2:02 p.m. UTC | #1 
On Thu, 12 Dec 2024 10:12:38 +0000, abdellatif.elkhlifi@arm.com wrote:
> Set the Upstream-Status to Denied because the community suggests a different design
> 
> The external system implementation in Corstone-1000 is user-defined.
> In the implementation provided by he FPGA board and by the FVP, the
> Cortex-A35 (Linux) can not access the memory of the external system (Cortex-M3).
> So, Linux can not load the external system firmware and can not communicate
> with the external system using Rpmsg over remoteproc subsystem. The reason is Rpmsg
> needs vrings memory buffers to be shared between both cores.
> The community prefers that the HW is updated with memory sharing before they
> consider merging the remoteproc driver.
> 
> [...]

Applied, thanks!

[1/1] arm-bsp/linux-yocto: corstone1000: Update the Upstream-Status of the remoteproc patches
      commit: 258277bab5a716abf35071fd9ffd250166f4df1d

Best regards,
diff mbox series

Patch

diff --git a/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0001-remoteproc-Add-Arm-remoteproc-driver.patch b/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0001-remoteproc-Add-Arm-remoteproc-driver.patch
index 50a6fb2b..c99be74a 100644
--- a/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0001-remoteproc-Add-Arm-remoteproc-driver.patch
+++ b/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0001-remoteproc-Add-Arm-remoteproc-driver.patch
@@ -18,7 +18,7 @@  The driver also supports control of multiple remote processors at the
 same time.
 
 Signed-off-by: Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
-Upstream-Status: Submitted [cover letter: https://lore.kernel.org/all/20240301164227.339208-1-abdellatif.elkhlifi@arm.com/]
+Upstream-Status: Denied [Agreement reached: https://lore.kernel.org/all/20241009094635.GA14639@e130802.arm.com/]
 ---
  MAINTAINERS                    |   6 +
  drivers/remoteproc/Kconfig     |  18 ++
diff --git a/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0002-arm64-dts-Add-corstone1000-external-system-device-no.patch b/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0002-arm64-dts-Add-corstone1000-external-system-device-no.patch
index 5c1f3de4..c4dc7ff6 100644
--- a/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0002-arm64-dts-Add-corstone1000-external-system-device-no.patch
+++ b/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0002-arm64-dts-Add-corstone1000-external-system-device-no.patch
@@ -6,7 +6,7 @@  Subject: [PATCH 2/6] arm64: dts: Add corstone1000 external system device node
 add device tree node for the external system core in Corstone-1000
 
 Signed-off-by: Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
-Upstream-Status: Submitted [cover letter: https://lore.kernel.org/all/20240301164227.339208-1-abdellatif.elkhlifi@arm.com/]
+Upstream-Status: Denied [Agreement reached: https://lore.kernel.org/all/20241009094635.GA14639@e130802.arm.com/]
 ---
  arch/arm64/boot/dts/arm/corstone1000.dtsi | 10 +++++++++-
  1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0003-dt-bindings-remoteproc-Add-Arm-remoteproc.patch b/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0003-dt-bindings-remoteproc-Add-Arm-remoteproc.patch
index 3c7bb0dd..cf0d5f09 100644
--- a/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0003-dt-bindings-remoteproc-Add-Arm-remoteproc.patch
+++ b/meta-arm-bsp/recipes-kernel/linux/files/corstone1000/0003-dt-bindings-remoteproc-Add-Arm-remoteproc.patch
@@ -6,7 +6,7 @@  Subject: [PATCH 3/6] dt-bindings: remoteproc: Add Arm remoteproc
 introduce the bindings for Arm remoteproc support.
 
 Signed-off-by: Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
-Upstream-Status: Submitted [cover letter: https://lore.kernel.org/all/20240301164227.339208-1-abdellatif.elkhlifi@arm.com/]
+Upstream-Status: Denied [Agreement reached: https://lore.kernel.org/all/20241009094635.GA14639@e130802.arm.com/]
 ---
  .../bindings/remoteproc/arm,rproc.yaml        | 69 +++++++++++++++++++
  MAINTAINERS                                   |  1 +