All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Kumar, Udit" <u-kumar1@ti.com>
To: Andrew Davis <afd@ti.com>,
	"Richard Genoud (TI)" <richard.genoud@bootlin.com>,
	Tom Rini <trini@konsulko.com>,
	Manorit Chawdhry <m-chawdhry@ti.com>,
	Apurva Nandan <a-nandan@ti.com>,
	Vignesh Raghavendra <vigneshr@ti.com>, Bryan Brattlof <bb@ti.com>,
	Vaishnav Achath <vaishnav.a@ti.com>,
	Jayesh Choudhary <j-choudhary@ti.com>,
	Simon Glass <sjg@chromium.org>,
	Alper Nebi Yasak <alpernebiyasak@gmail.com>
Cc: Markus Schneider-Pargmann <msp@baylibre.com>,
	Abhash Kumar <a-kumar2@ti.com>,
	Thomas Richard <thomas.richard@bootlin.com>,
	"Gregory CLEMENT" <gregory.clement@bootlin.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	<u-boot@lists.denx.de>
Subject: Re: [PATCH 19/20] arm: dts: k3-j7200: Remove background firewall on DDR
Date: Mon, 11 May 2026 16:26:12 +0530	[thread overview]
Message-ID: <afbf5dbb-8d2f-45dc-b34f-d448e09353c4@ti.com> (raw)
In-Reply-To: <f5e29a2c-70ec-4569-8204-333a0efb5eec@ti.com>



On 4/30/2026 11:00 PM, Andrew Davis wrote:
> On 4/30/26 3:44 AM, Richard Genoud (TI) wrote:
>> From: Prasanth Babu Mantena <p-mantena@ti.com>
>>
>> With updated TIFS version tag v11.02.06, the background firewall
>> for DDR firewalls are applied by the TIFS. This removes the
>> dependency of applying firewalls through binman.
>>
> 
> Does this then create a dependency on using a TIFS firmware
> equal or newer than v11.02.06 when using a new U-Boot?
> 
> Andrew
> 

I am not sure, how this will behave but we have to keep backward and
forward compatibility
>> Remove background firewall on the whole DDR, as this
>> background firewall is applied by the TIFS init for j7200
>> and j784s4. Modify for j7200 only, as there are no firewalls applied
>> for j784s4 from SPL.
>>
>> Signed-off-by: Prasanth Babu Mantena <p-mantena@ti.com>
>> ---
>>   arch/arm/dts/k3-j7200-binman.dtsi | 14 --------------
>>   1 file changed, 14 deletions(-)
>>
>> diff --git a/arch/arm/dts/k3-j7200-binman.dtsi b/arch/arm/dts/k3-
>> j7200-binman.dtsi
>> index b4e0ce8bfcf2..c2b86339d593 100644
>> --- a/arch/arm/dts/k3-j7200-binman.dtsi
>> +++ b/arch/arm/dts/k3-j7200-binman.dtsi
>> @@ -294,13 +294,6 @@
>>                               region = <2>;
>>                           };
>>   -                        firewall-4762-0 {
>> -                            /* nb_slv2__mem0 Background Firewall - 0 */
>> -                            insert-template = <&firewall_bg_3>;
>> -                            id = <4762>;
>> -                            region = <0>;
>> -                        };
>> -
>>                           firewall-4762-1 {
>>                               /* nb_slv2__mem0 Foreground Firewall */
>>                               insert-template =
>> <&firewall_armv8_optee_fg>;
>> @@ -308,13 +301,6 @@
>>                               region = <1>;
>>                           };
>>   -                        firewall-4763-0 {
>> -                            /* nb_slv3__mem0 Background Firewall - 0 */
>> -                            insert-template = <&firewall_bg_3>;
>> -                            id = <4763>;
>> -                            region = <0>;
>> -                        };
>> -
>>                           firewall-4763-1 {
>>                               /* nb_slv3__mem0 Foreground Firewall */
>>                               insert-template =
>> <&firewall_armv8_optee_fg>;
> 


  reply	other threads:[~2026-05-11 10:56 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-30  8:43 [PATCH 00/20] Introduce resume for J7xx SoCs Richard Genoud (TI)
2026-04-30  8:43 ` [PATCH 01/20] configs: j784s4_evm_r5: enable TI_SCI_POWER_DOMAIN Richard Genoud (TI)
2026-05-11  9:49   ` Kumar, Udit
2026-04-30  8:43 ` [PATCH 02/20] global: k3: use gd to store the resume state Richard Genoud (TI)
2026-04-30  8:43 ` [PATCH 03/20] board: ti: j721e: Add resume detection for J721e/J7200 Richard Genoud (TI)
2026-05-11  9:53   ` Kumar, Udit
2026-04-30  8:43 ` [PATCH 04/20] board: ti: j784s4: Add resume detection Richard Genoud (TI)
2026-05-11 10:47   ` Kumar, Udit
2026-04-30  8:43 ` [PATCH 05/20] board: ti: j722s: " Richard Genoud (TI)
2026-04-30  8:43 ` [PATCH 06/20] board: ti: j721s2: " Richard Genoud (TI)
2026-05-11 10:48   ` Kumar, Udit
2026-04-30  8:44 ` [PATCH 07/20] ram: k3-ddrss: Add exit retention support Richard Genoud (TI)
2026-04-30  8:44 ` [PATCH 08/20] ram: k3-ddrss: Add j722s DDR resume sequence Richard Genoud (TI)
2026-04-30  8:44 ` [PATCH 09/20] ram: k3-ddrss: support j784s4/j721e/j721s2 DDR resume Richard Genoud (TI)
2026-04-30  8:44 ` [PATCH 10/20] firmware: ti_sci: add low power mode operations Richard Genoud (TI)
2026-05-11 10:50   ` Kumar, Udit
2026-04-30  8:44 ` [PATCH 11/20] mach-k3: r5: common: add helper functions needed in LPM resume sequence Richard Genoud (TI)
2026-05-11 10:52   ` Kumar, Udit
2026-04-30  8:44 ` [PATCH 12/20] arm: mach-k3: j721e: Enable LPM resume flow for J7200/J721e SOC Richard Genoud (TI)
2026-04-30  8:44 ` [PATCH 13/20] arm: mach-k3: j784s4: Enable LPM resume flow for J784s4/J742s2 SOCs Richard Genoud (TI)
2026-04-30  8:44 ` [PATCH 14/20] arm: mach-k3: j722s: Enable LPM resume flow Richard Genoud (TI)
2026-04-30  8:44 ` [PATCH 15/20] arm: mach-k3: j721s2: " Richard Genoud (TI)
2026-04-30  8:44 ` [PATCH 16/20] arm: mach-k3: Update pm-boardcfg for all k3 platforms Richard Genoud (TI)
2026-05-11 10:54   ` Kumar, Udit
2026-04-30  8:44 ` [PATCH 17/20] board: evm: Enable de-isolation of IOs at resume for j7200 and j784s4 Richard Genoud (TI)
2026-04-30  8:44 ` [PATCH 18/20] board: ti: evm: Store wakeup reason on scratchpad memory Richard Genoud (TI)
2026-04-30 17:32   ` Andrew Davis
2026-04-30  8:44 ` [PATCH 19/20] arm: dts: k3-j7200: Remove background firewall on DDR Richard Genoud (TI)
2026-04-30 17:30   ` Andrew Davis
2026-05-11 10:56     ` Kumar, Udit [this message]
2026-04-30  8:44 ` [PATCH 20/20] arm: dts: k3-j7200: Extend firewall for ATF region to TIFS Richard Genoud (TI)
2026-04-30 17:28   ` Andrew Davis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=afbf5dbb-8d2f-45dc-b34f-d448e09353c4@ti.com \
    --to=u-kumar1@ti.com \
    --cc=a-kumar2@ti.com \
    --cc=a-nandan@ti.com \
    --cc=afd@ti.com \
    --cc=alpernebiyasak@gmail.com \
    --cc=bb@ti.com \
    --cc=gregory.clement@bootlin.com \
    --cc=j-choudhary@ti.com \
    --cc=m-chawdhry@ti.com \
    --cc=msp@baylibre.com \
    --cc=richard.genoud@bootlin.com \
    --cc=sjg@chromium.org \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=thomas.richard@bootlin.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=vaishnav.a@ti.com \
    --cc=vigneshr@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.