Please pull u-boot-marvell/remove-config-nr-dram-banks-v9-2020-08-26

Please pull u-boot-marvell/remove-config-nr-dram-banks-v9-2020-08-26

[Stefan Roese]

Hi Tom,

I've prepared a branch for you to pull the "Remove CONFIG_NR_DRAM_BANKS
option and bi_memstart/memsize from bd_info" patchset, if you care to
pull from it. I've rebased it on top of latest TOT, which was necessary
because of a merge conflict in pci-uclass.c. I think it's ready to go
upstream now.

Here the Azure build, without any issues:

https://dev.azure.com/sr0718/u-boot/_build/results?buildId=46&view=results

Thanks,
Stefan


The following changes since commit 9f9ecd3e4d7839e24c182fb7b24937e19b670f1b:

   Merge https://gitlab.denx.de/u-boot/custodians/u-boot-marvell 
(2020-08-25 13:38:29 -0400)

are available in the Git repository at:

   git at gitlab.denx.de:u-boot/custodians/u-boot-marvell.git 
remove-config-nr-dram-banks-v9-2020-08-26

for you to fetch changes up to bac9da46c57511e574aa7e763e17fdd6f55db5bb:

   asm-generic/u-boot.h: Remove bi_memstart & bi_memsize from bd_info 
(2020-08-26 09:20:05 +0200)

----------------------------------------------------------------
Stefan Roese (10):
       CONFIG_NR_DRAM_BANKS: Remove unreferenced code as its always defined
       image: Use gd->ram_base/_size in env_get_bootm_size()
       board_f: Add default values for bi_dram[] in dram_init_banksize()
       global: Move from bi_memstart/memsize -> gd->ram_base/ram_size
       xtensa: Remove arch_setup_bdinfo()
       xtensa: Remove local no-op dram_init_banksize()
       video: cfb_console.c: Use bi_dram[] values on all platforms
       powerpc: Remove bi_memstart & bi_memsize assignments in spl.c
       cmd: bdinfo: Remove print of superseeded bi_memstart / bi_memsize 
values
       asm-generic/u-boot.h: Remove bi_memstart & bi_memsize from bd_info

  api/api_platform-mips.c               |  3 +--
  api/api_platform-powerpc.c            |  2 +-
  arch/mips/lib/boot.c                  |  2 +-
  arch/mips/lib/bootm.c                 |  2 +-
  arch/powerpc/cpu/mpc83xx/fdt.c        |  2 +-
  arch/powerpc/cpu/mpc83xx/traps.c      |  2 +-
  arch/powerpc/cpu/mpc85xx/fdt.c        |  4 ++--
  arch/powerpc/cpu/mpc85xx/traps.c      |  2 +-
  arch/powerpc/cpu/mpc86xx/fdt.c        |  4 +++-
  arch/powerpc/cpu/mpc86xx/traps.c      |  2 +-
  arch/powerpc/cpu/mpc8xx/fdt.c         |  2 +-
  arch/powerpc/lib/bootm.c              |  4 ++--
  arch/x86/cpu/broadwell/cpu_from_spl.c |  2 --
  arch/xtensa/lib/Makefile              |  2 +-
  arch/xtensa/lib/bdinfo.c              | 22 ----------------------
  arch/xtensa/lib/bootm.c               |  5 ++---
  board/Arcturus/ucp1020/spl.c          |  2 --
  board/cadence/xtfpga/xtfpga.c         |  5 -----
  board/freescale/p1010rdb/spl.c        |  2 --
  board/freescale/p1_p2_rdb_pc/spl.c    |  2 --
  board/freescale/t102xrdb/spl.c        |  2 --
  board/freescale/t104xrdb/spl.c        |  2 --
  board/freescale/t208xqds/spl.c        |  2 --
  board/freescale/t208xrdb/spl.c        |  2 --
  board/freescale/t4rdb/spl.c           |  2 --
  board/xilinx/zynqmp/zynqmp.c          |  2 --
  cmd/bdinfo.c                          | 10 ----------
  cmd/bedbug.c                          |  2 +-
  common/board_f.c                      | 12 +-----------
  common/image.c                        | 10 ++--------
  common/init/handoff.c                 | 33 
