On 10:13-20230824, Mattijs Korpershoek wrote: > Hi Nishanth, > > On mer., août 23, 2023 at 22:10, Nishanth Menon <n...@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 <n...@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. > > > > > 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 -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
