arm64: a37xx: add distro boot compability

U-Boot New Andre Heider 3 participants 23 messages Dec 2, 2017 – Jan 10, 2018

Share & tools

Andre Heider

author

2017-12-02 09:46 UTC #0 lore raw

Hi,

this small set improves the usability to ease booting distro shipped kernels. Tested on espressobin with debian's flash-kernel. I could successfully boot from usb.

Thanks,
Andre

ps: please keep me cc:'ed, not subscribed

Andre Heider

author

2017-12-02 09:46 UTC #1 lore raw

PATCH 1/4 arm64: a37xx: use disto defaults

Signed-off-by Andre Heider <[email protected]>

---
 include/configs/mvebu_armada-37xx.h | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

include/configs/mvebu_armada-37xx.h +2 -3

index 1b2e0d71b4..3e1ff02ed7 100644
@@ -44,9 +44,6 @@
 /*
  * Other required minimal configurations
  */
-#define CONFIG_SYS_LONGHELP
-#define CONFIG_AUTO_COMPLETE
-#define CONFIG_CMDLINE_EDITING
 #define CONFIG_ARCH_CPU_INIT		/* call arch_cpu_init() */
 #define CONFIG_SYS_LOAD_ADDR	0x00800000	/* default load adr- 8M */
 #define CONFIG_SYS_MEMTEST_START 0x00800000	/* 8M */

@@ -109,4 +106,6 @@
 
 #define CONFIG_SUPPORT_VFAT
 
+#include <config_distro_defaults.h>
+
 #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */

Signature · 1 line

2.15.1

Kostya Porotchkin

2017-12-14 10:58 UTC #2 lore raw

PATCH 1/4 arm64: a37xx: use disto defaults

Quoted 40 lines

-----Original Message-----
From: Andre Heider [mailto:a.heider at gmail.com]
Sent: Saturday, December 02, 2017 11:47
To: u-boot at lists.denx.de
Cc: Stefan Roese; Kostya Porotchkin
Subject: [EXT] [PATCH 1/4] arm64: a37xx: use disto defaults
External Email
----------------------------------------------------------------------
Signed-off-by Andre Heider <[email protected]>
---
 include/configs/mvebu_armada-37xx.h | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
include/configs/mvebu_armada-37xx.h +2 -4
index 1b2e0d71b4..3e1ff02ed7 100644
@@ -44,9 +44,6 @@
 /*
  * Other required minimal configurations
  */
-#define CONFIG_SYS_LONGHELP
-#define CONFIG_AUTO_COMPLETE
-#define CONFIG_CMDLINE_EDITING
 #define CONFIG_ARCH_CPU_INIT		/* call arch_cpu_init() */
 #define CONFIG_SYS_LOAD_ADDR	0x00800000	/* default load adr- 8M */
 #define CONFIG_SYS_MEMTEST_START 0x00800000	/* 8M */
@@ -109,4 +106,6 @@
 
 #define CONFIG_SUPPORT_VFAT
 
+#include <config_distro_defaults.h>
+
 #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */
--

Quoted 1 line

2.15.1

Reviewed-by Konstantin Porotchkin <[email protected]>

Stefan Roese

2018-01-10 11:31 UTC #3 lore raw

PATCH 1/4 arm64: a37xx: use disto defaults

In reply to Andre Heider

Quoted 21 lines · Andre Heider

Signed-off-by Andre Heider <[email protected]>

---
  include/configs/mvebu_armada-37xx.h | 5 ++---
  1 file changed, 2 insertions(+), 3 deletions(-)

include/configs/mvebu_armada-37xx.h +0 -3

index 1b2e0d71b4..3e1ff02ed7 100644
@@ -44,9 +44,6 @@
  /*
   * Other required minimal configurations
   */
-#define CONFIG_SYS_LONGHELP
-#define CONFIG_AUTO_COMPLETE
-#define CONFIG_CMDLINE_EDITING
  #define CONFIG_ARCH_CPU_INIT		/* call arch_cpu_init() */
  #define CONFIG_SYS_LOAD_ADDR	0x00800000	/* default load adr- 8M */
  #define CONFIG_SYS_MEMTEST_START 0x00800000	/* 8M */

include/configs/mvebu_armada-37xx.h line 111

Quoted 6 lines

include/configs/mvebu_armada-37xx.h +2 -0

@@ -109,4 +106,6 @@
 
  #define CONFIG_SUPPORT_VFAT
 
+#include <config_distro_defaults.h>
+
  #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */

Applied to u-boot-marvell/master

Thanks,
Stefan

Andre Heider

author

2017-12-02 09:46 UTC #4 lore raw

PATCH 2/4 arm64: a37xx: add distro compatible env vars

In reply to Andre Heider

the values of dt_addr_r/kernel_addr_r/ramdisk_addr_r are taken from the downstream 'u-boot-2017.03-armada-17.10' release.

the chosen values of scriptaddr and pxefile_addr_r are below fdt_addr_r, in 1MB steps.

Signed-off-by Andre Heider <[email protected]>

---
 include/configs/mvebu_armada-37xx.h | 7 +++++++
 1 file changed, 7 insertions(+)

include/configs/mvebu_armada-37xx.h +7 -0

index 3e1ff02ed7..383e6dfaf4 100644
@@ -108,4 +108,11 @@
 
 #include <config_distro_defaults.h>
 
+#define CONFIG_EXTRA_ENV_SETTINGS	\
+	"scriptaddr=0x4d00000\0"	\
+	"pxefile_addr_r=0x4e00000\0"	\
+	"fdt_addr_r=0x4f00000\0"	\
+	"kernel_addr_r=0x5000000\0"	\
+	"ramdisk_addr_r=0x8000000"
+
 #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */

Signature · 1 line

2.15.1

Kostya Porotchkin

2017-12-14 10:59 UTC #5 lore raw

PATCH 2/4 arm64: a37xx: add distro compatible env vars

Quoted 41 lines

-----Original Message-----
From: Andre Heider [mailto:a.heider at gmail.com]
Sent: Saturday, December 02, 2017 11:47
To: u-boot at lists.denx.de
Cc: Stefan Roese; Kostya Porotchkin
Subject: [EXT] [PATCH 2/4] arm64: a37xx: add distro compatible env vars
External Email
----------------------------------------------------------------------
the values of dt_addr_r/kernel_addr_r/ramdisk_addr_r are taken from the downstream 'u-boot-2017.03-armada-17.10' release.
the chosen values of scriptaddr and pxefile_addr_r are below fdt_addr_r, in 1MB steps.
Signed-off-by Andre Heider <[email protected]>
---
 include/configs/mvebu_armada-37xx.h | 7 +++++++
 1 file changed, 7 insertions(+)
include/configs/mvebu_armada-37xx.h +7 -1
index 3e1ff02ed7..383e6dfaf4 100644
@@ -108,4 +108,11 @@
 
 #include <config_distro_defaults.h>
 
+#define CONFIG_EXTRA_ENV_SETTINGS	\
+	"scriptaddr=0x4d00000\0"	\
+	"pxefile_addr_r=0x4e00000\0"	\
+	"fdt_addr_r=0x4f00000\0"	\
+	"kernel_addr_r=0x5000000\0"	\
+	"ramdisk_addr_r=0x8000000"
+
 #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */
--

Quoted 1 line

2.15.1

Reviewed-by Konstantin Porotchkin <[email protected]>

Stefan Roese

2018-01-10 11:32 UTC #6 lore raw

PATCH 2/4 arm64: a37xx: add distro compatible env vars

In reply to Andre Heider

Quoted 18 lines · Andre Heider

the values of dt_addr_r/kernel_addr_r/ramdisk_addr_r are taken from the downstream 'u-boot-2017.03-armada-17.10' release.
the chosen values of scriptaddr and pxefile_addr_r are below fdt_addr_r, in 1MB steps.
Signed-off-by Andre Heider <[email protected]>
---
  include/configs/mvebu_armada-37xx.h | 7 +++++++
  1 file changed, 7 insertions(+)
include/configs/mvebu_armada-37xx.h +0 -0
index 3e1ff02ed7..383e6dfaf4 100644
@@ -108,4 +108,11 @@
 

include/configs/mvebu_armada-37xx.h line 118

Quoted 10 lines

include/configs/mvebu_armada-37xx.h +7 -0

  #include <config_distro_defaults.h>
 
+#define CONFIG_EXTRA_ENV_SETTINGS	\
+	"scriptaddr=0x4d00000\0"	\
+	"pxefile_addr_r=0x4e00000\0"	\
+	"fdt_addr_r=0x4f00000\0"	\
+	"kernel_addr_r=0x5000000\0"	\
+	"ramdisk_addr_r=0x8000000"
+
  #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */

Applied to u-boot-marvell/master

Thanks,
Stefan

Andre Heider

author

2017-12-02 09:46 UTC #7 lore raw

PATCH 3/4 arm64: a37xx: defconfigs: enable CONFIG_DISTRO_DEFAULTS

In reply to Andre Heider

Signed-off-by Andre Heider <[email protected]>

---
 configs/mvebu_db-88f3720_defconfig          | 1 +
 configs/mvebu_espressobin-88f3720_defconfig | 1 +
 2 files changed, 2 insertions(+)

configs/mvebu_db-88f3720_defconfig +1 -0

index 0fd4514c37..321151e41d 100644
@@ -3,6 +3,7 @@ CONFIG_ARCH_MVEBU=y
 CONFIG_SYS_MALLOC_F_LEN=0x2000
 CONFIG_TARGET_MVEBU_ARMADA_37XX=y
 CONFIG_DEFAULT_DEVICE_TREE="armada-3720-db"
+CONFIG_DISTRO_DEFAULTS=y
 CONFIG_DEBUG_UART=y
 CONFIG_AHCI=y
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set

configs/mvebu_espressobin-88f3720_defconfig +1 -0

index 95db6c610e..5d9fb75b17 100644
@@ -3,6 +3,7 @@ CONFIG_ARCH_MVEBU=y
 CONFIG_SYS_MALLOC_F_LEN=0x2000
 CONFIG_TARGET_MVEBU_ARMADA_37XX=y
 CONFIG_DEFAULT_DEVICE_TREE="armada-3720-espressobin"
+CONFIG_DISTRO_DEFAULTS=y
 CONFIG_DEBUG_UART=y
 CONFIG_AHCI=y
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set

Signature · 1 line

2.15.1

Kostya Porotchkin

2017-12-14 10:59 UTC #8 lore raw

PATCH 3/4 arm64: a37xx: defconfigs: enable CONFIG_DISTRO_DEFAULTS

Quoted 46 lines

-----Original Message-----
From: Andre Heider [mailto:a.heider at gmail.com]
Sent: Saturday, December 02, 2017 11:47
To: u-boot at lists.denx.de
Cc: Stefan Roese; Kostya Porotchkin
Subject: [EXT] [PATCH 3/4] arm64: a37xx: defconfigs: enable CONFIG_DISTRO_DEFAULTS
External Email
----------------------------------------------------------------------
Signed-off-by Andre Heider <[email protected]>
---
 configs/mvebu_db-88f3720_defconfig          | 1 +
 configs/mvebu_espressobin-88f3720_defconfig | 1 +
 2 files changed, 2 insertions(+)
configs/mvebu_db- renamed from configs/mvebu_db-88f3720_defconfig +0 -0
88f3720_defconfig
index 0fd4514c37..321151e41d 100644
configs/mvebu_db-88f3720_defconfig +1 -0
@@ -3,6 +3,7 @@ CONFIG_ARCH_MVEBU=y
 CONFIG_SYS_MALLOC_F_LEN=0x2000
 CONFIG_TARGET_MVEBU_ARMADA_37XX=y
 CONFIG_DEFAULT_DEVICE_TREE="armada-3720-db"
+CONFIG_DISTRO_DEFAULTS=y
 CONFIG_DEBUG_UART=y
 CONFIG_AHCI=y
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set diff --git
a/configs/mvebu_espressobin-88f3720_defconfig
b/configs/mvebu_espressobin-88f3720_defconfig
index 95db6c610e..5d9fb75b17 100644
configs/mvebu_espressobin-88f3720_defconfig +1 -1
@@ -3,6 +3,7 @@ CONFIG_ARCH_MVEBU=y
 CONFIG_SYS_MALLOC_F_LEN=0x2000
 CONFIG_TARGET_MVEBU_ARMADA_37XX=y
 CONFIG_DEFAULT_DEVICE_TREE="armada-3720-espressobin"
+CONFIG_DISTRO_DEFAULTS=y
 CONFIG_DEBUG_UART=y
 CONFIG_AHCI=y
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
--

Quoted 1 line

2.15.1

Reviewed-by Konstantin Porotchkin <[email protected]>

