On Thu, 2023-08-24 at 07:05 -0500, Nishanth Menon wrote:
On 10:13-20230824, Mattijs Korpershoek wrote:
Hi Nishanth,
On mer., août 23, 2023 at 22:10, Nishanth Menon nm@ti.com wrote:
Drop unused macro. This was meant for a second region of DDR which we do not need for AM62x evm configurations.
Signed-off-by: Nishanth Menon nm@ti.com
New patch (part of bootstd transition)
include/configs/am62x_evm.h | 3 --- 1 file changed, 3 deletions(-)
What about include/configs/verdin-am62.h ?
It seems that that board, which is am62x based, also has the un-needed include.
Since i'm not 100% sure, I have added Marcel (who posted the verdin am62 support) here.
Toradex team, Can you handle that cleanup? I stayed focussed on am62x_evm side of things.
Yes, of course.
diff --git a/include/configs/am62x_evm.h b/include/configs/am62x_evm.h index 44180dc7687b..379e0c13a395 100644 --- a/include/configs/am62x_evm.h +++ b/include/configs/am62x_evm.h @@ -12,9 +12,6 @@ #include <config_distro_bootcmd.h> #include <env/ti/mmc.h> -/* DDR Configuration */ -#define CFG_SYS_SDRAM_BASE1 0x880000000
/* Now for the remaining common defines */ #include <configs/ti_armv7_common.h> -- 2.40.0