Re: [PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-11-01 Thread Tom Rini
On Tue, Nov 01, 2022 at 08:55:09PM +0100, Pali Rohár wrote: > On Tuesday 01 November 2022 15:48:34 Tom Rini wrote: > > On Tue, Nov 01, 2022 at 06:35:04PM +0100, Pali Rohár wrote: > > > On Monday 31 October 2022 11:51:45 Max Krummenacher wrote: > > > > On Fri, Oct 28, 2022 at 6:40 PM Pali Rohár wro

Re: [PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-11-01 Thread Pali Rohár
On Tuesday 01 November 2022 15:48:34 Tom Rini wrote: > On Tue, Nov 01, 2022 at 06:35:04PM +0100, Pali Rohár wrote: > > On Monday 31 October 2022 11:51:45 Max Krummenacher wrote: > > > On Fri, Oct 28, 2022 at 6:40 PM Pali Rohár wrote: > > > > On Friday 28 October 2022 18:18:49 Max Krummenacher wrot

Re: [PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-11-01 Thread Tom Rini
On Tue, Nov 01, 2022 at 06:35:04PM +0100, Pali Rohár wrote: > On Monday 31 October 2022 11:51:45 Max Krummenacher wrote: > > On Fri, Oct 28, 2022 at 6:40 PM Pali Rohár wrote: > > > On Friday 28 October 2022 18:18:49 Max Krummenacher wrote: > > > > quiet_cmd_genenv = GENENV $@ > > > > -cmd_genenv

Re: [PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-11-01 Thread Pali Rohár
On Monday 31 October 2022 11:51:45 Max Krummenacher wrote: > On Fri, Oct 28, 2022 at 6:40 PM Pali Rohár wrote: > > On Friday 28 October 2022 18:18:49 Max Krummenacher wrote: > > > quiet_cmd_genenv = GENENV $@ > > > -cmd_genenv = $(OBJCOPY) --dump-section .rodata.default_environment=$@ > > > env

Re: [PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-11-01 Thread Tom Rini
On Mon, Oct 31, 2022 at 11:51:45AM +0100, Max Krummenacher wrote: > Hi > > On Fri, Oct 28, 2022 at 6:40 PM Pali Rohár wrote: > > > > Hello! This is really much better solution! Few comments are below. > > > > On Friday 28 October 2022 18:18:49 Max Krummenacher wrote: > > > From: Max Krummenacher

Re: [PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-10-31 Thread Max Krummenacher
Hi On Fri, Oct 28, 2022 at 6:40 PM Pali Rohár wrote: > > Hello! This is really much better solution! Few comments are below. > > On Friday 28 October 2022 18:18:49 Max Krummenacher wrote: > > From: Max Krummenacher > > > > With LTO enabled the U-Boot initial environment is no longer stored > > i

Re: [PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-10-28 Thread Pali Rohár
Hello! This is really much better solution! Few comments are below. On Friday 28 October 2022 18:18:49 Max Krummenacher wrote: > From: Max Krummenacher > > With LTO enabled the U-Boot initial environment is no longer stored > in an easy accessible section in env/common.o. I.e. the section name >

[PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-10-28 Thread Max Krummenacher
From: Max Krummenacher With LTO enabled the U-Boot initial environment is no longer stored in an easy accessible section in env/common.o. I.e. the section name changes from build to build, its content maybe compressed and it is annotated with additional data. Drop trying to read the initial env