Re: [U-Boot] [PATCH] dfu: fix: Add the absolute path to the file name for ext4 write operation

2014-07-22 Thread Lukasz Majewski
Hi Stephen,

 On 07/16/2014 03:38 AM, Lukasz Majewski wrote:
  Commit 1151b7ac10b81ecbb has cleaned up read and write operations.
  Unfortunately, for correct operation the write for ext4 fs requires
  absolute patch.
  This patch fixes this case.
 
 Shouldn't the user simply supply the absolute patch name as the entity
 name? That's what I've been doing.
 
 This way, it makes files in sub-directories look more natural; you'd
 pass in e.g. /boot/zImage rather than boot/zImage, which looks like a
 relative path.

Hmm.. this is the angle from which I didn't look. 

Thanks for the tip. I will adjust envs accordingly. Please discard this
patch.

-- 
Best regards,

Lukasz Majewski

Samsung RD Institute Poland (SRPOL) | Linux Platform Group
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] dfu: fix: Add the absolute path to the file name for ext4 write operation

2014-07-22 Thread Marek Vasut
On Tuesday, July 22, 2014 at 10:00:05 AM, Lukasz Majewski wrote:
 Hi Stephen,
 
  On 07/16/2014 03:38 AM, Lukasz Majewski wrote:
   Commit 1151b7ac10b81ecbb has cleaned up read and write operations.
   Unfortunately, for correct operation the write for ext4 fs requires
   absolute patch.
   This patch fixes this case.
  
  Shouldn't the user simply supply the absolute patch name as the entity
  name? That's what I've been doing.
  
  This way, it makes files in sub-directories look more natural; you'd
  pass in e.g. /boot/zImage rather than boot/zImage, which looks like a
  relative path.
 
 Hmm.. this is the angle from which I didn't look.
 
 Thanks for the tip. I will adjust envs accordingly. Please discard this
 patch.

Done.

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] dfu: fix: Add the absolute path to the file name for ext4 write operation

2014-07-21 Thread Stephen Warren
On 07/16/2014 03:38 AM, Lukasz Majewski wrote:
 Commit 1151b7ac10b81ecbb has cleaned up read and write operations.
 Unfortunately, for correct operation the write for ext4 fs requires
 absolute patch.
 This patch fixes this case.

Shouldn't the user simply supply the absolute patch name as the entity
name? That's what I've been doing.

This way, it makes files in sub-directories look more natural; you'd
pass in e.g. /boot/zImage rather than boot/zImage, which looks like a
relative path.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot