Re: [U-Boot] [PATCH 047/126] spl: Allow SPL/TPL to use of-platdata without libfdt
Hi Bin, On Mon, 7 Oct 2019 at 23:46, Bin Meng wrote: > > On Mon, Oct 7, 2019 at 9:55 AM Bin Meng wrote: > > > > On Sat, Oct 5, 2019 at 11:30 PM Bin Meng wrote: > > > > > > On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > > > > > > > At present libfdt is included in SPL/TPL if SPL/TPL_OF_CONTROL is > > > > enabled. > > > > But if of-platdata is in use this is not required. Update the condition > > > > to > > > > avoid building this extra code. This ensures that if a libfdt function > > > > is > > > > used it will produce a link error rather than silently increasing the > > > > build size. > > > > > > > > Signed-off-by: Simon Glass > > > > --- > > > > > > > > lib/Kconfig | 4 ++-- > > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > > > > > Reviewed-by: Bin Meng > > > > applied to u-boot-x86/next, thanks! > > This patch unfortunately breaks some ARM boards build, eg: chromebook_jerry > > See > https://gitlab.denx.de/u-boot/custodians/u-boot-x86/-/jobs/17131 > https://gitlab.denx.de/u-boot/custodians/u-boot-x86/-/jobs/17132 > > I will have to drop this patch from my queue. I'll put this one in a separate series. Regards, SImon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 047/126] spl: Allow SPL/TPL to use of-platdata without libfdt
On Mon, Oct 7, 2019 at 9:55 AM Bin Meng wrote: > > On Sat, Oct 5, 2019 at 11:30 PM Bin Meng wrote: > > > > On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > > > > > At present libfdt is included in SPL/TPL if SPL/TPL_OF_CONTROL is enabled. > > > But if of-platdata is in use this is not required. Update the condition to > > > avoid building this extra code. This ensures that if a libfdt function is > > > used it will produce a link error rather than silently increasing the > > > build size. > > > > > > Signed-off-by: Simon Glass > > > --- > > > > > > lib/Kconfig | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > > Reviewed-by: Bin Meng > > applied to u-boot-x86/next, thanks! This patch unfortunately breaks some ARM boards build, eg: chromebook_jerry See https://gitlab.denx.de/u-boot/custodians/u-boot-x86/-/jobs/17131 https://gitlab.denx.de/u-boot/custodians/u-boot-x86/-/jobs/17132 I will have to drop this patch from my queue. Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 047/126] spl: Allow SPL/TPL to use of-platdata without libfdt
On Sat, Oct 5, 2019 at 11:30 PM Bin Meng wrote: > > On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > > > At present libfdt is included in SPL/TPL if SPL/TPL_OF_CONTROL is enabled. > > But if of-platdata is in use this is not required. Update the condition to > > avoid building this extra code. This ensures that if a libfdt function is > > used it will produce a link error rather than silently increasing the > > build size. > > > > Signed-off-by: Simon Glass > > --- > > > > lib/Kconfig | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > Reviewed-by: Bin Meng applied to u-boot-x86/next, thanks! ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 047/126] spl: Allow SPL/TPL to use of-platdata without libfdt
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > At present libfdt is included in SPL/TPL if SPL/TPL_OF_CONTROL is enabled. > But if of-platdata is in use this is not required. Update the condition to > avoid building this extra code. This ensures that if a libfdt function is > used it will produce a link error rather than silently increasing the > build size. > > Signed-off-by: Simon Glass > --- > > lib/Kconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Reviewed-by: Bin Meng ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH 047/126] spl: Allow SPL/TPL to use of-platdata without libfdt
At present libfdt is included in SPL/TPL if SPL/TPL_OF_CONTROL is enabled. But if of-platdata is in use this is not required. Update the condition to avoid building this extra code. This ensures that if a libfdt function is used it will produce a link error rather than silently increasing the build size. Signed-off-by: Simon Glass --- lib/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/Kconfig b/lib/Kconfig index 135f0b372b0..bab57eb3059 100644 --- a/lib/Kconfig +++ b/lib/Kconfig @@ -473,7 +473,7 @@ config OF_LIBFDT_OVERLAY config SPL_OF_LIBFDT bool "Enable the FDT library for SPL" - default y if SPL_OF_CONTROL + default y if SPL_OF_CONTROL && !SPL_OF_PLATDATA help This enables the FDT library (libfdt). It provides functions for accessing binary device tree images in memory, such as adding and @@ -483,7 +483,7 @@ config SPL_OF_LIBFDT config TPL_OF_LIBFDT bool "Enable the FDT library for TPL" - default y if TPL_OF_CONTROL + default y if TPL_OF_CONTROL && !TPL_OF_PLATDATA help This enables the FDT library (libfdt). It provides functions for accessing binary device tree images in memory, such as adding and -- 2.23.0.444.g18eeb5a265-goog ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot