From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefano Babic <sbabic@denx.de>
Date: Sun, 2 Sep 2018 19:06:06 +0200
Subject: [U-Boot] [PULL] Please pull u-boot-imx
In-Reply-To: <20180901113746.GB26633@bill-the-cat>
References: <48c52f49-8882-821c-70e5-3dfa34c7722b@denx.de>
 <20180901113746.GB26633@bill-the-cat>
Message-ID: <671d2b53-4f1e-12a8-22fd-2b446b1c5037@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
To: u-boot@lists.denx.de



On 01/09/2018 13:37, Tom Rini wrote:
> On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:
>=20
>> Hi Tom,
>>
>> please pull from u-boot-imx, thanks !
>>
>> The following changes since commit 11ed312896c5f5814064c5d45dcb2f53dc121=
437:
>>
>>   configs: am57xx: change default board name to beagle_x15 (2018-08-26
>> 12:26:16 -0400)
>>
>> are available in the Git repository at:
>>
>>   git://www.denx.de/git/u-boot-imx.git master
>>
>> for you to fetch changes up to 2846e663fd62200a189bba357135e284a379a38b:
>>
>>   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

It looks wrong how I verify the whole build. Running buildman on my
host, all boards were built. I will

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

I saw this, but if the board maintainer won't use "make savedefonfig" to
generate its own defconfig, I have not blocked this if this lead to a
successful build.

> - So I start reading the whole diff and I see:
> commit 8e00d802e402d2a6734a88ebeb77a70efcfc354c
> Author: S=C3=A9bastien Szymanski <sebastien.szymanski@armadeus.com>
> Date:   Wed Jul 25 14:47:53 2018 +0200
>=20
>     ARM: opos6ul: make the board boot again
>=20
> Which is wrong.  The -u-boot.dtsi file is automatically included and
> should have all of the U-Boot specific DTS changes.

I have to understand why my build was successful...

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

Note this, they will be fixed.

> Rather than fix it myself like I was going to before I found the other
> issues.
>=20

Regards,
Stefano

--=20
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
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
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D