[PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains

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(+)

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

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

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(+)

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.
participants (3)
-
Amjad Ouled-Ameur
-
Suman Anna
-
Tom Rini