[PATCH V5 02/17] include: configs: am62x_evm: Drop unused SDRAM address

Marcel Ziswiler marcel.ziswiler at toradex.com
Thu Aug 24 15:02:21 CEST 2023


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 at 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 at 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


More information about the U-Boot mailing list