On 09/15/2017 08:10 AM, Jean-Jacques Hiblot wrote:
> Hi York,
>
>
> On 14/09/2017 21:41, York Sun wrote:
>> common/spl/spl_fit.c:201:12: warning: passing argument 4 of ‘gunzip’
>> from incompatible pointer type [-Wincompatible-pointer-types]
>> src, &length))
>>
>> Signed-off-by: York Sun <york....@nxp.com>
>> Reported-by: Heinrich Schuchardt <xypron.g...@gmx.de>
>> ---
>>
>> common/spl/spl_fit.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/common/spl/spl_fit.c b/common/spl/spl_fit.c
>> index 49ccf1c..82943cd 100644
>> --- a/common/spl/spl_fit.c
>> +++ b/common/spl/spl_fit.c
>> @@ -135,6 +135,7 @@ static int spl_load_fit_image(struct spl_load_info
>> *info, ulong sector,
>> int offset;
>> size_t length;
>> int len;
>> + ulong size;
>> ulong load_addr, load_ptr;
>> void *src;
>> ulong overhead;
>> @@ -197,8 +198,9 @@ static int spl_load_fit_image(struct spl_load_info
>> *info, ulong sector,
>> IS_ENABLED(CONFIG_SPL_GZIP) &&
>> image_comp == IH_COMP_GZIP &&
>> type == IH_TYPE_KERNEL) {
>> + size = length;
>> if (gunzip((void *)load_addr, CONFIG_SYS_BOOTM_LEN,
>> - src, &length)) {
>> + src, &size)) {
>> puts("Uncompressing error\n");
>> return -EIO;
>> }
> Here I think you should add a length = size;
> because "length" is used a few lines after (ln 211) : image_info->size =
> length;
You are correct. I will send an update. Thanks.
York
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
