[U-Boot] [PATCH] zynq: Use full tftpboot command instead of shortcut tftp
Michal Simek
monstr at monstr.eu
Mon Feb 10 10:30:37 CET 2014
Hi Albert,
On 02/10/2014 10:11 AM, Albert ARIBAUD wrote:
> Hi Michal,
>
> On Wed, 5 Feb 2014 07:56:07 +0100, Michal Simek
> <michal.simek at xilinx.com> wrote:
>
>> The reason is enabling tftpput command where
>> tftp shorcut stops to work for tftpboot.
>
> Do you think you could extend this patch to all config files where
> the 'tftp' shortcut exists? This would prevent future issues on
> targets other than Zynq and hopefully it is only a matter of grepping /
> sed'ing through include/configs/*.h.
Based on my experience it is better to use full commands in variables
but on the other hand I have met with the case when adding these 4 chars
caused that max variable size has been exceed with error.
It means for short variables I can't see any problem with it
and definitely will be better to use full command names instead of shortcut.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140210/4b7f8d66/attachment.pgp>
More information about the U-Boot
mailing list