On Thu, Jan 18, 2024 at 05:31:20PM +0100, Max Krummenacher wrote:
On Thu, Jan 18, 2024 at 10:50:07AM +0100, Max Krummenacher wrote:
On Wed, Jan 17, 2024 at 11:16:49AM +0100, Max Krummenacher wrote:
From: Max Krummenacher max.krummenacher@toradex.com
Speed grade T requires the VDD_CORE voltage to be 0.85V if using the maximum core frequency.
Speed grades G, K, S allow the VDD_CORE voltage to be 0.75V up to the maximum core frequency but allow to run at 0.85V.
For efficiency in manufacturing and code maintenance we use 0.85V for the PMIC defaults and device tree settings and dynamically adjust the voltage in the PMIC and device tree to 0.75V for lower speed SKU to gain more than 100mW power consumption reduction.
Signed-off-by: Max Krummenacher max.krummenacher@toradex.com
board/toradex/verdin-am62/verdin-am62.c | 47 +++++++++++++++++++++++++ 1 file changed, 47 insertions(+)
diff --git a/board/toradex/verdin-am62/verdin-am62.c b/board/toradex/verdin-am62/verdin-am62.c index 4e912b5f32f..8b9db224069 100644 --- a/board/toradex/verdin-am62/verdin-am62.c +++ b/board/toradex/verdin-am62/verdin-am62.c @@ -13,10 +13,13 @@ #include <fdt_support.h> #include <init.h> #include <k3-ddrss.h> +#include <power/regulator.h>
In the !DM_REGULATOR case regulator.h assumes errno.h to be already included. This happens when compiling the R5 SPL.
Will fix this in a v2 of the series.
Actually we can go on with the series. While it is true that errno.h is not included in regulator.h, the required header is included and its definitions available.
I will create a separate patch which cleans up the headers, so that they become self contained.
FTR, https://lore.kernel.org/all/20240118181106.4133924-1-max.oss.09@gmail.com/
As Max wrote this series does not require that cleanup.
Francesco