> -----Original Message-----
> From: Fabio Estevam [mailto:feste...@gmail.com]
> Sent: Saturday, December 02, 2017 4:53 AM
> To: Peng Fan <peng....@nxp.com>
> Cc: Peter Robinson <pbrobin...@gmail.com>; Fabio Estevam
> <fabio.este...@nxp.com>; u-boot@lists.denx.de; Diego Dorta
> <diego.do...@nxp.com>
> Subject: Re: [U-Boot] [PATCH 00/23] imx: add i.MX8M support and i.MX8MQ
> EVK
> 
> Hi Peng,
> 
> On Thu, Nov 30, 2017 at 12:53 AM, Peng Fan <peng....@nxp.com> wrote:
> 
> >> I don't see a README in here, I'm wondering what the
> >> details/requirements are around ARM Trusted Firmware and the build
> >> process, for this platform does it use the open ARM ATF, or is it
> >> similar to other platforms that early boot is proprietary and u-boot
> >> is just chain loaded? Something like the sunxi 64 bit readme [1] would be
> useful.
> >
> > We are using ATF and use a dedicated package tool to generate the final
> image.
> >
> > Then the boot flow is SPL->ATF/TEE->U-Boot->Kernel. I do not have much
> > time to write that down now. You may get a very good user guide from
> > NXP sw release, if you want to boot your board or else.
> 
> I agree with Peter that a README file is really important for this series.
> 
> Without it we can't really test it on the real hardware.
> 
> My colleague Diego tried to test it and was not able to boot it, so please 
> add a
> README file on your v2.

Ok. I'll add a README file, but arm trusted firmware code and imx-mkimage
code are not public available now.

Another thing is the ddr script is not using structure based style code.
It is hard for me to convert it from register offset to structure based.
So In V2, I prefer to leave that piece code as TODO. I'll still include the
ddr part in the V2 patchset, but I am not sure it will be accepted or not.

Regards,
Peng.

> 
> Thanks
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to