[PATCH v1 0/1] Adds support for OCP master port configuration of PRU-ICSS

[PATCH v1 0/1] Adds support for OCP master port configuration of PRU-ICSS

[Parvathi Pudi]

Hi,

This patch updates OCP master port configuration to enable memory access
outside of the PRU-ICSS subsystem.

It configures PRUSS_SYSCFG.STANDBY_INIT bit to enable the OCP master ports
on AM335x, AM437x and AM57x devices.

Thanks and Regards,
Parvathi.

Parvathi Pudi (1):
  bus: ti-sysc: PRUSS OCP configuration

 drivers/bus/ti-sysc.c | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

-- 
2.34.1

[PATCH v1 1/1] bus: ti-sysc: PRUSS OCP configuration

[Parvathi Pudi]

Updates OCP master port configuration to enable memory access outside
of the PRU-ICSS subsystem.

This set of changes configures PRUSS_SYSCFG.STANDBY_INIT bit to enable
the OCP master ports during resume sequence and disables the OCP master
ports during suspend sequence (applicable only on SoCs using OCP
interconnect like the OMAP family).

Signed-off-by: Parvathi Pudi <parvathi@couthit.com>
---
 drivers/bus/ti-sysc.c | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/drivers/bus/ti-sysc.c b/drivers/bus/ti-sysc.c
index f67b927ae4ca..51caae611acf 100644
--- a/drivers/bus/ti-sysc.c
+++ b/drivers/bus/ti-sysc.c
@@ -2036,6 +2036,19 @@ static void sysc_module_disable_quirk_pruss(struct sysc *ddata)
 	sysc_write(ddata, ddata->offsets[SYSC_SYSCONFIG], reg);
 }
 
+static void sysc_module_enable_quirk_pruss(struct sysc *ddata)
+{
+	u32 reg;
+
+	reg = sysc_read(ddata, ddata->offsets[SYSC_SYSCONFIG]);
+	/* Clearing the SYSC_PRUSS_STANDBY_INIT bit - Updates OCP master
+	 * port configuration to enable memory access outside of the
+	 * PRU-ICSS subsystem.
+	 */
+	reg &= (~SYSC_PRUSS_STANDBY_INIT);
+	sysc_write(ddata, ddata->offsets[SYSC_SYSCONFIG], reg);
+}
+
 static void sysc_init_module_quirks(struct sysc *ddata)
 {
 	if (ddata->legacy_mode || !ddata->name)
@@ -2088,8 +2101,10 @@ static void sysc_init_module_quirks(struct sysc *ddata)
 		ddata->module_disable_quirk = sysc_reset_done_quirk_wdt;
 	}
 
-	if (ddata->cfg.quirks & SYSC_MODULE_QUIRK_PRUSS)
+	if (ddata->cfg.quirks & SYSC_MODULE_QUIRK_PRUSS) {
+		ddata->module_enable_quirk = sysc_module_enable_quirk_pruss;
 		ddata->module_disable_quirk = sysc_module_disable_quirk_pruss;
+	}
 }
 
 static int sysc_clockdomain_init(struct sysc *ddata)
-- 
2.34.1

Re: [PATCH v1 0/1] Adds support for OCP master port configuration of PRU-ICSS

[Parvathi Pudi]

Hi,

> 
> This patch updates OCP master port configuration to enable memory access
> outside of the PRU-ICSS subsystem.
> 
> It configures PRUSS_SYSCFG.STANDBY_INIT bit to enable the OCP master ports
> on AM335x, AM437x and AM57x devices.
> 
> Thanks and Regards,
> Parvathi.
> 
> Parvathi Pudi (1):
>  bus: ti-sysc: PRUSS OCP configuration
> 
> drivers/bus/ti-sysc.c | 17 ++++++++++++++++-
> 1 file changed, 16 insertions(+), 1 deletion(-)
> 

This is a follow-up request for feedback in case this patch is missed, as it's
been two weeks since the patch was submitted and no feedback has been received.


Thanks and Regards,
Parvathi.

Re: [PATCH v1 1/1] bus: ti-sysc: PRUSS OCP configuration

[Andreas Kemnade]

Am Mon,  7 Apr 2025 12:51:34 +0530
schrieb Parvathi Pudi <parvathi@couthit.com>:

> Updates OCP master port configuration to enable memory access outside
> of the PRU-ICSS subsystem.
> 
> This set of changes configures PRUSS_SYSCFG.STANDBY_INIT bit to enable
> the OCP master ports during resume sequence and disables the OCP master
> ports during suspend sequence (applicable only on SoCs using OCP
> interconnect like the OMAP family).
> 
> Signed-off-by: Parvathi Pudi <parvathi@couthit.com>

mirrors what is done on module disable, so it looks sane.

Reviewed-by: Andreas Kemnade <andreas@kemnade.info>

Re: [PATCH v1 1/1] bus: ti-sysc: PRUSS OCP configuration

[Parvathi Pudi]

Hi,
> Am Mon,  7 Apr 2025 12:51:34 +0530
> schrieb Parvathi Pudi <parvathi@couthit.com>:
> 
>> Updates OCP master port configuration to enable memory access outside
>> of the PRU-ICSS subsystem.
>> 
>> This set of changes configures PRUSS_SYSCFG.STANDBY_INIT bit to enable
>> the OCP master ports during resume sequence and disables the OCP master
>> ports during suspend sequence (applicable only on SoCs using OCP
>> interconnect like the OMAP family).
>> 
>> Signed-off-by: Parvathi Pudi <parvathi@couthit.com>
> 
> mirrors what is done on module disable, so it looks sane.
> 

Yes.

> Reviewed-by: Andreas Kemnade <andreas@kemnade.info>

Thanks for review.


Thanks and Regards,
Parvathi.

Re: [PATCH v1 1/1] bus: ti-sysc: PRUSS OCP configuration

[Kevin Hilman]

Parvathi Pudi <parvathi@couthit.com> writes:

> Updates OCP master port configuration to enable memory access outside
> of the PRU-ICSS subsystem.
>
> This set of changes configures PRUSS_SYSCFG.STANDBY_INIT bit to enable
> the OCP master ports during resume sequence and disables the OCP master
> ports during suspend sequence (applicable only on SoCs using OCP
> interconnect like the OMAP family).
>
> Signed-off-by: Parvathi Pudi <parvathi@couthit.com>
> ---
>  drivers/bus/ti-sysc.c | 17 ++++++++++++++++-
>  1 file changed, 16 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/bus/ti-sysc.c b/drivers/bus/ti-sysc.c
> index f67b927ae4ca..51caae611acf 100644
> --- a/drivers/bus/ti-sysc.c
> +++ b/drivers/bus/ti-sysc.c
> @@ -2036,6 +2036,19 @@ static void sysc_module_disable_quirk_pruss(struct sysc *ddata)
>  	sysc_write(ddata, ddata->offsets[SYSC_SYSCONFIG], reg);
>  }
>  
> +static void sysc_module_enable_quirk_pruss(struct sysc *ddata)
> +{
> +	u32 reg;
> +
> +	reg = sysc_read(ddata, ddata->offsets[SYSC_SYSCONFIG]);
> +	/* Clearing the SYSC_PRUSS_STANDBY_INIT bit - Updates OCP master
> +	 * port configuration to enable memory access outside of the
> +	 * PRU-ICSS subsystem.
> +	 */

minor nit: incorrect multi-line comment style (first /* should be on
line of its own)

But I fixed this up locally before applying.

Kevin

Re: [PATCH v1 1/1] bus: ti-sysc: PRUSS OCP configuration

[Kevin Hilman]

On Mon, 07 Apr 2025 12:51:34 +0530, Parvathi Pudi wrote:
> Updates OCP master port configuration to enable memory access outside
> of the PRU-ICSS subsystem.
> 
> This set of changes configures PRUSS_SYSCFG.STANDBY_INIT bit to enable
> the OCP master ports during resume sequence and disables the OCP master
> ports during suspend sequence (applicable only on SoCs using OCP
> interconnect like the OMAP family).
> 
> [...]

Applied, thanks!

[1/1] bus: ti-sysc: PRUSS OCP configuration
      commit: 7d25c4e23763298f46f1ac955bf9b0a872662316

Best regards,
-- 
Kevin Hilman <khilman@baylibre.com>