public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [PATCH] configs: k3_*: Add config fragments for inline ECC and BIST
@ 2026-02-20  9:15 Neha Malcom Francis
  2026-02-20 14:45 ` Tom Rini
  2026-02-22  1:54 ` Kumar, Udit
  0 siblings, 2 replies; 5+ messages in thread
From: Neha Malcom Francis @ 2026-02-20  9:15 UTC (permalink / raw)
  To: trini; +Cc: bb, reatmon, u-boot, u-kumar1, gehariprasath, s-k6, n-francis

Add config fragment support for enabling inline ECC and/or BIST on TI K3
supported platforms.

Signed-off-by: Neha Malcom Francis <n-francis@ti.com>
---
Typically we do not enable these configs by default but would still like to have
the option to start building them in our default build flow for testing. Also
there is the added advantage of users being able to see what is needed in case
they choose to enable these features.

 configs/k3_bist.config       | 1 +
 configs/k3_inline_ecc.config | 1 +
 2 files changed, 2 insertions(+)
 create mode 100644 configs/k3_bist.config
 create mode 100644 configs/k3_inline_ecc.config

diff --git a/configs/k3_bist.config b/configs/k3_bist.config
new file mode 100644
index 00000000000..671dda1a9dc
--- /dev/null
+++ b/configs/k3_bist.config
@@ -0,0 +1 @@
+CONFIG_K3_BIST=y
diff --git a/configs/k3_inline_ecc.config b/configs/k3_inline_ecc.config
new file mode 100644
index 00000000000..143c814b41c
--- /dev/null
+++ b/configs/k3_inline_ecc.config
@@ -0,0 +1 @@
+CONFIG_K3_INLINE_ECC=y
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] configs: k3_*: Add config fragments for inline ECC and BIST
  2026-02-20  9:15 [PATCH] configs: k3_*: Add config fragments for inline ECC and BIST Neha Malcom Francis
@ 2026-02-20 14:45 ` Tom Rini
  2026-02-24 10:46   ` Neha Malcom Francis
  2026-02-22  1:54 ` Kumar, Udit
  1 sibling, 1 reply; 5+ messages in thread
From: Tom Rini @ 2026-02-20 14:45 UTC (permalink / raw)
  To: Neha Malcom Francis; +Cc: bb, reatmon, u-boot, u-kumar1, gehariprasath, s-k6

[-- Attachment #1: Type: text/plain, Size: 700 bytes --]

On Fri, Feb 20, 2026 at 02:45:32PM +0530, Neha Malcom Francis wrote:

> Add config fragment support for enabling inline ECC and/or BIST on TI K3
> supported platforms.
> 
> Signed-off-by: Neha Malcom Francis <n-francis@ti.com>
> ---
> Typically we do not enable these configs by default but would still like to have
> the option to start building them in our default build flow for testing. Also
> there is the added advantage of users being able to see what is needed in case
> they choose to enable these features.

If we want to partly justify this as helping users, we need to update
the board documentation as well. Also, can these live under
board/ti/common perhaps?

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] configs: k3_*: Add config fragments for inline ECC and BIST
  2026-02-20  9:15 [PATCH] configs: k3_*: Add config fragments for inline ECC and BIST Neha Malcom Francis
  2026-02-20 14:45 ` Tom Rini
@ 2026-02-22  1:54 ` Kumar, Udit
  2026-02-24 10:06   ` Neha Malcom Francis
  1 sibling, 1 reply; 5+ messages in thread
From: Kumar, Udit @ 2026-02-22  1:54 UTC (permalink / raw)
  To: Neha Malcom Francis, trini
  Cc: bb, reatmon, u-boot, gehariprasath, s-k6, u-kumar1


On 2/20/2026 2:45 PM, Neha Malcom Francis wrote:
> Add config fragment support for enabling inline ECC and/or BIST on TI K3
> supported platforms.
>
> Signed-off-by: Neha Malcom Francis <n-francis@ti.com>
> ---
> Typically we do not enable these configs by default but would still like to have
> the option to start building them in our default build flow for testing. Also
> there is the added advantage of users being able to see what is needed in case
> they choose to enable these features.

Can these fragments exits together ? , if yes then one config fragment 
should be ok


>
>   configs/k3_bist.config       | 1 +
>   configs/k3_inline_ecc.config | 1 +
>   2 files changed, 2 insertions(+)
>   create mode 100644 configs/k3_bist.config
>   create mode 100644 configs/k3_inline_ecc.config
>
> diff --git a/configs/k3_bist.config b/configs/k3_bist.config
> new file mode 100644
> index 00000000000..671dda1a9dc
> --- /dev/null
> +++ b/configs/k3_bist.config
> @@ -0,0 +1 @@
> +CONFIG_K3_BIST=y
> diff --git a/configs/k3_inline_ecc.config b/configs/k3_inline_ecc.config
> new file mode 100644
> index 00000000000..143c814b41c
> --- /dev/null
> +++ b/configs/k3_inline_ecc.config
> @@ -0,0 +1 @@
> +CONFIG_K3_INLINE_ECC=y

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] configs: k3_*: Add config fragments for inline ECC and BIST
  2026-02-22  1:54 ` Kumar, Udit
@ 2026-02-24 10:06   ` Neha Malcom Francis
  0 siblings, 0 replies; 5+ messages in thread
From: Neha Malcom Francis @ 2026-02-24 10:06 UTC (permalink / raw)
  To: Kumar, Udit, trini; +Cc: bb, reatmon, u-boot, gehariprasath, s-k6

Hi Udit,

On 22/02/26 07:24, Kumar, Udit wrote:
> 
> On 2/20/2026 2:45 PM, Neha Malcom Francis wrote:
>> Add config fragment support for enabling inline ECC and/or BIST on TI K3
>> supported platforms.
>>
>> Signed-off-by: Neha Malcom Francis <n-francis@ti.com>
>> ---
>> Typically we do not enable these configs by default but would still
>> like to have
>> the option to start building them in our default build flow for
>> testing. Also
>> there is the added advantage of users being able to see what is needed
>> in case
>> they choose to enable these features.
> 
> Can these fragments exits together ? , if yes then one config fragment
> should be ok

No since INLINE_ECC is applicable only to R5 builds while BIST is
applicable for only A72.
> 
> 
>>
>>   configs/k3_bist.config       | 1 +
>>   configs/k3_inline_ecc.config | 1 +
>>   2 files changed, 2 insertions(+)
>>   create mode 100644 configs/k3_bist.config
>>   create mode 100644 configs/k3_inline_ecc.config
>>
>> diff --git a/configs/k3_bist.config b/configs/k3_bist.config
>> new file mode 100644
>> index 00000000000..671dda1a9dc
>> --- /dev/null
>> +++ b/configs/k3_bist.config
>> @@ -0,0 +1 @@
>> +CONFIG_K3_BIST=y
>> diff --git a/configs/k3_inline_ecc.config b/configs/k3_inline_ecc.config
>> new file mode 100644
>> index 00000000000..143c814b41c
>> --- /dev/null
>> +++ b/configs/k3_inline_ecc.config
>> @@ -0,0 +1 @@
>> +CONFIG_K3_INLINE_ECC=y

-- 
Thanking You
Neha Malcom Francis


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] configs: k3_*: Add config fragments for inline ECC and BIST
  2026-02-20 14:45 ` Tom Rini
@ 2026-02-24 10:46   ` Neha Malcom Francis
  0 siblings, 0 replies; 5+ messages in thread
From: Neha Malcom Francis @ 2026-02-24 10:46 UTC (permalink / raw)
  To: Tom Rini; +Cc: bb, reatmon, u-boot, u-kumar1, gehariprasath, s-k6

Hi Tom

On 20/02/26 20:15, Tom Rini wrote:
> On Fri, Feb 20, 2026 at 02:45:32PM +0530, Neha Malcom Francis wrote:
> 
>> Add config fragment support for enabling inline ECC and/or BIST on TI K3
>> supported platforms.
>>
>> Signed-off-by: Neha Malcom Francis <n-francis@ti.com>
>> ---
>> Typically we do not enable these configs by default but would still like to have
>> the option to start building them in our default build flow for testing. Also
>> there is the added advantage of users being able to see what is needed in case
>> they choose to enable these features.
> 
> If we want to partly justify this as helping users, we need to update
> the board documentation as well. Also, can these live under
> board/ti/common perhaps?

Yes I can add documentation.

And yes putting it under board/ti/common can work.

> 

-- 
Thanking You
Neha Malcom Francis


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2026-02-24 10:47 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-20  9:15 [PATCH] configs: k3_*: Add config fragments for inline ECC and BIST Neha Malcom Francis
2026-02-20 14:45 ` Tom Rini
2026-02-24 10:46   ` Neha Malcom Francis
2026-02-22  1:54 ` Kumar, Udit
2026-02-24 10:06   ` Neha Malcom Francis

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox