u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains
@ 2021-09-27 12:34 Amjad Ouled-Ameur
  2021-09-27 12:34 ` [PATCH 1/2] ARM: DRA7: Enable OPP_HIGH for " Amjad Ouled-Ameur
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Amjad Ouled-Ameur @ 2021-09-27 12:34 UTC (permalink / raw)
  To: u-boot
  Cc: khilman, Amjad Ouled-Ameur, Andrew F. Davis, Subhajit Paul,
	Suman Anna, Tom Rini

This patchset enables the OPP_HIGH configuration for GPU, DSPEVE and IVA
voltage domains by default for various TI DRA7xx and AM57xx boards. This
is being done to meet the performance needs of 1080p MultiMedia usecases

Amjad Ouled-Ameur (2):
  ARM: DRA7: Enable OPP_HIGH for DSPEVE and IVA voltage domains
  ARM: DRA7: Enable OPP_HIGH for GPU voltage domain

 configs/am57xx_evm_defconfig        | 3 +++
 configs/am57xx_hs_evm_defconfig     | 3 +++
 configs/am57xx_hs_evm_usb_defconfig | 3 +++
 configs/dra7xx_evm_defconfig        | 3 +++
 configs/dra7xx_hs_evm_defconfig     | 3 +++
 configs/dra7xx_hs_evm_usb_defconfig | 3 +++
 6 files changed, 18 insertions(+)

-- 
2.25.1


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

* [PATCH 1/2] ARM: DRA7: Enable OPP_HIGH for DSPEVE and IVA voltage domains
  2021-09-27 12:34 [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains Amjad Ouled-Ameur
@ 2021-09-27 12:34 ` Amjad Ouled-Ameur
  2021-09-27 12:34 ` [PATCH 2/2] ARM: DRA7: Enable OPP_HIGH for GPU voltage domain Amjad Ouled-Ameur
  2021-09-27 14:43 ` [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains Suman Anna
  2 siblings, 0 replies; 5+ messages in thread
From: Amjad Ouled-Ameur @ 2021-09-27 12:34 UTC (permalink / raw)
  To: u-boot
  Cc: khilman, Amjad Ouled-Ameur, Suman Anna, Andrew F. Davis,
	Subhajit Paul, Tom Rini

Enable the OPP_HIGH configuration for DSPEVE and IVA voltage
domains by default for various TI DRA7xx and AM57xx boards. This
is being done to meet the performance needs of 1080p MultiMedia
usecases and other DSP usecases. These domains do not support
DVFS and the kernel will continue to run at the boot OPPs chosen
here.

Signed-off-by: Suman Anna <s-anna@ti.com>
Signed-off-by: Amjad Ouled-Ameur <aouledameur@baylibre.com>
---

 configs/am57xx_evm_defconfig        | 2 ++
 configs/am57xx_hs_evm_defconfig     | 2 ++
 configs/am57xx_hs_evm_usb_defconfig | 2 ++
 configs/dra7xx_evm_defconfig        | 2 ++
 configs/dra7xx_hs_evm_defconfig     | 2 ++
 configs/dra7xx_hs_evm_usb_defconfig | 2 ++
 6 files changed, 12 insertions(+)

diff --git a/configs/am57xx_evm_defconfig b/configs/am57xx_evm_defconfig
index cc11323b9d51..62fe2e5b9037 100644
--- a/configs/am57xx_evm_defconfig
+++ b/configs/am57xx_evm_defconfig
@@ -9,6 +9,8 @@ CONFIG_DEFAULT_DEVICE_TREE="am572x-idk"
 CONFIG_SPL_TEXT_BASE=0x40300000
 CONFIG_OMAP54XX=y
 CONFIG_TARGET_AM57XX_EVM=y
+CONFIG_DRA7_DSPEVE_OPP_HIGH=y
+CONFIG_DRA7_IVA_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/am57xx_hs_evm_defconfig b/configs/am57xx_hs_evm_defconfig
index 5763264adb1a..55bc28a533c1 100644
--- a/configs/am57xx_hs_evm_defconfig
+++ b/configs/am57xx_hs_evm_defconfig
@@ -12,6 +12,8 @@ CONFIG_TI_SECURE_EMIF_REGION_START=0xbdb00000
 CONFIG_TI_SECURE_EMIF_TOTAL_REGION_SIZE=0x02000000
 CONFIG_TI_SECURE_EMIF_PROTECTED_REGION_SIZE=0x01c00000
 CONFIG_TARGET_AM57XX_EVM=y
+CONFIG_DRA7_DSPEVE_OPP_HIGH=y
+CONFIG_DRA7_IVA_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/am57xx_hs_evm_usb_defconfig b/configs/am57xx_hs_evm_usb_defconfig
index f2ae04583739..cdb3af3cb69d 100644
--- a/configs/am57xx_hs_evm_usb_defconfig
+++ b/configs/am57xx_hs_evm_usb_defconfig
@@ -14,6 +14,8 @@ CONFIG_TI_SECURE_EMIF_REGION_START=0xbdb00000
 CONFIG_TI_SECURE_EMIF_TOTAL_REGION_SIZE=0x02000000
 CONFIG_TI_SECURE_EMIF_PROTECTED_REGION_SIZE=0x01c00000
 CONFIG_TARGET_AM57XX_EVM=y
+CONFIG_DRA7_DSPEVE_OPP_HIGH=y
+CONFIG_DRA7_IVA_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/dra7xx_evm_defconfig b/configs/dra7xx_evm_defconfig
index 0c253da80cb9..71f6b0b6c3ff 100644
--- a/configs/dra7xx_evm_defconfig
+++ b/configs/dra7xx_evm_defconfig
@@ -9,6 +9,8 @@ CONFIG_DEFAULT_DEVICE_TREE="dra7-evm"
 CONFIG_SPL_TEXT_BASE=0x40300000
 CONFIG_OMAP54XX=y
 CONFIG_TARGET_DRA7XX_EVM=y
+CONFIG_DRA7_DSPEVE_OPP_HIGH=y
+CONFIG_DRA7_IVA_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/dra7xx_hs_evm_defconfig b/configs/dra7xx_hs_evm_defconfig
index 885b236f1886..5462a79f10c5 100644
--- a/configs/dra7xx_hs_evm_defconfig
+++ b/configs/dra7xx_hs_evm_defconfig
@@ -12,6 +12,8 @@ CONFIG_TI_SECURE_EMIF_REGION_START=0xbdb00000
 CONFIG_TI_SECURE_EMIF_TOTAL_REGION_SIZE=0x02000000
 CONFIG_TI_SECURE_EMIF_PROTECTED_REGION_SIZE=0x01c00000
 CONFIG_TARGET_DRA7XX_EVM=y
+CONFIG_DRA7_DSPEVE_OPP_HIGH=y
+CONFIG_DRA7_IVA_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/dra7xx_hs_evm_usb_defconfig b/configs/dra7xx_hs_evm_usb_defconfig
index efdd9f31c74a..eff9346221f4 100644
--- a/configs/dra7xx_hs_evm_usb_defconfig
+++ b/configs/dra7xx_hs_evm_usb_defconfig
@@ -14,6 +14,8 @@ CONFIG_TI_SECURE_EMIF_REGION_START=0xbdb00000
 CONFIG_TI_SECURE_EMIF_TOTAL_REGION_SIZE=0x02000000
 CONFIG_TI_SECURE_EMIF_PROTECTED_REGION_SIZE=0x01c00000
 CONFIG_TARGET_DRA7XX_EVM=y
+CONFIG_DRA7_DSPEVE_OPP_HIGH=y
+CONFIG_DRA7_IVA_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
-- 
2.25.1


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

* [PATCH 2/2] ARM: DRA7: Enable OPP_HIGH for GPU voltage domain
  2021-09-27 12:34 [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains Amjad Ouled-Ameur
  2021-09-27 12:34 ` [PATCH 1/2] ARM: DRA7: Enable OPP_HIGH for " Amjad Ouled-Ameur
@ 2021-09-27 12:34 ` Amjad Ouled-Ameur
  2021-09-27 14:43 ` [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains Suman Anna
  2 siblings, 0 replies; 5+ messages in thread
From: Amjad Ouled-Ameur @ 2021-09-27 12:34 UTC (permalink / raw)
  To: u-boot
  Cc: khilman, Amjad Ouled-Ameur, Subhajit Paul, Suman Anna,
	Andrew F. Davis, Tom Rini

Enable the OPP_HIGH configuration for GPU voltage domain
by default for various TI DRA7xx and AM57xx boards. This
is being done to meet the performance needs of 1080p
GFX/MultiMedia usecases. This domain does not support DVFS
and the kernel will continue to run at the boot OPP chosen
here.

Based on logic similar to that of DSPEVE and IVA voltage
domains in commit 58a8921fe34fd5 ("ARM: DRA7: Enable OPP_HIGH
for DSPEVE and IVA voltage domains")

Signed-off-by: Subhajit Paul <subhajit_paul@ti.com>
Signed-off-by: Suman Anna <s-anna@ti.com>
Signed-off-by: Amjad Ouled-Ameur <aouledameur@baylibre.com>

---

 configs/am57xx_evm_defconfig        | 1 +
 configs/am57xx_hs_evm_defconfig     | 1 +
 configs/am57xx_hs_evm_usb_defconfig | 1 +
 configs/dra7xx_evm_defconfig        | 1 +
 configs/dra7xx_hs_evm_defconfig     | 1 +
 configs/dra7xx_hs_evm_usb_defconfig | 1 +
 6 files changed, 6 insertions(+)

diff --git a/configs/am57xx_evm_defconfig b/configs/am57xx_evm_defconfig
index 62fe2e5b9037..cfd742a3d80a 100644
--- a/configs/am57xx_evm_defconfig
+++ b/configs/am57xx_evm_defconfig
@@ -11,6 +11,7 @@ CONFIG_OMAP54XX=y
 CONFIG_TARGET_AM57XX_EVM=y
 CONFIG_DRA7_DSPEVE_OPP_HIGH=y
 CONFIG_DRA7_IVA_OPP_HIGH=y
+CONFIG_DRA7_GPU_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/am57xx_hs_evm_defconfig b/configs/am57xx_hs_evm_defconfig
index 55bc28a533c1..edc3e8e34449 100644
--- a/configs/am57xx_hs_evm_defconfig
+++ b/configs/am57xx_hs_evm_defconfig
@@ -14,6 +14,7 @@ CONFIG_TI_SECURE_EMIF_PROTECTED_REGION_SIZE=0x01c00000
 CONFIG_TARGET_AM57XX_EVM=y
 CONFIG_DRA7_DSPEVE_OPP_HIGH=y
 CONFIG_DRA7_IVA_OPP_HIGH=y
+CONFIG_DRA7_GPU_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/am57xx_hs_evm_usb_defconfig b/configs/am57xx_hs_evm_usb_defconfig
index cdb3af3cb69d..ba1449a6ef07 100644
--- a/configs/am57xx_hs_evm_usb_defconfig
+++ b/configs/am57xx_hs_evm_usb_defconfig
@@ -16,6 +16,7 @@ CONFIG_TI_SECURE_EMIF_PROTECTED_REGION_SIZE=0x01c00000
 CONFIG_TARGET_AM57XX_EVM=y
 CONFIG_DRA7_DSPEVE_OPP_HIGH=y
 CONFIG_DRA7_IVA_OPP_HIGH=y
+CONFIG_DRA7_GPU_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/dra7xx_evm_defconfig b/configs/dra7xx_evm_defconfig
index 71f6b0b6c3ff..95d4244595ab 100644
--- a/configs/dra7xx_evm_defconfig
+++ b/configs/dra7xx_evm_defconfig
@@ -11,6 +11,7 @@ CONFIG_OMAP54XX=y
 CONFIG_TARGET_DRA7XX_EVM=y
 CONFIG_DRA7_DSPEVE_OPP_HIGH=y
 CONFIG_DRA7_IVA_OPP_HIGH=y
+CONFIG_DRA7_GPU_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/dra7xx_hs_evm_defconfig b/configs/dra7xx_hs_evm_defconfig
index 5462a79f10c5..537d7d40fdcf 100644
--- a/configs/dra7xx_hs_evm_defconfig
+++ b/configs/dra7xx_hs_evm_defconfig
@@ -14,6 +14,7 @@ CONFIG_TI_SECURE_EMIF_PROTECTED_REGION_SIZE=0x01c00000
 CONFIG_TARGET_DRA7XX_EVM=y
 CONFIG_DRA7_DSPEVE_OPP_HIGH=y
 CONFIG_DRA7_IVA_OPP_HIGH=y
+CONFIG_DRA7_GPU_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/dra7xx_hs_evm_usb_defconfig b/configs/dra7xx_hs_evm_usb_defconfig
index eff9346221f4..cab7f9f5394d 100644
--- a/configs/dra7xx_hs_evm_usb_defconfig
+++ b/configs/dra7xx_hs_evm_usb_defconfig
@@ -16,6 +16,7 @@ CONFIG_TI_SECURE_EMIF_PROTECTED_REGION_SIZE=0x01c00000
 CONFIG_TARGET_DRA7XX_EVM=y
 CONFIG_DRA7_DSPEVE_OPP_HIGH=y
 CONFIG_DRA7_IVA_OPP_HIGH=y
+CONFIG_DRA7_GPU_OPP_HIGH=y
 CONFIG_SPL=y
 CONFIG_ENV_OFFSET_REDUND=0x280000
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
-- 
2.25.1


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

* Re: [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains
  2021-09-27 12:34 [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains Amjad Ouled-Ameur
  2021-09-27 12:34 ` [PATCH 1/2] ARM: DRA7: Enable OPP_HIGH for " Amjad Ouled-Ameur
  2021-09-27 12:34 ` [PATCH 2/2] ARM: DRA7: Enable OPP_HIGH for GPU voltage domain Amjad Ouled-Ameur
@ 2021-09-27 14:43 ` Suman Anna
  2021-09-27 14:50   ` Tom Rini
  2 siblings, 1 reply; 5+ messages in thread
From: Suman Anna @ 2021-09-27 14:43 UTC (permalink / raw)
  To: Amjad Ouled-Ameur, u-boot
  Cc: khilman, Tom Rini, Bajjuri, Praneeth, Lokesh Vutla

Hi Amjad,

On 9/27/21 7:34 AM, Amjad Ouled-Ameur wrote:
> This patchset enables the OPP_HIGH configuration for GPU, DSPEVE and IVA
> voltage domains by default for various TI DRA7xx and AM57xx boards. This
> is being done to meet the performance needs of 1080p MultiMedia usecases
> 
> Amjad Ouled-Ameur (2):
>   ARM: DRA7: Enable OPP_HIGH for DSPEVE and IVA voltage domains
>   ARM: DRA7: Enable OPP_HIGH for GPU voltage domain

These were intentionally not posted to upstream all these years, and we prefer
not to enable these by default.

regards
Suman

> 
>  configs/am57xx_evm_defconfig        | 3 +++
>  configs/am57xx_hs_evm_defconfig     | 3 +++
>  configs/am57xx_hs_evm_usb_defconfig | 3 +++
>  configs/dra7xx_evm_defconfig        | 3 +++
>  configs/dra7xx_hs_evm_defconfig     | 3 +++
>  configs/dra7xx_hs_evm_usb_defconfig | 3 +++
>  6 files changed, 18 insertions(+)
> 


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

* Re: [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains
  2021-09-27 14:43 ` [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains Suman Anna
@ 2021-09-27 14:50   ` Tom Rini
  0 siblings, 0 replies; 5+ messages in thread
From: Tom Rini @ 2021-09-27 14:50 UTC (permalink / raw)
  To: Suman Anna
  Cc: Amjad Ouled-Ameur, u-boot, khilman, Bajjuri, Praneeth, Lokesh Vutla

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

On Mon, Sep 27, 2021 at 09:43:01AM -0500, Suman Anna wrote:
> Hi Amjad,
> 
> On 9/27/21 7:34 AM, Amjad Ouled-Ameur wrote:
> > This patchset enables the OPP_HIGH configuration for GPU, DSPEVE and IVA
> > voltage domains by default for various TI DRA7xx and AM57xx boards. This
> > is being done to meet the performance needs of 1080p MultiMedia usecases
> > 
> > Amjad Ouled-Ameur (2):
> >   ARM: DRA7: Enable OPP_HIGH for DSPEVE and IVA voltage domains
> >   ARM: DRA7: Enable OPP_HIGH for GPU voltage domain
> 
> These were intentionally not posted to upstream all these years, and we prefer
> not to enable these by default.

Can you please elaborate a bit on why?  Thanks.

-- 
Tom

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

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

end of thread, other threads:[~2021-09-27 14:50 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-27 12:34 [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains Amjad Ouled-Ameur
2021-09-27 12:34 ` [PATCH 1/2] ARM: DRA7: Enable OPP_HIGH for " Amjad Ouled-Ameur
2021-09-27 12:34 ` [PATCH 2/2] ARM: DRA7: Enable OPP_HIGH for GPU voltage domain Amjad Ouled-Ameur
2021-09-27 14:43 ` [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains Suman Anna
2021-09-27 14:50   ` Tom Rini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).