On 09/09/23 16:28, Roger Quadros wrote:

On 08/09/2023 14:05, Apurva Nandan wrote:
From: Hari Nagalla <hnaga...@ti.com>

The K3 J784S4 SoCs have four C71x DSP subsystems in MAIN voltage domain
missing period.

SoCs have/SoC has

The C71x DSPs are 64 bit machine with fixed and floating point DSP> operations. 
The C71x DSP subsystem is a slighly updated version of the
slighly/slightly

C71x DSP subsystem on J721e. And, same ip data can be used for J7200,
J721e/J721E
ip/IP

J721S2 and J784S4.

Extend support to the C71x DSPs for J784S4 with J721S2 compatible
string.

Signed-off-by: Hari Nagalla <hnaga...@ti.com>
Signed-off-by: Apurva Nandan <a-nan...@ti.com>
---
  drivers/remoteproc/ti_k3_dsp_rproc.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/remoteproc/ti_k3_dsp_rproc.c 
b/drivers/remoteproc/ti_k3_dsp_rproc.c
index 1a2e9dd1f5..d856e31db8 100644
--- a/drivers/remoteproc/ti_k3_dsp_rproc.c
+++ b/drivers/remoteproc/ti_k3_dsp_rproc.c
@@ -327,7 +327,8 @@ static int k3_dsp_of_get_memories(struct udevice *dev)
for (i = 0; i < dsp->num_mems; i++) {
                /* C71 cores only have a L1P Cache, there are no L1P SRAMs */
-               if (device_is_compatible(dev, "ti,j721e-c71-dsp") &&
+               if (((device_is_compatible(dev, "ti,j721e-c71-dsp")) ||
+                    (device_is_compatible(dev, "ti,j721s2-c71-dsp"))) &&
                    !strcmp(mem_names[i], "l1pram")) {
                        dsp->mem[i].bus_addr = FDT_ADDR_T_NONE;
                        dsp->mem[i].dev_addr = FDT_ADDR_T_NONE;
@@ -446,6 +447,7 @@ static const struct k3_dsp_boot_data c71_data = {
  static const struct udevice_id k3_dsp_ids[] = {
        { .compatible = "ti,j721e-c66-dsp", .data = (ulong)&c66_data, },
        { .compatible = "ti,j721e-c71-dsp", .data = (ulong)&c71_data, },
+       { .compatible = "ti,j721s2-c71-dsp", .data = (ulong)&c71_data, },
        {}
  };
Okay, will fix all, thanks!

--
Thanks and regards,
Apurva Nandan,
Texas Instruments India.

Reply via email to