Hi Peng, On 05/11/18 14:06, Fabio Estevam wrote: > Hi Peng, > > On Mon, Nov 5, 2018 at 7:53 AM Peng Fan <[email protected]> wrote: >> >> Check lseek return value. >> >> Fix Coverity CID: 184236 184235 184232 >> >> Reported-by: Coverity >> Signed-off-by: Peng Fan <[email protected]> > > Thanks for the respin. > > Reviewed-by: Fabio Estevam <[email protected]> >
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: [email protected] ===================================================================== _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
