[U-Boot] [PATCH V2 1/4] tools: imx8image: check lseek return value

Stefano Babic sbabic at denx.de
Tue Nov 6 10:32:48 UTC 2018


Hi Peng,

On 05/11/18 14:06, Fabio Estevam wrote:
> Hi Peng,
> 
> On Mon, Nov 5, 2018 at 7:53 AM Peng Fan <peng.fan at nxp.com> wrote:
>>
>> Check lseek return value.
>>
>> Fix Coverity CID: 184236 184235 184232
>>
>> Reported-by: Coverity
>> Signed-off-by: Peng Fan <peng.fan at nxp.com>
> 
> Thanks for the respin.
> 
> Reviewed-by: Fabio Estevam <festevam at gmail.com>
> 

Yes, thanks for this. I admit that I prefer in this way as Fabio, but I
admit that it is not thinkable that lseek() will change the API, and I
agree that change could be more or less a question of taste. Anyway, I
prefer in this way...

I merged the series together with:

- MAINTAINERS: add NXP linux team maillist as i.MX reviewer
- ARM: dts: fsl-imx8qxp-mek: Move regulator outside "simple-bus"
- mx8mq_evk: README: Delete file introduced by mistake

I do not see any other open points and I will send PR to Tom.

Best regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list