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

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=resul...
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@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

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.

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

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.

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

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.

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=resul...
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@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!
participants (3)
-
Peter Robinson
-
Stefan Roese
-
Tom Rini