From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Sun, 27 Mar 2016 11:33:18 +0200 Subject: [U-Boot] ENV library broken with setenv/printenv argument structs In-Reply-To: References: <56F5056F.50304@denx.de> Message-ID: <56F7A8DE.5020602@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Andreas, On 25/03/2016 15:00, Andreas Fenkart wrote: > HI Stefano, > > I was not aware of the use case. I have sent out a simple patch that > moves the definition of the structs from fw_main_env.c to fw_env.c, > which should avoid the linking problem. Do you have test application > you could send me. > I without doing the option parsing as done in > fw_env_main, you probably use only use a very limited subset of the > library. Yes, currently I am using just fw_parse_script. But exporting a library, it can be possible that someone else need other options. I see you sent a patch, I send an answer. Stefano > > /Andi > > 2016-03-25 10:31 GMT+01:00 Stefano Babic : >> Hi Andreas, >> >> your series for the U-Boot environment tools does not let to use anymore >> the tools as library to be linked to custom program. >> >> In fact, tools/env/Makefile generates a library too: >> >> lib-y += fw_env.o \ >> crc32.o ctype.o linux_string.o \ >> env_attr.o env_flags.o aes.o >> >> that means that fw_main should not maintain any structure that is >> referenced by the library. >> In fact, linking with the library (renamed as ibubootenv.a), gives the >> errors: >> >> undefined reference to `printenv_args' >> undefined reference to `common_args' >> >> The two functions fw_printenv() and fw_setenv() are then supposed to be >> called by a custom program, what it does not work anymore. >> >> Can you take a look at it ? >> >> Thanks, >> Stefano Babic >> >> -- >> ===================================================================== >> DENX Software Engineering GmbH, Managing Director: Wolfgang Denk >> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany >> Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de >> ===================================================================== -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================