From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Sat, 1 Sep 2018 07:37:46 -0400 Subject: [U-Boot] [PULL] Please pull u-boot-imx In-Reply-To: <48c52f49-8882-821c-70e5-3dfa34c7722b@denx.de> References: <48c52f49-8882-821c-70e5-3dfa34c7722b@denx.de> Message-ID: <20180901113746.GB26633@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote: > Hi Tom, >=20 > please pull from u-boot-imx, thanks ! >=20 > The following changes since commit 11ed312896c5f5814064c5d45dcb2f53dc1214= 37: >=20 > configs: am57xx: change default board name to beagle_x15 (2018-08-26 > 12:26:16 -0400) >=20 > are available in the Git repository at: >=20 > git://www.denx.de/git/u-boot-imx.git master >=20 > for you to fetch changes up to 2846e663fd62200a189bba357135e284a379a38b: >=20 > imx: missing CONFIG_MII in mx7dsabresd_qspi_defconfig (2018-08-31 > 12:08:43 +0200) >=20 OK, NAK for the following problems: - Fail to build: https://travis-ci.org/trini/u-boot/jobs/423197300 The failing board is ids8313 and it's failing to configure as CONFIG_SYS_BOOTCOUNT_I2C_BUS needs to be set, and isn't - In looking into the above I see configs/ge_bx50v3_defconfig and configs/mx53ppd_defconfig and a few more have a bunch of comments added to it. These will be blown away on the next re-sync. Comments need to go into the board README or something not an auto-generated file. - So I start reading the whole diff and I see: commit 8e00d802e402d2a6734a88ebeb77a70efcfc354c Author: S=C3=A9bastien Szymanski Date: Wed Jul 25 14:47:53 2018 +0200 ARM: opos6ul: make the board boot again Which is wrong. The -u-boot.dtsi file is automatically included and should have all of the U-Boot specific DTS changes. - And since the above big real problems exist I'm going to point out for you to fix: WARNING: no status info for 'mx7dsabresd_qspi' WARNING: no maintainers for 'mx7dsabresd_qspi' Rather than fix it myself like I was going to before I found the other issues. --=20 Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: