Message ID | 1517156391-11353-18-git-send-email-jagan@amarulasolutions.com |
---|---|
State | Changes Requested |
Delegated to: | Marek Vasut |
Headers | show |
Series | musb: sunxi: Add OTG-Peripheral support for Allwineer H3/H5/A64 | expand |
On Sun, Jan 28, 2018 at 09:49:38PM +0530, Jagan Teki wrote: > U-Boot size is crossing MMC environment offset due to adding > new features on arm64 boards. So update the ENV_OFFSET to > 0x100000 so the u-boot size limit increased to 0xf6000 > > MMC layout(updated): > =================== > part: sector: size > unused: 0: 8KiB > SPL: 16: 32KiB > U-Boot: 80: 984KiB > uENV: 2048: 128KiB > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > --- > env/Kconfig | 3 ++- > include/configs/sunxi-common.h | 2 +- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/env/Kconfig b/env/Kconfig > index 692f863..e2b32c2 100644 > --- a/env/Kconfig > +++ b/env/Kconfig > @@ -441,7 +441,8 @@ config ENV_OFFSET > hex "Environment Offset" > depends on !ENV_IS_IN_UBI > depends on !ENV_IS_NOWHERE > - default 0x88000 if ARCH_SUNXI > + default 0x88000 if ARCH_SUNXI && !ARM64 > + default 0x100000 if ARCH_SUNXI && ARM64 This breaks all existing platforms. NAK. Why do you think we went into so great troubles recently? Maxime
On Mon, Jan 29, 2018 at 1:32 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > On Sun, Jan 28, 2018 at 09:49:38PM +0530, Jagan Teki wrote: >> U-Boot size is crossing MMC environment offset due to adding >> new features on arm64 boards. So update the ENV_OFFSET to >> 0x100000 so the u-boot size limit increased to 0xf6000 >> >> MMC layout(updated): >> =================== >> part: sector: size >> unused: 0: 8KiB >> SPL: 16: 32KiB >> U-Boot: 80: 984KiB >> uENV: 2048: 128KiB >> >> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> >> --- >> env/Kconfig | 3 ++- >> include/configs/sunxi-common.h | 2 +- >> 2 files changed, 3 insertions(+), 2 deletions(-) >> >> diff --git a/env/Kconfig b/env/Kconfig >> index 692f863..e2b32c2 100644 >> --- a/env/Kconfig >> +++ b/env/Kconfig >> @@ -441,7 +441,8 @@ config ENV_OFFSET >> hex "Environment Offset" >> depends on !ENV_IS_IN_UBI >> depends on !ENV_IS_NOWHERE >> - default 0x88000 if ARCH_SUNXI >> + default 0x88000 if ARCH_SUNXI && !ARM64 >> + default 0x100000 if ARCH_SUNXI && ARM64 > > This breaks all existing platforms. NAK. Can you elaborate, what it breaks. 32 bit sunxi start with 0x88000 and 64-sun start with 0x100000 this is what I expecting here. > > Why do you think we went into so great troubles recently? Certainly yes, MUSB_GADGET increasing u-boot.itb 548K u-boot.itb exceeds file size limit: limit: 516096 bytes actual: 560088 bytes excess: 43992 bytes make: *** [u-boot.itb] Error 1 make: *** Waiting for unfinished jobs..
On Mon, Jan 29, 2018 at 01:49:14PM +0530, Jagan Teki wrote: > On Mon, Jan 29, 2018 at 1:32 PM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: > > On Sun, Jan 28, 2018 at 09:49:38PM +0530, Jagan Teki wrote: > >> U-Boot size is crossing MMC environment offset due to adding > >> new features on arm64 boards. So update the ENV_OFFSET to > >> 0x100000 so the u-boot size limit increased to 0xf6000 > >> > >> MMC layout(updated): > >> =================== > >> part: sector: size > >> unused: 0: 8KiB > >> SPL: 16: 32KiB > >> U-Boot: 80: 984KiB > >> uENV: 2048: 128KiB > >> > >> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > >> --- > >> env/Kconfig | 3 ++- > >> include/configs/sunxi-common.h | 2 +- > >> 2 files changed, 3 insertions(+), 2 deletions(-) > >> > >> diff --git a/env/Kconfig b/env/Kconfig > >> index 692f863..e2b32c2 100644 > >> --- a/env/Kconfig > >> +++ b/env/Kconfig > >> @@ -441,7 +441,8 @@ config ENV_OFFSET > >> hex "Environment Offset" > >> depends on !ENV_IS_IN_UBI > >> depends on !ENV_IS_NOWHERE > >> - default 0x88000 if ARCH_SUNXI > >> + default 0x88000 if ARCH_SUNXI && !ARM64 > >> + default 0x100000 if ARCH_SUNXI && ARM64 > > > > This breaks all existing platforms. NAK. > > Can you elaborate, what it breaks. 32 bit sunxi start with 0x88000 and > 64-sun start with 0x100000 this is what I expecting here. If you have an environment already flashed, flashing the new u-boot will overwrite it, and might break any partition layout you have set. > > Why do you think we went into so great troubles recently? > > Certainly yes, MUSB_GADGET increasing u-boot.itb 548K > > u-boot.itb exceeds file size limit: > limit: 516096 bytes > actual: 560088 bytes > excess: 43992 bytes > make: *** [u-boot.itb] Error 1 > make: *** Waiting for unfinished jobs.. That's not my question. Why do you think we had some many discussions, debates and patches if we could have done so from the beginning? Maxime
On Tue, Jan 30, 2018 at 1:54 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > On Mon, Jan 29, 2018 at 01:49:14PM +0530, Jagan Teki wrote: >> On Mon, Jan 29, 2018 at 1:32 PM, Maxime Ripard >> <maxime.ripard@free-electrons.com> wrote: >> > On Sun, Jan 28, 2018 at 09:49:38PM +0530, Jagan Teki wrote: >> >> U-Boot size is crossing MMC environment offset due to adding >> >> new features on arm64 boards. So update the ENV_OFFSET to >> >> 0x100000 so the u-boot size limit increased to 0xf6000 >> >> >> >> MMC layout(updated): >> >> =================== >> >> part: sector: size >> >> unused: 0: 8KiB >> >> SPL: 16: 32KiB >> >> U-Boot: 80: 984KiB >> >> uENV: 2048: 128KiB >> >> >> >> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> >> >> --- >> >> env/Kconfig | 3 ++- >> >> include/configs/sunxi-common.h | 2 +- >> >> 2 files changed, 3 insertions(+), 2 deletions(-) >> >> >> >> diff --git a/env/Kconfig b/env/Kconfig >> >> index 692f863..e2b32c2 100644 >> >> --- a/env/Kconfig >> >> +++ b/env/Kconfig >> >> @@ -441,7 +441,8 @@ config ENV_OFFSET >> >> hex "Environment Offset" >> >> depends on !ENV_IS_IN_UBI >> >> depends on !ENV_IS_NOWHERE >> >> - default 0x88000 if ARCH_SUNXI >> >> + default 0x88000 if ARCH_SUNXI && !ARM64 >> >> + default 0x100000 if ARCH_SUNXI && ARM64 >> > >> > This breaks all existing platforms. NAK. >> >> Can you elaborate, what it breaks. 32 bit sunxi start with 0x88000 and >> 64-sun start with 0x100000 this is what I expecting here. > > If you have an environment already flashed, flashing the new u-boot > will overwrite it, and might break any partition layout you have set. True, but we can't control this adding new stuff ultimately increasing size. either we can stop working new features or increasing partition size, do you have any other suggestions? > >> > Why do you think we went into so great troubles recently? >> >> Certainly yes, MUSB_GADGET increasing u-boot.itb 548K >> >> u-boot.itb exceeds file size limit: >> limit: 516096 bytes >> actual: 560088 bytes >> excess: 43992 bytes >> make: *** [u-boot.itb] Error 1 >> make: *** Waiting for unfinished jobs.. > > That's not my question. Why do you think we had some many discussions, > debates and patches if we could have done so from the beginning? Sorry, can't understand please explain it little bit.
On Tue, Jan 30, 2018 at 02:29:27PM +0530, Jagan Teki wrote: > On Tue, Jan 30, 2018 at 1:54 PM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: > > On Mon, Jan 29, 2018 at 01:49:14PM +0530, Jagan Teki wrote: > >> On Mon, Jan 29, 2018 at 1:32 PM, Maxime Ripard > >> <maxime.ripard@free-electrons.com> wrote: > >> > On Sun, Jan 28, 2018 at 09:49:38PM +0530, Jagan Teki wrote: > >> >> U-Boot size is crossing MMC environment offset due to adding > >> >> new features on arm64 boards. So update the ENV_OFFSET to > >> >> 0x100000 so the u-boot size limit increased to 0xf6000 > >> >> > >> >> MMC layout(updated): > >> >> =================== > >> >> part: sector: size > >> >> unused: 0: 8KiB > >> >> SPL: 16: 32KiB > >> >> U-Boot: 80: 984KiB > >> >> uENV: 2048: 128KiB > >> >> > >> >> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > >> >> --- > >> >> env/Kconfig | 3 ++- > >> >> include/configs/sunxi-common.h | 2 +- > >> >> 2 files changed, 3 insertions(+), 2 deletions(-) > >> >> > >> >> diff --git a/env/Kconfig b/env/Kconfig > >> >> index 692f863..e2b32c2 100644 > >> >> --- a/env/Kconfig > >> >> +++ b/env/Kconfig > >> >> @@ -441,7 +441,8 @@ config ENV_OFFSET > >> >> hex "Environment Offset" > >> >> depends on !ENV_IS_IN_UBI > >> >> depends on !ENV_IS_NOWHERE > >> >> - default 0x88000 if ARCH_SUNXI > >> >> + default 0x88000 if ARCH_SUNXI && !ARM64 > >> >> + default 0x100000 if ARCH_SUNXI && ARM64 > >> > > >> > This breaks all existing platforms. NAK. > >> > >> Can you elaborate, what it breaks. 32 bit sunxi start with 0x88000 and > >> 64-sun start with 0x100000 this is what I expecting here. > > > > If you have an environment already flashed, flashing the new u-boot > > will overwrite it, and might break any partition layout you have set. > > True, but we can't control this adding new stuff ultimately increasing > size. either we can stop working new features or increasing partition > size, do you have any other suggestions? https://lists.denx.de/pipermail/u-boot/2017-December/314940.html > >> > Why do you think we went into so great troubles recently? > >> > >> Certainly yes, MUSB_GADGET increasing u-boot.itb 548K > >> > >> u-boot.itb exceeds file size limit: > >> limit: 516096 bytes > >> actual: 560088 bytes > >> excess: 43992 bytes > >> make: *** [u-boot.itb] Error 1 > >> make: *** Waiting for unfinished jobs.. > > > > That's not my question. Why do you think we had some many discussions, > > debates and patches if we could have done so from the beginning? > > Sorry, can't understand please explain it little bit. We've had: https://patchwork.ozlabs.org/cover/828100/ https://patchwork.ozlabs.org/cover/851904/ https://patchwork.ozlabs.org/cover/864994/ That were doing everything we could to *not* move the location of the environment in order to address the issue. Why do you think we did that? Maxime
On Wed, Jan 31, 2018 at 9:25 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > On Tue, Jan 30, 2018 at 02:29:27PM +0530, Jagan Teki wrote: >> On Tue, Jan 30, 2018 at 1:54 PM, Maxime Ripard >> <maxime.ripard@free-electrons.com> wrote: >> > On Mon, Jan 29, 2018 at 01:49:14PM +0530, Jagan Teki wrote: >> >> On Mon, Jan 29, 2018 at 1:32 PM, Maxime Ripard >> >> <maxime.ripard@free-electrons.com> wrote: >> >> > On Sun, Jan 28, 2018 at 09:49:38PM +0530, Jagan Teki wrote: >> >> >> U-Boot size is crossing MMC environment offset due to adding >> >> >> new features on arm64 boards. So update the ENV_OFFSET to >> >> >> 0x100000 so the u-boot size limit increased to 0xf6000 >> >> >> >> >> >> MMC layout(updated): >> >> >> =================== >> >> >> part: sector: size >> >> >> unused: 0: 8KiB >> >> >> SPL: 16: 32KiB >> >> >> U-Boot: 80: 984KiB >> >> >> uENV: 2048: 128KiB >> >> >> >> >> >> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> >> >> >> --- >> >> >> env/Kconfig | 3 ++- >> >> >> include/configs/sunxi-common.h | 2 +- >> >> >> 2 files changed, 3 insertions(+), 2 deletions(-) >> >> >> >> >> >> diff --git a/env/Kconfig b/env/Kconfig >> >> >> index 692f863..e2b32c2 100644 >> >> >> --- a/env/Kconfig >> >> >> +++ b/env/Kconfig >> >> >> @@ -441,7 +441,8 @@ config ENV_OFFSET >> >> >> hex "Environment Offset" >> >> >> depends on !ENV_IS_IN_UBI >> >> >> depends on !ENV_IS_NOWHERE >> >> >> - default 0x88000 if ARCH_SUNXI >> >> >> + default 0x88000 if ARCH_SUNXI && !ARM64 >> >> >> + default 0x100000 if ARCH_SUNXI && ARM64 >> >> > >> >> > This breaks all existing platforms. NAK. >> >> >> >> Can you elaborate, what it breaks. 32 bit sunxi start with 0x88000 and >> >> 64-sun start with 0x100000 this is what I expecting here. >> > >> > If you have an environment already flashed, flashing the new u-boot >> > will overwrite it, and might break any partition layout you have set. >> >> True, but we can't control this adding new stuff ultimately increasing >> size. either we can stop working new features or increasing partition >> size, do you have any other suggestions? > > https://lists.denx.de/pipermail/u-boot/2017-December/314940.html > >> >> > Why do you think we went into so great troubles recently? >> >> >> >> Certainly yes, MUSB_GADGET increasing u-boot.itb 548K >> >> >> >> u-boot.itb exceeds file size limit: >> >> limit: 516096 bytes >> >> actual: 560088 bytes >> >> excess: 43992 bytes >> >> make: *** [u-boot.itb] Error 1 >> >> make: *** Waiting for unfinished jobs.. >> > >> > That's not my question. Why do you think we had some many discussions, >> > debates and patches if we could have done so from the beginning? >> >> Sorry, can't understand please explain it little bit. > > We've had: > https://patchwork.ozlabs.org/cover/828100/ > https://patchwork.ozlabs.org/cover/851904/ > https://patchwork.ozlabs.org/cover/864994/ > > That were doing everything we could to *not* move the location of the > environment in order to address the issue. Why do you think we did that? > > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux and Kernel engineering > http://free-electrons.com
On Wed, Jan 31, 2018 at 9:25 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > On Tue, Jan 30, 2018 at 02:29:27PM +0530, Jagan Teki wrote: >> On Tue, Jan 30, 2018 at 1:54 PM, Maxime Ripard >> <maxime.ripard@free-electrons.com> wrote: >> > On Mon, Jan 29, 2018 at 01:49:14PM +0530, Jagan Teki wrote: >> >> On Mon, Jan 29, 2018 at 1:32 PM, Maxime Ripard >> >> <maxime.ripard@free-electrons.com> wrote: >> >> > On Sun, Jan 28, 2018 at 09:49:38PM +0530, Jagan Teki wrote: >> >> >> U-Boot size is crossing MMC environment offset due to adding >> >> >> new features on arm64 boards. So update the ENV_OFFSET to >> >> >> 0x100000 so the u-boot size limit increased to 0xf6000 >> >> >> >> >> >> MMC layout(updated): >> >> >> =================== >> >> >> part: sector: size >> >> >> unused: 0: 8KiB >> >> >> SPL: 16: 32KiB >> >> >> U-Boot: 80: 984KiB >> >> >> uENV: 2048: 128KiB >> >> >> >> >> >> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> >> >> >> --- >> >> >> env/Kconfig | 3 ++- >> >> >> include/configs/sunxi-common.h | 2 +- >> >> >> 2 files changed, 3 insertions(+), 2 deletions(-) >> >> >> >> >> >> diff --git a/env/Kconfig b/env/Kconfig >> >> >> index 692f863..e2b32c2 100644 >> >> >> --- a/env/Kconfig >> >> >> +++ b/env/Kconfig >> >> >> @@ -441,7 +441,8 @@ config ENV_OFFSET >> >> >> hex "Environment Offset" >> >> >> depends on !ENV_IS_IN_UBI >> >> >> depends on !ENV_IS_NOWHERE >> >> >> - default 0x88000 if ARCH_SUNXI >> >> >> + default 0x88000 if ARCH_SUNXI && !ARM64 >> >> >> + default 0x100000 if ARCH_SUNXI && ARM64 >> >> > >> >> > This breaks all existing platforms. NAK. >> >> >> >> Can you elaborate, what it breaks. 32 bit sunxi start with 0x88000 and >> >> 64-sun start with 0x100000 this is what I expecting here. >> > >> > If you have an environment already flashed, flashing the new u-boot >> > will overwrite it, and might break any partition layout you have set. >> >> True, but we can't control this adding new stuff ultimately increasing >> size. either we can stop working new features or increasing partition >> size, do you have any other suggestions? > > https://lists.denx.de/pipermail/u-boot/2017-December/314940.html > >> >> > Why do you think we went into so great troubles recently? >> >> >> >> Certainly yes, MUSB_GADGET increasing u-boot.itb 548K >> >> >> >> u-boot.itb exceeds file size limit: >> >> limit: 516096 bytes >> >> actual: 560088 bytes >> >> excess: 43992 bytes >> >> make: *** [u-boot.itb] Error 1 >> >> make: *** Waiting for unfinished jobs.. >> > >> > That's not my question. Why do you think we had some many discussions, >> > debates and patches if we could have done so from the beginning? >> >> Sorry, can't understand please explain it little bit. > > We've had: > https://patchwork.ozlabs.org/cover/828100/ > https://patchwork.ozlabs.org/cover/851904/ > https://patchwork.ozlabs.org/cover/864994/ Now we have transition IN, and we can able to change the SIZE check right?
On Tue, Feb 06, 2018 at 03:49:28PM +0530, Jagan Teki wrote: > On Wed, Jan 31, 2018 at 9:25 PM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: > > On Tue, Jan 30, 2018 at 02:29:27PM +0530, Jagan Teki wrote: > >> On Tue, Jan 30, 2018 at 1:54 PM, Maxime Ripard > >> <maxime.ripard@free-electrons.com> wrote: > >> > On Mon, Jan 29, 2018 at 01:49:14PM +0530, Jagan Teki wrote: > >> >> On Mon, Jan 29, 2018 at 1:32 PM, Maxime Ripard > >> >> <maxime.ripard@free-electrons.com> wrote: > >> >> > On Sun, Jan 28, 2018 at 09:49:38PM +0530, Jagan Teki wrote: > >> >> >> U-Boot size is crossing MMC environment offset due to adding > >> >> >> new features on arm64 boards. So update the ENV_OFFSET to > >> >> >> 0x100000 so the u-boot size limit increased to 0xf6000 > >> >> >> > >> >> >> MMC layout(updated): > >> >> >> =================== > >> >> >> part: sector: size > >> >> >> unused: 0: 8KiB > >> >> >> SPL: 16: 32KiB > >> >> >> U-Boot: 80: 984KiB > >> >> >> uENV: 2048: 128KiB > >> >> >> > >> >> >> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > >> >> >> --- > >> >> >> env/Kconfig | 3 ++- > >> >> >> include/configs/sunxi-common.h | 2 +- > >> >> >> 2 files changed, 3 insertions(+), 2 deletions(-) > >> >> >> > >> >> >> diff --git a/env/Kconfig b/env/Kconfig > >> >> >> index 692f863..e2b32c2 100644 > >> >> >> --- a/env/Kconfig > >> >> >> +++ b/env/Kconfig > >> >> >> @@ -441,7 +441,8 @@ config ENV_OFFSET > >> >> >> hex "Environment Offset" > >> >> >> depends on !ENV_IS_IN_UBI > >> >> >> depends on !ENV_IS_NOWHERE > >> >> >> - default 0x88000 if ARCH_SUNXI > >> >> >> + default 0x88000 if ARCH_SUNXI && !ARM64 > >> >> >> + default 0x100000 if ARCH_SUNXI && ARM64 > >> >> > > >> >> > This breaks all existing platforms. NAK. > >> >> > >> >> Can you elaborate, what it breaks. 32 bit sunxi start with 0x88000 and > >> >> 64-sun start with 0x100000 this is what I expecting here. > >> > > >> > If you have an environment already flashed, flashing the new u-boot > >> > will overwrite it, and might break any partition layout you have set. > >> > >> True, but we can't control this adding new stuff ultimately increasing > >> size. either we can stop working new features or increasing partition > >> size, do you have any other suggestions? > > > > https://lists.denx.de/pipermail/u-boot/2017-December/314940.html > > > >> >> > Why do you think we went into so great troubles recently? > >> >> > >> >> Certainly yes, MUSB_GADGET increasing u-boot.itb 548K > >> >> > >> >> u-boot.itb exceeds file size limit: > >> >> limit: 516096 bytes > >> >> actual: 560088 bytes > >> >> excess: 43992 bytes > >> >> make: *** [u-boot.itb] Error 1 > >> >> make: *** Waiting for unfinished jobs.. > >> > > >> > That's not my question. Why do you think we had some many discussions, > >> > debates and patches if we could have done so from the beginning? > >> > >> Sorry, can't understand please explain it little bit. > > > > We've had: > > https://patchwork.ozlabs.org/cover/828100/ > > https://patchwork.ozlabs.org/cover/851904/ > > https://patchwork.ozlabs.org/cover/864994/ > > Now we have transition IN, and we can able to change the SIZE check right? Not until the transition is over. Maxime
=================== part: sector: size unused: 0: 8KiB SPL: 16: 32KiB U-Boot: 80: 984KiB uENV: 2048: 128KiB Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> --- env/Kconfig | 3 ++- include/configs/sunxi-common.h | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/env/Kconfig b/env/Kconfig index 692f863..e2b32c2 100644 --- a/env/Kconfig +++ b/env/Kconfig @@ -441,7 +441,8 @@ config ENV_OFFSET hex "Environment Offset" depends on !ENV_IS_IN_UBI depends on !ENV_IS_NOWHERE - default 0x88000 if ARCH_SUNXI + default 0x88000 if ARCH_SUNXI && !ARM64 + default 0x100000 if ARCH_SUNXI && ARM64 help Offset from the start of the device (or partition) diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h index 6236e12..687c7a3 100644 --- a/include/configs/sunxi-common.h +++ b/include/configs/sunxi-common.h @@ -154,7 +154,7 @@ * (CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR * 512)), but the value will be used * directly in a makefile, without the preprocessor expansion. */ -#define CONFIG_BOARD_SIZE_LIMIT 0x7e000 +#define CONFIG_BOARD_SIZE_LIMIT 0xf6000 #endif #if CONFIG_MMC_SUNXI_SLOT_EXTRA != -1