[U-Boot] [PATCH V2 02/12] arm: dts: include dts for imx6sabresd

[Peng Fan]

> -----Original Message-----
> From: Stefano Babic [mailto:sbabic at denx.de]
> Sent: Monday, August 28, 2017 3:07 PM
> To: Peng Fan <peng.fan at nxp.com>; Jagan Teki <jagannadh.teki at gmail.com>
> Cc: Stefano Babic <sbabic at denx.de>; u-boot at lists.denx.de; Fabio Estevam
> <fabio.estevam at freescale.com>
> Subject: Re: [U-Boot] [PATCH V2 02/12] arm: dts: include dts for imx6sabresd
> 
> Hi Jagan, Peng, Fabio,
> 
> On 28/08/2017 08:45, Peng Fan wrote:
> >
> >
> >> -----Original Message-----
> >> From: Jagan Teki [mailto:jagannadh.teki at gmail.com]
> >> Sent: Monday, August 28, 2017 2:41 PM
> >> To: Peng Fan <peng.fan at nxp.com>
> >> Cc: Stefano Babic <sbabic at denx.de>; u-boot at lists.denx.de
> >> Subject: Re: [U-Boot] [PATCH V2 02/12] arm: dts: include dts for
> >> imx6sabresd
> >>
> >> On Mon, Aug 28, 2017 at 11:58 AM, Peng Fan <peng.fan at nxp.com> wrote:
> >>> Import dts from commit aae4e7a8bc("Linux 4.13-rc4") for imx6sabresd
> board.
> >>>
> >>> Signed-off-by: Peng Fan <peng.fan at nxp.com>
> >>> Cc: Stefano Babic <sbabic at denx.de>
> >>> ---
> >>
> >> Similar series[1] with DTB support for sabresd
> >>
> >>
> >> [1] https://lists.denx.de/pipermail/u-boot/2017-May/293206.html
> >
> > I thought that thread was not active, since past 3 months. I'll leave Stefano to
> decide.
> >
> 
> In my understanding, Jagan's series has still open coments from Joe regarding
> phy-reset. I have not seen a new post, and I confess I have then forgotten this
> series. I have not seen a review by Fabio, too, and he's the maintainer for sabre
> boards.
> 
> Peng, I have just tried to build with your series, but it breaks many boards due
> to DTC issue. I have also tried to build with last DTC from git, but it does not
> help. Is it known to you ?

I only build mx6sabresd_defconfig, did not run buildman to build other boards,
Because I always met
/usr/include/python2.7/pyconfig.h:1207:0: warning: "_XOPEN_SOURCE" redefined [enabled by default]
 #define _XOPEN_SOURCE 600
 ^
In file included from /usr/include/stdint.h:25:0,

I'll build other boards and fix the issue.

Thanks,
Peng.

> 
> 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
> ============================================================
> =========