mbox series

[v4,0/2] Calculate SDRAM size for Actions OWL SoCs

Message ID 1589012108-11064-1-git-send-email-amittomer25@gmail.com
Headers show
Series Calculate SDRAM size for Actions OWL SoCs | expand

Message

Amit Tomer May 9, 2020, 8:15 a.m. UTC 
Mani pointed out that changes in previous version were not good enough
for S900 and he provide snippet that seems to work on S900.

This series v4 fixes the S900 Support by taking in changes suggested
by Mani. 

---------------------------------------------------------------------
 
Realized that sent the wrong version(v2) that has typos and didn't get
compile for S900. Just fixed this in v3.

This small series allows us to calculate SDRAM size instead
of guessing it.

Patch (1/2) is re-worked to support S900 SoC along with S700.

These changes have been tested on S700 based Cubieboard7-lite board, and
it would be great if this can be tested on S900.

Amit Singh Tomar (2):
  Actions: OWL: Calculate SDRAM size
  arm: actions: remove "CONFIG_SYS_SDRAM_SIZE" for Actions Owl Semi SoCs

 arch/arm/mach-owl/soc.c      | 22 +++++++++++++++++++++-
 include/configs/owl-common.h |  1 -
 2 files changed, 21 insertions(+), 2 deletions(-)

Comments

Amit Tomer June 12, 2020, 6:26 p.m. UTC | #1 
Hi Tom

On Sat, May 9, 2020 at 1:45 PM Amit Singh Tomar <amittomer25@gmail.com> wrote:
>
> Mani pointed out that changes in previous version were not good enough
> for S900 and he provide snippet that seems to work on S900.
>
> This series v4 fixes the S900 Support by taking in changes suggested
> by Mani.
>
> ---------------------------------------------------------------------
>
> Realized that sent the wrong version(v2) that has typos and didn't get
> compile for S900. Just fixed this in v3.
>
> This small series allows us to calculate SDRAM size instead
> of guessing it.
>
> Patch (1/2) is re-worked to support S900 SoC along with S700.
>
> These changes have been tested on S700 based Cubieboard7-lite board, and
> it would be great if this can be tested on S900.
>
> Amit Singh Tomar (2):
>   Actions: OWL: Calculate SDRAM size
>   arm: actions: remove "CONFIG_SYS_SDRAM_SIZE" for Actions Owl Semi SoCs
>
>  arch/arm/mach-owl/soc.c      | 22 +++++++++++++++++++++-
>  include/configs/owl-common.h |  1 -
>  2 files changed, 21 insertions(+), 2 deletions(-)
>
> --
> 2.7.4
>

Do you have a plan to take these in ?

Thanks
-Amit.
Tom Rini June 12, 2020, 6:42 p.m. UTC | #2 
On Fri, Jun 12, 2020 at 11:56:47PM +0530, Amit Tomer wrote:
> Hi Tom
> 
> On Sat, May 9, 2020 at 1:45 PM Amit Singh Tomar <amittomer25@gmail.com> wrote:
> >
> > Mani pointed out that changes in previous version were not good enough
> > for S900 and he provide snippet that seems to work on S900.
> >
> > This series v4 fixes the S900 Support by taking in changes suggested
> > by Mani.
> >
> > ---------------------------------------------------------------------
> >
> > Realized that sent the wrong version(v2) that has typos and didn't get
> > compile for S900. Just fixed this in v3.
> >
> > This small series allows us to calculate SDRAM size instead
> > of guessing it.
> >
> > Patch (1/2) is re-worked to support S900 SoC along with S700.
> >
> > These changes have been tested on S700 based Cubieboard7-lite board, and
> > it would be great if this can be tested on S900.
> >
> > Amit Singh Tomar (2):
> >   Actions: OWL: Calculate SDRAM size
> >   arm: actions: remove "CONFIG_SYS_SDRAM_SIZE" for Actions Owl Semi SoCs
> >
> >  arch/arm/mach-owl/soc.c      | 22 +++++++++++++++++++++-
> >  include/configs/owl-common.h |  1 -
> >  2 files changed, 21 insertions(+), 2 deletions(-)
> >
> > --
> > 2.7.4
> >
> 
> Do you have a plan to take these in ?

Of what I call from the thread, it's OK for this to go in for v2020.07
so it's on my TODO list for -next but only part of the queue of
hopefully easy to grab and use things, which isn't where I start :)