Stefan Roese

2018-01-10 11:32 UTC #9 lore raw

PATCH 3/4 arm64: a37xx: defconfigs: enable CONFIG_DISTRO_DEFAULTS

In reply to Andre Heider

Quoted 24 lines · Andre Heider

Signed-off-by Andre Heider <[email protected]>

---
  configs/mvebu_db-88f3720_defconfig          | 1 +
  configs/mvebu_espressobin-88f3720_defconfig | 1 +
  2 files changed, 2 insertions(+)

configs/mvebu_db-88f3720_defconfig +1 -0

index 0fd4514c37..321151e41d 100644
@@ -3,6 +3,7 @@ CONFIG_ARCH_MVEBU=y
  CONFIG_SYS_MALLOC_F_LEN=0x2000
  CONFIG_TARGET_MVEBU_ARMADA_37XX=y
  CONFIG_DEFAULT_DEVICE_TREE="armada-3720-db"
+CONFIG_DISTRO_DEFAULTS=y
  CONFIG_DEBUG_UART=y
  CONFIG_AHCI=y
  # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set

configs/mvebu_espressobin-88f3720_defconfig +0 -0

index 95db6c610e..5d9fb75b17 100644

configs/mvebu_espressobin-88f3720_defconfig CONFIG_ARCH_MVEBU=y · line 9

Quoted 7 lines

configs/mvebu_espressobin-88f3720_defconfig +1 -0

@@ -3,6 +3,7 @@ CONFIG_ARCH_MVEBU=y
  CONFIG_SYS_MALLOC_F_LEN=0x2000
  CONFIG_TARGET_MVEBU_ARMADA_37XX=y
  CONFIG_DEFAULT_DEVICE_TREE="armada-3720-espressobin"
+CONFIG_DISTRO_DEFAULTS=y
  CONFIG_DEBUG_UART=y
  CONFIG_AHCI=y
  # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set

Applied to u-boot-marvell/master

Thanks,
Stefan

Andre Heider

author

2017-12-02 09:46 UTC #10 lore raw

PATCH 4/4 arm64: a37xx: use distro bootcmd

In reply to Andre Heider

Signed-off-by Andre Heider <[email protected]>

---
 include/configs/mvebu_armada-37xx.h | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

include/configs/mvebu_armada-37xx.h +12 -1

index 383e6dfaf4..f5b4f72118 100644
@@ -108,11 +108,22 @@
 
 #include <config_distro_defaults.h>
 
+#define BOOT_TARGET_DEVICES(func) \
+	func(MMC, mmc, 1) \
+	func(MMC, mmc, 0) \
+	func(USB, usb, 0) \
+	func(SCSI, scsi, 0) \
+	func(PXE, pxe, na) \
+	func(DHCP, dhcp, na)
+
+#include <config_distro_bootcmd.h>
+
 #define CONFIG_EXTRA_ENV_SETTINGS	\
 	"scriptaddr=0x4d00000\0"	\
 	"pxefile_addr_r=0x4e00000\0"	\
 	"fdt_addr_r=0x4f00000\0"	\
 	"kernel_addr_r=0x5000000\0"	\
-	"ramdisk_addr_r=0x8000000"
+	"ramdisk_addr_r=0x8000000\0"	\
+	BOOTENV
 
 #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */

Signature · 1 line

2.15.1

Kostya Porotchkin

2017-12-14 11:00 UTC #11 lore raw

PATCH 4/4 arm64: a37xx: use distro bootcmd

Quoted 47 lines

-----Original Message-----
From: Andre Heider [mailto:a.heider at gmail.com]
Sent: Saturday, December 02, 2017 11:47
To: u-boot at lists.denx.de
Cc: Stefan Roese; Kostya Porotchkin
Subject: [EXT] [PATCH 4/4] arm64: a37xx: use distro bootcmd
External Email
----------------------------------------------------------------------
Signed-off-by Andre Heider <[email protected]>
---
 include/configs/mvebu_armada-37xx.h | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)
include/configs/mvebu_armada-37xx.h +12 -2
index 383e6dfaf4..f5b4f72118 100644
@@ -108,11 +108,22 @@
 
 #include <config_distro_defaults.h>
 
+#define BOOT_TARGET_DEVICES(func) \
+	func(MMC, mmc, 1) \
+	func(MMC, mmc, 0) \
+	func(USB, usb, 0) \
+	func(SCSI, scsi, 0) \
+	func(PXE, pxe, na) \
+	func(DHCP, dhcp, na)
+
+#include <config_distro_bootcmd.h>
+
 #define CONFIG_EXTRA_ENV_SETTINGS	\
 	"scriptaddr=0x4d00000\0"	\
 	"pxefile_addr_r=0x4e00000\0"	\
 	"fdt_addr_r=0x4f00000\0"	\
 	"kernel_addr_r=0x5000000\0"	\
-	"ramdisk_addr_r=0x8000000"
+	"ramdisk_addr_r=0x8000000\0"	\
+	BOOTENV
 
 #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */
--

Quoted 1 line

2.15.1

Reviewed-by Konstantin Porotchkin <[email protected]>

Stefan Roese

2018-01-10 11:32 UTC #12 lore raw

PATCH 4/4 arm64: a37xx: use distro bootcmd

In reply to Andre Heider

Quoted 24 lines · Andre Heider

Signed-off-by Andre Heider <[email protected]>

---
  include/configs/mvebu_armada-37xx.h | 13 ++++++++++++-
  1 file changed, 12 insertions(+), 1 deletion(-)

include/configs/mvebu_armada-37xx.h +10 -0

index 383e6dfaf4..f5b4f72118 100644
@@ -108,11 +108,22 @@
 
  #include <config_distro_defaults.h>
 
+#define BOOT_TARGET_DEVICES(func) \
+	func(MMC, mmc, 1) \
+	func(MMC, mmc, 0) \
+	func(USB, usb, 0) \
+	func(SCSI, scsi, 0) \
+	func(PXE, pxe, na) \
+	func(DHCP, dhcp, na)
+
+#include <config_distro_bootcmd.h>
+

include/configs/mvebu_armada-37xx.h line 129

Quoted 10 lines

include/configs/mvebu_armada-37xx.h +2 -1

  #define CONFIG_EXTRA_ENV_SETTINGS	\
  	"scriptaddr=0x4d00000\0"	\
  	"pxefile_addr_r=0x4e00000\0"	\
  	"fdt_addr_r=0x4f00000\0"	\
  	"kernel_addr_r=0x5000000\0"	\
-	"ramdisk_addr_r=0x8000000"
+	"ramdisk_addr_r=0x8000000\0"	\
+	BOOTENV
 
  #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */

Applied to u-boot-marvell/master

Thanks,
Stefan

Andre Heider

author

2017-12-14 10:17 UTC #13 lore raw

In reply to Andre Heider

Quoted 3 lines · Andre Heider

this small set improves the usability to ease booting distro shipped kernels. Tested on espressobin with debian's flash-kernel. I could successfully boot from usb.

Anyone? Bueller?

Stefan Roese

2017-12-14 10:23 UTC #14 lore raw

Hi Andre,

Quoted 6 lines · Andre Heider

Quoted 3 lines · Andre Heider
this small set improves the usability to ease booting distro shipped kernels. Tested on espressobin with debian's flash-kernel. I could successfully boot from usb.
Anyone? Bueller?

I'm waiting for Kosta to comment on this (already triggered). So please allow us a bit more time.

Thanks,
Stefan

Andre Heider

author

2017-12-14 10:26 UTC #15 lore raw

Hi,

Quoted 4 lines · Stefan Roese

Hi Andre,
Quoted 1 line · Andre Heider
On 02/12/17 10:46, Andre Heider wrote:

Quoted 8 lines

Quoted 5 lines
Quoted 3 lines
this small set improves the usability to ease booting distro shipped kernels. Tested on espressobin with debian's flash-kernel. I could successfully boot from usb.
Anyone? Bueller?
I'm waiting for Kosta to comment on this (already triggered). So please allow us a bit more time.

Alright, thanks! I just wanted to make sure this isn't falling through the cracks.

Regards,
Andre

Stefan Roese

2017-12-14 10:30 UTC #16 lore raw

Quoted 9 lines · Andre Heider

Quoted 8 lines · Stefan Roese
Hi Andre,
Quoted 5 lines · Andre Heider
Quoted 3 lines · Andre Heider
this small set improves the usability to ease booting distro shipped kernels. Tested on espressobin with debian's flash-kernel. I could successfully boot from usb.

Quoted 7 lines

Quoted 4 lines
Quoted 1 line
Anyone? Bueller?
I'm waiting for Kosta to comment on this (already triggered). So please allow us a bit more time.
Alright, thanks! I just wanted to make sure this isn't falling through the cracks.

No problem. Checking for the status is always possible. My current plan is to push this patch (and other pending Marvell patches) after this upcoming release, as Tom wanted to be a bit more strict with only accepting bug fixes after rc1.

Thanks,
Stefan

Andre Heider

author

2017-12-14 12:03 UTC #17 lore raw

Quoted 15 lines · Stefan Roese

Quoted 14 lines · Andre Heider
Quoted 12 lines · Stefan Roese
Hi Andre,
Quoted 6 lines · Andre Heider
Quoted 3 lines · Andre Heider
this small set improves the usability to ease booting distro shipped kernels. Tested on espressobin with debian's flash-kernel. I could successfully boot from usb.
Anyone? Bueller?
I'm waiting for Kosta to comment on this (already triggered). So please allow us a bit more time.

Quoted 7 lines

Quoted 2 lines
Alright, thanks! I just wanted to make sure this isn't falling through the cracks.
No problem. Checking for the status is always possible. My current plan is to push this patch (and other pending Marvell patches) after this upcoming release, as Tom wanted to be a bit more strict with only accepting bug fixes after rc1.

Thanks for the reviews, Konstantin!

For the record, I wasn't aiming at this release, next is just fine.

While having the attention of both of you:
Downstream has diverged quite a bit compared to upstream, is there any ongoing work to upstream the missing bits?

Konstantin, since the discrepancy got pretty big I guess most users are currently using a downstream release. Is there any chance to get this set in there? I already got these patches against that tree if it helps.

Thanks,
Andre

Kostya Porotchkin

2017-12-14 12:21 UTC #18 lore raw

Hi, Andre,

Quoted 29 lines

-----Original Message-----
From: Andre Heider [mailto:a.heider at gmail.com]
Sent: Thursday, December 14, 2017 14:03
To: Stefan Roese; Kostya Porotchkin
Cc: u-boot at lists.denx.de
Subject: [EXT] Re: arm64: a37xx: add distro boot compability
External Email
----------------------------------------------------------------------
Quoted 18 lines · Stefan Roese
Quoted 16 lines · Andre Heider
Quoted 4 lines · Stefan Roese
Hi Andre,
Quoted 1 line · Andre Heider
On 02/12/17 10:46, Andre Heider wrote:
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
this small set improves the usability to ease booting distro shipped kernels. Tested on espressobin with debian's flash-kernel. I could successfully boot from usb.
Anyone? Bueller?
I'm waiting for Kosta to comment on this (already triggered). So please allow us a bit more time.
Alright, thanks! I just wanted to make sure this isn't falling through the cracks.

Quoted 6 lines

Quoted 4 lines
No problem. Checking for the status is always possible. My current plan is to push this patch (and other pending Marvell patches) after this upcoming release, as Tom wanted to be a bit more strict with only accepting bug fixes after rc1.
Thanks for the reviews, Konstantin!

[Konstantin Porotchkin] You are welcome!

Quoted 6 lines

For the record, I wasn't aiming at this release, next is just fine.
While having the attention of both of you:
Downstream has diverged quite a bit compared to upstream, is there any ongoing work to upstream the missing bits?

[Konstantin Porotchkin] I need to understand the terminology first :-) The upstream is the DENX release and the downstream is Marvell's 2017.03?

Quoted 4 lines

Konstantin, since the discrepancy got pretty big I guess most users are currently using a downstream release. Is there any chance to get this set in there? I already got these patches against that tree if it helps.

[Konstantin Porotchkin] If my assumption above is right, you are asking when we are planning to sync with the current mainline, right? Unfortunately this is out of my control. We are planning to do so as soon as we have time for it, but we got pretty busy supporting new SoC devices at this moment.

Regards
Kosta

Quoted 3 lines

Thanks,
Andre

Andre Heider

author

2017-12-14 12:49 UTC #19 lore raw

Hi Kostya,

Quoted 5 lines · Kostya Porotchkin

Quoted 3 lines
While having the attention of both of you:
Downstream has diverged quite a bit compared to upstream, is there any ongoing work to upstream the missing bits?
[Konstantin Porotchkin] I need to understand the terminology first :-) The upstream is the DENX release and the downstream is Marvell's 2017.03?