+++++++++++++--------------------
  drivers/pci/pci-uclass.c              |  3 ++-
  drivers/video/cfb_console.c           |  8 +-------
  include/asm-generic/u-boot.h          |  4 ----
  include/handoff.h                     |  2 --
  lib/fdtdec.c                          |  5 -----
  lib/lmb.c                             |  9 ++-------
  37 files changed, 41 insertions(+), 142 deletions(-)
  delete mode 100644 arch/xtensa/lib/bdinfo.c

Please pull u-boot-marvell/remove-config-nr-dram-banks-v9-2020-08-26

[Tom Rini]

On Wed, Aug 26, 2020 at 09:31:07AM +0200, Stefan Roese wrote:

> Hi Tom,
> 
> I've prepared a branch for you to pull the "Remove CONFIG_NR_DRAM_BANKS
> option and bi_memstart/memsize from bd_info" patchset, if you care to
> pull from it. I've rebased it on top of latest TOT, which was necessary
> because of a merge conflict in pci-uclass.c. I think it's ready to go
> upstream now.

This I think needs to wait for the merge window.  There's _probably_ not
anymore run-time gotchas on platforms that we haven't been able to test
yet, but it's too late in the cycle for "probably" changes like this.
Sorry.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200826/e4008fe6/attachment.sig>

Please pull u-boot-marvell/remove-config-nr-dram-banks-v9-2020-08-26

[Stefan Roese]

On 26.08.20 13:42, Tom Rini wrote:
> On Wed, Aug 26, 2020 at 09:31:07AM +0200, Stefan Roese wrote:
> 
>> Hi Tom,
>>
>> I've prepared a branch for you to pull the "Remove CONFIG_NR_DRAM_BANKS
>> option and bi_memstart/memsize from bd_info" patchset, if you care to
>> pull from it. I've rebased it on top of latest TOT, which was necessary
>> because of a merge conflict in pci-uclass.c. I think it's ready to go
>> upstream now.
> 
> This I think needs to wait for the merge window.  There's _probably_ not
> anymore run-time gotchas on platforms that we haven't been able to test
> yet, but it's too late in the cycle for "probably" changes like this.
> Sorry.

Okay. How about pulling this into -next then?

Thanks,
Stefan

Please pull u-boot-marvell/remove-config-nr-dram-banks-v9-2020-08-26

[Tom Rini]

On Wed, Aug 26, 2020 at 01:52:14PM +0200, Stefan Roese wrote:
> On 26.08.20 13:42, Tom Rini wrote:
> > On Wed, Aug 26, 2020 at 09:31:07AM +0200, Stefan Roese wrote:
> > 
> > > Hi Tom,
> > > 
> > > I've prepared a branch for you to pull the "Remove CONFIG_NR_DRAM_BANKS
> > > option and bi_memstart/memsize from bd_info" patchset, if you care to
> > > pull from it. I've rebased it on top of latest TOT, which was necessary
> > > because of a merge conflict in pci-uclass.c. I think it's ready to go
> > > upstream now.
> > 
> > This I think needs to wait for the merge window.  There's _probably_ not
> > anymore run-time gotchas on platforms that we haven't been able to test
> > yet, but it's too late in the cycle for "probably" changes like this.
> > Sorry.
> 
> Okay. How about pulling this into -next then?

Yes, I will take this for -next.  I need to see if I should have opened
that on -rc3 back on Monday, or with -rc4 in about 2 weeks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200826/dd8e6150/attachment.sig>

Please pull u-boot-marvell/remove-config-nr-dram-banks-v9-2020-08-26

[Peter Robinson]

