Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181217

[Stefano Babic]

On 27/12/18 20:16, Fabio Estevam wrote:
> Hi Tom,
> 
> On Wed, Dec 26, 2018 at 9:27 PM Tom Rini  wrote:
> 
>> There's the list of boards that it might be.  Doing a for B in ...;do
>> make O=/tmp/$B ${B}_defconfig oldconfig and I see that yup, ids8313
>> trips on the new symbols without having a value.  So those two bootcount
>> patches from Denis need a slight rework (the second depends on the
>> first) or to be dropped for now.
> 
> As we are getting closer to the 2019.01 release, I think we should
> drop Denis' patches for now.
> 

Agree, I drop Denis' patches and I reporst the PR.

Thanks,
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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181217

[Fabio Estevam]

Hi Tom,

On Wed, Dec 26, 2018 at 9:27 PM Tom Rini  wrote:

> There's the list of boards that it might be.  Doing a for B in ...;do
> make O=/tmp/$B ${B}_defconfig oldconfig and I see that yup, ids8313
> trips on the new symbols without having a value.  So those two bootcount
> patches from Denis need a slight rework (the second depends on the
> first) or to be dropped for now.

As we are getting closer to the 2019.01 release, I think we should
drop Denis' patches for now.

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

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181217

[Tom Rini]

On Mon, Dec 17, 2018 at 04:57:26PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, u-boot-imx-20181217, thanks !
> 
> - fix missing MAINTANERS for embestmx6boards
> - drop Wolfgang's patch
> - apply V4 of "pico-imx7d: Increase the CONFIG_ENV_OFFSET size " that
> use fix values for size.
> 
> The following changes since commit d117d8f19b0625f88309e47a8a32c2faa384dddc:
> 
>   Merge branch 'master' of git://git.denx.de/u-boot-i2c (2018-12-13
> 09:36:55 -0500)
> 
> are available in the Git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181217
> 
> for you to fetch changes up to 22f219feb807df91d958c3cc5a8634b6218b2565:
> 
>   pico-imx7d: Increase the CONFIG_ENV_OFFSET size (2018-12-17 16:51:02
> +0100)
> 
> 
> imx for 2019.01
> 
> - introduce support for i.MX8M
> - fix size limit for Vhybrid / pico boards
> - several board fixes
> - w1 driver for MX2x / MX5x
> 
> 

So, this breaks travis.  I know I've mentioned some of this off-list,
but for the record and to aid in future debugging of issues like this,
if we look at https://travis-ci.org/trini/u-boot/builds/472348357 we can
see just a single job fails.  So I rebased and put in as the first
commit a hack to .travis.yml where I remove every other job in the
matrix so it runs (relatively) quick.  Then I did a git bisect where for
the test I did:
$ git push -f github HEAD:TRAVIS-imx-PR-bisect
And then bisect good/bad depending on what happens.  This narrowed down
things to:
Author: Denis Zalevskiy 
Date:   Wed Oct 17 10:33:28 2018 +0200

bootcount: i2c: Add bus switching to the I2C bootcount driver

Being the culprit.  My _guess_ is that after looking at the commit we
have some boards that are prompting for
SYS_BOOTCOUNT_I2C_BUS/SYS_BOOTCOUNT_I2C_ADDR now and not having a value
set to they don't get built, but don't quite show up as an obvious
failure either (aside, Simon, any idea why?).  So, if we do:
$ ./tools/buildman/buildman -v --dry-run powerpc -x freescale
boards.cfg is up to date. Nothing to do.
Dry run, so not doing much. But I would do this:

Building current source for 47 boards (16 threads, 1 job per thread)
Build directory: ..

powerpc : 47 boards
   mpc8308_p1m sbc8349 sbc8349_PCI_33 sbc8349_PCI_66 ve8313 caddy2
   vme8349 hrcon hrcon_dh strider_con strider_con_dp strider_cpu
   strider_cpu_dp ids8313 kmcoge5ne kmeter1 kmopti2 kmsupx5 kmtegr1
   kmtepr2 kmvect1 suvd3 tuge1 tuxx1 TQM834x sbc8548 sbc8548_PCI_33
   sbc8548_PCI_33_PCIE sbc8548_PCI_66 sbc8548_PCI_66_PCIE socrates
   UCP1020 UCP1020_SPIFLASH controlcenterd_36BIT_SDCARD
   controlcenterd_36BIT_SDCARD_DEVELOP controlcenterd_TRAILBLAZER
   controlcenterd_TRAILBLAZER_DEVELOP kmcoge4 kmlion1 Cyrus_P5020
   Cyrus_P5040 xpedite520x xpedite537x xpedite550x sbc8641d xpedite517x
   MCR3000
Total boards to build for each commit: 47

There's the list of boards that it might be.  Doing a for B in ...;do
make O=/tmp/$B ${B}_defconfig oldconfig and I see that yup, ids8313
trips on the new symbols without having a value.  So those two bootcount
patches from Denis need a slight rework (the second depends on the
first) or to be dropped for now.

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181214

[Stefano Babic]

On 16/12/18 22:12, Lukasz Majewski wrote:
> Dear Stefano,
> 
>> On Fri, Dec 14, 2018 at 02:39:04PM +0100, Stefano Babic wrote:
>>
>>> Hi Tom,
>>>
>>> please pull from u-boot-imx, thanks !
>>>
>>>
>>> The following changes since commit
>>> c49aff3e66b930aa06936afee401cf5e19377958:
>>>
>>>   Merge branch 'master' of git://git.denx.de/u-boot-sunxi
>>> (2018-12-07 19:02:01 -0500)
>>>
>>> are available in the Git repository at:
>>>
>>>   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181214
>>>
>>> for you to fetch changes up to
>>> 5d0f0b567c78197db6f2f47c89a18b44ade28284:
>>>
>>>   pico-imx7d: Increase the CONFIG_ENV_OFFSET size (2018-12-14
>>> 14:30:57 +0100)
>>>   
>>
>> NAK.  You're failing the "no new defconfigs without MAINTAINERS" test
>> in Travis: https://travis-ci.org/trini/u-boot/jobs/468004873#L975
>>
>> Also, Wolfgang's patch is breaking
>> guruplug,sheevaplug,linkit-smart-7688-ram,colibri_pxa270,linkit-smart-7688,gardena-smart-gateway-mt7688-ram,gardena-smart-gateway-mt7688
>> in my specific Debian chroot and I need to dig in more to provide
>> something useful there other than "it fails".  So please drop that for
>> now too.  Thanks!
>>
> 
> As this PR is not accepted; Stefano, could you also pull following
> changes (relevant for vybrid):
> 
> http://patchwork.ozlabs.org/patch/1006731/
> http://patchwork.ozlabs.org/patch/1006732/
> http://patchwork.ozlabs.org/patch/1006733/
> 
> One of it:
> "nand: vybrid: Use calloc() instead of malloc() to allocate struct nfc"
> is a fix.
> 
> The other two just add support for Vybrid's NFC controller in device
> tree. There shall be no issues with this code for in-tree boards.
> 
> Thanks in advance and sorry for a late reply to this thread.
> 

I have picked up the 3 patches, my build is running..

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181214

[Lukasz Majewski]

Dear Stefano,

> On Fri, Dec 14, 2018 at 02:39:04PM +0100, Stefano Babic wrote:
> 
> > Hi Tom,
> > 
> > please pull from u-boot-imx, thanks !
> > 
> > 
> > The following changes since commit
> > c49aff3e66b930aa06936afee401cf5e19377958:
> > 
> >   Merge branch 'master' of git://git.denx.de/u-boot-sunxi
> > (2018-12-07 19:02:01 -0500)
> > 
> > are available in the Git repository at:
> > 
> >   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181214
> > 
> > for you to fetch changes up to
> > 5d0f0b567c78197db6f2f47c89a18b44ade28284:
> > 
> >   pico-imx7d: Increase the CONFIG_ENV_OFFSET size (2018-12-14
> > 14:30:57 +0100)
> >   
> 
> NAK.  You're failing the "no new defconfigs without MAINTAINERS" test
> in Travis: https://travis-ci.org/trini/u-boot/jobs/468004873#L975
> 
> Also, Wolfgang's patch is breaking
> guruplug,sheevaplug,linkit-smart-7688-ram,colibri_pxa270,linkit-smart-7688,gardena-smart-gateway-mt7688-ram,gardena-smart-gateway-mt7688
> in my specific Debian chroot and I need to dig in more to provide
> something useful there other than "it fails".  So please drop that for
> now too.  Thanks!
> 

As this PR is not accepted; Stefano, could you also pull following
changes (relevant for vybrid):

http://patchwork.ozlabs.org/patch/1006731/
http://patchwork.ozlabs.org/patch/1006732/
http://patchwork.ozlabs.org/patch/1006733/

One of it:
"nand: vybrid: Use calloc() instead of malloc() to allocate struct nfc"
is a fix.

The other two just add support for Vybrid's NFC controller in device
tree. There shall be no issues with this code for in-tree boards.

Thanks in advance and sorry for a late reply to this thread.


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de


pgpAkftD6XP13.pgp
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181214

[Fabio Estevam]

Hi Tom,

On Fri, Dec 14, 2018 at 8:06 PM Tom Rini  wrote:

> I replied to Wolfgang's patch with the problem and I suggest that
> pico-7d simply stop using >> as it's not going to work once we Kconfig
> migrate anyhow.

pico-imx7d does not use >>.

I sent 5 different versions of this patch and the initial ones do not
depend on Wolfgang's patch, as I hardcoded the offset.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181214

[Tom Rini]

On Fri, Dec 14, 2018 at 08:09:18PM -0200, Fabio Estevam wrote:
> Hi Tom,
> 
> On Fri, Dec 14, 2018 at 8:06 PM Tom Rini  wrote:
> 
> > I replied to Wolfgang's patch with the problem and I suggest that
> > pico-7d simply stop using >> as it's not going to work once we Kconfig
> > migrate anyhow.
> 
> pico-imx7d does not use >>.
> 
> I sent 5 different versions of this patch and the initial ones do not
> depend on Wolfgang's patch, as I hardcoded the offset.

OK, thanks.

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181214

[Tom Rini]

On Fri, Dec 14, 2018 at 11:03:59PM +0100, Stefano Babic wrote:
> 
> 
> On 14/12/18 19:37, Tom Rini wrote:
> > On Fri, Dec 14, 2018 at 02:39:04PM +0100, Stefano Babic wrote:
> > 
> >> Hi Tom,
> >>
> >> please pull from u-boot-imx, thanks !
> >>
> >>
> >> The following changes since commit 
> >> c49aff3e66b930aa06936afee401cf5e19377958:
> >>
> >>   Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2018-12-07
> >> 19:02:01 -0500)
> >>
> >> are available in the Git repository at:
> >>
> >>   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181214
> >>
> >> for you to fetch changes up to 5d0f0b567c78197db6f2f47c89a18b44ade28284:
> >>
> >>   pico-imx7d: Increase the CONFIG_ENV_OFFSET size (2018-12-14 14:30:57
> >> +0100)
> >>
> > 
> > NAK.  You're failing the "no new defconfigs without MAINTAINERS" test in
> > Travis: https://travis-ci.org/trini/u-boot/jobs/468004873#L975
> > 
> 
> I see, I'll ix it.
> 
> > Also, Wolfgang's patch is breaking
> > guruplug,sheevaplug,linkit-smart-7688-ram,colibri_pxa270,linkit-smart-7688,gardena-smart-gateway-mt7688-ram,gardena-smart-gateway-mt7688
> > in my specific Debian chroot and I need to dig in more to provide
> > something useful there other than "it fails".  So please drop that for
> > now too. 
> 
> I do, I put Fabio's pico-7d in wait queue as this depends on Wolfgang's

I replied to Wolfgang's patch with the problem and I suggest that
pico-7d simply stop using >> as it's not going to work once we Kconfig
migrate anyhow.

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181214

[Stefano Babic]

On 14/12/18 19:37, Tom Rini wrote:
> On Fri, Dec 14, 2018 at 02:39:04PM +0100, Stefano Babic wrote:
> 
>> Hi Tom,
>>
>> please pull from u-boot-imx, thanks !
>>
>>
>> The following changes since commit c49aff3e66b930aa06936afee401cf5e19377958:
>>
>>   Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2018-12-07
>> 19:02:01 -0500)
>>
>> are available in the Git repository at:
>>
>>   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181214
>>
>> for you to fetch changes up to 5d0f0b567c78197db6f2f47c89a18b44ade28284:
>>
>>   pico-imx7d: Increase the CONFIG_ENV_OFFSET size (2018-12-14 14:30:57
>> +0100)
>>
> 
> NAK.  You're failing the "no new defconfigs without MAINTAINERS" test in
> Travis: https://travis-ci.org/trini/u-boot/jobs/468004873#L975
> 

I see, I'll ix it.

> Also, Wolfgang's patch is breaking
> guruplug,sheevaplug,linkit-smart-7688-ram,colibri_pxa270,linkit-smart-7688,gardena-smart-gateway-mt7688-ram,gardena-smart-gateway-mt7688
> in my specific Debian chroot and I need to dig in more to provide
> something useful there other than "it fails".  So please drop that for
> now too. 

I do, I put Fabio's pico-7d in wait queue as this depends on Wolfgang's

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181214

[Tom Rini]

On Fri, Dec 14, 2018 at 02:39:04PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> 
> The following changes since commit c49aff3e66b930aa06936afee401cf5e19377958:
> 
>   Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2018-12-07
> 19:02:01 -0500)
> 
> are available in the Git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181214
> 
> for you to fetch changes up to 5d0f0b567c78197db6f2f47c89a18b44ade28284:
> 
>   pico-imx7d: Increase the CONFIG_ENV_OFFSET size (2018-12-14 14:30:57
> +0100)
> 

NAK.  You're failing the "no new defconfigs without MAINTAINERS" test in
Travis: https://travis-ci.org/trini/u-boot/jobs/468004873#L975

Also, Wolfgang's patch is breaking
guruplug,sheevaplug,linkit-smart-7688-ram,colibri_pxa270,linkit-smart-7688,gardena-smart-gateway-mt7688-ram,gardena-smart-gateway-mt7688
in my specific Debian chroot and I need to dig in more to provide
something useful there other than "it fails".  So please drop that for
now too.  Thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Peter Robinson]

On Mon, Nov 19, 2018 at 11:17 AM Fabio Estevam  wrote:
>
> Ricardo/Peter,
>
> On Mon, Nov 19, 2018 at 5:44 AM Baruch Siach  wrote:
>
> > I guess Fabio's patch would fix the issue for you. Indefinite loops are not 
> > a
> > good idea in general, I think.
>
> I have just sent a patch that avoids the infinite loop.
>
> Please test it when you have a chance.

Replied with my tested by on the patch but 2018.11 with that patch
boots fine for me on the HB2 imx6q rev 1.5 SOM.

Thanks,
Peter
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Fabio Estevam]

Ricardo/Peter,

On Mon, Nov 19, 2018 at 5:44 AM Baruch Siach  wrote:

> I guess Fabio's patch would fix the issue for you. Indefinite loops are not a
> good idea in general, I think.

I have just sent a patch that avoids the infinite loop.

Please test it when you have a chance.

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

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Baruch Siach]

Hi Ricardo,

On Sun, Nov 18, 2018 at 05:43:02PM -0200, Ricardo Salveti wrote:
> On Sun, Nov 18, 2018 at 7:51 AM Baruch Siach  wrote:
> > Peter Robinson writes:
> > > On Fri, Nov 16, 2018 at 1:06 AM Ricardo Salveti  
> > > wrote:
> > >> On Wed, Nov 14, 2018 at 5:07 AM Baruch Siach  wrote:
> > >> > On Tue, Nov 13, 2018 at 11:42:44AM -0200, Ricardo Salveti wrote:
> > >> > > On Thu, Sep 6, 2018 at 12:52 PM Fabio Estevam  
> > >> > > wrote:
> > >> > > > On Thu, Sep 6, 2018 at 12:42 PM, Baruch Siach  
> > >> > > > wrote:
> > >> > > >
> > >> > > > > I tested current master successfully on Hummingboard2 with i.MX6 
> > >> > > > > Solo
> > >> > > > > (SOM rev 1.5):
> > >> > > >
> > >> > > > Thanks for testing.
> > >> > > >
> > >> > > > It seems we need more information from Peter about the regression 
> > >> > > > he reported.
> > >> > > >
> > >> > > > It would be helpful if Peter could run a bisect so that we could
> > >> > > > understand where the regression is coming from.
> > >> > >
> > >> > > Finally got the time to test u-boot 2018.09 on my hummingboard 2 and 
> > >> > > I
> > >> > > can also confirm the boot issue with imx6q (Hummingboard 2 MicroSOM
> > >> > > i2eX iMX6D - rev 1.3).
> > >> > >
> > >> > > The patch set that introduced this regression was part of another 
> > >> > > pull
> > >> > > request, the one that introduces eMMC booting support (from Jon
> > >> > > Nettleton, e.g. 86e5a7fc13 and 19ed6063a5). After doing some more
> > >> > > testing it seems that the hang happens when trying to verify if the
> > >> > > board has eMMC during runtime (has_emmc -> mmc_get_op_cond(mmc)),
> > >> > > which is not the case for this SOM in particular (and probably why it
> > >> > > works fine on most rev 1.5-based SOMs, as eMMC is usually available
> > >> > > there).
> > >> > >
> > >> > > Tested with current u-boot master and the issue is still valid.
> > >> > >
> > >> > > Jon, did you have any issue when testing that patch set on SOMs
> > >> > > without eMMC support?
> > >> >
> > >> > I tested U-Boot successfully with SOM rev 1.3 (no eMMC) on 
> > >> > Hummingboard2, as
> > >> > shown in my previous message on this thread.
> > >>
> > >> Indeed, you tested with i.MX6Q, only difference is that mine is iMX6D,
> > >> but both without eMMC.
> > >
> > > I see the issue with a .IMX6Q wirth SOM rev 1.5 (TI wifi, no EMMC) on
> > > a hummingboard2
> >
> > I could not reproduce with SOM rev 1.3 (no eMMC) on Hummingboard2.
> >
> > >> > What toolchain are you using?
> > >>
> > >> Using GCC 8.2 from latest OpenEmbedded. Will try building with the
> > >> version you used to see if I get any different behavior.
> > >
> > > gcc 8.2.x from Fedora 29
> >
> > I am using the ARM (Ltd) provided GNU toolchain version 8.2:
> >
> > => version
> > U-Boot 2018.11 (Nov 18 2018 - 11:22:16 +0200)
> >
> > arm-linux-gnueabihf-gcc (GNU Toolchain for the A-profile Architecture 
> > 8.2-2018-08 (arm-rel-8.23)) 8.2.1 20180802
> > GNU ld (GNU Toolchain for the A-profile Architecture 8.2-2018-08 
> > (arm-rel-8.23)) 2.30.0.20180625
> >
> > >> > What do you see on the serial console?
> > >>
> > >> It boots up to the point when it tries to find the emmc, and then it
> > >> basically hangs completely (tested with current master):
> > >>
> > >> U-Boot SPL 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
> > >> Trying to boot from MMC1
> > >>
> > >> U-Boot 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
> > >>
> > >> CPU:   Freescale i.MX6D rev1.5 996 MHz (running at 792 MHz)
> > >> CPU:   Extended Commercial temperature grade (-20C to 105C) at 33C
> > >> Reset cause: POR
> > >> Board: MX6 HummingBoard2
> > >> DRAM:  1 GiB
> > >> MMC:   FSL_SDHC: 0, FSL_SDHC: 1
> > >> Loading Environment from MMC... *** Warning - bad CRC, using default 
> > >> environment
> > >>
> > >> No panel detected: default to HDMI
> > >> Display: HDMI (1024x768)
> > >> In:serial
> > >> Out:   serial
> > >> Err:   serial
> > >> ---> hangs
> > >
> > > Exactly the same as I saw.
> >
> > Here is what I get at this point:
> >
> > In:serial
> > Out:   serial
> > Err:   serial
> > Card did not respond to voltage select!
> > Net:   FEC
> > ...
> >
> > The error message is from drivers/mmc/mmc.c:mmc_get_op_cond(). The code
> > around this point might be the source of your issue. Can you add a few
> > prints in mmc_get_op_cond() to pinpoint the call that hangs the code?
> 
> Also tried with ARM's pre-built toolchain (same version), and got the
> same hang. Looking a bit further, it basically looks up while waiting
> the first mmc command to complete:
> 
> has_emmc -> mmc_get_op_cond -> mmc_send_if_cond (testing for SD
> version 2) -> mmc_send_cmd -> esdhc_send_cmd_common -> while
> (!(esdhc_read32(>irqstat) & flags))
> 
> And the boot log with MMC_TRACE:
> 
> No panel detected: default to HDMI
> Display: HDMI (1024x768)
> In:serial
> Out:   serial
> Err:   serial
> CMD_SEND:0
> ARG  0x
> MMC_RSP_NONE
> CMD_SEND:8

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Fabio Estevam]

Hi Ricardo,

On Sun, Nov 18, 2018 at 5:44 PM Ricardo Salveti  wrote:

> Also tried with ARM's pre-built toolchain (same version), and got the
> same hang. Looking a bit further, it basically looks up while waiting
> the first mmc command to complete:
>
> has_emmc -> mmc_get_op_cond -> mmc_send_if_cond (testing for SD
> version 2) -> mmc_send_cmd -> esdhc_send_cmd_common -> while
> (!(esdhc_read32(>irqstat) & flags))

Does the change below help?

--- a/drivers/mmc/fsl_esdhc.c
+++ b/drivers/mmc/fsl_esdhc.c
@@ -396,6 +396,7 @@ static int esdhc_send_cmd_common(struct
fsl_esdhc_priv *priv, struct mmc *mmc,
uintirqstat;
u32 flags = IRQSTAT_CC | IRQSTAT_CTOE;
struct fsl_esdhc *regs = priv->esdhc_regs;
+   unsigned long start;

 #ifdef CONFIG_SYS_FSL_ERRATUM_ESDHC111
if (cmd->cmdidx == MMC_CMD_STOP_TRANSMISSION)
@@ -453,8 +454,11 @@ static int esdhc_send_cmd_common(struct
fsl_esdhc_priv *priv, struct mmc *mmc,
flags = IRQSTAT_BRR;

/* Wait for the command to complete */
-   while (!(esdhc_read32(>irqstat) & flags))
-   ;
+   start = get_timer(0);
+   while (!(esdhc_read32(>irqstat) & flags)) {
+   if (get_timer(start) > 1000)
+   return -ETIMEDOUT;
+   }

irqstat = esdhc_read32(>irqstat);
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Ricardo Salveti]

Hi Baruch,

On Sun, Nov 18, 2018 at 6:29 PM Baruch Siach  wrote:
>
> Hi Ricardo,
>
> Ricardo Salveti writes:
> > The line where it locks up waiting for the command to complete:
> > http://git.denx.de/?p=u-boot.git;a=blob;f=drivers/mmc/fsl_esdhc.c;h=3cdfa7f5a689789ea2f24da867d7c947ab78c002;hb=HEAD#l455
> >
> > Unclear why this only happens with this som/soc, maybe hardware/errata
> > differences?
> >
> > Let me know if you need any extra info.
>
> I'll take a look tomorrow.
>
> Can you reproduce this hang with these binary images:
>
>   https://images.solid-build.xyz/IMX6/U-Boot/spl-imx6-sdhc.bin
>   https://images.solid-build.xyz/IMX6/U-Boot/u-boot-imx6-sdhc.img

Yup, looks like it hangs at the same place:

U-Boot SPL 2018.01-02296-g457cdd60c3 (May 17 2018 - 22:59:22)
Trying to boot from MMC1

U-Boot 2018.01-02296-g457cdd60c3 (May 17 2018 - 22:59:22 +0200),
Build: jenkins-u-boot-imx6-variant=sdhc-2

CPU:   Freescale i.MX6D rev1.5 996 MHz (running at 792 MHz)
CPU:   Extended Commercial temperature grade (-20C to 105C) at 26C
Reset cause: POR
Board: MX6 HummingBoard2
   Watchdog enabled
DRAM:  1 GiB
MMC:   FSL_SDHC: 0, FSL_SDHC: 1
No panel detected: default to HDMI
Display: HDMI (1024x768)
In:serial
Out:   serial
Err:   serial
---> hangs
Cheers,
--
Ricardo Salveti de Araujo
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Baruch Siach]

Hi Ricardo,

Ricardo Salveti writes:
> On Sun, Nov 18, 2018 at 7:51 AM Baruch Siach  wrote:
>> Peter Robinson writes:
>> > On Fri, Nov 16, 2018 at 1:06 AM Ricardo Salveti  
>> > wrote:
>> >> On Wed, Nov 14, 2018 at 5:07 AM Baruch Siach  wrote:
>> >> > On Tue, Nov 13, 2018 at 11:42:44AM -0200, Ricardo Salveti wrote:
>> >> > > On Thu, Sep 6, 2018 at 12:52 PM Fabio Estevam  
>> >> > > wrote:
>> >> > > > On Thu, Sep 6, 2018 at 12:42 PM, Baruch Siach  
>> >> > > > wrote:
>> >> > > >
>> >> > > > > I tested current master successfully on Hummingboard2 with i.MX6 
>> >> > > > > Solo
>> >> > > > > (SOM rev 1.5):
>> >> > > >
>> >> > > > Thanks for testing.
>> >> > > >
>> >> > > > It seems we need more information from Peter about the regression 
>> >> > > > he reported.
>> >> > > >
>> >> > > > It would be helpful if Peter could run a bisect so that we could
>> >> > > > understand where the regression is coming from.
>> >> > >
>> >> > > Finally got the time to test u-boot 2018.09 on my hummingboard 2 and I
>> >> > > can also confirm the boot issue with imx6q (Hummingboard 2 MicroSOM
>> >> > > i2eX iMX6D - rev 1.3).
>> >> > >
>> >> > > The patch set that introduced this regression was part of another pull
>> >> > > request, the one that introduces eMMC booting support (from Jon
>> >> > > Nettleton, e.g. 86e5a7fc13 and 19ed6063a5). After doing some more
>> >> > > testing it seems that the hang happens when trying to verify if the
>> >> > > board has eMMC during runtime (has_emmc -> mmc_get_op_cond(mmc)),
>> >> > > which is not the case for this SOM in particular (and probably why it
>> >> > > works fine on most rev 1.5-based SOMs, as eMMC is usually available
>> >> > > there).
>> >> > >
>> >> > > Tested with current u-boot master and the issue is still valid.
>> >> > >
>> >> > > Jon, did you have any issue when testing that patch set on SOMs
>> >> > > without eMMC support?
>> >> >
>> >> > I tested U-Boot successfully with SOM rev 1.3 (no eMMC) on 
>> >> > Hummingboard2, as
>> >> > shown in my previous message on this thread.
>> >>
>> >> Indeed, you tested with i.MX6Q, only difference is that mine is iMX6D,
>> >> but both without eMMC.
>> >
>> > I see the issue with a .IMX6Q wirth SOM rev 1.5 (TI wifi, no EMMC) on
>> > a hummingboard2
>>
>> I could not reproduce with SOM rev 1.3 (no eMMC) on Hummingboard2.
>>
>> >> > What toolchain are you using?
>> >>
>> >> Using GCC 8.2 from latest OpenEmbedded. Will try building with the
>> >> version you used to see if I get any different behavior.
>> >
>> > gcc 8.2.x from Fedora 29
>>
>> I am using the ARM (Ltd) provided GNU toolchain version 8.2:
>>
>> => version
>> U-Boot 2018.11 (Nov 18 2018 - 11:22:16 +0200)
>>
>> arm-linux-gnueabihf-gcc (GNU Toolchain for the A-profile Architecture 
>> 8.2-2018-08 (arm-rel-8.23)) 8.2.1 20180802
>> GNU ld (GNU Toolchain for the A-profile Architecture 8.2-2018-08 
>> (arm-rel-8.23)) 2.30.0.20180625
>>
>> >> > What do you see on the serial console?
>> >>
>> >> It boots up to the point when it tries to find the emmc, and then it
>> >> basically hangs completely (tested with current master):
>> >>
>> >> U-Boot SPL 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
>> >> Trying to boot from MMC1
>> >>
>> >> U-Boot 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
>> >>
>> >> CPU:   Freescale i.MX6D rev1.5 996 MHz (running at 792 MHz)
>> >> CPU:   Extended Commercial temperature grade (-20C to 105C) at 33C
>> >> Reset cause: POR
>> >> Board: MX6 HummingBoard2
>> >> DRAM:  1 GiB
>> >> MMC:   FSL_SDHC: 0, FSL_SDHC: 1
>> >> Loading Environment from MMC... *** Warning - bad CRC, using default 
>> >> environment
>> >>
>> >> No panel detected: default to HDMI
>> >> Display: HDMI (1024x768)
>> >> In:serial
>> >> Out:   serial
>> >> Err:   serial
>> >> ---> hangs
>> >
>> > Exactly the same as I saw.
>>
>> Here is what I get at this point:
>>
>> In:serial
>> Out:   serial
>> Err:   serial
>> Card did not respond to voltage select!
>> Net:   FEC
>> ...
>>
>> The error message is from drivers/mmc/mmc.c:mmc_get_op_cond(). The code
>> around this point might be the source of your issue. Can you add a few
>> prints in mmc_get_op_cond() to pinpoint the call that hangs the code?
>
> Also tried with ARM's pre-built toolchain (same version), and got the
> same hang. Looking a bit further, it basically looks up while waiting
> the first mmc command to complete:
>
> has_emmc -> mmc_get_op_cond -> mmc_send_if_cond (testing for SD
> version 2) -> mmc_send_cmd -> esdhc_send_cmd_common -> while
> (!(esdhc_read32(>irqstat) & flags))
>
> And the boot log with MMC_TRACE:
>
> No panel detected: default to HDMI
> Display: HDMI (1024x768)
> In:serial
> Out:   serial
> Err:   serial
> CMD_SEND:0
> ARG  0x
> MMC_RSP_NONE
> CMD_SEND:8
> ARG  0x01AA
>
> The line where it locks up waiting for the command to complete:
> 

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Ricardo Salveti]

Hi Baruch,

On Sun, Nov 18, 2018 at 7:51 AM Baruch Siach  wrote:
>
> Hi Peter, Ricardo,
>
> Peter Robinson writes:
> > On Fri, Nov 16, 2018 at 1:06 AM Ricardo Salveti  
> > wrote:
> >> On Wed, Nov 14, 2018 at 5:07 AM Baruch Siach  wrote:
> >> > On Tue, Nov 13, 2018 at 11:42:44AM -0200, Ricardo Salveti wrote:
> >> > > On Thu, Sep 6, 2018 at 12:52 PM Fabio Estevam  
> >> > > wrote:
> >> > > > On Thu, Sep 6, 2018 at 12:42 PM, Baruch Siach  
> >> > > > wrote:
> >> > > >
> >> > > > > I tested current master successfully on Hummingboard2 with i.MX6 
> >> > > > > Solo
> >> > > > > (SOM rev 1.5):
> >> > > >
> >> > > > Thanks for testing.
> >> > > >
> >> > > > It seems we need more information from Peter about the regression he 
> >> > > > reported.
> >> > > >
> >> > > > It would be helpful if Peter could run a bisect so that we could
> >> > > > understand where the regression is coming from.
> >> > >
> >> > > Finally got the time to test u-boot 2018.09 on my hummingboard 2 and I
> >> > > can also confirm the boot issue with imx6q (Hummingboard 2 MicroSOM
> >> > > i2eX iMX6D - rev 1.3).
> >> > >
> >> > > The patch set that introduced this regression was part of another pull
> >> > > request, the one that introduces eMMC booting support (from Jon
> >> > > Nettleton, e.g. 86e5a7fc13 and 19ed6063a5). After doing some more
> >> > > testing it seems that the hang happens when trying to verify if the
> >> > > board has eMMC during runtime (has_emmc -> mmc_get_op_cond(mmc)),
> >> > > which is not the case for this SOM in particular (and probably why it
> >> > > works fine on most rev 1.5-based SOMs, as eMMC is usually available
> >> > > there).
> >> > >
> >> > > Tested with current u-boot master and the issue is still valid.
> >> > >
> >> > > Jon, did you have any issue when testing that patch set on SOMs
> >> > > without eMMC support?
> >> >
> >> > I tested U-Boot successfully with SOM rev 1.3 (no eMMC) on 
> >> > Hummingboard2, as
> >> > shown in my previous message on this thread.
> >>
> >> Indeed, you tested with i.MX6Q, only difference is that mine is iMX6D,
> >> but both without eMMC.
> >
> > I see the issue with a .IMX6Q wirth SOM rev 1.5 (TI wifi, no EMMC) on
> > a hummingboard2
>
> I could not reproduce with SOM rev 1.3 (no eMMC) on Hummingboard2.
>
> >> > What toolchain are you using?
> >>
> >> Using GCC 8.2 from latest OpenEmbedded. Will try building with the
> >> version you used to see if I get any different behavior.
> >
> > gcc 8.2.x from Fedora 29
>
> I am using the ARM (Ltd) provided GNU toolchain version 8.2:
>
> => version
> U-Boot 2018.11 (Nov 18 2018 - 11:22:16 +0200)
>
> arm-linux-gnueabihf-gcc (GNU Toolchain for the A-profile Architecture 
> 8.2-2018-08 (arm-rel-8.23)) 8.2.1 20180802
> GNU ld (GNU Toolchain for the A-profile Architecture 8.2-2018-08 
> (arm-rel-8.23)) 2.30.0.20180625
>
> >> > What do you see on the serial console?
> >>
> >> It boots up to the point when it tries to find the emmc, and then it
> >> basically hangs completely (tested with current master):
> >>
> >> U-Boot SPL 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
> >> Trying to boot from MMC1
> >>
> >> U-Boot 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
> >>
> >> CPU:   Freescale i.MX6D rev1.5 996 MHz (running at 792 MHz)
> >> CPU:   Extended Commercial temperature grade (-20C to 105C) at 33C
> >> Reset cause: POR
> >> Board: MX6 HummingBoard2
> >> DRAM:  1 GiB
> >> MMC:   FSL_SDHC: 0, FSL_SDHC: 1
> >> Loading Environment from MMC... *** Warning - bad CRC, using default 
> >> environment
> >>
> >> No panel detected: default to HDMI
> >> Display: HDMI (1024x768)
> >> In:serial
> >> Out:   serial
> >> Err:   serial
> >> ---> hangs
> >
> > Exactly the same as I saw.
>
> Here is what I get at this point:
>
> In:serial
> Out:   serial
> Err:   serial
> Card did not respond to voltage select!
> Net:   FEC
> ...
>
> The error message is from drivers/mmc/mmc.c:mmc_get_op_cond(). The code
> around this point might be the source of your issue. Can you add a few
> prints in mmc_get_op_cond() to pinpoint the call that hangs the code?

Also tried with ARM's pre-built toolchain (same version), and got the
same hang. Looking a bit further, it basically looks up while waiting
the first mmc command to complete:

has_emmc -> mmc_get_op_cond -> mmc_send_if_cond (testing for SD
version 2) -> mmc_send_cmd -> esdhc_send_cmd_common -> while
(!(esdhc_read32(>irqstat) & flags))

And the boot log with MMC_TRACE:

No panel detected: default to HDMI
Display: HDMI (1024x768)
In:serial
Out:   serial
Err:   serial
CMD_SEND:0
ARG  0x
MMC_RSP_NONE
CMD_SEND:8
ARG  0x01AA

The line where it locks up waiting for the command to complete:
http://git.denx.de/?p=u-boot.git;a=blob;f=drivers/mmc/fsl_esdhc.c;h=3cdfa7f5a689789ea2f24da867d7c947ab78c002;hb=HEAD#l455

Unclear why this only happens with this som/soc, maybe hardware/errata
differences?

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Baruch Siach]

Hi Peter, Ricardo,

Peter Robinson writes:
> On Fri, Nov 16, 2018 at 1:06 AM Ricardo Salveti  wrote:
>> On Wed, Nov 14, 2018 at 5:07 AM Baruch Siach  wrote:
>> > On Tue, Nov 13, 2018 at 11:42:44AM -0200, Ricardo Salveti wrote:
>> > > On Thu, Sep 6, 2018 at 12:52 PM Fabio Estevam  wrote:
>> > > > On Thu, Sep 6, 2018 at 12:42 PM, Baruch Siach  
>> > > > wrote:
>> > > >
>> > > > > I tested current master successfully on Hummingboard2 with i.MX6 Solo
>> > > > > (SOM rev 1.5):
>> > > >
>> > > > Thanks for testing.
>> > > >
>> > > > It seems we need more information from Peter about the regression he 
>> > > > reported.
>> > > >
>> > > > It would be helpful if Peter could run a bisect so that we could
>> > > > understand where the regression is coming from.
>> > >
>> > > Finally got the time to test u-boot 2018.09 on my hummingboard 2 and I
>> > > can also confirm the boot issue with imx6q (Hummingboard 2 MicroSOM
>> > > i2eX iMX6D - rev 1.3).
>> > >
>> > > The patch set that introduced this regression was part of another pull
>> > > request, the one that introduces eMMC booting support (from Jon
>> > > Nettleton, e.g. 86e5a7fc13 and 19ed6063a5). After doing some more
>> > > testing it seems that the hang happens when trying to verify if the
>> > > board has eMMC during runtime (has_emmc -> mmc_get_op_cond(mmc)),
>> > > which is not the case for this SOM in particular (and probably why it
>> > > works fine on most rev 1.5-based SOMs, as eMMC is usually available
>> > > there).
>> > >
>> > > Tested with current u-boot master and the issue is still valid.
>> > >
>> > > Jon, did you have any issue when testing that patch set on SOMs
>> > > without eMMC support?
>> >
>> > I tested U-Boot successfully with SOM rev 1.3 (no eMMC) on Hummingboard2, 
>> > as
>> > shown in my previous message on this thread.
>>
>> Indeed, you tested with i.MX6Q, only difference is that mine is iMX6D,
>> but both without eMMC.
>
> I see the issue with a .IMX6Q wirth SOM rev 1.5 (TI wifi, no EMMC) on
> a hummingboard2

I could not reproduce with SOM rev 1.3 (no eMMC) on Hummingboard2.

>> > What toolchain are you using?
>>
>> Using GCC 8.2 from latest OpenEmbedded. Will try building with the
>> version you used to see if I get any different behavior.
>
> gcc 8.2.x from Fedora 29

I am using the ARM (Ltd) provided GNU toolchain version 8.2:

=> version
U-Boot 2018.11 (Nov 18 2018 - 11:22:16 +0200)

arm-linux-gnueabihf-gcc (GNU Toolchain for the A-profile Architecture 
8.2-2018-08 (arm-rel-8.23)) 8.2.1 20180802
GNU ld (GNU Toolchain for the A-profile Architecture 8.2-2018-08 
(arm-rel-8.23)) 2.30.0.20180625

>> > What do you see on the serial console?
>>
>> It boots up to the point when it tries to find the emmc, and then it
>> basically hangs completely (tested with current master):
>>
>> U-Boot SPL 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
>> Trying to boot from MMC1
>>
>> U-Boot 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
>>
>> CPU:   Freescale i.MX6D rev1.5 996 MHz (running at 792 MHz)
>> CPU:   Extended Commercial temperature grade (-20C to 105C) at 33C
>> Reset cause: POR
>> Board: MX6 HummingBoard2
>> DRAM:  1 GiB
>> MMC:   FSL_SDHC: 0, FSL_SDHC: 1
>> Loading Environment from MMC... *** Warning - bad CRC, using default 
>> environment
>>
>> No panel detected: default to HDMI
>> Display: HDMI (1024x768)
>> In:serial
>> Out:   serial
>> Err:   serial
>> ---> hangs
>
> Exactly the same as I saw.

Here is what I get at this point:

In:serial
Out:   serial
Err:   serial
Card did not respond to voltage select!
Net:   FEC
...

The error message is from drivers/mmc/mmc.c:mmc_get_op_cond(). The code
around this point might be the source of your issue. Can you add a few
prints in mmc_get_op_cond() to pinpoint the call that hangs the code?

Thanks,
baruch

-- 
 http://baruch.siach.name/blog/  ~. .~   Tk Open Systems
=}ooO--U--Ooo{=
   - bar...@tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Peter Robinson]

On Fri, Nov 16, 2018 at 1:06 AM Ricardo Salveti  wrote:
>
> Hi Baruch,
>
> On Wed, Nov 14, 2018 at 5:07 AM Baruch Siach  wrote:
> >
> > Hi Ricardo,
> >
> > On Tue, Nov 13, 2018 at 11:42:44AM -0200, Ricardo Salveti wrote:
> > > On Thu, Sep 6, 2018 at 12:52 PM Fabio Estevam  wrote:
> > > > On Thu, Sep 6, 2018 at 12:42 PM, Baruch Siach  wrote:
> > > >
> > > > > I tested current master successfully on Hummingboard2 with i.MX6 Solo
> > > > > (SOM rev 1.5):
> > > >
> > > > Thanks for testing.
> > > >
> > > > It seems we need more information from Peter about the regression he 
> > > > reported.
> > > >
> > > > It would be helpful if Peter could run a bisect so that we could
> > > > understand where the regression is coming from.
> > >
> > > Finally got the time to test u-boot 2018.09 on my hummingboard 2 and I
> > > can also confirm the boot issue with imx6q (Hummingboard 2 MicroSOM
> > > i2eX iMX6D - rev 1.3).
> > >
> > > The patch set that introduced this regression was part of another pull
> > > request, the one that introduces eMMC booting support (from Jon
> > > Nettleton, e.g. 86e5a7fc13 and 19ed6063a5). After doing some more
> > > testing it seems that the hang happens when trying to verify if the
> > > board has eMMC during runtime (has_emmc -> mmc_get_op_cond(mmc)),
> > > which is not the case for this SOM in particular (and probably why it
> > > works fine on most rev 1.5-based SOMs, as eMMC is usually available
> > > there).
> > >
> > > Tested with current u-boot master and the issue is still valid.
> > >
> > > Jon, did you have any issue when testing that patch set on SOMs
> > > without eMMC support?
> >
> > I tested U-Boot successfully with SOM rev 1.3 (no eMMC) on Hummingboard2, as
> > shown in my previous message on this thread.
>
> Indeed, you tested with i.MX6Q, only difference is that mine is iMX6D,
> but both without eMMC.

I see the issue with a .IMX6Q wirth SOM rev 1.5 (TI wifi, no EMMC) on
a hummingboard2

> > What toolchain are you using?
>
> Using GCC 8.2 from latest OpenEmbedded. Will try building with the
> version you used to see if I get any different behavior.

gcc 8.2.x from Fedora 29

> > What do you see on the serial console?
>
> It boots up to the point when it tries to find the emmc, and then it
> basically hangs completely (tested with current master):
>
> U-Boot SPL 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
> Trying to boot from MMC1
>
> U-Boot 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
>
> CPU:   Freescale i.MX6D rev1.5 996 MHz (running at 792 MHz)
> CPU:   Extended Commercial temperature grade (-20C to 105C) at 33C
> Reset cause: POR
> Board: MX6 HummingBoard2
> DRAM:  1 GiB
> MMC:   FSL_SDHC: 0, FSL_SDHC: 1
> Loading Environment from MMC... *** Warning - bad CRC, using default 
> environment
>
> No panel detected: default to HDMI
> Display: HDMI (1024x768)
> In:serial
> Out:   serial
> Err:   serial
> ---> hangs

Exactly the same as I saw.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Ricardo Salveti]

Hi Baruch,

On Wed, Nov 14, 2018 at 5:07 AM Baruch Siach  wrote:
>
> Hi Ricardo,
>
> On Tue, Nov 13, 2018 at 11:42:44AM -0200, Ricardo Salveti wrote:
> > On Thu, Sep 6, 2018 at 12:52 PM Fabio Estevam  wrote:
> > > On Thu, Sep 6, 2018 at 12:42 PM, Baruch Siach  wrote:
> > >
> > > > I tested current master successfully on Hummingboard2 with i.MX6 Solo
> > > > (SOM rev 1.5):
> > >
> > > Thanks for testing.
> > >
> > > It seems we need more information from Peter about the regression he 
> > > reported.
> > >
> > > It would be helpful if Peter could run a bisect so that we could
> > > understand where the regression is coming from.
> >
> > Finally got the time to test u-boot 2018.09 on my hummingboard 2 and I
> > can also confirm the boot issue with imx6q (Hummingboard 2 MicroSOM
> > i2eX iMX6D - rev 1.3).
> >
> > The patch set that introduced this regression was part of another pull
> > request, the one that introduces eMMC booting support (from Jon
> > Nettleton, e.g. 86e5a7fc13 and 19ed6063a5). After doing some more
> > testing it seems that the hang happens when trying to verify if the
> > board has eMMC during runtime (has_emmc -> mmc_get_op_cond(mmc)),
> > which is not the case for this SOM in particular (and probably why it
> > works fine on most rev 1.5-based SOMs, as eMMC is usually available
> > there).
> >
> > Tested with current u-boot master and the issue is still valid.
> >
> > Jon, did you have any issue when testing that patch set on SOMs
> > without eMMC support?
>
> I tested U-Boot successfully with SOM rev 1.3 (no eMMC) on Hummingboard2, as
> shown in my previous message on this thread.

Indeed, you tested with i.MX6Q, only difference is that mine is iMX6D,
but both without eMMC.

> What toolchain are you using?

Using GCC 8.2 from latest OpenEmbedded. Will try building with the
version you used to see if I get any different behavior.

> What do you see on the serial console?

It boots up to the point when it tries to find the emmc, and then it
basically hangs completely (tested with current master):

U-Boot SPL 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)
Trying to boot from MMC1

U-Boot 2018.11+gf6206f8587 (Nov 16 2018 - 00:56:34 +)

CPU:   Freescale i.MX6D rev1.5 996 MHz (running at 792 MHz)
CPU:   Extended Commercial temperature grade (-20C to 105C) at 33C
Reset cause: POR
Board: MX6 HummingBoard2
DRAM:  1 GiB
MMC:   FSL_SDHC: 0, FSL_SDHC: 1
Loading Environment from MMC... *** Warning - bad CRC, using default environment

No panel detected: default to HDMI
Display: HDMI (1024x768)
In:serial
Out:   serial
Err:   serial
---> hangs

Cheers,
-- 
Ricardo Salveti de Araujo
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Baruch Siach]

Hi Ricardo,

On Tue, Nov 13, 2018 at 11:42:44AM -0200, Ricardo Salveti wrote:
> On Thu, Sep 6, 2018 at 12:52 PM Fabio Estevam  wrote:
> > On Thu, Sep 6, 2018 at 12:42 PM, Baruch Siach  wrote:
> >
> > > I tested current master successfully on Hummingboard2 with i.MX6 Solo
> > > (SOM rev 1.5):
> >
> > Thanks for testing.
> >
> > It seems we need more information from Peter about the regression he 
> > reported.
> >
> > It would be helpful if Peter could run a bisect so that we could
> > understand where the regression is coming from.
> 
> Finally got the time to test u-boot 2018.09 on my hummingboard 2 and I
> can also confirm the boot issue with imx6q (Hummingboard 2 MicroSOM
> i2eX iMX6D - rev 1.3).
> 
> The patch set that introduced this regression was part of another pull
> request, the one that introduces eMMC booting support (from Jon
> Nettleton, e.g. 86e5a7fc13 and 19ed6063a5). After doing some more
> testing it seems that the hang happens when trying to verify if the
> board has eMMC during runtime (has_emmc -> mmc_get_op_cond(mmc)),
> which is not the case for this SOM in particular (and probably why it
> works fine on most rev 1.5-based SOMs, as eMMC is usually available
> there).
> 
> Tested with current u-boot master and the issue is still valid.
> 
> Jon, did you have any issue when testing that patch set on SOMs
> without eMMC support?

I tested U-Boot successfully with SOM rev 1.3 (no eMMC) on Hummingboard2, as 
shown in my previous message on this thread.

What toolchain are you using?

What do you see on the serial console?

baruch

-- 
 http://baruch.siach.name/blog/  ~. .~   Tk Open Systems
=}ooO--U--Ooo{=
   - bar...@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Ricardo Salveti]

On Thu, Sep 6, 2018 at 12:52 PM Fabio Estevam  wrote:
>
> Hi Baruch,
>
> On Thu, Sep 6, 2018 at 12:42 PM, Baruch Siach  wrote:
>
> > I tested current master successfully on Hummingboard2 with i.MX6 Solo
> > (SOM rev 1.5):
>
> Thanks for testing.
>
> It seems we need more information from Peter about the regression he reported.
>
> It would be helpful if Peter could run a bisect so that we could
> understand where the regression is coming from.

Finally got the time to test u-boot 2018.09 on my hummingboard 2 and I
can also confirm the boot issue with imx6q (Hummingboard 2 MicroSOM
i2eX iMX6D - rev 1.3).

The patch set that introduced this regression was part of another pull
request, the one that introduces eMMC booting support (from Jon
Nettleton, e.g. 86e5a7fc13 and 19ed6063a5). After doing some more
testing it seems that the hang happens when trying to verify if the
board has eMMC during runtime (has_emmc -> mmc_get_op_cond(mmc)),
which is not the case for this SOM in particular (and probably why it
works fine on most rev 1.5-based SOMs, as eMMC is usually available
there).

Tested with current u-boot master and the issue is still valid.

Jon, did you have any issue when testing that patch set on SOMs
without eMMC support?

Cheers,
-- 
Ricardo Salveti de Araujo
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181112

[Tom Rini]

On Mon, Nov 12, 2018 at 01:03:12PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> still a couple of fixes (colibri_vf could not be built):
> 
> The following changes since commit c5bbfaf05dc8592b479a44df6abaadbab54fec2b:
> 
>   imx8qxp_mek: Disable CONFIG_DISPLAY_CPUINFO (2018-11-07 12:13:45 -0500)
> 
> are available in the Git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181112
> 
> for you to fetch changes up to 43e6f94cbcaf193aeedcf86e85a3ff4c79f66773:
> 
>   imx: mkimage: add size check to the u-boot.imx make target (2018-11-12
> 11:08:53 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181106

[Tom Rini]

On Tue, Nov 06, 2018 at 04:57:46PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> this solves issue reported by coverity for i.MX8:
> 
> The following changes since commit 5ef76e59c12c79d106ebda70b710468aa6bd8b75:
> 
>   Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-11-04
> 08:12:21 -0500)
> 
> are available in the Git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181106
> 
> for you to fetch changes up to 0ea82ba2b082475090e11872ec11409a9fc8d486:
> 
>   MAINTAINERS: add NXP linux team maillist as i.MX reviewer (2018-11-06
> 11:26:21 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181025

[Tom Rini]

On Thu, Oct 25, 2018 at 12:01:10PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx (tag: u-boot-imx-20181024). It contains the
> fixes we discussed:
> 
> - fix build i.MX8 for Travis
> - fix warnings (Warp)
> - fix warning in imx8 DTS
> 
> The following changes since commit 0a60a81ba3860946551cb79aa6486aa076e357f3:
> 
>   Kconfig: sandbox: enable cmd_avb and dependencies (2018-10-07 13:34:19
> -0400)
> 
> are available in the Git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181025
> 
> for you to fetch changes up to 3c28576bb0f0990d699fd330089412e620706941:
> 
>   arm: dts: imx8qxp: fix build warining (2018-10-25 11:48:13 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Tom,

please just wait a little for this. Fix for i.MX8 is still WIP, and now
it is in test. I will send a new PR later.

Thanks,
Stefano

On 24/10/18 15:38, Stefano Babic wrote:
> Hi Tom,
> 
> new PR (and new tag: u-boot-imx-20181024) with the fixes we discussed:
> 
> - fix build i.MX8 for Travis
> - fix warnings (Warp)
> 
> The following changes since commit 0a60a81ba3860946551cb79aa6486aa076e357f3:
> 
>   Kconfig: sandbox: enable cmd_avb and dependencies (2018-10-07 13:34:19
> -0400)
> 
> are available in the Git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181024
> 
> for you to fetch changes up to cf0f7fde7b2cd3944364c1e70a65603cdd3fa889:
> 
>   imx: mkimage: avoid stop CI when required files not exists (2018-10-24
> 13:23:36 +0200)
> 
> 
> Merged imx8 architecture, fix build for imx8 + warnings
> 
> 
> Anatolij Gustschin (3):
>   imx8: cpu: add function for reading FEC MAC from fuse
>   imx8: cpu: add uclass based CPU driver
>   i2c: imx_lpi2c: fix typo and register base address format
> 
> Bernhard Messerklinger (1):
>   serial: mxc: Add match string for i.mx6 quad/dual lite serial
> 
> Breno Matheus Lima (7):
>   doc: imx: reorganize i.MX documentation
>   doc: imx: Move SPD related info to the appropriate doc
>   doc: imx: mkimage: reorganize i.MX mkimage documentation
>   doc: imx: Reorganize i.MX SoC common documentation
>   doc: imx: hab: Reorganize High Assurance Boot documentation
>   doc: imx: misc: Reorganize miscellaneous documentation
>   doc: imx: Improve i.MX documentation naming
> 
> Dan Cimpoca (1):
>   board: ge: bx50v3: fix initialization of i2c bus0
> 
> Fabien Lahoudere (1):
>   board: ge: bx50v3: Change maintainer
> 
> Fabio Estevam (5):
>   pico-imx7d: Make SPL binary fit into 64kB
>   pico-imx7d: Add USB Host support
>   colibri_imx7: prime get_ram_size() using imx_ddr_size()
>   configs: mx7: Remove CONFIG_DBG_MONITOR
>   warp7: MAINTAINERS: Add an entry for warp7_bl33_defconfig
> 
> Ian Ray (2):
>   board: ge: bx50v3: correct LDB clock
>   board: ge: bx50v3: b{4,6}50v3 modeline
> 
> Marcel Ziswiler (1):
>   imx: mx7: fix potential overflow in imx_ddr_size()
> 
> Marek Vasut (3):
>   arm: imx: mx5: Make videoskip available on MX5
>   arm: mx5: Add LDB clock config code
>   arm: mx5: Add M53Menlo board
> 
> Otavio Salvador (2):
>   pico-imx6ul, pico-imx7d: Use eMMC user partition by default
>   pico-imx6ul, pico-imx7d: Enable USB and PXE boot support
> 
> Peng Fan (34):
>   dt-bindings: pinctrl: add i.MX8QXP pads definition
>   dt-bindings: clock: dt-bindings: pinctrl: add i.MX8QXP clocks
> definition
>   dt-bindings: soc: add i.MX8QXP pm and rsrc definition
>   imx8: add scfw macro definition
>   imx: add Kconfig entry for i.MX8QXP
>   arm: build mach-imx for i.MX8
>   arm: global_data: add scu_dev for i.MX8
>   misc: add i.MX8 misc driver
>   misc: imx8: add scfw api impementation
>   imx: boot_mode: Add FLEXSPI boot entry
>   imx8: add imx-regs header file
>   imx8: pins: include i.MX8QXP pin header when CONFIG_IMX8QXP defined
>   imx: add i.MX8 cpu type
>   armv8: add cpu core helper functions
>   imx8: add basic cpu support
>   imx8: add boot device detection
>   imx8: implement mmc_get_env_dev
>   imx8: add mmu and dram related functions
>   imx8: add arch_cpu_init arch_cpu_init_dm
>   imx8: add iomux configuration api
>   imx8: add dummy clock
>   gpio: mxc_gpio: add support for i.MX8
>   pinctrl: Add pinctrl driver for i.MX8
>   power: Add power domain driver for i.MX8
>   clk: imx: add clk driver for i.MX8QXP
>   serial_lpuart: Update lpuart driver to support i.MX8
>   serial: lpuart: support uclass clk api
>   mmc: fsl_esdhc: add uclass clk support
>   arm: dts: introduce dtsi for i.MX8QXP
>   imx: add i.MX8QXP MEK board support
>   tools: add i.MX8/8X image support
>   arm: imx: include imx8image support
>   imx: imx8qxp_mek: update to build image in U-Boot
>   imx: mkimage: avoid stop CI when required files not exists
> 
> Rui Miguel Silva (4):
>   imx: mx7: avoid some initialization if low level is skipped
>   optee: adjust dependencies and default values for dram
>   warp7: include: configs: set skip low level init
>   warp7: configs: add bl33 defconfig
> 
> Sébastien Szymanski (1):
>   ARM: opos6ul: make the board boot again
> 
> Xiaoliang Yang (1):
>   Kconfig: Convert CONFIG_IMX_WATCHDOG to Kconfig
> 
> Ye Li (2):
>   serial: lpuart: Enable RX and TX FIFO
>   fsl_esdhc: Update usdhc driver to support i.MX8
> 
>  arch/arm/Kconfig  |   8 +
>  arch/arm/Makefile |   

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Tom,

new PR (and new tag: u-boot-imx-20181024) with the fixes we discussed:

- fix build i.MX8 for Travis
- fix warnings (Warp)

The following changes since commit 0a60a81ba3860946551cb79aa6486aa076e357f3:

  Kconfig: sandbox: enable cmd_avb and dependencies (2018-10-07 13:34:19
-0400)

are available in the Git repository at:

  git://www.denx.de/git/u-boot-imx.git tags/u-boot-imx-20181024

for you to fetch changes up to cf0f7fde7b2cd3944364c1e70a65603cdd3fa889:

  imx: mkimage: avoid stop CI when required files not exists (2018-10-24
13:23:36 +0200)


Merged imx8 architecture, fix build for imx8 + warnings


Anatolij Gustschin (3):
  imx8: cpu: add function for reading FEC MAC from fuse
  imx8: cpu: add uclass based CPU driver
  i2c: imx_lpi2c: fix typo and register base address format

Bernhard Messerklinger (1):
  serial: mxc: Add match string for i.mx6 quad/dual lite serial

Breno Matheus Lima (7):
  doc: imx: reorganize i.MX documentation
  doc: imx: Move SPD related info to the appropriate doc
  doc: imx: mkimage: reorganize i.MX mkimage documentation
  doc: imx: Reorganize i.MX SoC common documentation
  doc: imx: hab: Reorganize High Assurance Boot documentation
  doc: imx: misc: Reorganize miscellaneous documentation
  doc: imx: Improve i.MX documentation naming

Dan Cimpoca (1):
  board: ge: bx50v3: fix initialization of i2c bus0

Fabien Lahoudere (1):
  board: ge: bx50v3: Change maintainer

Fabio Estevam (5):
  pico-imx7d: Make SPL binary fit into 64kB
  pico-imx7d: Add USB Host support
  colibri_imx7: prime get_ram_size() using imx_ddr_size()
  configs: mx7: Remove CONFIG_DBG_MONITOR
  warp7: MAINTAINERS: Add an entry for warp7_bl33_defconfig

Ian Ray (2):
  board: ge: bx50v3: correct LDB clock
  board: ge: bx50v3: b{4,6}50v3 modeline

Marcel Ziswiler (1):
  imx: mx7: fix potential overflow in imx_ddr_size()

Marek Vasut (3):
  arm: imx: mx5: Make videoskip available on MX5
  arm: mx5: Add LDB clock config code
  arm: mx5: Add M53Menlo board

Otavio Salvador (2):
  pico-imx6ul, pico-imx7d: Use eMMC user partition by default
  pico-imx6ul, pico-imx7d: Enable USB and PXE boot support

Peng Fan (34):
  dt-bindings: pinctrl: add i.MX8QXP pads definition
  dt-bindings: clock: dt-bindings: pinctrl: add i.MX8QXP clocks
definition
  dt-bindings: soc: add i.MX8QXP pm and rsrc definition
  imx8: add scfw macro definition
  imx: add Kconfig entry for i.MX8QXP
  arm: build mach-imx for i.MX8
  arm: global_data: add scu_dev for i.MX8
  misc: add i.MX8 misc driver
  misc: imx8: add scfw api impementation
  imx: boot_mode: Add FLEXSPI boot entry
  imx8: add imx-regs header file
  imx8: pins: include i.MX8QXP pin header when CONFIG_IMX8QXP defined
  imx: add i.MX8 cpu type
  armv8: add cpu core helper functions
  imx8: add basic cpu support
  imx8: add boot device detection
  imx8: implement mmc_get_env_dev
  imx8: add mmu and dram related functions
  imx8: add arch_cpu_init arch_cpu_init_dm
  imx8: add iomux configuration api
  imx8: add dummy clock
  gpio: mxc_gpio: add support for i.MX8
  pinctrl: Add pinctrl driver for i.MX8
  power: Add power domain driver for i.MX8
  clk: imx: add clk driver for i.MX8QXP
  serial_lpuart: Update lpuart driver to support i.MX8
  serial: lpuart: support uclass clk api
  mmc: fsl_esdhc: add uclass clk support
  arm: dts: introduce dtsi for i.MX8QXP
  imx: add i.MX8QXP MEK board support
  tools: add i.MX8/8X image support
  arm: imx: include imx8image support
  imx: imx8qxp_mek: update to build image in U-Boot
  imx: mkimage: avoid stop CI when required files not exists

Rui Miguel Silva (4):
  imx: mx7: avoid some initialization if low level is skipped
  optee: adjust dependencies and default values for dram
  warp7: include: configs: set skip low level init
  warp7: configs: add bl33 defconfig

Sébastien Szymanski (1):
  ARM: opos6ul: make the board boot again

Xiaoliang Yang (1):
  Kconfig: Convert CONFIG_IMX_WATCHDOG to Kconfig

Ye Li (2):
  serial: lpuart: Enable RX and TX FIFO
  fsl_esdhc: Update usdhc driver to support i.MX8

 arch/arm/Kconfig  |   8 +
 arch/arm/Makefile |   2 +-
 arch/arm/dts/Makefile |   2 +
 arch/arm/dts/fsl-imx8-ca35.dtsi   |  66 +++
 arch/arm/dts/fsl-imx8dx.dtsi  | 499
+++
 arch/arm/dts/fsl-imx8dxp.dtsi |  11 ++
 arch/arm/dts/fsl-imx8qxp-mek.dts  | 246
++
 arch/arm/dts/fsl-imx8qxp.dtsi   

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Tue, Oct 23, 2018 at 04:42:53PM +0200, Stefano Babic wrote:
> Hi Tom,
> 
> On 23/10/18 16:29, Tom Rini wrote:
> > On Tue, Oct 23, 2018 at 09:52:58AM +0200, Stefano Babic wrote:
> > 
> >> Hi Tom,
> >>
> >> please pull from u-boot-imx (tag: u-boot-imx-20181023) thanks !
> >>
> >> The tree contains fixes for i.MX6 and improvement in docs, but the most
> >> important part is the introduction of the i.MX8 architecture (thanks to
> >> Peng and Antolji for their work !). Build on Travis reported no error,
> >> but I get a build issue with the VF610 boards - this will be require a
> >> fix in next days. i.MX8 can be built following exactly the instruction
> >> of the README file (NXP firmware requires to acknowledge the EULA).
> > 
> > I know the i.MX8 stuff has been kicking around for a while so.. OK, yes,
> > I won't be too upset about pushing it so late in the cycle.  But, my
> > travis builds do see an error:
> > https://travis-ci.org/trini/u-boot/jobs/445069389#L1038 which is "Fail
> > open first container file ahab-container.img" for imx8qxp_mek and I also
> > see that locally.
> 
> I know: to build i.MX8, we need to download external firmware. In fact,
> we have to provide the files according to board/freescale/mx8mq_evk/README.
> 
> Near "imx-atf" (this is not a problem, loaded and built), we need two
> additional "closed" firmware, that is firmware-imx (currently, 7.6) and
> imx-sc-firmware. They can be downloaded but they underly the acknowledge
> of NXP's EULA. They are self extracting scripts and we could hack it to
> have an automatic build, but I *guess* this breaks the license. I do not
> know if we are allowed to do this, and both of these firmwares do not
> provide a way to "implicitely" skip the EULA as we do in OE by setting a
> variable in local.conf.
> 
> >  This feels like the same class of error that we have
> > to deal with on other AArch64 platforms, namely of throwing a user
> > visible error that what they just built will not boot due to lack of
> > other files.
> 
> I am unsure: mkimage packs all files together as part of the build, when
> they are available. User cannot even try to boot, because the build
> stops if these files are not available. The error at mkimage step is
> saying us that build is not complete due to missing files (atf,
> firmware-imx and imx-sc-firmware).

Yup, this is the same problem we have in other platforms.  Take a look
at board/sunxi/mksunxi_fit_atf.sh or tools/k3_fit_atf.sh for what we do
on some other platforms so that CI build completes but it's clear to the
end user that the binary will not work and what they need to do / read
to get a functional build.  There's also
arch/arm/mach-omap2/config_secure.mk as an example of all the fun we
have for building the secure TI platforms that more closely follow the
problem you have here, namely of needing EULA'd packages (and in those
cases, also NDA, iirc) installed to complete a functional build.  I
think you'll need to do something closer to the _fit_atf.sh scripts
in this case, to check for required blobs and if not found, not call
mkimage and echo something loud and visible to the user.

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Tom,

On 23/10/18 16:29, Tom Rini wrote:
> On Tue, Oct 23, 2018 at 09:52:58AM +0200, Stefano Babic wrote:
> 
>> Hi Tom,
>>
>> please pull from u-boot-imx (tag: u-boot-imx-20181023) thanks !
>>
>> The tree contains fixes for i.MX6 and improvement in docs, but the most
>> important part is the introduction of the i.MX8 architecture (thanks to
>> Peng and Antolji for their work !). Build on Travis reported no error,
>> but I get a build issue with the VF610 boards - this will be require a
>> fix in next days. i.MX8 can be built following exactly the instruction
>> of the README file (NXP firmware requires to acknowledge the EULA).
> 
> I know the i.MX8 stuff has been kicking around for a while so.. OK, yes,
> I won't be too upset about pushing it so late in the cycle.  But, my
> travis builds do see an error:
> https://travis-ci.org/trini/u-boot/jobs/445069389#L1038 which is "Fail
> open first container file ahab-container.img" for imx8qxp_mek and I also
> see that locally.

I know: to build i.MX8, we need to download external firmware. In fact,
we have to provide the files according to board/freescale/mx8mq_evk/README.

Near "imx-atf" (this is not a problem, loaded and built), we need two
additional "closed" firmware, that is firmware-imx (currently, 7.6) and
imx-sc-firmware. They can be downloaded but they underly the acknowledge
of NXP's EULA. They are self extracting scripts and we could hack it to
have an automatic build, but I *guess* this breaks the license. I do not
know if we are allowed to do this, and both of these firmwares do not
provide a way to "implicitely" skip the EULA as we do in OE by setting a
variable in local.conf.

>  This feels like the same class of error that we have
> to deal with on other AArch64 platforms, namely of throwing a user
> visible error that what they just built will not boot due to lack of
> other files.

I am unsure: mkimage packs all files together as part of the build, when
they are available. User cannot even try to boot, because the build
stops if these files are not available. The error at mkimage step is
saying us that build is not complete due to missing files (atf,
firmware-imx and imx-sc-firmware).

> 
> Also:
> $ ./tools/genboardscfg.py -o /dev/null
> WARNING: no status info for 'warp7_bl33'
> WARNING: no maintainers for 'warp7_bl33'

I have not seen, thanks, this must be fixed.

> 
> which I would have just fixed-up, but since we need to address that
> imx8qxp_mek problem, please update the appropriate MAINTAINERS file too
> for warp7_bl33.  Thanks!

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Tue, Oct 23, 2018 at 09:52:58AM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx (tag: u-boot-imx-20181023) thanks !
> 
> The tree contains fixes for i.MX6 and improvement in docs, but the most
> important part is the introduction of the i.MX8 architecture (thanks to
> Peng and Antolji for their work !). Build on Travis reported no error,
> but I get a build issue with the VF610 boards - this will be require a
> fix in next days. i.MX8 can be built following exactly the instruction
> of the README file (NXP firmware requires to acknowledge the EULA).

I know the i.MX8 stuff has been kicking around for a while so.. OK, yes,
I won't be too upset about pushing it so late in the cycle.  But, my
travis builds do see an error:
https://travis-ci.org/trini/u-boot/jobs/445069389#L1038 which is "Fail
open first container file ahab-container.img" for imx8qxp_mek and I also
see that locally.  This feels like the same class of error that we have
to deal with on other AArch64 platforms, namely of throwing a user
visible error that what they just built will not boot due to lack of
other files.

Also:
$ ./tools/genboardscfg.py -o /dev/null
WARNING: no status info for 'warp7_bl33'
WARNING: no maintainers for 'warp7_bl33'

which I would have just fixed-up, but since we need to address that
imx8qxp_mek problem, please update the appropriate MAINTAINERS file too
for warp7_bl33.  Thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Wed, Sep 19, 2018 at 05:16:17PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx with following changes:
> 
> - changes in pico-* boards
> - fix imx6ull pinmux
> 
> Thanks,
> Stefano
> 
> 
> The following changes since commit b57f1895b669e3fbdca486e7c40dfea00f22bb93:
> 
>   Merge git://git.denx.de/u-boot-x86 (2018-09-17 08:04:25 -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 f7df8932ccaec97c8c422de8adadb3a0491543ed:
> 
>   pico-imx7d: Add a new defconfig with PICO-Hobbit selected (2018-09-18
> 16:55:21 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Peter Robinson]

> > I tested current master successfully on Hummingboard2 with i.MX6 Solo
> > (SOM rev 1.5):
>
> Thanks for testing.
>
> It seems we need more information from Peter about the regression he reported.
>
> It would be helpful if Peter could run a bisect so that we could
> understand where the regression is coming from.

We're using gcc 8.2 and binutils 2.31 and there have been a few
interesting bits there on some other platforms, I wonder if something
like that is coming along here with the latest toolchains. I'll try
building with Fedora 27 (gcc 7.3.1 / binutils 2.29) to rule them out
first.

P
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Fabio Estevam]

Hi Baruch,

On Thu, Sep 6, 2018 at 12:42 PM, Baruch Siach  wrote:

> I tested current master successfully on Hummingboard2 with i.MX6 Solo
> (SOM rev 1.5):

Thanks for testing.

It seems we need more information from Peter about the regression he reported.

It would be helpful if Peter could run a bisect so that we could
understand where the regression is coming from.

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

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Baruch Siach]

Hi Fabio,

Fabio Estevam writes:
> Hi Peter,
>
> [Adding Jon and Baruch]
>
> On Thu, Sep 6, 2018 at 9:37 AM, Peter Robinson  wrote:
>
>> It seems like the Wandboard Quad revB I have also works fine so It
>> might just be a regression in the mx6cuboxi target.
>
> I don't have access to a hummingboard2 to investigate it.
>
> Does this issue also happen on a mx6q cuboxi?
>
> Are you able to run a bisect?

I tested current master successfully on Hummingboard2 with i.MX6 Solo
(SOM rev 1.5):

U-Boot SPL 2018.09-rc3-00026-g4cdeda511f80 (Sep 06 2018 - 18:30:30 +0300)
Trying to boot from MMC1


U-Boot 2018.09-rc3-00026-g4cdeda511f80 (Sep 06 2018 - 18:30:30 +0300)

CPU:   Freescale i.MX6SOLO rev1.3 996 MHz (running at 792 MHz)
CPU:   Commercial temperature grade (0C to 95C) at 38C
Reset cause: POR
Board: MX6 HummingBoard2 (som rev 1.5)
DRAM:  512 MiB
MMC:   FSL_SDHC: 0, FSL_SDHC: 1
Loading Environment from MMC... *** Warning - bad CRC, using default environment

No panel detected: default to HDMI
Display: HDMI (1024x768)
In:serial
Out:   serial
Err:   serial
Net:   FEC
Error: FEC address not set.

Hit any key to stop autoboot:  0
=>

... and with i.MX6Q (SOM rev 1.3):

U-Boot SPL 2018.09-rc3-00026-g4cdeda511f80 (Sep 06 2018 - 18:30:30 +0300)
Trying to boot from MMC1


U-Boot 2018.09-rc3-00026-g4cdeda511f80 (Sep 06 2018 - 18:30:30 +0300)

CPU:   Freescale i.MX6Q rev1.5 996 MHz (running at 792 MHz)
CPU:   Extended Commercial temperature grade (-20C to 105C) at 27C
Reset cause: POR
Board: MX6 HummingBoard2
DRAM:  2 GiB
MMC:   FSL_SDHC: 0, FSL_SDHC: 1
Loading Environment from MMC... *** Warning - bad CRC, using default environment

No panel detected: default to HDMI
Display: HDMI (1024x768)
In:serial
Out:   serial
Err:   serial
Card did not respond to voltage select!
Net:   FEC
Hit any key to stop autoboot:  0
=>

I don't have handy at the moment the exact same combination of HB2 with
i.MX6Q SOM rev 1.5.

I built mx6cuboxi_defconfig with no changes using Linaro GCC 7.3-2018.05.

baruch

--
 http://baruch.siach.name/blog/  ~. .~   Tk Open Systems
=}ooO--U--Ooo{=
   - bar...@tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Fabio Estevam]

Hi Peter,

[Adding Jon and Baruch]

On Thu, Sep 6, 2018 at 9:37 AM, Peter Robinson  wrote:

> It seems like the Wandboard Quad revB I have also works fine so It
> might just be a regression in the mx6cuboxi target.

I don't have access to a hummingboard2 to investigate it.

Does this issue also happen on a mx6q cuboxi?

Are you able to run a bisect?

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

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Peter Robinson]

Hi Stefano,
> >> a new PR. I dropped the patches causing breakages on PowerPC boards, as
> >> well as the issue on GE boards. It ran on travis successfully.
> >>
> >>
> >> The following changes since commit 
> >> 11ed312896c5f5814064c5d45dcb2f53dc121437:
> >>
> >>   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 c1d1543ebc6e1fb026d0d7ac96d865faa7567555:
> >>
> >>   mx7dsabresd: Add the qspi target to the list of supported defconfigs
> >> (2018-09-04 08:47:23 +0200)
> >>
> >> 
> >> Alex Kiernan (1):
> >>   Cleanup CONFIG_BOOTDELAY on cl-som-imx7
> >>
> >> Anson Huang (3):
> >>   imx: mx7: psci: improve cpu hotplug flow
> >>   imx: mx7: add gpc initialization for low power mode
> >>   imx: mx7: add system suspend/resume support
> >>
> >> Fabio Estevam (1):
> >>   mx7dsabresd: Add the qspi target to the list of supported defconfigs
> >>
> >> Martin Kaiser (1):
> >>   watchdog: mx25: use the imx_watchdog driver for mx25
> >>
> >> Stefan Agner (2):
> >>   board: toradex: common: fail gracefully on missing NAND chip
> >>   colibri_imx7_emmc: add Colibri iMX7D 1GB (eMMC) module support
> >>
> >> Stefano Babic (1):
> >>   imx: missing CONFIG_MII in mx7dsabresd_qspi_defconfig
> >>
> >> Ye Li (6):
> >>   imx: imx6sx-sdb: Enable DM QSPI driver
> >>   imx: imx6sx-sabreauto: convert to use DM QSPI driver
> >>   imx: imx7d-sdb: Add DM QSPI support
> >>   dts: imx6ul: Update alias to support DM
> >>   dts: imx6ul_evk: Add DTS files for 14x14 EVK and 9x9 EVK boards
> >>   imx: imx6ul_evk: Enable DM driver for iMX6UL EVK u-boot
> >
> > Are you aware of any issues on some imx6 devices in 2018.09?
>
> No.
>
> > The
> > hummingboard2 (imx6q) hangs for me just after the output below, even
> > with the above PR, it works fine with a imx6sx device (udoo neo). I'm
> > sure I've seen something on the list that might be related but I can't
> > find it.
>
> :-(
>
> >
> > Sorry for picking this up so late in the cycle.
>
> I can just try to test on another mx6q boar if I get enough time, but of
> course this is not a great test.

It seems like the Wandboard Quad revB I have also works fine so It
might just be a regression in the mx6cuboxi target.

P
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Peter,

On 06/09/2018 11:23, Peter Robinson wrote:
> Hi Stefano,
> 
>> a new PR. I dropped the patches causing breakages on PowerPC boards, as
>> well as the issue on GE boards. It ran on travis successfully.
>>
>>
>> The following changes since commit 11ed312896c5f5814064c5d45dcb2f53dc121437:
>>
>>   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 c1d1543ebc6e1fb026d0d7ac96d865faa7567555:
>>
>>   mx7dsabresd: Add the qspi target to the list of supported defconfigs
>> (2018-09-04 08:47:23 +0200)
>>
>> 
>> Alex Kiernan (1):
>>   Cleanup CONFIG_BOOTDELAY on cl-som-imx7
>>
>> Anson Huang (3):
>>   imx: mx7: psci: improve cpu hotplug flow
>>   imx: mx7: add gpc initialization for low power mode
>>   imx: mx7: add system suspend/resume support
>>
>> Fabio Estevam (1):
>>   mx7dsabresd: Add the qspi target to the list of supported defconfigs
>>
>> Martin Kaiser (1):
>>   watchdog: mx25: use the imx_watchdog driver for mx25
>>
>> Stefan Agner (2):
>>   board: toradex: common: fail gracefully on missing NAND chip
>>   colibri_imx7_emmc: add Colibri iMX7D 1GB (eMMC) module support
>>
>> Stefano Babic (1):
>>   imx: missing CONFIG_MII in mx7dsabresd_qspi_defconfig
>>
>> Ye Li (6):
>>   imx: imx6sx-sdb: Enable DM QSPI driver
>>   imx: imx6sx-sabreauto: convert to use DM QSPI driver
>>   imx: imx7d-sdb: Add DM QSPI support
>>   dts: imx6ul: Update alias to support DM
>>   dts: imx6ul_evk: Add DTS files for 14x14 EVK and 9x9 EVK boards
>>   imx: imx6ul_evk: Enable DM driver for iMX6UL EVK u-boot
> 
> Are you aware of any issues on some imx6 devices in 2018.09?

No.

> The
> hummingboard2 (imx6q) hangs for me just after the output below, even
> with the above PR, it works fine with a imx6sx device (udoo neo). I'm
> sure I've seen something on the list that might be related but I can't
> find it.

:-(

> 
> Sorry for picking this up so late in the cycle.

I can just try to test on another mx6q boar if I get enough time, but of
course this is not a great test.

Regards,
Stefano

> 
> Peter
> 
> U-Boot SPL 2018.09-rc3 (Sep 05 2018 - 20:28:15 +)
> Trying to boot from MMC1
> 
> 
> U-Boot 2018.09-rc3 (Sep 05 2018 - 20:28:15 +)
> 
> CPU:   Freescale i.MX6Q rev1.5 996 MHz (running at 792 MHz)
> CPU:   Extended Commercial temperature grade (-20C to 105C) at 25C
> Reset cause: POR
> Board: MX6 HummingBoard2 (som rev 1.5)
> DRAM:  2 GiB
> MMC:   FSL_SDHC: 0, FSL_SDHC: 1
> Loading Environment from MMC... *** Warning - bad CRC, using default 
> environment
> 
> No panel detected: default to HDMI
> Display: HDMI (1024x768)
> In:serial
> Out:   serial
> Err:   serial
> 


-- 
=
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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Peter Robinson]

Hi Stefano,

> a new PR. I dropped the patches causing breakages on PowerPC boards, as
> well as the issue on GE boards. It ran on travis successfully.
>
>
> The following changes since commit 11ed312896c5f5814064c5d45dcb2f53dc121437:
>
>   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 c1d1543ebc6e1fb026d0d7ac96d865faa7567555:
>
>   mx7dsabresd: Add the qspi target to the list of supported defconfigs
> (2018-09-04 08:47:23 +0200)
>
> 
> Alex Kiernan (1):
>   Cleanup CONFIG_BOOTDELAY on cl-som-imx7
>
> Anson Huang (3):
>   imx: mx7: psci: improve cpu hotplug flow
>   imx: mx7: add gpc initialization for low power mode
>   imx: mx7: add system suspend/resume support
>
> Fabio Estevam (1):
>   mx7dsabresd: Add the qspi target to the list of supported defconfigs
>
> Martin Kaiser (1):
>   watchdog: mx25: use the imx_watchdog driver for mx25
>
> Stefan Agner (2):
>   board: toradex: common: fail gracefully on missing NAND chip
>   colibri_imx7_emmc: add Colibri iMX7D 1GB (eMMC) module support
>
> Stefano Babic (1):
>   imx: missing CONFIG_MII in mx7dsabresd_qspi_defconfig
>
> Ye Li (6):
>   imx: imx6sx-sdb: Enable DM QSPI driver
>   imx: imx6sx-sabreauto: convert to use DM QSPI driver
>   imx: imx7d-sdb: Add DM QSPI support
>   dts: imx6ul: Update alias to support DM
>   dts: imx6ul_evk: Add DTS files for 14x14 EVK and 9x9 EVK boards
>   imx: imx6ul_evk: Enable DM driver for iMX6UL EVK u-boot

Are you aware of any issues on some imx6 devices in 2018.09? The
hummingboard2 (imx6q) hangs for me just after the output below, even
with the above PR, it works fine with a imx6sx device (udoo neo). I'm
sure I've seen something on the list that might be related but I can't
find it.

Sorry for picking this up so late in the cycle.

Peter

U-Boot SPL 2018.09-rc3 (Sep 05 2018 - 20:28:15 +)
Trying to boot from MMC1


U-Boot 2018.09-rc3 (Sep 05 2018 - 20:28:15 +)

CPU:   Freescale i.MX6Q rev1.5 996 MHz (running at 792 MHz)
CPU:   Extended Commercial temperature grade (-20C to 105C) at 25C
Reset cause: POR
Board: MX6 HummingBoard2 (som rev 1.5)
DRAM:  2 GiB
MMC:   FSL_SDHC: 0, FSL_SDHC: 1
Loading Environment from MMC... *** Warning - bad CRC, using default environment

No panel detected: default to HDMI
Display: HDMI (1024x768)
In:serial
Out:   serial
Err:   serial
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Tue, Sep 04, 2018 at 09:11:50AM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> a new PR. I dropped the patches causing breakages on PowerPC boards, as
> well as the issue on GE boards. It ran on travis successfully.
> 
> 
> The following changes since commit 11ed312896c5f5814064c5d45dcb2f53dc121437:
> 
>   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 c1d1543ebc6e1fb026d0d7ac96d865faa7567555:
> 
>   mx7dsabresd: Add the qspi target to the list of supported defconfigs
> (2018-09-04 08:47:23 +0200)

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

On 02/09/2018 16:15, Tom Rini wrote:
> On Sun, Sep 02, 2018 at 03:20:25PM +0200, Sébastien Szymanski wrote:
> 
>> Hi Tom,
>>
>>> On 1 Sep 2018, at 13:37, Tom Rini  wrote:
>>>
>>> - So I start reading the whole diff and I see:
>>> commit 8e00d802e402d2a6734a88ebeb77a70efcfc354c
>>> Author: Sébastien 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.
>>>
>>
>> :(
>>
>> I don’t understand your comment. What -u-boot.dtsi file is
>> automatically included ? I have put all the U-Boot specific DTS
>> changes in imx6ul-opos6ul-u-boot.dtsi and imx6u-opos6uldev-u-boot.dts
>> files.
> 
> Take a look at the u_boot_dtsi_options logic in scripts/Makefile.lib for
> all of the possible dtsi files that will be automatically included if
> found.  This is so that end the end for a given board you should be able
> to drop in the dts file(s) from Linux, create a -u-boot.dtsi and then
> have things work, no changes to the upstream files required nor
> specifying an otherwise unusual name.
> 

I revert the patch on u-boot.imx to get a clean build again that can be
accepted by Tom.

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

On 02/09/2018 19:11, Tom Rini wrote:
> On Sun, Sep 02, 2018 at 07:06:06PM +0200, Stefano Babic wrote:
>>
>>
>> On 01/09/2018 13:37, Tom Rini wrote:
>>> On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:
>>>
 Hi Tom,

 please pull from u-boot-imx, thanks !

 The following changes since commit 
 11ed312896c5f5814064c5d45dcb2f53dc121437:

   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)

>>>
>>> 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
> 
> That's odd.  Are you building the world, or just arm? 

I am building i.MX, that could be the cause:

./tools/buildman/buildman imx mx25 mx27 mx31 mx35 mxs mx5 mx6 mx7 vf610


> I noticed this
> since my non-travis world build got stuck and then confirmed it just
> trying to build that single PowerPC board.

ok

> 
>>> - 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.
> 
> It needs to be a rejection error, I try and remember to re-sync all of
> the defconfigs at least before the final release and if I'm doing it
> right, before the -rcs too.

ok

> 
>>> - So I start reading the whole diff and I see:
>>> commit 8e00d802e402d2a6734a88ebeb77a70efcfc354c
>>> Author: Sébastien 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.
>>
>> I have to understand why my build was successful...
> 
> It's not a fatal problem, it's just not doing things the right way.
> 
>>> - 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'
>>>
>>
>> Note this, they will be fixed.
> 
> Fabio posted a patch for this one BTW.

Thanks, I pick it up.

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Sun, Sep 02, 2018 at 07:06:06PM +0200, Stefano Babic wrote:
> 
> 
> On 01/09/2018 13:37, Tom Rini wrote:
> > On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:
> > 
> >> Hi Tom,
> >>
> >> please pull from u-boot-imx, thanks !
> >>
> >> The following changes since commit 
> >> 11ed312896c5f5814064c5d45dcb2f53dc121437:
> >>
> >>   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)
> >>
> > 
> > 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

That's odd.  Are you building the world, or just arm?  I noticed this
since my non-travis world build got stuck and then confirmed it just
trying to build that single PowerPC board.

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

It needs to be a rejection error, I try and remember to re-sync all of
the defconfigs at least before the final release and if I'm doing it
right, before the -rcs too.

> > - So I start reading the whole diff and I see:
> > commit 8e00d802e402d2a6734a88ebeb77a70efcfc354c
> > Author: Sébastien 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.
> 
> I have to understand why my build was successful...

It's not a fatal problem, it's just not doing things the right way.

> > - 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'
> > 
> 
> Note this, they will be fixed.

Fabio posted a patch for this one BTW.

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

On 01/09/2018 13:37, Tom Rini wrote:
> On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:
> 
>> Hi Tom,
>>
>> please pull from u-boot-imx, thanks !
>>
>> The following changes since commit 11ed312896c5f5814064c5d45dcb2f53dc121437:
>>
>>   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)
>>
> 
> 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ébastien 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.

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

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

Note this, they will be fixed.

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

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Sun, Sep 02, 2018 at 03:20:25PM +0200, Sébastien Szymanski wrote:

> Hi Tom,
> 
> > On 1 Sep 2018, at 13:37, Tom Rini  wrote:
> > 
> > - So I start reading the whole diff and I see:
> > commit 8e00d802e402d2a6734a88ebeb77a70efcfc354c
> > Author: Sébastien 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.
> > 
> 
> :(
> 
> I don’t understand your comment. What -u-boot.dtsi file is
> automatically included ? I have put all the U-Boot specific DTS
> changes in imx6ul-opos6ul-u-boot.dtsi and imx6u-opos6uldev-u-boot.dts
> files.

Take a look at the u_boot_dtsi_options logic in scripts/Makefile.lib for
all of the possible dtsi files that will be automatically included if
found.  This is so that end the end for a given board you should be able
to drop in the dts file(s) from Linux, create a -u-boot.dtsi and then
have things work, no changes to the upstream files required nor
specifying an otherwise unusual name.

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Sébastien Szymanski]

Hi Tom,

> On 1 Sep 2018, at 13:37, Tom Rini  wrote:
> 
> - So I start reading the whole diff and I see:
> commit 8e00d802e402d2a6734a88ebeb77a70efcfc354c
> Author: Sébastien 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.
> 

:(

I don’t understand your comment. What -u-boot.dtsi file is automatically 
included ? I have put all the U-Boot specific DTS changes in 
imx6ul-opos6ul-u-boot.dtsi and imx6u-opos6uldev-u-boot.dts files.

Regards, 

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

--
Sébastien Szymanski
Software engineer, Armadeus Systems
Tel: +33 (0)9 72 29 41 44
Fax: +33 (0)9 72 28 79 26
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit 11ed312896c5f5814064c5d45dcb2f53dc121437:
> 
>   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)
> 

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

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

On 31/08/2018 15:07, Tom Rini wrote:
> On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:
>> Hi Tom,
>>
>> please pull from u-boot-imx, thanks !
>>
>> The following changes since commit 11ed312896c5f5814064c5d45dcb2f53dc121437:
>>
>>   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)
>>
>> 
>> Alex Kiernan (1):
>>   Cleanup CONFIG_BOOTDELAY on cl-som-imx7
>>
>> Anson Huang (3):
>>   imx: mx7: psci: improve cpu hotplug flow
>>   imx: mx7: add gpc initialization for low power mode
>>   imx: mx7: add system suspend/resume support
>>
>> Denis Zalevskiy (6):
>>   board: ge: Remove EEPROM bus param from read_vpd()
>>   board: ge: Move VPD EEPROM configuration to the defconfig
>>   bootcount: i2c: Add bus switching to the I2C bootcount driver
>>   bootcount: Configure length limit for I2C bootcount
>>   board: ge: Move VPD reading to the vpd_reader
>>   board: ge: Store bootcount in EEPROM on PPD and Bx50v3
>>
>> Martin Kaiser (1):
>>   watchdog: mx25: use the imx_watchdog driver for mx25
>>
>> Stefan Agner (2):
>>   board: toradex: common: fail gracefully on missing NAND chip
>>   colibri_imx7_emmc: add Colibri iMX7D 1GB (eMMC) module support
>>
>> Stefano Babic (2):
>>   Merge branch 'master' of git://git.denx.de/u-boot into master
>>   imx: missing CONFIG_MII in mx7dsabresd_qspi_defconfig
>>
>> Sébastien Szymanski (1):
>>   ARM: opos6ul: make the board boot again
>>
>> Ye Li (6):
>>   imx: imx6sx-sdb: Enable DM QSPI driver
>>   imx: imx6sx-sabreauto: convert to use DM QSPI driver
>>   imx: imx7d-sdb: Add DM QSPI support
>>   dts: imx6ul: Update alias to support DM
>>   dts: imx6ul_evk: Add DTS files for 14x14 EVK and 9x9 EVK boards
>>   imx: imx6ul_evk: Enable DM driver for iMX6UL EVK u-boot
>>
>>  arch/arm/dts/Makefile |   7 +-
>>  arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi |  16 +
>>  arch/arm/dts/imx6sx-sabreauto.dts |  40 ++
>>  arch/arm/dts/imx6sx-sdb-u-boot.dtsi   |  16 +
>>  arch/arm/dts/imx6sx.dtsi  |  12 +-
>>  arch/arm/dts/imx6ul-14x14-evk-u-boot.dtsi |  10 +
>>  arch/arm/dts/imx6ul-14x14-evk.dts | 427 +++
>>  arch/arm/dts/imx6ul-9x9-evk-u-boot.dtsi   |  10 +
>>  arch/arm/dts/imx6ul-9x9-evk.dts   | 471
>> +
>>  arch/arm/dts/imx6ul-opos6ul-u-boot.dtsi   |  28 ++
>>  arch/arm/dts/imx6ul-opos6ul.dtsi  |   2 -
>>  arch/arm/dts/imx6ul-opos6uldev-u-boot.dts |  25 ++
>>  arch/arm/dts/imx6ul-opos6uldev.dts|   2 -
>>  arch/arm/dts/imx6ul.dtsi  |  13 +-
>>  arch/arm/dts/imx7d-sdb-qspi-u-boot.dtsi   |  10 +
>>  arch/arm/dts/imx7d-sdb-qspi.dts   |  44 ++
>>  arch/arm/dts/imx7d-sdb.dts|   6 +-
>>  arch/arm/dts/imx7d.dtsi   |  12 +
>>  arch/arm/dts/imx7s.dtsi   |  22 +-
>>  arch/arm/include/asm/arch-mx25/imx-regs.h |   1 +
>>  arch/arm/mach-imx/mx7/Makefile|   2 +-
>>  arch/arm/mach-imx/mx7/psci-mx7.c  | 472
>> +-
>>  arch/arm/mach-imx/mx7/psci-suspend.S  |  67 +++
>>  arch/arm/mach-imx/mx7/soc.c   | 103 +
>>  board/freescale/mx6sxsabreauto/mx6sxsabreauto.c   |  24 --
>>  board/freescale/mx6sxsabresd/mx6sxsabresd.c   |  25 --
>>  board/freescale/mx6ul_14x14_evk/mx6ul_14x14_evk.c | 208 ++
>>  board/freescale/mx7dsabresd/mx7dsabresd.c |  16 -
>>  board/ge/bx50v3/Kconfig   |   2 +
>>  board/ge/bx50v3/bx50v3.c  |  57 +--
>>  board/ge/common/Kconfig   |  14 +
>>  board/ge/common/vpd_reader.c  |  57 ++-
>>  board/ge/common/vpd_reader.h  |  16 +-
>>  board/ge/mx53ppd/Kconfig  |   2 +
>>  board/ge/mx53ppd/mx53ppd.c|  50 +--
>>  board/toradex/colibri_imx7/Kconfig|  42 +-
>>  board/toradex/colibri_imx7/MAINTAINERS|   4 +
>>  board/toradex/colibri_imx7/colibri_imx7.c |  41 +-
>>  board/toradex/common/tdx-cfg-block.c  |   7 +-
>>  configs/ge_bx50v3_defconfig   |  27 +-
>>  configs/mx53ppd_defconfig |  29 +-
>>  configs/mx6sxsabreauto_defconfig  |   2 +
>>  configs/mx6sxsabresd_defconfig|   7 +
>>  

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Otavio Salvador]

On Fri, Aug 31, 2018 at 4:21 PM Stefano Babic  wrote:
> On 31/08/2018 20:53, Otavio Salvador wrote:
> > On Fri, Aug 31, 2018 at 10:08 AM Tom Rini  wrote:
> >> On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:
> >>> please pull from u-boot-imx, thanks !
> >
> > We sent few patches for PICO and we'd like to get them merged for next
> > release if possible. Is it possible for you to review them and see if
> > they are fine for merging or need adjustments?
>
> I saw your patches, but Tom is already complaining that I have sent a PR
> with more changes as he expected. You just sent your patchset yesterday
> and, even if they are just for pico-imx6ul, I will pick them after release.

Being a board specific I really see no high right but postponing it,
is fine for sure. If possible, provide some feedback so I can rework
anything need.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Otavio,

On 31/08/2018 20:53, Otavio Salvador wrote:
> Hello Stefano,
> 
> On Fri, Aug 31, 2018 at 10:08 AM Tom Rini  wrote:
>> On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:
>>> please pull from u-boot-imx, thanks !
> 
> We sent few patches for PICO and we'd like to get them merged for next
> release if possible. Is it possible for you to review them and see if
> they are fine for merging or need adjustments?

I saw your patches, but Tom is already complaining that I have sent a PR
with more changes as he expected. You just sent your patchset yesterday
and, even if they are just for pico-imx6ul, I will pick them after release.

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Otavio Salvador]

Hello Stefano,

On Fri, Aug 31, 2018 at 10:08 AM Tom Rini  wrote:
> On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:
> > please pull from u-boot-imx, thanks !

We sent few patches for PICO and we'd like to get them merged for next
release if possible. Is it possible for you to review them and see if
they are fine for merging or need adjustments?

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Fri, Aug 31, 2018 at 03:04:31PM +0200, Stefano Babic wrote:
> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit 11ed312896c5f5814064c5d45dcb2f53dc121437:
> 
>   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)
> 
> 
> Alex Kiernan (1):
>   Cleanup CONFIG_BOOTDELAY on cl-som-imx7
> 
> Anson Huang (3):
>   imx: mx7: psci: improve cpu hotplug flow
>   imx: mx7: add gpc initialization for low power mode
>   imx: mx7: add system suspend/resume support
> 
> Denis Zalevskiy (6):
>   board: ge: Remove EEPROM bus param from read_vpd()
>   board: ge: Move VPD EEPROM configuration to the defconfig
>   bootcount: i2c: Add bus switching to the I2C bootcount driver
>   bootcount: Configure length limit for I2C bootcount
>   board: ge: Move VPD reading to the vpd_reader
>   board: ge: Store bootcount in EEPROM on PPD and Bx50v3
> 
> Martin Kaiser (1):
>   watchdog: mx25: use the imx_watchdog driver for mx25
> 
> Stefan Agner (2):
>   board: toradex: common: fail gracefully on missing NAND chip
>   colibri_imx7_emmc: add Colibri iMX7D 1GB (eMMC) module support
> 
> Stefano Babic (2):
>   Merge branch 'master' of git://git.denx.de/u-boot into master
>   imx: missing CONFIG_MII in mx7dsabresd_qspi_defconfig
> 
> Sébastien Szymanski (1):
>   ARM: opos6ul: make the board boot again
> 
> Ye Li (6):
>   imx: imx6sx-sdb: Enable DM QSPI driver
>   imx: imx6sx-sabreauto: convert to use DM QSPI driver
>   imx: imx7d-sdb: Add DM QSPI support
>   dts: imx6ul: Update alias to support DM
>   dts: imx6ul_evk: Add DTS files for 14x14 EVK and 9x9 EVK boards
>   imx: imx6ul_evk: Enable DM driver for iMX6UL EVK u-boot
> 
>  arch/arm/dts/Makefile |   7 +-
>  arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi |  16 +
>  arch/arm/dts/imx6sx-sabreauto.dts |  40 ++
>  arch/arm/dts/imx6sx-sdb-u-boot.dtsi   |  16 +
>  arch/arm/dts/imx6sx.dtsi  |  12 +-
>  arch/arm/dts/imx6ul-14x14-evk-u-boot.dtsi |  10 +
>  arch/arm/dts/imx6ul-14x14-evk.dts | 427 +++
>  arch/arm/dts/imx6ul-9x9-evk-u-boot.dtsi   |  10 +
>  arch/arm/dts/imx6ul-9x9-evk.dts   | 471
> +
>  arch/arm/dts/imx6ul-opos6ul-u-boot.dtsi   |  28 ++
>  arch/arm/dts/imx6ul-opos6ul.dtsi  |   2 -
>  arch/arm/dts/imx6ul-opos6uldev-u-boot.dts |  25 ++
>  arch/arm/dts/imx6ul-opos6uldev.dts|   2 -
>  arch/arm/dts/imx6ul.dtsi  |  13 +-
>  arch/arm/dts/imx7d-sdb-qspi-u-boot.dtsi   |  10 +
>  arch/arm/dts/imx7d-sdb-qspi.dts   |  44 ++
>  arch/arm/dts/imx7d-sdb.dts|   6 +-
>  arch/arm/dts/imx7d.dtsi   |  12 +
>  arch/arm/dts/imx7s.dtsi   |  22 +-
>  arch/arm/include/asm/arch-mx25/imx-regs.h |   1 +
>  arch/arm/mach-imx/mx7/Makefile|   2 +-
>  arch/arm/mach-imx/mx7/psci-mx7.c  | 472
> +-
>  arch/arm/mach-imx/mx7/psci-suspend.S  |  67 +++
>  arch/arm/mach-imx/mx7/soc.c   | 103 +
>  board/freescale/mx6sxsabreauto/mx6sxsabreauto.c   |  24 --
>  board/freescale/mx6sxsabresd/mx6sxsabresd.c   |  25 --
>  board/freescale/mx6ul_14x14_evk/mx6ul_14x14_evk.c | 208 ++
>  board/freescale/mx7dsabresd/mx7dsabresd.c |  16 -
>  board/ge/bx50v3/Kconfig   |   2 +
>  board/ge/bx50v3/bx50v3.c  |  57 +--
>  board/ge/common/Kconfig   |  14 +
>  board/ge/common/vpd_reader.c  |  57 ++-
>  board/ge/common/vpd_reader.h  |  16 +-
>  board/ge/mx53ppd/Kconfig  |   2 +
>  board/ge/mx53ppd/mx53ppd.c|  50 +--
>  board/toradex/colibri_imx7/Kconfig|  42 +-
>  board/toradex/colibri_imx7/MAINTAINERS|   4 +
>  board/toradex/colibri_imx7/colibri_imx7.c |  41 +-
>  board/toradex/common/tdx-cfg-block.c  |   7 +-
>  configs/ge_bx50v3_defconfig   |  27 +-
>  configs/mx53ppd_defconfig |  29 +-
>  configs/mx6sxsabreauto_defconfig  |   2 +
>  configs/mx6sxsabresd_defconfig|   7 +
>  configs/mx6ul_14x14_evk_defconfig |  17 +-
>  configs/mx6ul_9x9_evk_defconfig   |  20 +-
>  

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Jul 23, 2018 at 11:44:04AM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit 474ecd2c84d97314b8145fbe3a57887f41b2edb3:
> 
>   env: Simplify Makefile using $(SPL_TPL_) (2018-07-21 12:24:31 -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 f97f167107b33fc6596561dae1309571ade39055:
> 
>   configs: imx6q_logic: Cleanup ramdiskaddr and fdtaddr (2018-07-23
> 11:05:54 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Wed, Jun 27, 2018 at 05:01:09PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> As discussed in previous thread, I have merged some NAND patches, too
> (see Stefan's and Jorg's patches).
> 
> 
> -- The following changes since commit
> a715415bb5948c84cc44c601b193188990f7238b:
> 
>   Merge branch 'master' of git://git.denx.de/u-boot-usb (2018-06-16
> 00:07:37 -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 b4cb809289235eb39dc2a3e521d8254da0ad752e:
> 
>   imx: bx50v3: fix Maintainers (2018-06-27 13:02:36 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Fri, May 18, 2018 at 10:23:18PM -0400, Tom Rini wrote:
> On Fri, May 18, 2018 at 08:51:42AM +0200, Stefano Babic wrote:
> 
> > Hi Tom,
> > 
> > please pull from u-boot-imx, thanks !
> > 
> > The following changes since commit f2d0f5e7ab3b8a7b4bf6e2ac499b4867c701d52d:
> > 
> >   ARM: re-enable MVGBE for edminiv2 (2018-05-16 11:38:08 -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 ee943655576f4a9e0af832e00a682a8d9f425bb1:
> > 
> >   arm: imx53: Add support for imx53 boards from K+P (2018-05-18 08:29:38
> > +0200)
> > 
> 
> Applied to u-boot/master, along with a follow up to fix SDPX tags,
> thanks!

And, oops, I should have complained:
WARNING: no status info for 'ge_bx50v3'
WARNING: no maintainers for 'ge_bx50v3'

were introduced, please follow-up with a fix, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Fri, May 18, 2018 at 08:51:42AM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit f2d0f5e7ab3b8a7b4bf6e2ac499b4867c701d52d:
> 
>   ARM: re-enable MVGBE for edminiv2 (2018-05-16 11:38:08 -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 ee943655576f4a9e0af832e00a682a8d9f425bb1:
> 
>   arm: imx53: Add support for imx53 boards from K+P (2018-05-18 08:29:38
> +0200)
> 

Applied to u-boot/master, along with a follow up to fix SDPX tags,
thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Lukasz Majewski]

Hi Tom, Stefano,

> On Thu, May 17, 2018 at 07:08:37PM +0200, Stefano Babic wrote:
> > On 17/05/2018 18:33, Lukasz Majewski wrote:  
> > > Hi Stefano, Tom,
> > >   
> > 
> > Tom was not in CC, added
> >   
> > > On 17 May 2018 17:50, Stefano Babic  wrote:
> > > 
> > > Hi Tom,
> > > 
> > > please pull from u-boot-imx, thanks !
> > > 
> > > The following changes since commit
> > > f2d0f5e7ab3b8a7b4bf6e2ac499b4867c701d52d:
> > > 
> > >   ARM: re-enable MVGBE for edminiv2 (2018-05-16 11:38:08
> > > -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
> > > 2f5988533d5c447d7bda79b0d9719f84db9e1835:
> > > 
> > >   imx6: sabrelite: update defconfig to use distro defaults
> > > (2018-05-17 17:47:17 +0200)
> > > 
> > > 
> > > Guillaume GARDET (2):
> > >   imx6: Convert sabrelite and nitrogen6x boards to distro
> > > boot support
> > >   imx6: sabrelite: update defconfig to use distro defaults
> > > 
> > > Ian Ray (6):
> > >   board: ge: bx50v3: add winbond SPI NOR support
> > >   board: ge: bx50v3: rename detect_baseboard function
> > >   board: ge: bx50v3: fix display support for b{46}50v3
> > >   board: ge: bx50v3: use VPD instead of compile-time
> > > checks board: ge: bx50v3: configure video arguments using VPD
> > >   board: ge: bx50v3: remove redundant targets
> > > 
> > > Lukasz Majewski (8):
> > >   dts: imx53: Add gpio and i2c nodes to imx53.dtsi file
> > >   dts: pinctrl: Provide IMX_PAD_SION definition for imx53
> > > pinctrl
> > > 
> > > 
> > > Two above patches are correct.
> > > 
> > >   pmic: fsl: Provide some more definitions for MC34708
> > > PMIC pmic: fsl: Define number of bytes sent at once by MC34708
> > > PMIC pmic: dm: Provide *trans_len() callback for pmic-uclass
> > >   pmic: dm: Rewrite pmic_reg_{read|write} and
> > > pmic_clrsetbits to support transmissions larger than 1 byte
> > >   pmic: dm: Add support for MC34708 for PMIC DM
> > >   arm: imx53: Add support for imx53 boards from K+P
> > > 
> > > 
> > > Stefano, Tom please drop above patches as they were superseded by
> > > v2 and v3 (after discussion with Simon).  
> > 
> > I have just added the pmic patches due to the dependency in "arm:
> > imx53: Add support for imx53 boards from K+P", and I have not seen
> > any change request in them. I drop the series, Tom please wait, I
> > will send a new PR after.  
> 
> OK.
> 
> > > IMHO v3 is eligible for pulling:
> > > https://patchwork.ozlabs.org/patch/913670/
> > >   
> > 
> > Should they flow into Jaehoon's tree instead of u-boot-imx ?  
> 
> I'm fine with this coming along via the imx tree so we can get it in
> now.
> 

I would also opt for this option.

For the future - I've looked at:

http://git.denx.de/?p=u-boot/u-boot-pmic.git;a=summary
https://www.denx.de/wiki/U-Boot/Custodians   (which seems outdated
anyway)

It seems like u-boot-pmic repo is dead (as Przemek is not doing any
open source work since 2017). If you think that it helps
- I can take it and rise my hand to be PMIC maintainer (again).


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de


pgp81HknDHloT.pgp
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Thu, May 17, 2018 at 07:08:37PM +0200, Stefano Babic wrote:
> On 17/05/2018 18:33, Lukasz Majewski wrote:
> > Hi Stefano, Tom,
> > 
> 
> Tom was not in CC, added
> 
> > On 17 May 2018 17:50, Stefano Babic  wrote:
> > 
> > Hi Tom,
> > 
> > please pull from u-boot-imx, thanks !
> > 
> > The following changes since commit
> > f2d0f5e7ab3b8a7b4bf6e2ac499b4867c701d52d:
> > 
> >   ARM: re-enable MVGBE for edminiv2 (2018-05-16 11:38:08 -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 2f5988533d5c447d7bda79b0d9719f84db9e1835:
> > 
> >   imx6: sabrelite: update defconfig to use distro defaults (2018-05-17
> > 17:47:17 +0200)
> > 
> > 
> > Guillaume GARDET (2):
> >   imx6: Convert sabrelite and nitrogen6x boards to distro boot
> > support
> >   imx6: sabrelite: update defconfig to use distro defaults
> > 
> > Ian Ray (6):
> >   board: ge: bx50v3: add winbond SPI NOR support
> >   board: ge: bx50v3: rename detect_baseboard function
> >   board: ge: bx50v3: fix display support for b{46}50v3
> >   board: ge: bx50v3: use VPD instead of compile-time checks
> >   board: ge: bx50v3: configure video arguments using VPD
> >   board: ge: bx50v3: remove redundant targets
> > 
> > Lukasz Majewski (8):
> >   dts: imx53: Add gpio and i2c nodes to imx53.dtsi file
> >   dts: pinctrl: Provide IMX_PAD_SION definition for imx53 pinctrl
> > 
> > 
> > Two above patches are correct.
> > 
> >   pmic: fsl: Provide some more definitions for MC34708 PMIC
> >   pmic: fsl: Define number of bytes sent at once by MC34708 PMIC
> >   pmic: dm: Provide *trans_len() callback for pmic-uclass
> >   pmic: dm: Rewrite pmic_reg_{read|write} and pmic_clrsetbits to
> > support transmissions larger than 1 byte
> >   pmic: dm: Add support for MC34708 for PMIC DM
> >   arm: imx53: Add support for imx53 boards from K+P
> > 
> > 
> > Stefano, Tom please drop above patches as they were superseded by v2 and
> > v3 (after discussion with Simon).
> 
> I have just added the pmic patches due to the dependency in "arm: imx53:
> Add support for imx53 boards from K+P", and I have not seen any change
> request in them. I drop the series, Tom please wait, I will send a new
> PR after.

OK.

> > IMHO v3 is eligible for pulling:
> > https://patchwork.ozlabs.org/patch/913670/
> > 
> 
> Should they flow into Jaehoon's tree instead of u-boot-imx ?

I'm fine with this coming along via the imx tree so we can get it in
now.

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

On 17/05/2018 18:33, Lukasz Majewski wrote:
> Hi Stefano, Tom,
> 

Tom was not in CC, added

> On 17 May 2018 17:50, Stefano Babic  wrote:
> 
> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit
> f2d0f5e7ab3b8a7b4bf6e2ac499b4867c701d52d:
> 
>   ARM: re-enable MVGBE for edminiv2 (2018-05-16 11:38:08 -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 2f5988533d5c447d7bda79b0d9719f84db9e1835:
> 
>   imx6: sabrelite: update defconfig to use distro defaults (2018-05-17
> 17:47:17 +0200)
> 
> 
> Guillaume GARDET (2):
>   imx6: Convert sabrelite and nitrogen6x boards to distro boot
> support
>   imx6: sabrelite: update defconfig to use distro defaults
> 
> Ian Ray (6):
>   board: ge: bx50v3: add winbond SPI NOR support
>   board: ge: bx50v3: rename detect_baseboard function
>   board: ge: bx50v3: fix display support for b{46}50v3
>   board: ge: bx50v3: use VPD instead of compile-time checks
>   board: ge: bx50v3: configure video arguments using VPD
>   board: ge: bx50v3: remove redundant targets
> 
> Lukasz Majewski (8):
>   dts: imx53: Add gpio and i2c nodes to imx53.dtsi file
>   dts: pinctrl: Provide IMX_PAD_SION definition for imx53 pinctrl
> 
> 
> Two above patches are correct.
> 
>   pmic: fsl: Provide some more definitions for MC34708 PMIC
>   pmic: fsl: Define number of bytes sent at once by MC34708 PMIC
>   pmic: dm: Provide *trans_len() callback for pmic-uclass
>   pmic: dm: Rewrite pmic_reg_{read|write} and pmic_clrsetbits to
> support transmissions larger than 1 byte
>   pmic: dm: Add support for MC34708 for PMIC DM
>   arm: imx53: Add support for imx53 boards from K+P
> 
> 
> Stefano, Tom please drop above patches as they were superseded by v2 and
> v3 (after discussion with Simon).

I have just added the pmic patches due to the dependency in "arm: imx53:
Add support for imx53 boards from K+P", and I have not seen any change
request in them. I drop the series, Tom please wait, I will send a new
PR after.

> 
> IMHO v3 is eligible for pulling:
> https://patchwork.ozlabs.org/patch/913670/
> 

Should they flow into Jaehoon's tree instead of u-boot-imx ?

Best regards,
Stefano

> BR,
> Łukasz Majewski
> 
> Magnus Lilja (2):
>   mx31pdk: Convert CONFIG_MX31 flag to use Kconfig.
>   mx31: Convert MX31_HCLK_FREQ and MX31_CLK32 to Kconfig.
> 
> Nandor Han (2):
>   board: ge: bx50v3: unify two switch statements
>   board: ge: bx50v3: detect the monitor type by reading VPD earlier
> 
> Peter Robinson (4):
>   mx6 common: remove dangling comment
>   mx7: remove empty ifndef statement
>   mx6: Select CONFIG_MP with MX6_SMP
>   mx6: remove duplicated BOUNCE_BUFFER defines
> 
> Sebastian Reichel (1):
>   ge: ppd: move CONFIG_ENV_IS_IN_MMC to defconfig
> 
> arch/arm/Kconfig |  14 +++
> arch/arm/dts/imx53-kp.dts    | 135
> 
> arch/arm/dts/imx53-pinfunc.h |   1 +
> arch/arm/dts/imx53.dtsi  | 101
> +++-
> arch/arm/include/asm/arch-mx31/clock.h   |   8 
> arch/arm/mach-imx/mx3/Kconfig    |  34
> +
> arch/arm/mach-imx/mx5/Kconfig    |  12 ++
> arch/arm/mach-imx/mx6/Kconfig    |  15 ++--
> board/ge/bx50v3/Kconfig  |   2 +-
> board/ge/bx50v3/bx50v3.c |  71
> ++
> board/k+p/kp_imx53/Kconfig   |  15 
> board/k+p/kp_imx53/MAINTAINERS   |   6 +++
> board/k+p/kp_imx53/Makefile  |   8 
> board/k+p/kp_imx53/kp_id_rev.c   | 121
> ++
> board/k+p/kp_imx53/kp_id_rev.h   |  28
> ++
> board/k+p/kp_imx53/kp_imx53.c    | 212
> 
> +
> configs/ge_b650v3_defconfig  |  43
> -
> configs/ge_b850v3_defconfig  |  43
> -
> configs/{ge_b450v3_defconfig => ge_bx50v3_defconfig} |   4 +-
> configs/kp_imx53_defconfig   |  40
> 

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Lukasz Majewski]

Hi Tom,

> On Sun, Apr 15, 2018 at 12:28:54PM +0200, Stefano Babic wrote:
> 
> > Hi Tom,
> > 
> > please pull from u-boot-imx, thanks !
> > 
> > 
> > -- The following changes since commit
> > 6d7403bf72b5ea46497fe8222d0303cb79563379:
> > 
> >   doc: mxc_hab: Update i.MX HAB documentation (2018-03-11 16:00:21
> > +0100)
> > 
> > are available in the git repository at:
> > 
> >   git://www.denx.de/git/u-boot-imx.git master
> > 
> > for you to fetch changes up to
> > b4e9bdcd05ac83146ad16802a94db3e78f530d1a:
> > 
> >   mx6cuboxi: Fix some memory configuration errors (2018-04-15
> > 11:56:21 +0200)
> >   
> 
> Applied to u-boot/master.
> 
> For the record, the K+P KP_IMX6Q_TPC i.MX6Q commit is missing a
> Signed-off-by line.  I'm willing to assume it was just a thinko, right
> Lukasz ?  Thanks!
> 

Yes, this is my commit :-)



Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de


pgpJG62Q8wnD9.pgp
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Sun, Apr 15, 2018 at 12:28:54PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> 
> -- The following changes since commit
> 6d7403bf72b5ea46497fe8222d0303cb79563379:
> 
>   doc: mxc_hab: Update i.MX HAB documentation (2018-03-11 16:00:21 +0100)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to b4e9bdcd05ac83146ad16802a94db3e78f530d1a:
> 
>   mx6cuboxi: Fix some memory configuration errors (2018-04-15 11:56:21
> +0200)
> 

Applied to u-boot/master.

For the record, the K+P KP_IMX6Q_TPC i.MX6Q commit is missing a
Signed-off-by line.  I'm willing to assume it was just a thinko, right
Lukasz ?  Thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Sun, Mar 11, 2018 at 04:07:29PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> this contains the last fixes sent on Friday. There are still concerns
> last Bryan's patch:
>   
>   imx: hab: Convert DCD non-NULL error to warning
> 
> This is still under discussion (and it is not part of my PR).
> 
> The following changes since commit 2e5c42c630c4c550abf9d8da8cc11d8a7928ab0a:
> 
>   Merge git://git.denx.de/u-boot-imx (2018-03-09 13:29:58 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to 6d7403bf72b5ea46497fe8222d0303cb79563379:
> 
>   doc: mxc_hab: Update i.MX HAB documentation (2018-03-11 16:00:21 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Fabio Estevam]

Hi Stefano,

On Fri, Mar 9, 2018 at 8:11 AM, Stefano Babic  wrote:
If you like I can resent a v2 of option 1 where the error message is removed.
>
> It is a hack, but we can do just for the release. I have already applied
> V1, I will drop it from u-boot-imx. I will also apply "imx: syscounter:
> make sure asm is volatile" as you told me today, and then I send a PR
> with these two patches to Tom.

Sounds good! I have sent a v3 patch.

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

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Fabio,

On 09/03/2018 11:42, Fabio Estevam wrote:
> Hi Stefano,
> 
> On Fri, Mar 9, 2018 at 7:13 AM, Stefano Babic  wrote:
>> Hi Tom,
>>
>> Fabio fixed a build issue yesterday. It is just a single patch, but it
>> should be flow into the release. You can pick it udirectly p, but I have
>> also applied to u-boot-imx to make things easier.
> 
> Troy made a good observation yesterday: he asked me if after applying
> this patch the error is still shown when BOOT_FROM is not passed in
> the cfg file.
> 
> I tested it and it does not.

Something strange happens...

> 
> So we have some options here that I would like to discuss:
> 
> 1. Drop the error message completely. This message is only helpful for
> developers during bring up phase
> 
> 2. Understand better the problem and fix the parallel build issue
> 
> I think 1 can be done for the upcoming release as the negative effect
> of breaking builds is worse than showing the lack of BOOT_FROM tag.

My concern is more that build is broken, yes. Release is now upcoming,
we can do just a few changes.

> 
> Option number 2 would require more time to implement.
> 
> If you like I can resent a v2 of option 1 where the error message is removed.

It is a hack, but we can do just for the release. I have already applied
V1, I will drop it from u-boot-imx. I will also apply "imx: syscounter:
make sure asm is volatile" as you told me today, and then I send a PR
with these two patches 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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Fabio Estevam]

Hi Stefano,

On Fri, Mar 9, 2018 at 7:13 AM, Stefano Babic  wrote:
> Hi Tom,
>
> Fabio fixed a build issue yesterday. It is just a single patch, but it
> should be flow into the release. You can pick it udirectly p, but I have
> also applied to u-boot-imx to make things easier.

Troy made a good observation yesterday: he asked me if after applying
this patch the error is still shown when BOOT_FROM is not passed in
the cfg file.

I tested it and it does not.

So we have some options here that I would like to discuss:

1. Drop the error message completely. This message is only helpful for
developers during bring up phase

2. Understand better the problem and fix the parallel build issue

I think 1 can be done for the upcoming release as the negative effect
of breaking builds is worse than showing the lack of BOOT_FROM tag.

Option number 2 would require more time to implement.

If you like I can resent a v2 of option 1 where the error message is removed.

Please let me know your thoughts.

Thanks,

Fabio Estevam
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Sat, Feb 24, 2018 at 04:58:14PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> some fixes, please pull from u-boot-imx, thanks!
> 
> The following changes since commit 02b0895c215e252cff50d9edf53a972d2bbbfd30:
> 
>   Merge git://git.denx.de/u-boot-sh (2018-02-17 16:06:59 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to 3d409153502f25a1ea3b8b4e9940148b7871273d:
> 
>   imx6q: engicam: Use SPL_LOAD_FIT for MMC boards (2018-02-22 14:51:01
> +0100)
> 

For the updated version, applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Tom,

On 05/02/2018 04:57, Tom Rini wrote:
> On Sun, Feb 04, 2018 at 07:16:07PM -0500, Tom Rini wrote:
>> On Sun, Feb 04, 2018 at 12:34:29PM +0100, Stefano Babic wrote:
>>
>>> Hi Tom,
>>>
>>> please pull from u-boot-imx, thanks !
>>>
>>>
>>> checkpatch warnings in this PR are just related to long lines (in DTS
>>> and pinmux). These are accepted exceptions. In a couple of cases long
>>> lines are also accepted because code ramains more readable if not split
>>> into more lines).
>>>
>>> The following changes since commit b2153075f42c2d46d310778e226bcb11f0af47f5:
>>>
>>>   Kconfig: usb: rockchip: Remove not needed *_defconfig USB/gadget
>>> entries (2018-02-02 07:13:48 -0500)
>>>
>>> are available in the git repository at:
>>>
>>>   git://www.denx.de/git/u-boot-imx.git master
>>>
>>> for you to fetch changes up to d1ceb0c4881332cb0586920f0a40f8e4a48d99a9:
>>>
>>>   imx: mx7: run sec_init for CAAM RNG (2018-02-04 12:14:11 +0100)
>>>
>>
>> Applied to u-boot/master, thanks!
> 
> That said, oops!
> WARNING: no status info for 'imx6qdl_icore_mipi'
> WARNING: no maintainers for 'imx6qdl_icore_mipi'
> 
> Please fix, thanks!
> 

Yes, thanks, I have sent a patch.

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Sun, Feb 04, 2018 at 07:16:07PM -0500, Tom Rini wrote:
> On Sun, Feb 04, 2018 at 12:34:29PM +0100, Stefano Babic wrote:
> 
> > Hi Tom,
> > 
> > please pull from u-boot-imx, thanks !
> > 
> > 
> > checkpatch warnings in this PR are just related to long lines (in DTS
> > and pinmux). These are accepted exceptions. In a couple of cases long
> > lines are also accepted because code ramains more readable if not split
> > into more lines).
> > 
> > The following changes since commit b2153075f42c2d46d310778e226bcb11f0af47f5:
> > 
> >   Kconfig: usb: rockchip: Remove not needed *_defconfig USB/gadget
> > entries (2018-02-02 07:13:48 -0500)
> > 
> > are available in the git repository at:
> > 
> >   git://www.denx.de/git/u-boot-imx.git master
> > 
> > for you to fetch changes up to d1ceb0c4881332cb0586920f0a40f8e4a48d99a9:
> > 
> >   imx: mx7: run sec_init for CAAM RNG (2018-02-04 12:14:11 +0100)
> > 
> 
> Applied to u-boot/master, thanks!

That said, oops!
WARNING: no status info for 'imx6qdl_icore_mipi'
WARNING: no maintainers for 'imx6qdl_icore_mipi'

Please fix, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Sun, Feb 04, 2018 at 12:34:29PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> 
> checkpatch warnings in this PR are just related to long lines (in DTS
> and pinmux). These are accepted exceptions. In a couple of cases long
> lines are also accepted because code ramains more readable if not split
> into more lines).
> 
> The following changes since commit b2153075f42c2d46d310778e226bcb11f0af47f5:
> 
>   Kconfig: usb: rockchip: Remove not needed *_defconfig USB/gadget
> entries (2018-02-02 07:13:48 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to d1ceb0c4881332cb0586920f0a40f8e4a48d99a9:
> 
>   imx: mx7: run sec_init for CAAM RNG (2018-02-04 12:14:11 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Tom,

On 16/01/2018 01:00, Tom Rini wrote:
> On Mon, Jan 15, 2018 at 06:52:01PM +0100, Stefano Babic wrote:
> 
>> Hi Tom,
>>
>> please pull from u-boot-imx, thanks !
>>
>> The following changes since commit 373b9003410b44a1133060c2e63483b278fb476b:
>>
>>   Merge git://git.denx.de/u-boot-sunxi (2018-01-11 14:14:19 -0500)
>>
>> are available in the git repository at:
>>
>>   git://www.denx.de/git/u-boot-imx.git master
>>
>> for you to fetch changes up to 9587b0d61133c025d48ebf480aa021fc2fdd53a7:
>>
>>   arm: imx: hab: Add hab_failsafe console command (2018-01-14 17:26:30
>> +0100)
>>
> 
> Applied to u-boot/master, thanks!
> 
> But please note that checkpatch has a lot to say, some of which should
> probably be addressed:
>

Ouch.

I will check it :-)

Regards,
Stefano

> WARNING: please write a paragraph that describes the config symbol fully
> #51: FILE: arch/arm/mach-imx/mx6/Kconfig:380:
> +config TARGET_SKSIMX6
> 
> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> #67: 
> new file mode 100644
> 
> WARNING: line over 80 characters
> #171: FILE: board/sks-kinkel/sksimx6/sksimx6.c:62:
> + IOMUX_PADS(PAD_NANDF_D0__GPIO2_IO00 | MUX_PAD_CTRL(NO_PAD_CTRL)),/* CD 
> */
> 
> CHECK: Blank lines aren't necessary after an open brace '{'
> #215: FILE: board/sks-kinkel/sksimx6/sksimx6.c:106:
> +{
> +
> 
> CHECK: Blank lines aren't necessary before a close brace '}'
> #245: FILE: board/sks-kinkel/sksimx6/sksimx6.c:136:
> +
> +}
> 
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> #249: FILE: board/sks-kinkel/sksimx6/sksimx6.c:140:
> + uint32_t base = IMX_FEC_BASE;
> 
> CHECK: Alignment should match open parenthesis
> #261: FILE: board/sks-kinkel/sksimx6/sksimx6.c:152:
> + phydev = phy_find_by_mask(bus, (0xf << CONFIG_FEC_MXC_PHYADDR),
> + PHY_INTERFACE_MODE_RGMII);
> 
> WARNING: unnecessary whitespace before a quoted newline
> #333: FILE: board/sks-kinkel/sksimx6/sksimx6.c:224:
> + printf("Warning: failed to initialize mmc dev \n");
> 
> CHECK: Please don't use multiple blank lines
> #350: FILE: board/sks-kinkel/sksimx6/sksimx6.c:241:
> +
> +
> 
> WARNING: line over 80 characters
> #471: FILE: board/sks-kinkel/sksimx6/sksimx6.c:362:
> + mx6_dram_cfg(_qdl, _1g_mmdc_calib, 
> _125);
> 
> CHECK: Please don't use multiple blank lines
> #601: FILE: include/configs/sksimx6.h:7:
> +
> +
> 
> WARNING: please, no space before tabs
> #670: FILE: include/configs/sksimx6.h:76:
> +^I"addmisc=setenv bootargs ${bootargs} ${miscargs}\0" ^I^I\$
> 
> WARNING: please, no space before tabs
> #675: FILE: include/configs/sksimx6.h:81:
> +^I"fdt_addr_r=0x1800\0" ^I^I^I^I^I\$
> 
> WARNING: please, no space before tabs
> #677: FILE: include/configs/sksimx6.h:83:
> +^I"fdt_high=0x\0" ^I^I^I^I^I\$
> 
> WARNING: please, no space before tabs
> #678: FILE: include/configs/sksimx6.h:84:
> +^I"kernel_addr_r=" __stringify(CONFIG_LOADADDR) "\0" ^I^I\$
> 
> WARNING: please, no space before tabs
> #681: FILE: include/configs/sksimx6.h:87:
> +^I"mmcboot=if run mmcload;then " ^I^I^I^I^I\$
> 
> WARNING: please, no space before tabs
> #683: FILE: include/configs/sksimx6.h:89:
> +^I^I^I"bootm;fi\0" ^I^I^I^I^I\$
> 
> total: 0 errors, 11 warnings, 6 checks, 611 lines checked
> WARNING: line over 80 characters
> #38: FILE: drivers/misc/mxc_ocotp.c:356:
> + printf("mxc_ocotp: The word has been programmed, no 
> more write\n");
> 
> total: 0 errors, 1 warnings, 0 checks, 23 lines checked
> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> #34: 
> new file mode 100644
> 
> WARNING: line over 80 characters
> #549: FILE: arch/arm/dts/imx6sx-sdb.dtsi:367:
> + MX6SX_PAD_ENET2_RX_CLK__ENET2_REF_CLK_25M   
> 0x91
> 
> WARNING: line over 80 characters
> #555: FILE: arch/arm/dts/imx6sx-sdb.dtsi:373:
> + MX6SX_PAD_ENET2_COL__GPIO2_IO_6 
> 0x8000
> 
> WARNING: line over 80 characters
> #585: FILE: arch/arm/dts/imx6sx-sdb.dtsi:403:
> + MX6SX_PAD_GPIO1_IO01__I2C1_SDA  
> 0x4001b8b1
> 
> WARNING: line over 80 characters
> #586: FILE: arch/arm/dts/imx6sx-sdb.dtsi:404:
> + MX6SX_PAD_GPIO1_IO00__I2C1_SCL  
> 0x4001b8b1
> 
> WARNING: line over 80 characters
> #592: FILE: arch/arm/dts/imx6sx-sdb.dtsi:410:
> + MX6SX_PAD_KEY_ROW4__I2C3_SDA
> 0x4001b8b1
> 
> WARNING: line over 80 characters
> #593: FILE: arch/arm/dts/imx6sx-sdb.dtsi:411:
> + MX6SX_PAD_KEY_COL4__I2C3_SCL
> 0x4001b8b1
> 
> WARNING: line over 80 characters
> #599: FILE: arch/arm/dts/imx6sx-sdb.dtsi:417:
> + MX6SX_PAD_CSI_DATA07__I2C4_SDA  
> 0x4001b8b1
> 
> WARNING: line over 80 characters
> #600: FILE: arch/arm/dts/imx6sx-sdb.dtsi:418:
> + MX6SX_PAD_CSI_DATA06__I2C4_SCL   

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Jan 15, 2018 at 06:52:01PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit 373b9003410b44a1133060c2e63483b278fb476b:
> 
>   Merge git://git.denx.de/u-boot-sunxi (2018-01-11 14:14:19 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to 9587b0d61133c025d48ebf480aa021fc2fdd53a7:
> 
>   arm: imx: hab: Add hab_failsafe console command (2018-01-14 17:26:30
> +0100)
> 

Applied to u-boot/master, thanks!

But please note that checkpatch has a lot to say, some of which should
probably be addressed:

WARNING: please write a paragraph that describes the config symbol fully
#51: FILE: arch/arm/mach-imx/mx6/Kconfig:380:
+config TARGET_SKSIMX6

WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#67: 
new file mode 100644

WARNING: line over 80 characters
#171: FILE: board/sks-kinkel/sksimx6/sksimx6.c:62:
+   IOMUX_PADS(PAD_NANDF_D0__GPIO2_IO00 | MUX_PAD_CTRL(NO_PAD_CTRL)),/* CD 
*/

CHECK: Blank lines aren't necessary after an open brace '{'
#215: FILE: board/sks-kinkel/sksimx6/sksimx6.c:106:
+{
+

CHECK: Blank lines aren't necessary before a close brace '}'
#245: FILE: board/sks-kinkel/sksimx6/sksimx6.c:136:
+
+}

CHECK: Prefer kernel type 'u32' over 'uint32_t'
#249: FILE: board/sks-kinkel/sksimx6/sksimx6.c:140:
+   uint32_t base = IMX_FEC_BASE;

CHECK: Alignment should match open parenthesis
#261: FILE: board/sks-kinkel/sksimx6/sksimx6.c:152:
+   phydev = phy_find_by_mask(bus, (0xf << CONFIG_FEC_MXC_PHYADDR),
+   PHY_INTERFACE_MODE_RGMII);

WARNING: unnecessary whitespace before a quoted newline
#333: FILE: board/sks-kinkel/sksimx6/sksimx6.c:224:
+   printf("Warning: failed to initialize mmc dev \n");

CHECK: Please don't use multiple blank lines
#350: FILE: board/sks-kinkel/sksimx6/sksimx6.c:241:
+
+

WARNING: line over 80 characters
#471: FILE: board/sks-kinkel/sksimx6/sksimx6.c:362:
+   mx6_dram_cfg(_qdl, _1g_mmdc_calib, 
_125);

CHECK: Please don't use multiple blank lines
#601: FILE: include/configs/sksimx6.h:7:
+
+

WARNING: please, no space before tabs
#670: FILE: include/configs/sksimx6.h:76:
+^I"addmisc=setenv bootargs ${bootargs} ${miscargs}\0" ^I^I\$

WARNING: please, no space before tabs
#675: FILE: include/configs/sksimx6.h:81:
+^I"fdt_addr_r=0x1800\0" ^I^I^I^I^I\$

WARNING: please, no space before tabs
#677: FILE: include/configs/sksimx6.h:83:
+^I"fdt_high=0x\0" ^I^I^I^I^I\$

WARNING: please, no space before tabs
#678: FILE: include/configs/sksimx6.h:84:
+^I"kernel_addr_r=" __stringify(CONFIG_LOADADDR) "\0" ^I^I\$

WARNING: please, no space before tabs
#681: FILE: include/configs/sksimx6.h:87:
+^I"mmcboot=if run mmcload;then " ^I^I^I^I^I\$

WARNING: please, no space before tabs
#683: FILE: include/configs/sksimx6.h:89:
+^I^I^I"bootm;fi\0" ^I^I^I^I^I\$

total: 0 errors, 11 warnings, 6 checks, 611 lines checked
WARNING: line over 80 characters
#38: FILE: drivers/misc/mxc_ocotp.c:356:
+   printf("mxc_ocotp: The word has been programmed, no 
more write\n");

total: 0 errors, 1 warnings, 0 checks, 23 lines checked
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#34: 
new file mode 100644

WARNING: line over 80 characters
#549: FILE: arch/arm/dts/imx6sx-sdb.dtsi:367:
+   MX6SX_PAD_ENET2_RX_CLK__ENET2_REF_CLK_25M   
0x91

WARNING: line over 80 characters
#555: FILE: arch/arm/dts/imx6sx-sdb.dtsi:373:
+   MX6SX_PAD_ENET2_COL__GPIO2_IO_6 
0x8000

WARNING: line over 80 characters
#585: FILE: arch/arm/dts/imx6sx-sdb.dtsi:403:
+   MX6SX_PAD_GPIO1_IO01__I2C1_SDA  
0x4001b8b1

WARNING: line over 80 characters
#586: FILE: arch/arm/dts/imx6sx-sdb.dtsi:404:
+   MX6SX_PAD_GPIO1_IO00__I2C1_SCL  
0x4001b8b1

WARNING: line over 80 characters
#592: FILE: arch/arm/dts/imx6sx-sdb.dtsi:410:
+   MX6SX_PAD_KEY_ROW4__I2C3_SDA
0x4001b8b1

WARNING: line over 80 characters
#593: FILE: arch/arm/dts/imx6sx-sdb.dtsi:411:
+   MX6SX_PAD_KEY_COL4__I2C3_SCL
0x4001b8b1

WARNING: line over 80 characters
#599: FILE: arch/arm/dts/imx6sx-sdb.dtsi:417:
+   MX6SX_PAD_CSI_DATA07__I2C4_SDA  
0x4001b8b1

WARNING: line over 80 characters
#600: FILE: arch/arm/dts/imx6sx-sdb.dtsi:418:
+   MX6SX_PAD_CSI_DATA06__I2C4_SCL  
0x4001b8b1

WARNING: line over 80 characters
#640: FILE: arch/arm/dts/imx6sx-sdb.dtsi:458:
+   MX6SX_PAD_QSPI1A_DATA0__GPIO4_IO_16 
0x8000

WARNING: line over 80 characters
#740: FILE: arch/arm/dts/imx6sx-sdb.dtsi:558:
+   MX6SX_PAD_KEY_COL0__GPIO2_IO_10 0x17059 

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Jan 08, 2018 at 05:54:38PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> some fixes, please pull from u-boot-imx, thanks !
> 
> The following changes since commit 1314bd1192b4c67d28bdae7eee639588e88090cd:
> 
>   boards: amlogic: khadas-vim: Typo fixup (2018-01-02 07:57:34 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to ca9d211e2c7801bc3e194d325ece0d3b583b32d2:
> 
>   mtd: nand: mxs_nand_spl: Remove nand size print (2018-01-08 17:37:12
> +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Jan 08, 2018 at 06:50:36PM +0100, Stefano Babic wrote:
> Hi Tom,
> 
> On 08/01/2018 18:38, Tom Rini wrote:
> > On Mon, Jan 08, 2018 at 05:54:38PM +0100, Stefano Babic wrote:
> >> Hi Tom,
> >>
> >> some fixes, please pull from u-boot-imx, thanks !
> >>
> >> The following changes since commit 
> >> 1314bd1192b4c67d28bdae7eee639588e88090cd:
> >>
> >>   boards: amlogic: khadas-vim: Typo fixup (2018-01-02 07:57:34 -0500)
> >>
> >> are available in the git repository at:
> >>
> >>   git://www.denx.de/git/u-boot-imx.git master
> >>
> >> for you to fetch changes up to ca9d211e2c7801bc3e194d325ece0d3b583b32d2:
> >>
> >>   mtd: nand: mxs_nand_spl: Remove nand size print (2018-01-08 17:37:12
> >> +0100)
> >>
> >> 
> >> Christopher Spinrath (3):
> >>   ARM: imx: cm_fx6: env: use standard variables
> >>   ARM: imx: cm_fx6: env: support distro boot command
> >>   ARM: imx: cm_fx6: env: don't run boot scripts twice
> >>
> >> Eran Matityahu (1):
> >>   imx: spl: Fix NAND bootmode detection
> >>
> >> Eric Nelson (1):
> >>   mx6: Add board mx6memcal for use in validating DDR
> >>
> >> Fabio Estevam (1):
> >>   mx6ull: Handle the CONFIG_MX6ULL cases correctly
> >>
> >> Jagan Teki (2):
> >>   board: engicam: Fix to remove legacy board/icorem6_rqs
> >>   mtd: nand: mxs_nand_spl: Remove nand size print
> >>
> >> Stefan Agner (3):
> >>   imx: move CONFIG_SYSCOUNTER_TIMER to Kconfig
> >>   imx: introduce CONFIG_GPT_TIMER
> >>   imx: initialize and use generic timer on i.MX 6UL/ULL
> > 
> > It's release day.  How many of these are release day candidate fixes?  I
> > know the MX6ULL thing is important, and the SPL bootmode thing looks so
> > too.
> 
> Yes, both are important - for MX6ULL this means both Fabio's and
> Stefan's patches. Eran's is a fix, too, and it should be in.
> 
> Jagan's is just a cleanup - this drops dead code in a Engicam board, and
> drop a couple of nasty print when booting from NAND. No big issue, but
> also no effects on the rest of code.
> 
> >  I do not know about the rest, are they?  Thanks!
> 
> It remains Cristopher's (small fixes just for cm_fx6 board) and Eric's.
> The last one is a very interesting thing because it replaces the
> proprietary ddr-stress tool from Freescale. It is not a fix, but it is
> very nice to have. It adds another target, so there is no changes in
> common code and if you do not argue, it will be nice to have in release.

OK.  I'm going to take it (applied message coming shortly) since we're
just now exiting the general US/EU winter holiday time-frame and I'll
use that as why most of this wasn't sent much sooner.  Thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Tom,

On 08/01/2018 18:38, Tom Rini wrote:
> On Mon, Jan 08, 2018 at 05:54:38PM +0100, Stefano Babic wrote:
>> Hi Tom,
>>
>> some fixes, please pull from u-boot-imx, thanks !
>>
>> The following changes since commit 1314bd1192b4c67d28bdae7eee639588e88090cd:
>>
>>   boards: amlogic: khadas-vim: Typo fixup (2018-01-02 07:57:34 -0500)
>>
>> are available in the git repository at:
>>
>>   git://www.denx.de/git/u-boot-imx.git master
>>
>> for you to fetch changes up to ca9d211e2c7801bc3e194d325ece0d3b583b32d2:
>>
>>   mtd: nand: mxs_nand_spl: Remove nand size print (2018-01-08 17:37:12
>> +0100)
>>
>> 
>> Christopher Spinrath (3):
>>   ARM: imx: cm_fx6: env: use standard variables
>>   ARM: imx: cm_fx6: env: support distro boot command
>>   ARM: imx: cm_fx6: env: don't run boot scripts twice
>>
>> Eran Matityahu (1):
>>   imx: spl: Fix NAND bootmode detection
>>
>> Eric Nelson (1):
>>   mx6: Add board mx6memcal for use in validating DDR
>>
>> Fabio Estevam (1):
>>   mx6ull: Handle the CONFIG_MX6ULL cases correctly
>>
>> Jagan Teki (2):
>>   board: engicam: Fix to remove legacy board/icorem6_rqs
>>   mtd: nand: mxs_nand_spl: Remove nand size print
>>
>> Stefan Agner (3):
>>   imx: move CONFIG_SYSCOUNTER_TIMER to Kconfig
>>   imx: introduce CONFIG_GPT_TIMER
>>   imx: initialize and use generic timer on i.MX 6UL/ULL
> 
> It's release day.  How many of these are release day candidate fixes?  I
> know the MX6ULL thing is important, and the SPL bootmode thing looks so
> too.

Yes, both are important - for MX6ULL this means both Fabio's and
Stefan's patches. Eran's is a fix, too, and it should be in.

Jagan's is just a cleanup - this drops dead code in a Engicam board, and
drop a couple of nasty print when booting from NAND. No big issue, but
also no effects on the rest of code.

>  I do not know about the rest, are they?  Thanks!

It remains Cristopher's (small fixes just for cm_fx6 board) and Eric's.
The last one is a very interesting thing because it replaces the
proprietary ddr-stress tool from Freescale. It is not a fix, but it is
very nice to have. It adds another target, so there is no changes in
common code and if you do not argue, it will be nice to have in release.

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Jan 08, 2018 at 05:54:38PM +0100, Stefano Babic wrote:
> Hi Tom,
> 
> some fixes, please pull from u-boot-imx, thanks !
> 
> The following changes since commit 1314bd1192b4c67d28bdae7eee639588e88090cd:
> 
>   boards: amlogic: khadas-vim: Typo fixup (2018-01-02 07:57:34 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to ca9d211e2c7801bc3e194d325ece0d3b583b32d2:
> 
>   mtd: nand: mxs_nand_spl: Remove nand size print (2018-01-08 17:37:12
> +0100)
> 
> 
> Christopher Spinrath (3):
>   ARM: imx: cm_fx6: env: use standard variables
>   ARM: imx: cm_fx6: env: support distro boot command
>   ARM: imx: cm_fx6: env: don't run boot scripts twice
> 
> Eran Matityahu (1):
>   imx: spl: Fix NAND bootmode detection
> 
> Eric Nelson (1):
>   mx6: Add board mx6memcal for use in validating DDR
> 
> Fabio Estevam (1):
>   mx6ull: Handle the CONFIG_MX6ULL cases correctly
> 
> Jagan Teki (2):
>   board: engicam: Fix to remove legacy board/icorem6_rqs
>   mtd: nand: mxs_nand_spl: Remove nand size print
> 
> Stefan Agner (3):
>   imx: move CONFIG_SYSCOUNTER_TIMER to Kconfig
>   imx: introduce CONFIG_GPT_TIMER
>   imx: initialize and use generic timer on i.MX 6UL/ULL

It's release day.  How many of these are release day candidate fixes?  I
know the MX6ULL thing is important, and the SPL bootmode thing looks so
too.  I do not know about the rest, are they?  Thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Fri, Dec 29, 2017 at 11:51:43AM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks! Series with patch causing warning:
> 
> Hannu Lounento (5):
>   net: e1000: add support for writing to EEPROM
> 
> is dropped from PR until issues will be fixed. This caused a rebasing of
> my tree.
> 
> 
> The following changes since commit 48a346061d1f0a7eff566f6a7cc7eeeba7efb1e2:
> 
>   Merge git://git.denx.de/u-boot-x86 (2017-12-21 09:54:10 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to d6abd1d53992cf863019aed1251924c45893a46b:
> 
>   arm: imx: add tzic interrupt controller for imx53 (2017-12-29 11:18:59
> +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Peng Fan]

> -Original Message-
> From: Fabio Estevam [mailto:feste...@gmail.com]
> Sent: Monday, December 18, 2017 8:04 AM
> To: Tom Rini <tr...@konsulko.com>; Peng Fan <peng@nxp.com>
> Cc: Stefano Babic <sba...@denx.de>; U-Boot Mailing List  b...@lists.denx.de>
> Subject: Re: [U-Boot] [PULL] Please pull u-boot-imx
> 
> Adding Peng
> 
> On Sun, Dec 17, 2017 at 5:26 PM, Tom Rini <tr...@konsulko.com> wrote:
> > On Sun, Dec 17, 2017 at 11:39:33AM +0100, Stefano Babic wrote:
> >> Hi Tom,
> >>
> >> please pull from u-boot-imx, thanks !
> >>
> >> The following changes since commit
> fcc8250c2f7c982f3593a8eecf737f8e2c95f222:
> >>
> >>   Merge git://git.denx.de/u-boot-mips (2017-11-28 16:54:30 -0500)
> >>
> >> are available in the git repository at:
> >>
> >>   git://www.denx.de/git/u-boot-imx.git master
> >
> > NAK.
> >
> > [snip]
> >> Peng Fan (2):
> >>   power: pmic.h: include dm/ofnode.h
> >>   power: pmic/regulator allow dm be omitted by SPL
> >
> > The second patch here breaks rockchip (phycore-rk3288 at least) and
> > does not introduce a 'config SPL_DM_PMIC' symbol that would allow for
> > platforms to enable that still so that it's not dropped when required.
> > If you cannot defer that patch for now (I know there are some critical
> > patches in your PR), please re-work it so we do have SPL_DM_PMIC/etc
> > so that we have no size differences on rockchip (buildman --step 0
> > -SBCdevelk foo && buildman -SsBdevelk --step 0 foo is how I do that).
> > Thanks!

Share on me. Please drop the second patch.

> 
> Peng, please fix this build issue for your v3 series.

I'll fix it in my second patch.

Thanks,
Peng.

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

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Fabio Estevam]

Adding Peng

On Sun, Dec 17, 2017 at 5:26 PM, Tom Rini  wrote:
> On Sun, Dec 17, 2017 at 11:39:33AM +0100, Stefano Babic wrote:
>> Hi Tom,
>>
>> please pull from u-boot-imx, thanks !
>>
>> The following changes since commit fcc8250c2f7c982f3593a8eecf737f8e2c95f222:
>>
>>   Merge git://git.denx.de/u-boot-mips (2017-11-28 16:54:30 -0500)
>>
>> are available in the git repository at:
>>
>>   git://www.denx.de/git/u-boot-imx.git master
>
> NAK.
>
> [snip]
>> Peng Fan (2):
>>   power: pmic.h: include dm/ofnode.h
>>   power: pmic/regulator allow dm be omitted by SPL
>
> The second patch here breaks rockchip (phycore-rk3288 at least) and does
> not introduce a 'config SPL_DM_PMIC' symbol that would allow for
> platforms to enable that still so that it's not dropped when required.
> If you cannot defer that patch for now (I know there are some critical
> patches in your PR), please re-work it so we do have SPL_DM_PMIC/etc so
> that we have no size differences on rockchip (buildman --step 0
> -SBCdevelk foo && buildman -SsBdevelk --step 0 foo is how I do that).
> Thanks!

Peng, please fix this build issue for your v3 series.

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

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

Hi Tom,

On 17/12/2017 20:26, Tom Rini wrote:
> On Sun, Dec 17, 2017 at 11:39:33AM +0100, Stefano Babic wrote:
>> Hi Tom,
>>
>> please pull from u-boot-imx, thanks !
>>
>> The following changes since commit fcc8250c2f7c982f3593a8eecf737f8e2c95f222:
>>
>>   Merge git://git.denx.de/u-boot-mips (2017-11-28 16:54:30 -0500)
>>
>> are available in the git repository at:
>>
>>   git://www.denx.de/git/u-boot-imx.git master
> 
> NAK.
> 
> [snip]
>> Peng Fan (2):
>>   power: pmic.h: include dm/ofnode.h
>>   power: pmic/regulator allow dm be omitted by SPL
> 
> The second patch here breaks rockchip (phycore-rk3288 at least) and does
> not introduce a 'config SPL_DM_PMIC' symbol that would allow for
> platforms to enable that still so that it's not dropped when required.
> If you cannot defer that patch for now

This was just a preparation for MX8, discussed with Peng. I have not
seen that it breaks rockchip, sorry. It can be dropped for the moment,
waiting until Peng will resend a new version fixing the issues.

> (I know there are some critical
> patches in your PR), please re-work it so we do have SPL_DM_PMIC/etc so
> that we have no size differences on rockchip (buildman --step 0
> -SBCdevelk foo && buildman -SsBdevelk --step 0 foo is how I do that).
> Thanks!

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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Sun, Dec 17, 2017 at 11:39:33AM +0100, Stefano Babic wrote:
> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit fcc8250c2f7c982f3593a8eecf737f8e2c95f222:
> 
>   Merge git://git.denx.de/u-boot-mips (2017-11-28 16:54:30 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master

NAK.

[snip]
> Peng Fan (2):
>   power: pmic.h: include dm/ofnode.h
>   power: pmic/regulator allow dm be omitted by SPL

The second patch here breaks rockchip (phycore-rk3288 at least) and does
not introduce a 'config SPL_DM_PMIC' symbol that would allow for
platforms to enable that still so that it's not dropped when required.
If you cannot defer that patch for now (I know there are some critical
patches in your PR), please re-work it so we do have SPL_DM_PMIC/etc so
that we have no size differences on rockchip (buildman --step 0
-SBCdevelk foo && buildman -SsBdevelk --step 0 foo is how I do that).
Thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Nov 27, 2017 at 11:04:11AM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> 
> The following changes since commit c253573f3e269fd9a24ee6684d87dd91106018a5:
> 
>   Prepare v2017.11 (2017-11-13 20:08:06 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to 5451ca4da99f09bce7a693056865c93c1182f183:
> 
>   configs: icore-rqs: Enable falcon mode (2017-11-27 10:36:40 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Thu, Nov 09, 2017 at 11:35:56AM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> some fixes for 2017.11. Please pull from u-boot-imx, thanks !
> 
> 
> The following changes since commit 7a69604bce9a9a9476753af64e5a1870880c1333:
> 
>   Prepare v2017.11-rc4 (2017-11-06 18:25:37 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to 9c8979cdb76dd65bb0983eb4cb1a895be0f0b1dc:
> 
>   imx7: Add include guards for include/asm/arch-mx7/sys_proto.h file
> (2017-11-09 11:32:49 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Tue, Oct 31, 2017 at 11:53:03AM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> please pull these fixes from u-boot-imx, thanks !
> 
> The following changes since commit b79372ae94fbc9e30d014ad8ce830d2062539eb9:
> 
>   scripts/get_maintainer.pl: enable find_maintainer_files (2017-10-29
> 10:13:10 -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 40b0dae15159bf14f24cf7714ddd690ae3ca65e3:
> 
>   mx6slevk: Call gpio_request() (2017-10-31 11:35:01 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Thu, Oct 12, 2017 at 06:29:05PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> 
> The following changes since commit 1b22c5ba496ffc9b0702919d58c410ed1527ab63:
> 
>   Merge git://git.denx.de/u-boot-i2c (2017-10-11 08:38:20 -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 0a333602df9fd9dcd6f58c8c098b29a3bd1dffcc:
> 
>   ARM: imx6: Add DHCOM i.MX6 PDK board support (2017-10-12 18:01:06 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Wed, Sep 20, 2017 at 03:47:53PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit 08cebeeaadd9192dd501308ac6a8b858ffa255c1:
> 
>   Merge git://git.denx.de/u-boot-fdt (2017-09-15 22:34:34 -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 031426a7af63d3c939fc963311e6dc8e904a0440:
> 
>   mx6sabresd: Add Serial Download Protocol support (2017-09-20 15:34:59
> +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Wed, Aug 30, 2017 at 12:35:02PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit 90ae53ce1ae665a1b43ecf5ccafc339839392427:
> 
>   fastboot: Ensure we treat CONFIG_FASTBOOT_BUF_ADDR as long (2017-08-22
> 08:20:02 -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 63af4b0ad0666e0797306593813b6272e1995bde:
> 
>   i.MX6Q: icorem6: Add falcon mode (2017-08-30 12:18:41 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Wed, Aug 16, 2017 at 01:18:40PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit f19955a01482d118eac0fdaf530ec6e5b08cf414:
> 
>   Merge git://git.denx.de/u-boot-uniphier (2017-07-26 11:29:25 -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 e872f27a020f353dd334fc025999466adf5ae18c:
> 
>   imx: mx7: fix build warning when CONFIG_IMX_RDC not enabled
> (2017-08-16 12:43:09 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Fri, Jul 14, 2017 at 05:46:02PM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> All boards compile clean, there are still warning for 9 boards dur to
> fec_get_miibus. Patch was already sent by Lothar and will follow the
> normal way through Joe's tree. At the moment, it is ok to still have
> these warnings.

OK, but we better get them resolved sooner rather than later.

> 
> The following changes since commit 8d3a25685e4aac7070365a2b3c53c2c81b27930f:
> 
>   Merge git://git.denx.de/u-boot-dm (2017-07-11 20:28:46 -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 651782a08b5bb4e710bda1b11fff774b8ade384f:
> 
>   mx6cuboxi: Move CONFIG_CMD_SATA to Kconfig (2017-07-14 15:16:01 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Tue, Jun 27, 2017 at 09:53:35AM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> this should be my last PR for the release. I checked which fixes should
> go into release and all other patches can wait. I am starting to merge
> pending patches into -next.
> 
> Most of commits are fixes (mx25, pciimx,..). Exception is pico-imx7d: I
> merged since a lot of time, and I convinced I have already sent a
> PR...sorry, for that.
> 
> For all imxers: let me know if I missed aome patches that *must* flow
> into 2017.07. IMHO all of the pending patches can wait and I merge now
> in -next.
> 
> The following changes since commit ccbbada0a59fead35495409d0c2c7bcb22a40278:
> 
>   Merge branch 'master' of git://git.denx.de/u-boot-mmc (2017-05-30
> 14:07:23 -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 65496a34835cb4c9547bd02dd15b018c333add9d:
> 
>   mx6: soc: Fix typo in temperature unit name (2017-06-27 09:06:09 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Thu, Apr 13, 2017 at 10:01:28AM +0200, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit d5abcf94c7123167725fc22ace342f0d455093c1:
> 
>   ti: boot: Register the MMC controllers in SPL in the same way as in
> u-boot (2017-03-19 22:17:14 -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 f479cec3b623778c26b23f66dc28cf33100ce089:
> 
>   imx: i.mx6q: add the initial support for LogicPD i.MX6Q SOM
> (2017-04-12 18:59:12 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Sun, Mar 19, 2017 at 06:07:47PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit 2808576491ae36b6ea96743005058f370d936beb:
> 
>   arm64: booti: allow to place kernel image anywhere in physical memory
> (2017-03-14 20:40:23 -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 d883fcc6bbb2fcc3df90857fee99c2f543a0289c:
> 
>   imx: ventana: add EMMC configuration (2017-03-19 17:39:59 +0100)
> 

With a small fixup to arch/arm/dts/imx6ul-isiot.dtsi, applied to
u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Feb 27, 2017 at 06:08:59PM +0100, Stefano Babic wrote:

> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> 
> The following changes since commit 66c246cce7c66019a93ff7105157c3e2126dd277:
> 
>   ARM: DRA7xx: Fix memory allocation overflow (2017-02-17 17:24:35 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to dc05e47a10b11f4266e3a41df96748d486193e6d:
> 
>   tqma6: [cosmetic] sanitize environment defines (2017-02-19 17:16:51 +0100)
> 

Applied prior to v2017.03-rc3 to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Tue, Feb 28, 2017 at 09:39:01AM +0530, Jagan Teki wrote:
> On Mon, Feb 27, 2017 at 2:42 PM, Stefano Babic  wrote:
> > Hi Tom,
> >
> > please pull from u-boot-imx, thanks !
> >
> > The following changes since commit b24cf8540a85a9bf97975aadd6a7542f166c78a3:
> >
> >   video: mxsfb: Fix reset hang when videomode variable is not present
> > (2017-02-22 21:47:59 +0100)
> >
> > are available in the git repository at:
> >
> >   git://www.denx.de/git/u-boot-imx.git master
> >
> > for you to fetch changes up to 60a38e423b21a897b8fa0d9376d9370ea915ddd8:
> >
> >   i.MX6Q: isiot: Switch the mmc env based on devno (2017-02-26 13:00:22
> > +0100)
> >
> > 
> > Andrey Yurovsky (1):
> >   mtd: nand: build MXS driver for MX7 as well
> >
> > Jagan Teki (25):
> >   configs: imx6: Don't define USDHC2_BASE_ADDR
> >   arm: imx6ul: Add Engicam Is.IoT MX6UL Starter Kit initial support
> >   arm: dts: imx6ul-isiot: Add I2C nodes
> >   imx6: isiotmx6ul: Add I2C support
> >   arm: dts: imx6ul-isiot: Add FEC node
> >   imx6: isiotmx6ul: Add FEC support
> >   imx6: isiotmx6ul: Add NAND support
> >   imx6: isiotmx6ul: Add nandboot env support
> >   imx6ul: isiotmx6ul: Enable I2C support
> >   i.MX6: engicam: Include dts files under MAINTAINERS
> >   imx6: Add imx6_src_get_boot_mode
> >   imx: spl: Update NAND bootmode detection bit
> >   imx: Use IMX6_BMODE_* macros instead of numericals
> >   imx6: Add src_base structure define macro
> >   imx6: isiotmx6ul: Update SPL board boot order for eMMC
> >   i.MX6UL: isiot: Add eMMC boot support
> >   i.MX6UL: isiot: Add modeboot env via board_late_init
> >   i.MX6UL: isiot: Add mmc_late_init
> >   i.MX6UL: isiot: Switch the mmc env based on devno
> >   arm: dts: imx6qdl-icore-rqs: Add eMMC node
> >   imx6: icorem6_rqs: Update SPL board boot order for eMMC
> >   imx6: icorem6_rqs: Add eMMC boot support
> >   i.MX6Q: icorem6_rqs: Add modeboot env via board_late_init
> >   i.MX6Q: icorem6_rqs: Add mmc_late_init
> >   i.MX6Q: isiot: Switch the mmc env based on devno
> 
> These patches were initiated during early on MW, any possibility to
> merge these as well?

As I was saying in my -rc3 release email, no.  The next window is just
around the corner.

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Jagan Teki]

On Mon, Feb 27, 2017 at 2:42 PM, Stefano Babic  wrote:
> Hi Tom,
>
> please pull from u-boot-imx, thanks !
>
> The following changes since commit b24cf8540a85a9bf97975aadd6a7542f166c78a3:
>
>   video: mxsfb: Fix reset hang when videomode variable is not present
> (2017-02-22 21:47:59 +0100)
>
> are available in the git repository at:
>
>   git://www.denx.de/git/u-boot-imx.git master
>
> for you to fetch changes up to 60a38e423b21a897b8fa0d9376d9370ea915ddd8:
>
>   i.MX6Q: isiot: Switch the mmc env based on devno (2017-02-26 13:00:22
> +0100)
>
> 
> Andrey Yurovsky (1):
>   mtd: nand: build MXS driver for MX7 as well
>
> Jagan Teki (25):
>   configs: imx6: Don't define USDHC2_BASE_ADDR
>   arm: imx6ul: Add Engicam Is.IoT MX6UL Starter Kit initial support
>   arm: dts: imx6ul-isiot: Add I2C nodes
>   imx6: isiotmx6ul: Add I2C support
>   arm: dts: imx6ul-isiot: Add FEC node
>   imx6: isiotmx6ul: Add FEC support
>   imx6: isiotmx6ul: Add NAND support
>   imx6: isiotmx6ul: Add nandboot env support
>   imx6ul: isiotmx6ul: Enable I2C support
>   i.MX6: engicam: Include dts files under MAINTAINERS
>   imx6: Add imx6_src_get_boot_mode
>   imx: spl: Update NAND bootmode detection bit
>   imx: Use IMX6_BMODE_* macros instead of numericals
>   imx6: Add src_base structure define macro
>   imx6: isiotmx6ul: Update SPL board boot order for eMMC
>   i.MX6UL: isiot: Add eMMC boot support
>   i.MX6UL: isiot: Add modeboot env via board_late_init
>   i.MX6UL: isiot: Add mmc_late_init
>   i.MX6UL: isiot: Switch the mmc env based on devno
>   arm: dts: imx6qdl-icore-rqs: Add eMMC node
>   imx6: icorem6_rqs: Update SPL board boot order for eMMC
>   imx6: icorem6_rqs: Add eMMC boot support
>   i.MX6Q: icorem6_rqs: Add modeboot env via board_late_init
>   i.MX6Q: icorem6_rqs: Add mmc_late_init
>   i.MX6Q: isiot: Switch the mmc env based on devno

These patches were initiated during early on MW, any possibility to
merge these as well?

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Feb 27, 2017 at 06:02:18PM +0100, Stefano Babic wrote:
> On 27/02/2017 17:41, Fabio Estevam wrote:
> > Hi Stefano,
> > 
> > On Mon, Feb 27, 2017 at 10:29 AM, Stefano Babic  wrote:
> > 
> >> ok - I push it to -next, and I send my PR again after release.
> > 
> > Peter Robinson's patches are bug fixes for mx6sx udoo neo to work
> > correctly with distro config.
> > 
> > Can we get them in for 2017.03?
> > 
> > Thanks
> > 
> 
> In the tree together with Peter's there are also Markus', just fixes and
> related to tqma6. I will propose I reset my tree to
> 76cb4dbc5c1d33d1f251af2553db47d88c3e6a45, this generate a small PR as:
> 
> The following changes since commit b24cf8540a85a9bf97975aadd6a7542f166c78a3:
> 
>   video: mxsfb: Fix reset hang when videomode variable is not present
> (2017-02-22 21:47:59 +0100)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git
> 76cb4dbc5c1d33d1f251af2553db47d88c3e6a45
> 
> for you to fetch changes up to 76cb4dbc5c1d33d1f251af2553db47d88c3e6a45:
> 
>   Merge branch 'master' of git://git.denx.de/u-boot (2017-02-25 18:52:37
> +0100)
> 
> 
> Andrey Yurovsky (1):
>   mtd: nand: build MXS driver for MX7 as well
> 
> Markus Niebel (6):
>   arm: imx6: tqma6: add configurable CMA size
>   imx6: tqma6: rely on default setting for tftp and nfs
>   mx6: tqma6: fix typo in env
>   mx6: tqma6: add rootfsmode environment for mmc / sd
>   tqma6: fix rounding in env
>   tqma6: [cosmetic] sanitize environment defines
> 
> Peter Robinson (3):
>   mx6sx: udoo_neo: Define the default serial console
>   mx6sx: udoo_neo: use different load address for ramdisk
>   mx6sx: udoo_neo: Enable distro boot options in config
> 
> Stefano Babic (1):
>   Merge branch 'master' of git://git.denx.de/u-boot
> 
> 
> Tom, should this ok ?

Drop the merge commit and then yes, looks reasonable, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

On 27/02/2017 17:41, Fabio Estevam wrote:
> Hi Stefano,
> 
> On Mon, Feb 27, 2017 at 10:29 AM, Stefano Babic  wrote:
> 
>> ok - I push it to -next, and I send my PR again after release.
> 
> Peter Robinson's patches are bug fixes for mx6sx udoo neo to work
> correctly with distro config.
> 
> Can we get them in for 2017.03?
> 
> Thanks
> 

In the tree together with Peter's there are also Markus', just fixes and
related to tqma6. I will propose I reset my tree to
76cb4dbc5c1d33d1f251af2553db47d88c3e6a45, this generate a small PR as:

The following changes since commit b24cf8540a85a9bf97975aadd6a7542f166c78a3:

  video: mxsfb: Fix reset hang when videomode variable is not present
(2017-02-22 21:47:59 +0100)

are available in the git repository at:

  git://www.denx.de/git/u-boot-imx.git
76cb4dbc5c1d33d1f251af2553db47d88c3e6a45

for you to fetch changes up to 76cb4dbc5c1d33d1f251af2553db47d88c3e6a45:

  Merge branch 'master' of git://git.denx.de/u-boot (2017-02-25 18:52:37
+0100)


Andrey Yurovsky (1):
  mtd: nand: build MXS driver for MX7 as well

Markus Niebel (6):
  arm: imx6: tqma6: add configurable CMA size
  imx6: tqma6: rely on default setting for tftp and nfs
  mx6: tqma6: fix typo in env
  mx6: tqma6: add rootfsmode environment for mmc / sd
  tqma6: fix rounding in env
  tqma6: [cosmetic] sanitize environment defines

Peter Robinson (3):
  mx6sx: udoo_neo: Define the default serial console
  mx6sx: udoo_neo: use different load address for ramdisk
  mx6sx: udoo_neo: Enable distro boot options in config

Stefano Babic (1):
  Merge branch 'master' of git://git.denx.de/u-boot


Tom, should this ok ?

Stefano

-- 
Meet DENX at the Embedded World Trade Show
14 Mar - 16 Mar 2017, Nuremberg Trade Fair Centre, Hall 4, Booth 581
--
=
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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Fabio Estevam]

Hi Stefano,

On Mon, Feb 27, 2017 at 10:29 AM, Stefano Babic  wrote:

> ok - I push it to -next, and I send my PR again after release.

Peter Robinson's patches are bug fixes for mx6sx udoo neo to work
correctly with distro config.

Can we get them in for 2017.03?

Thanks
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Peter Robinson]

On Mon, Feb 27, 2017 at 1:29 PM, Stefano Babic  wrote:
> On 27/02/2017 14:22, Tom Rini wrote:
>> On Mon, Feb 27, 2017 at 10:12:59AM +0100, Stefano Babic wrote:
>>> Hi Tom,
>>>
>>> please pull from u-boot-imx, thanks !
>>>
>>> The following changes since commit b24cf8540a85a9bf97975aadd6a7542f166c78a3:
>>>
>>>   video: mxsfb: Fix reset hang when videomode variable is not present
>>> (2017-02-22 21:47:59 +0100)
>>>
>>> are available in the git repository at:
>>>
>>>   git://www.denx.de/git/u-boot-imx.git master
>>>
>>> for you to fetch changes up to 60a38e423b21a897b8fa0d9376d9370ea915ddd8:
>>>
>>>   i.MX6Q: isiot: Switch the mmc env based on devno (2017-02-26 13:00:22
>>> +0100)
>>>
>>> 
>
>>
>> This is too much too close to the release window, sorry.
>>
>
> ok - I push it to -next, and I send my PR again after release.

Maybe do a more focuses -fixes PR?
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Feb 27, 2017 at 02:29:38PM +0100, Stefano Babic wrote:
> On 27/02/2017 14:22, Tom Rini wrote:
> > On Mon, Feb 27, 2017 at 10:12:59AM +0100, Stefano Babic wrote:
> >> Hi Tom,
> >>
> >> please pull from u-boot-imx, thanks !
> >>
> >> The following changes since commit 
> >> b24cf8540a85a9bf97975aadd6a7542f166c78a3:
> >>
> >>   video: mxsfb: Fix reset hang when videomode variable is not present
> >> (2017-02-22 21:47:59 +0100)
> >>
> >> are available in the git repository at:
> >>
> >>   git://www.denx.de/git/u-boot-imx.git master
> >>
> >> for you to fetch changes up to 60a38e423b21a897b8fa0d9376d9370ea915ddd8:
> >>
> >>   i.MX6Q: isiot: Switch the mmc env based on devno (2017-02-26 13:00:22
> >> +0100)
> >>
> >> 
> 
> > 
> > This is too much too close to the release window, sorry.
> > 
> 
> ok - I push it to -next, and I send my PR again after release.

Thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Stefano Babic]

On 27/02/2017 14:22, Tom Rini wrote:
> On Mon, Feb 27, 2017 at 10:12:59AM +0100, Stefano Babic wrote:
>> Hi Tom,
>>
>> please pull from u-boot-imx, thanks !
>>
>> The following changes since commit b24cf8540a85a9bf97975aadd6a7542f166c78a3:
>>
>>   video: mxsfb: Fix reset hang when videomode variable is not present
>> (2017-02-22 21:47:59 +0100)
>>
>> are available in the git repository at:
>>
>>   git://www.denx.de/git/u-boot-imx.git master
>>
>> for you to fetch changes up to 60a38e423b21a897b8fa0d9376d9370ea915ddd8:
>>
>>   i.MX6Q: isiot: Switch the mmc env based on devno (2017-02-26 13:00:22
>> +0100)
>>
>> 

> 
> This is too much too close to the release window, sorry.
> 

ok - I push it to -next, and I send my PR again after release.

Regards,
Stefano


-- 
Meet DENX at the Embedded World Trade Show
14 Mar - 16 Mar 2017, Nuremberg Trade Fair Centre, Hall 4, Booth 581
--
=
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: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-imx

[Tom Rini]

On Mon, Feb 27, 2017 at 10:12:59AM +0100, Stefano Babic wrote:
> Hi Tom,
> 
> please pull from u-boot-imx, thanks !
> 
> The following changes since commit b24cf8540a85a9bf97975aadd6a7542f166c78a3:
> 
>   video: mxsfb: Fix reset hang when videomode variable is not present
> (2017-02-22 21:47:59 +0100)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-imx.git master
> 
> for you to fetch changes up to 60a38e423b21a897b8fa0d9376d9370ea915ddd8:
> 
>   i.MX6Q: isiot: Switch the mmc env based on devno (2017-02-26 13:00:22
> +0100)
> 
> 
> Andrey Yurovsky (1):
>   mtd: nand: build MXS driver for MX7 as well
> 
> Jagan Teki (25):
>   configs: imx6: Don't define USDHC2_BASE_ADDR
>   arm: imx6ul: Add Engicam Is.IoT MX6UL Starter Kit initial support
>   arm: dts: imx6ul-isiot: Add I2C nodes
>   imx6: isiotmx6ul: Add I2C support
>   arm: dts: imx6ul-isiot: Add FEC node
>   imx6: isiotmx6ul: Add FEC support
>   imx6: isiotmx6ul: Add NAND support
>   imx6: isiotmx6ul: Add nandboot env support
>   imx6ul: isiotmx6ul: Enable I2C support
>   i.MX6: engicam: Include dts files under MAINTAINERS
>   imx6: Add imx6_src_get_boot_mode
>   imx: spl: Update NAND bootmode detection bit
>   imx: Use IMX6_BMODE_* macros instead of numericals
>   imx6: Add src_base structure define macro
>   imx6: isiotmx6ul: Update SPL board boot order for eMMC
>   i.MX6UL: isiot: Add eMMC boot support
>   i.MX6UL: isiot: Add modeboot env via board_late_init
>   i.MX6UL: isiot: Add mmc_late_init
>   i.MX6UL: isiot: Switch the mmc env based on devno
>   arm: dts: imx6qdl-icore-rqs: Add eMMC node
>   imx6: icorem6_rqs: Update SPL board boot order for eMMC
>   imx6: icorem6_rqs: Add eMMC boot support
>   i.MX6Q: icorem6_rqs: Add modeboot env via board_late_init
>   i.MX6Q: icorem6_rqs: Add mmc_late_init
>   i.MX6Q: isiot: Switch the mmc env based on devno
> 
> Markus Niebel (13):
>   arm: imx6: tqma6: add configurable CMA size
>   imx6: tqma6: rely on default setting for tftp and nfs
>   mx6: tqma6: fix typo in env
>   mx6: tqma6: add rootfsmode environment for mmc / sd
>   tqma6: fix rounding in env
>   tqma6: [cosmetic] sanitize environment defines
>   arm: imx6: tqma6: use CONFIG_TQM6x for SOM specific settings
>   arm: imx6: tqma6: add support for TQMa6DL variant
>   imx6: tqma6: implement power_init_board
>   imx6: tqma6: use lower driver stength for I2C pins
>   imx6: tqma6: disable spi CS unused in U-Boot
>   imx6: tqma6: adjust ethernet phy reset delay
>   mx6: tqma6: clear enet clk sel for mba6
> 
> Peng Fan (16):
>   imx: mx7ulp: Add mx7ulp to Kconfig
>   imx: mx7ulp: add registers header file
>   imx: mx7ulp: add iomux driver to support IOMUXC0 and IOMUXC1
>   imx: mx7ulp: Add clock framework and functions
>   imx: mx7ulp: Add soc level initialization codes and functions
>   gpio: Add Rapid GPIO2P driver for i.MX7ULP
>   mxc_ocotp: Update driver to support OCOTP controller on i.MX7ULP
>   pinctrl: Add i.MX7ULP pinctrl driver
>   i2c: lpi2c: add lpi2c driver for i.MX7ULP
>   serial: lpuart: restructure lpuart driver
>   serial: lpuart: add i.MX7ULP support
>   mx7ulp: Add HAB boot support
>   arm: dts: add i.MX7ULP dtsi file
>   mmc: fsl_esdhc: support i.MX7ULP
>   imx: imx7ulp: add EVK board support
>   imx: mx7ulp_evk: enable mmc/regulator support
> 
> Peter Robinson (3):
>   mx6sx: udoo_neo: Define the default serial console
>   mx6sx: udoo_neo: use different load address for ramdisk
>   mx6sx: udoo_neo: Enable distro boot options in config
> 
> Stefano Babic (1):
>   Merge branch 'master' of git://git.denx.de/u-boot
> 
> Ye Li (3):
>   imx: mx7ulp: Implement the clock functions for i2c driver
>   mx7ulp: Add iomux pins header file
>   wdog: Add the watchdog driver for MX7ULP.
> 
>  arch/arm/Kconfig|7 +
>  arch/arm/Makefile   |2 +-
>  arch/arm/cpu/armv7/Makefile |3 +-
>  arch/arm/cpu/armv7/mx6/Kconfig  |   12 +
>  arch/arm/cpu/armv7/mx7ulp/Kconfig   |   17 ++
>  arch/arm/cpu/armv7/mx7ulp/Makefile  |8 +
>  arch/arm/cpu/armv7/mx7ulp/clock.c   |  365
> +++
>  arch/arm/cpu/armv7/mx7ulp/iomux.c   |   70 ++
>  arch/arm/cpu/armv7/mx7ulp/pcc.c |  286
> +
>  arch/arm/cpu/armv7/mx7ulp/scg.c | 1086
> ++
>  arch/arm/cpu/armv7/mx7ulp/soc.c |  247
> ++
>  arch/arm/cpu/armv7/vf610/generic.c  |5 +
>