Re: [U-Boot] [PATCH 0/2] Add atomic write to fw_setenv for environments on filesystems

2018-03-08 Thread Heinrich Schuchardt 

On 03/08/2018 12:52 PM, Alex Kiernan wrote:

For environments stored on filesystems where you can't have a redundant
configuration, rather than just over-writing the existing environment
in fw_setenv, do the tradtional create temporary file, rename, sync,
sync directory dance to achieve ACID semantics when writing through
fw_setenv.

Note that this series triggers large numbers of checkpatch warnings because
of the existing code style:

   warning: space prohibited between function name and open parenthesis '('
   check: Avoid CamelCase: 


So, please, change the patches to match the U-Boot coding style.

Regards

Heinrich




Alex Kiernan (2):
   tools: env: Refactor write path of flash_io()
   tools: env: Implement atomic replace for filesystem

  tools/env/fw_env.c | 159 -
  1 file changed, 121 insertions(+), 38 deletions(-)



___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 0/2] Add atomic write to fw_setenv for environments on filesystems

2018-03-08 Thread Alex Kiernan 

For environments stored on filesystems where you can't have a redundant
configuration, rather than just over-writing the existing environment
in fw_setenv, do the tradtional create temporary file, rename, sync,
sync directory dance to achieve ACID semantics when writing through
fw_setenv.

Note that this series triggers large numbers of checkpatch warnings because
of the existing code style:

  warning: space prohibited between function name and open parenthesis '('
  check: Avoid CamelCase: 


Alex Kiernan (2):
  tools: env: Refactor write path of flash_io()
  tools: env: Implement atomic replace for filesystem

 tools/env/fw_env.c | 159 -
 1 file changed, 121 insertions(+), 38 deletions(-)

-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot