* [PATCH] configs: j722s_evm_a53_defconfig: Allow configurability of MMCSD speeds @ 2026-02-20 7:25 Moteen Shah 2026-02-20 8:35 ` Kumar, Udit 0 siblings, 1 reply; 6+ messages in thread From: Moteen Shah @ 2026-02-20 7:25 UTC (permalink / raw) To: u-boot Cc: trini, m-chawdhry, n-francis, vigneshr, u-kumar1, gehariprasath, m-shah Add config to allow UHS support, in turn allowing to select UHS speed modes for the SD card Signed-off-by: Moteen Shah <m-shah@ti.com> --- Test logs: https://gist.github.com/Jamm02/e8b8b8c95cb62a7145d73f7ef50b2d8e configs/j722s_evm_a53_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/j722s_evm_a53_defconfig b/configs/j722s_evm_a53_defconfig index e84670c75ed..24ee293c803 100644 --- a/configs/j722s_evm_a53_defconfig +++ b/configs/j722s_evm_a53_defconfig @@ -107,6 +107,7 @@ CONFIG_MMC_SDHCI=y CONFIG_MMC_SDHCI_ADMA=y CONFIG_SPL_MMC_SDHCI_ADMA=y CONFIG_MMC_SDHCI_AM654=y +CONFIG_MMC_UHS_SUPPORT=y CONFIG_MTD=y CONFIG_DM_MTD=y CONFIG_MTD_SPI_NAND=y -- 2.34.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] configs: j722s_evm_a53_defconfig: Allow configurability of MMCSD speeds 2026-02-20 7:25 [PATCH] configs: j722s_evm_a53_defconfig: Allow configurability of MMCSD speeds Moteen Shah @ 2026-02-20 8:35 ` Kumar, Udit 2026-02-20 9:47 ` Moteen Shah 0 siblings, 1 reply; 6+ messages in thread From: Kumar, Udit @ 2026-02-20 8:35 UTC (permalink / raw) To: Moteen Shah, u-boot; +Cc: trini, m-chawdhry, n-francis, vigneshr, gehariprasath On 2/20/2026 12:55 PM, Moteen Shah wrote: > Add config to allow UHS support, in turn allowing to select > UHS speed modes for the SD card > > Signed-off-by: Moteen Shah <m-shah@ti.com> > --- > Test logs: https://gist.github.com/Jamm02/e8b8b8c95cb62a7145d73f7ef50b2d8e > > configs/j722s_evm_a53_defconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/configs/j722s_evm_a53_defconfig b/configs/j722s_evm_a53_defconfig > index e84670c75ed..24ee293c803 100644 > --- a/configs/j722s_evm_a53_defconfig > +++ b/configs/j722s_evm_a53_defconfig > @@ -107,6 +107,7 @@ CONFIG_MMC_SDHCI=y > CONFIG_MMC_SDHCI_ADMA=y > CONFIG_SPL_MMC_SDHCI_ADMA=y > CONFIG_MMC_SDHCI_AM654=y > +CONFIG_MMC_UHS_SUPPORT=y you may need at A72/SPL stage as well, and do you think , R5/SPL will need this too ? > CONFIG_MTD=y > CONFIG_DM_MTD=y > CONFIG_MTD_SPI_NAND=y ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] configs: j722s_evm_a53_defconfig: Allow configurability of MMCSD speeds 2026-02-20 8:35 ` Kumar, Udit @ 2026-02-20 9:47 ` Moteen Shah 2026-03-09 23:15 ` Tom Rini 0 siblings, 1 reply; 6+ messages in thread From: Moteen Shah @ 2026-02-20 9:47 UTC (permalink / raw) To: Kumar, Udit, u-boot; +Cc: trini, m-chawdhry, n-francis, vigneshr, gehariprasath On 20/02/26 14:05, Kumar, Udit wrote: > > On 2/20/2026 12:55 PM, Moteen Shah wrote: >> Add config to allow UHS support, in turn allowing to select >> UHS speed modes for the SD card >> >> Signed-off-by: Moteen Shah <m-shah@ti.com> >> --- >> Test logs: >> https://gist.github.com/Jamm02/e8b8b8c95cb62a7145d73f7ef50b2d8e >> >> configs/j722s_evm_a53_defconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/configs/j722s_evm_a53_defconfig >> b/configs/j722s_evm_a53_defconfig >> index e84670c75ed..24ee293c803 100644 >> --- a/configs/j722s_evm_a53_defconfig >> +++ b/configs/j722s_evm_a53_defconfig >> @@ -107,6 +107,7 @@ CONFIG_MMC_SDHCI=y >> CONFIG_MMC_SDHCI_ADMA=y >> CONFIG_SPL_MMC_SDHCI_ADMA=y >> CONFIG_MMC_SDHCI_AM654=y >> +CONFIG_MMC_UHS_SUPPORT=y > > > you may need at A72/SPL stage as well, Other J7 boards already have this config enabled in the a72 stage. > > and do you think , R5/SPL will need this too ? Any of the K3 devices does not enables this in R5 SPL, though we do have HS400 enabled in the R5 stage. Adding the config should result in a little performance gain I think. Regards, Moteen > > >> CONFIG_MTD=y >> CONFIG_DM_MTD=y >> CONFIG_MTD_SPI_NAND=y ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] configs: j722s_evm_a53_defconfig: Allow configurability of MMCSD speeds 2026-02-20 9:47 ` Moteen Shah @ 2026-03-09 23:15 ` Tom Rini 2026-03-12 9:16 ` Moteen Shah 0 siblings, 1 reply; 6+ messages in thread From: Tom Rini @ 2026-03-09 23:15 UTC (permalink / raw) To: Moteen Shah Cc: Kumar, Udit, u-boot, m-chawdhry, n-francis, vigneshr, gehariprasath [-- Attachment #1: Type: text/plain, Size: 1388 bytes --] On Fri, Feb 20, 2026 at 03:17:37PM +0530, Moteen Shah wrote: > > On 20/02/26 14:05, Kumar, Udit wrote: > > > > On 2/20/2026 12:55 PM, Moteen Shah wrote: > > > Add config to allow UHS support, in turn allowing to select > > > UHS speed modes for the SD card > > > > > > Signed-off-by: Moteen Shah <m-shah@ti.com> > > > --- > > > Test logs: > > > https://gist.github.com/Jamm02/e8b8b8c95cb62a7145d73f7ef50b2d8e > > > > > > configs/j722s_evm_a53_defconfig | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/configs/j722s_evm_a53_defconfig > > > b/configs/j722s_evm_a53_defconfig > > > index e84670c75ed..24ee293c803 100644 > > > --- a/configs/j722s_evm_a53_defconfig > > > +++ b/configs/j722s_evm_a53_defconfig > > > @@ -107,6 +107,7 @@ CONFIG_MMC_SDHCI=y > > > CONFIG_MMC_SDHCI_ADMA=y > > > CONFIG_SPL_MMC_SDHCI_ADMA=y > > > CONFIG_MMC_SDHCI_AM654=y > > > +CONFIG_MMC_UHS_SUPPORT=y > > > > > > you may need at A72/SPL stage as well, > > Other J7 boards already have this config enabled in the a72 stage. > > > > and do you think , R5/SPL will need this too ? > > Any of the K3 devices does not enables this in R5 SPL, though we do have > HS400 enabled in the R5 stage. > Adding the config should result in a little performance gain I think. Is this a function of board design, or a SoC-level feature? -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] configs: j722s_evm_a53_defconfig: Allow configurability of MMCSD speeds 2026-03-09 23:15 ` Tom Rini @ 2026-03-12 9:16 ` Moteen Shah 2026-03-12 15:50 ` Tom Rini 0 siblings, 1 reply; 6+ messages in thread From: Moteen Shah @ 2026-03-12 9:16 UTC (permalink / raw) To: Tom Rini Cc: Kumar, Udit, u-boot, m-chawdhry, n-francis, vigneshr, gehariprasath Hey Tom and Udit, On 10/03/26 04:45, Tom Rini wrote: > On Fri, Feb 20, 2026 at 03:17:37PM +0530, Moteen Shah wrote: >> On 20/02/26 14:05, Kumar, Udit wrote: >>> On 2/20/2026 12:55 PM, Moteen Shah wrote: >>>> Add config to allow UHS support, in turn allowing to select >>>> UHS speed modes for the SD card >>>> >>>> Signed-off-by: Moteen Shah <m-shah@ti.com> >>>> --- >>>> Test logs: >>>> https://gist.github.com/Jamm02/e8b8b8c95cb62a7145d73f7ef50b2d8e >>>> >>>> configs/j722s_evm_a53_defconfig | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/configs/j722s_evm_a53_defconfig >>>> b/configs/j722s_evm_a53_defconfig >>>> index e84670c75ed..24ee293c803 100644 >>>> --- a/configs/j722s_evm_a53_defconfig >>>> +++ b/configs/j722s_evm_a53_defconfig >>>> @@ -107,6 +107,7 @@ CONFIG_MMC_SDHCI=y >>>> CONFIG_MMC_SDHCI_ADMA=y >>>> CONFIG_SPL_MMC_SDHCI_ADMA=y >>>> CONFIG_MMC_SDHCI_AM654=y >>>> +CONFIG_MMC_UHS_SUPPORT=y >>> >>> you may need at A72/SPL stage as well, >> Other J7 boards already have this config enabled in the a72 stage. >>> and do you think , R5/SPL will need this too ? >> Any of the K3 devices does not enables this in R5 SPL, though we do have >> HS400 enabled in the R5 stage. >> Adding the config should result in a little performance gain I think. > Is this a function of board design, or a SoC-level feature? SOC level feature, thought it has dependency on the board to provide the right voltage to the card. @udit, I did some profiling with UHS enabled at the R5 stage on J784s4-EVM but I see no performance gain there, likely due to the fact that the R5 stage takes up very minimal percentage of the boot time. This patch can go in as it is. Regards, Moteen > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] configs: j722s_evm_a53_defconfig: Allow configurability of MMCSD speeds 2026-03-12 9:16 ` Moteen Shah @ 2026-03-12 15:50 ` Tom Rini 0 siblings, 0 replies; 6+ messages in thread From: Tom Rini @ 2026-03-12 15:50 UTC (permalink / raw) To: Moteen Shah Cc: Kumar, Udit, u-boot, m-chawdhry, n-francis, vigneshr, gehariprasath [-- Attachment #1: Type: text/plain, Size: 2129 bytes --] On Thu, Mar 12, 2026 at 02:46:27PM +0530, Moteen Shah wrote: > Hey Tom and Udit, > > On 10/03/26 04:45, Tom Rini wrote: > > On Fri, Feb 20, 2026 at 03:17:37PM +0530, Moteen Shah wrote: > > > On 20/02/26 14:05, Kumar, Udit wrote: > > > > On 2/20/2026 12:55 PM, Moteen Shah wrote: > > > > > Add config to allow UHS support, in turn allowing to select > > > > > UHS speed modes for the SD card > > > > > > > > > > Signed-off-by: Moteen Shah <m-shah@ti.com> > > > > > --- > > > > > Test logs: > > > > > https://gist.github.com/Jamm02/e8b8b8c95cb62a7145d73f7ef50b2d8e > > > > > > > > > > configs/j722s_evm_a53_defconfig | 1 + > > > > > 1 file changed, 1 insertion(+) > > > > > > > > > > diff --git a/configs/j722s_evm_a53_defconfig > > > > > b/configs/j722s_evm_a53_defconfig > > > > > index e84670c75ed..24ee293c803 100644 > > > > > --- a/configs/j722s_evm_a53_defconfig > > > > > +++ b/configs/j722s_evm_a53_defconfig > > > > > @@ -107,6 +107,7 @@ CONFIG_MMC_SDHCI=y > > > > > CONFIG_MMC_SDHCI_ADMA=y > > > > > CONFIG_SPL_MMC_SDHCI_ADMA=y > > > > > CONFIG_MMC_SDHCI_AM654=y > > > > > +CONFIG_MMC_UHS_SUPPORT=y > > > > > > > > you may need at A72/SPL stage as well, > > > Other J7 boards already have this config enabled in the a72 stage. > > > > and do you think , R5/SPL will need this too ? > > > Any of the K3 devices does not enables this in R5 SPL, though we do have > > > HS400 enabled in the R5 stage. > > > Adding the config should result in a little performance gain I think. > > Is this a function of board design, or a SoC-level feature? > > SOC level feature, thought it has dependency on the board to provide the > right voltage to the card. > > @udit, I did some profiling with UHS enabled at the R5 stage on J784s4-EVM > but I see no performance gain there, likely due to the fact that the R5 > stage takes up very minimal percentage of the boot time. This patch can go > in as it is. I think we should imply this under ARCH_K3 then, so that boards which do not meet the design requirements can turn it off, as I am assuming most do. -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-03-12 15:50 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-02-20 7:25 [PATCH] configs: j722s_evm_a53_defconfig: Allow configurability of MMCSD speeds Moteen Shah 2026-02-20 8:35 ` Kumar, Udit 2026-02-20 9:47 ` Moteen Shah 2026-03-09 23:15 ` Tom Rini 2026-03-12 9:16 ` Moteen Shah 2026-03-12 15:50 ` Tom Rini
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.