On Wed, Aug 26, 2020 at 1:22 PM Tom Rini <trini@konsulko.com> wrote:
>
> On Wed, Aug 26, 2020 at 01:52:14PM +0200, Stefan Roese wrote:
> > On 26.08.20 13:42, Tom Rini wrote:
> > > On Wed, Aug 26, 2020 at 09:31:07AM +0200, Stefan Roese wrote:
> > >
> > > > Hi Tom,
> > > >
> > > > I've prepared a branch for you to pull the "Remove CONFIG_NR_DRAM_BANKS
> > > > option and bi_memstart/memsize from bd_info" patchset, if you care to
> > > > pull from it. I've rebased it on top of latest TOT, which was necessary
> > > > because of a merge conflict in pci-uclass.c. I think it's ready to go
> > > > upstream now.
> > >
> > > This I think needs to wait for the merge window.  There's _probably_ not
> > > anymore run-time gotchas on platforms that we haven't been able to test
> > > yet, but it's too late in the cycle for "probably" changes like this.
> > > Sorry.
> >
> > Okay. How about pulling this into -next then?
>
> Yes, I will take this for -next.  I need to see if I should have opened
> that on -rc3 back on Monday, or with -rc4 in about 2 weeks.

There was a rc3 on Monday? I was expecting one but never saw announce/tag/etc

Please pull u-boot-marvell/remove-config-nr-dram-banks-v9-2020-08-26

[Tom Rini]

On Wed, Aug 26, 2020 at 01:55:55PM +0100, Peter Robinson wrote:
> On Wed, Aug 26, 2020 at 1:22 PM Tom Rini <trini@konsulko.com> wrote:
> >
> > On Wed, Aug 26, 2020 at 01:52:14PM +0200, Stefan Roese wrote:
> > > On 26.08.20 13:42, Tom Rini wrote:
> > > > On Wed, Aug 26, 2020 at 09:31:07AM +0200, Stefan Roese wrote:
> > > >
> > > > > Hi Tom,
> > > > >
> > > > > I've prepared a branch for you to pull the "Remove CONFIG_NR_DRAM_BANKS
> > > > > option and bi_memstart/memsize from bd_info" patchset, if you care to
> > > > > pull from it. I've rebased it on top of latest TOT, which was necessary
> > > > > because of a merge conflict in pci-uclass.c. I think it's ready to go
> > > > > upstream now.
> > > >
> > > > This I think needs to wait for the merge window.  There's _probably_ not
> > > > anymore run-time gotchas on platforms that we haven't been able to test
> > > > yet, but it's too late in the cycle for "probably" changes like this.
> > > > Sorry.
> > >
> > > Okay. How about pulling this into -next then?
> >
> > Yes, I will take this for -next.  I need to see if I should have opened
> > that on -rc3 back on Monday, or with -rc4 in about 2 weeks.
> 
> There was a rc3 on Monday? I was expecting one but never saw announce/tag/etc

There was supposed to be, yes.  I need to finish my review of the i.MX
PR, push that, and then push a late -rc3, hopefully today.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200826/5d69a800/attachment.sig>

Please pull u-boot-marvell/remove-config-nr-dram-banks-v9-2020-08-26

[Tom Rini]

On Wed, Aug 26, 2020 at 09:31:07AM +0200, Stefan Roese wrote:

> Hi Tom,
> 
> I've prepared a branch for you to pull the "Remove CONFIG_NR_DRAM_BANKS
> option and bi_memstart/memsize from bd_info" patchset, if you care to
> pull from it. I've rebased it on top of latest TOT, which was necessary
> because of a merge conflict in pci-uclass.c. I think it's ready to go
> upstream now.
> 
> Here the Azure build, without any issues:
> 
> https://dev.azure.com/sr0718/u-boot/_build/results?buildId=46&view=results
> 
> Thanks,
> Stefan
> 
> 
> The following changes since commit 9f9ecd3e4d7839e24c182fb7b24937e19b670f1b:
> 
>    Merge https://gitlab.denx.de/u-boot/custodians/u-boot-marvell 
> (2020-08-25 13:38:29 -0400)
> 
> are available in the Git repository at:
> 
>    git at gitlab.denx.de:u-boot/custodians/u-boot-marvell.git 
> remove-config-nr-dram-banks-v9-2020-08-26
> 
> for you to fetch changes up to bac9da46c57511e574aa7e763e17fdd6f55db5bb:
> 
>    asm-generic/u-boot.h: Remove bi_memstart & bi_memsize from bd_info 
> (2020-08-26 09:20:05 +0200)
> 

Applied to u-boot/next, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200907/9e4d70f2/attachment.sig>