Exactly ;)

Quoted 5 lines

Quoted 3 lines
Konstantin, since the discrepancy got pretty big I guess most users are currently using a downstream release. Is there any chance to get this set in there? I already got these patches against that tree if it helps.
[Konstantin Porotchkin] If my assumption above is right, you are asking when we are planning to sync with the current mainline, right? Unfortunately this is out of my control. We are planning to do so as soon as we have time for it, but we got pretty busy supporting new SoC devices at this moment.

Alright, and I can see that syncing would be a rather big task.

But instead I was offering the same patches against the Marvell tree, without the need for any syncing.

They're now in a pull request I just created on Marvell's public github:
https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/pull/8

Thanks,
Andre

Kostya Porotchkin

2017-12-20 16:08 UTC #20 lore raw

Hi, Andre,

Quoted 29 lines

-----Original Message-----
From: Andre Heider [mailto:a.heider at gmail.com]
Sent: Thursday, December 14, 2017 14:50
To: Kostya Porotchkin; Stefan Roese
Cc: u-boot at lists.denx.de; Nadav Haklai; Igal Liberman
Subject: Re: [EXT] Re: arm64: a37xx: add distro boot compability
Hi Kostya,
Quoted 5 lines · Kostya Porotchkin
Quoted 3 lines
While having the attention of both of you:
Downstream has diverged quite a bit compared to upstream, is there any ongoing work to upstream the missing bits?
[Konstantin Porotchkin] I need to understand the terminology first :-) The upstream is the DENX release and the downstream is Marvell's
2017.03?
Exactly ;)
Quoted 3 lines
Quoted 3 lines
Konstantin, since the discrepancy got pretty big I guess most users are currently using a downstream release. Is there any chance to get this set in there? I already got these patches against that tree if
it helps.
Quoted 1 line
[Konstantin Porotchkin] If my assumption above is right, you are
asking when we are planning to sync with the current mainline, right?
Quoted 1 line
Unfortunately this is out of my control. We are planning to do so as
soon as we have time for it, but we got pretty busy supporting new SoC devices at this moment.

Quoted 7 lines

Alright, and I can see that syncing would be a rather big task.
But instead I was offering the same patches against the Marvell tree, without the need for any syncing.
They're now in a pull request I just created on Marvell's public github:
https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/pull/8

[Konstantin Porotchkin] Thank you, I hope that I or Igal (in CC list) will have some time shortly for merging your changes into our GIT.

Regards
Kosta

Quoted 3 lines

Thanks,
Andre

Andre Heider

author

2017-12-21 11:34 UTC #21 lore raw

On 20/12/17 17:08, Kostya Porotchkin wrote:
[...]

Quoted 6 lines

Quoted 5 lines
But instead I was offering the same patches against the Marvell tree, without the need for any syncing.
They're now in a pull request I just created on Marvell's public github:
https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/pull/8
[Konstantin Porotchkin] Thank you, I hope that I or Igal (in CC list) will have some time shortly for merging your changes into our GIT.

Neat, thanks for merging! :)

Regards,
Andre

Kostya Porotchkin

2017-12-21 11:38 UTC #22 lore raw

Quoted 14 lines

-----Original Message-----
From: Andre Heider [mailto:a.heider at gmail.com]
Sent: Thursday, December 21, 2017 13:35
To: Kostya Porotchkin; Stefan Roese
Cc: u-boot at lists.denx.de; Nadav Haklai; Igal Liberman
Subject: Re: [EXT] Re: arm64: a37xx: add distro boot compability
On 20/12/17 17:08, Kostya Porotchkin wrote:
[...]
Quoted 3 lines
Quoted 3 lines
But instead I was offering the same patches against the Marvell tree, without the need for any syncing.

Quoted 7 lines

Quoted 1 line
Quoted 1 line
They're now in a pull request I just created on Marvell's public
github:
Quoted 2 lines
Quoted 1 line
https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/pull/8
[Konstantin Porotchkin] Thank you, I hope that I or Igal (in CC list)
will have some time shortly for merging your changes into our GIT.
Neat, thanks for merging! :)

[Konstantin Porotchkin] I pushed them internally to the current development branch, so once they passed the CI test, internal review and get merged, they will be available inside of the next u-boot release.

Quoted 3 lines

Regards,
Andre