| Message ID | 20210412000455.12996-5-andre.przywara@arm.com |
|---|---|
| State | Accepted |
| Commit | ff47d539f28bcae15bc680d2e32fec6cc9551eb4 |
| Delegated to: | Tom Rini |
| Headers | show |
| Series | highbank: Update to driver model | expand |
On Mon, Apr 12, 2021 at 01:04:53AM +0100, Andre Przywara wrote: > So far we were defining a somewhat confusing PHYS_SDRAM_1_SIZE variable, > which originally was only used for setting the memtest boundaries. This > definition in highbank.h has been removed about a year ago (moved to > Kconfig), so we also don't need the hard-coded size definition any longer. > > Get rid of the misleading memory size definition, which was actually wrong > anyway (it's 4088 MB for those machines with just 4GB of DRAM). > > Signed-off-by: Andre Przywara <andre.przywara@arm.com> Applied to u-boot/master, thanks!
diff --git a/include/configs/highbank.h b/include/configs/highbank.h index fbd26ddd0fc..ff92c4f5540 100644 --- a/include/configs/highbank.h +++ b/include/configs/highbank.h @@ -39,12 +39,6 @@ #define CONFIG_SYS_LOAD_ADDR 0x800000 #define CONFIG_SYS_64BIT_LBA -/*----------------------------------------------------------------------- - * Physical Memory Map - * The DRAM is already setup, so do not touch the DT node later. - */ -#define PHYS_SDRAM_1_SIZE (4089 << 20) - /* Environment data setup */ #define CONFIG_SYS_NVRAM_BASE_ADDR 0xfff88000 /* NVRAM base address */
So far we were defining a somewhat confusing PHYS_SDRAM_1_SIZE variable, which originally was only used for setting the memtest boundaries. This definition in highbank.h has been removed about a year ago (moved to Kconfig), so we also don't need the hard-coded size definition any longer. Get rid of the misleading memory size definition, which was actually wrong anyway (it's 4088 MB for those machines with just 4GB of DRAM). Signed-off-by: Andre Przywara <andre.przywara@arm.com> --- include/configs/highbank.h | 6 ------ 1 file changed, 6 deletions(-)