Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Hi Simon, reviving an old (dead) thread: On 20/11/18 22:11, Simon Goldschmidt wrote: > On 04.09.2018 12:30, Andreas Reichel wrote: >> Hi all, >> >> as Stefano Babic was so friendly and pointed out a few things already, >> we come the following problematic points: >> >> For SWupdate to access U-B

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Wolfgang, On 22.11.2018 17:00, Wolfgang Denk wrote: Dear Simon, In message you wrote: I don't know your board. It may be intentional, for example if there are size restrictions for the SPL wich don't allow to include the CRC32 code... I don't think this is intentional. I discussed th

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Simon, In message you wrote: > > > I don't know your board. > > > > It may be intentional, for example if there are size restrictions > > for the SPL wich don't allow to include the CRC32 code... > > > > I don't think this is intentional. I discussed this in another thread with > Marek by n

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Wolfgang, Am Do., 22. Nov. 2018, 15:23 hat Wolfgang Denk geschrieben: > Dear Simon, > > In message <97f8b0ec-103b-e9b7-42ed-e1922f6d4...@gmail.com> you wrote: > > > > Oops, I'll have to take that back. I just checked and SPL *does* boot > > the uImage U-Boot also if I deliberately corrupt the CR

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Simon, In message <97f8b0ec-103b-e9b7-42ed-e1922f6d4...@gmail.com> you wrote: > > Oops, I'll have to take that back. I just checked and SPL *does* boot > the uImage U-Boot also if I deliberately corrupt the CRC of the image. > So you're right, it seems to be not protected at all! > > Is tha

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On 21.11.2018 16:48, Simon Goldschmidt wrote: On 21.11.2018 16:41, Wolfgang Denk wrote: Dear Simon, In message you wrote: Only if you use signed images. With plain U-Boot, there is not even a checksum for it... When SPL loads U-Boot from a legacy image, isn't there a CRC involved over the

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On Wed, Nov 21, 2018 at 7:10 AM Stefano Babic wrote: > On 21/11/18 00:10, Marek Vasut wrote: > > On 11/20/2018 10:11 PM, Simon Goldschmidt wrote: > >> On 04.09.2018 12:30, Andreas Reichel wrote: > The library is already built (tools/env/lib.a), but IMHO the best thing > is to export it official an

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Simon, In message you wrote: > > Hmm, on my board, SPL loads U-Boot from NOR to SDRAM, then checks the > uimage CRC. I'd say this is enough checks. That might not hold for all > boards though. If you check the image in RAM, then yes. I've also seen implementations where it gets checked o

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On 21.11.2018 16:41, Wolfgang Denk wrote: Dear Simon, In message you wrote: Only if you use signed images. With plain U-Boot, there is not even a checksum for it... When SPL loads U-Boot from a legacy image, isn't there a CRC involved over the full image including the environment? Yes, but

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Simon, In message you wrote: > > > Only if you use signed images. With plain U-Boot, there is not even > > a checksum for it... > > When SPL loads U-Boot from a legacy image, isn't there a CRC involved over > the full image including the environment? Yes, but when - for example - the comp

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Am Mi., 21. Nov. 2018, 15:23 hat Wolfgang Denk geschrieben: > Dear Stefano, > > In message <151c30b1-d383-d212-a611-519150a80...@denx.de> you wrote: > > > > > I tend to disagree. As mentioned, the original intent of the > > > default environment was only (!) to make sure that a working > > > bau

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On 21/11/18 15:37, Simon Goldschmidt wrote: > > > Am Mi., 21. Nov. 2018, 15:27 hat Wolfgang Denk > geschrieben: > > Dear Stefano, > > In message <7089ef62-ed0f-87f4-e979-8c18a6ae4...@denx.de > > you wrote: >

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Hi Marek, On 21/11/18 15:31, Marek Vasut wrote: > On 11/21/2018 10:20 AM, Wolfgang Denk wrote: >> Dear Stefano, > > Hi, > >> In message <96836cc1-e4bb-a2a2-05ac-056053b4c...@denx.de> you wrote: >>> >>> I would like to see the library under LGPL instead of GPL2, too, and I >>> raised this issue w

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Am Mi., 21. Nov. 2018, 15:27 hat Wolfgang Denk geschrieben: > Dear Stefano, > > In message <7089ef62-ed0f-87f4-e979-8c18a6ae4...@denx.de> you wrote: > > > > > Right, when we sign (and check the signatures) of all other images, > > > then why not do the very same for some environment image? > > >

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On 11/21/2018 10:20 AM, Wolfgang Denk wrote: > Dear Stefano, Hi, > In message <96836cc1-e4bb-a2a2-05ac-056053b4c...@denx.de> you wrote: >> >> I would like to see the library under LGPL instead of GPL2, too, and I >> raised this issue when I started SWUpdate, but I was not very active to >> promot

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Stefano, In message <7089ef62-ed0f-87f4-e979-8c18a6ae4...@denx.de> you wrote: > > > Right, when we sign (and check the signatures) of all other images, > > then why not do the very same for some environment image? > > The weird thing is with "saveenv" - if we just read the env, it is fine, >

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Stefano, In message <151c30b1-d383-d212-a611-519150a80...@denx.de> you wrote: > > > I tend to disagree. As mentioned, the original intent of the > > default environment was only (!) to make sure that a working > > baudrate setting for the console is available on boards where the > > environm

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Hi Wolfgang, On 21/11/18 12:45, Wolfgang Denk wrote: > Dear Stefano, > > In message <9efc8990-118c-d5b9-802d-8190db232...@denx.de> you wrote: >> >> True, but this has side effect and limitation. You cannot change a >> variable in the deafult environment if you need it because you do not >> know i

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Hi Wolfgang, On 21/11/18 12:38, Wolfgang Denk wrote: > Dear Stefano, > > In message <6b07725e-51ba-7309-1506-e7211487b...@denx.de> you wrote: >> >>> One possible way to fix this is not to use a compiled in default >>> environment for the fw_env tools (or only use this as a very last >>> resort),

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Simon, In message you wrote: > > > > Right, when we sign (and check the signatures) of all other images, > > then why not do the very same for some environment image? > > You normally cannot sign the environment in the target when saving it > when using private/public keys. You mix things

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On Wed, Nov 21, 2018 at 12:45 PM Wolfgang Denk wrote: > > Dear Stefano, > > In message <9efc8990-118c-d5b9-802d-8190db232...@denx.de> you wrote: > > > > True, but this has side effect and limitation. You cannot change a > > variable in the deafult environment if you need it because you do not > >

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Hi Simon, On 21/11/18 12:13, Simon Goldschmidt wrote: > On Wed, Nov 21, 2018 at 12:05 PM Stefano Babic wrote: >> >> On 21/11/18 11:21, Simon Goldschmidt wrote: >>> On Wed, Nov 21, 2018 at 10:19 AM Stefano Babic wrote: Hi Simon, On 20/11/18 22:11, Simon Goldschmidt wrote:

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Stefano, In message <9efc8990-118c-d5b9-802d-8190db232...@denx.de> you wrote: > > True, but this has side effect and limitation. You cannot change a > variable in the deafult environment if you need it because you do not > know it. If some changes are needed, even if for very small things lik

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Stefano, In message <6b07725e-51ba-7309-1506-e7211487b...@denx.de> you wrote: > > > One possible way to fix this is not to use a compiled in default > > environment for the fw_env tools (or only use this as a very last > > resort), but instead provide a configuration option to read the > > de

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On Wed, Nov 21, 2018 at 12:05 PM Stefano Babic wrote: > > On 21/11/18 11:21, Simon Goldschmidt wrote: > > On Wed, Nov 21, 2018 at 10:19 AM Stefano Babic wrote: > >> > >> Hi Simon, > >> > >> On 20/11/18 22:11, Simon Goldschmidt wrote: > >>> On 04.09.2018 12:30, Andreas Reichel wrote: > Hi all

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On 21/11/18 11:21, Simon Goldschmidt wrote: > On Wed, Nov 21, 2018 at 10:19 AM Stefano Babic wrote: >> >> Hi Simon, >> >> On 20/11/18 22:11, Simon Goldschmidt wrote: >>> On 04.09.2018 12:30, Andreas Reichel wrote: Hi all, as Stefano Babic was so friendly and pointed out a few things

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Hi Simon, On 21/11/18 11:14, Simon Goldschmidt wrote: > On Wed, Nov 21, 2018 at 10:33 AM Stefano Babic wrote: >> >> Hi Wolfgang, >> >> On 21/11/18 09:31, Wolfgang Denk wrote: >>> Dear Simon, >>> >>> In message <90014ca9-64d8-8e38-a6c0-4c06e991c...@gmail.com> you wrote: > However, this ap

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On 21.11.18 11:14, Simon Goldschmidt wrote: - What prevents the creation of such a dev package that only builds the needed lib? I think Simon wants to prevent to rebuild u-boot. No, I want to prevent recompiling the u-boot-tools package (fw_setenv) and Swupdate after changing U-Boot's defa

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On Wed, Nov 21, 2018 at 10:19 AM Stefano Babic wrote: > > Hi Simon, > > On 20/11/18 22:11, Simon Goldschmidt wrote: > > On 04.09.2018 12:30, Andreas Reichel wrote: > >> Hi all, > >> > >> as Stefano Babic was so friendly and pointed out a few things already, > >> we come the following problematic p

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On Wed, Nov 21, 2018 at 10:33 AM Stefano Babic wrote: > > Hi Wolfgang, > > On 21/11/18 09:31, Wolfgang Denk wrote: > > Dear Simon, > > > > In message <90014ca9-64d8-8e38-a6c0-4c06e991c...@gmail.com> you wrote: > >> > >>> However, this approach creates several difficulties: > > > > As usual, "diffi

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

On 21.11.18 10:19, Stefano Babic wrote: Hi Simon, On 20/11/18 22:11, Simon Goldschmidt wrote: On 04.09.2018 12:30, Andreas Reichel wrote: Hi all, as Stefano Babic was so friendly and pointed out a few things already, we come the following problematic points: For SWupdate to access U-Boot's e

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Hi Wolfgang, On 21/11/18 09:31, Wolfgang Denk wrote: > Dear Simon, > > In message <90014ca9-64d8-8e38-a6c0-4c06e991c...@gmail.com> you wrote: >> >>> However, this approach creates several difficulties: > > As usual, "difficulties" are made for solving. > > I mean, none of these are technically

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Dear Stefano, In message <96836cc1-e4bb-a2a2-05ac-056053b4c...@denx.de> you wrote: > > I would like to see the library under LGPL instead of GPL2, too, and I > raised this issue when I started SWUpdate, but I was not very active to > promote this. Tom, Wolfgang, is there chances to switch license

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Hi Simon, On 20/11/18 22:11, Simon Goldschmidt wrote: > On 04.09.2018 12:30, Andreas Reichel wrote: >> Hi all, >> >> as Stefano Babic was so friendly and pointed out a few things already, >> we come the following problematic points: >> >> For SWupdate to access U-Boot's environment, it uses code f

Re: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

Hi Marek, On 21/11/18 00:10, Marek Vasut wrote: > On 11/20/2018 10:11 PM, Simon Goldschmidt wrote: >> On 04.09.2018 12:30, Andreas Reichel wrote: >>> Hi all, >>> >>> as Stefano Babic was so friendly and pointed out a few things already, >>> we come the following problematic points: >>> >>> For SWu