Re: [PATCH V5 02/17] include: configs: am62x_evm: Drop unused SDRAM address
On Thu, 2023-08-24 at 18:15 +0200, Francesco Dolcini wrote: [snip] > > > > > 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. > > > > I believe that this is used on verdin-am62, please double check marcel. > > No, likely I need to go home and not spread incomplete information here ... (;-p) We took care of it. https://lore.kernel.org/all/20230828215037.482278-1-mar...@ziswiler.com Thanks! Cheers Marcel
Re: [PATCH V5 02/17] include: configs: am62x_evm: Drop unused SDRAM address
On Thu, Aug 24, 2023 at 06:09:54PM +0200, Francesco Dolcini wrote: > On Thu, Aug 24, 2023 at 01:02:21PM +, Marcel Ziswiler wrote: > > 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 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 > > > > > --- > > > > > 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. > > I believe that this is used on verdin-am62, please double check marcel. No, likely I need to go home and not spread incomplete information here ...
Re: [PATCH V5 02/17] include: configs: am62x_evm: Drop unused SDRAM address
On Thu, Aug 24, 2023 at 01:02:21PM +, Marcel Ziswiler wrote: > 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 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 > > > > --- > > > > 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. I believe that this is used on verdin-am62, please double check marcel.
Re: [PATCH V5 02/17] include: configs: am62x_evm: Drop unused SDRAM address
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 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 > > > --- > > > 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 > > > #include > > > > > > -/* DDR Configuration */ > > > -#define CFG_SYS_SDRAM_BASE10x88000 > > > - > > > /* Now for the remaining common defines */ > > > #include > > > > > > -- > > > 2.40.0
Re: [PATCH V5 02/17] include: configs: am62x_evm: Drop unused SDRAM address
On 10:13-20230824, Mattijs Korpershoek wrote: > Hi Nishanth, > > On mer., août 23, 2023 at 22:10, Nishanth Menon 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 > > --- > > 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 > > #include > > > > -/* DDR Configuration */ > > -#define CFG_SYS_SDRAM_BASE10x88000 > > - > > /* Now for the remaining common defines */ > > #include > > > > -- > > 2.40.0 -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
Re: [PATCH V5 02/17] include: configs: am62x_evm: Drop unused SDRAM address
Hi Nishanth, On mer., août 23, 2023 at 22:10, Nishanth Menon 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 > --- > 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. > > 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 > #include > > -/* DDR Configuration */ > -#define CFG_SYS_SDRAM_BASE1 0x88000 > - > /* Now for the remaining common defines */ > #include > > -- > 2.40.0
[PATCH V5 02/17] include: configs: am62x_evm: Drop unused SDRAM address
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 --- New patch (part of bootstd transition) include/configs/am62x_evm.h | 3 --- 1 file changed, 3 deletions(-) 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 #include -/* DDR Configuration */ -#define CFG_SYS_SDRAM_BASE10x88000 - /* Now for the remaining common defines */ #include -- 2.40.0
