Re: [U-Boot] [PULL] Please pull u-boot-imx: u-boot-imx-20181217
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
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
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
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
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
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
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
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
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
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
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
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
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(®s->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:
Re: [U-Boot] [PULL] Please pull u-boot-imx
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(®s->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(®s->irqstat) & flags)) - ; + start = get_timer(0); + while (!(esdhc_read32(®s->irqstat) & flags)) { + if (get_timer(start) > 1000) + return -ETIMEDOUT; + } irqstat = esdhc_read32(®s->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
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
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(®s->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/mm
Re: [U-Boot] [PULL] Please pull u-boot-imx
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(®s->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
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
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
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
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
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
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
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
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
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 | 2
Re: [U-Boot] [PULL] Please pull u-boot-imx
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
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
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
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
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
> > 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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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 + >> configs/mx6ul_14x14_evk_defconfi
Re: [U-Boot] [PULL] Please pull u-boot-imx
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
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
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
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 +- > configs/mx7dsabresd_qspi_defconfig
Re: [U-Boot] [PULL] Please pull u-boot-imx
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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(&mem_qdl, &mx6dl_1g_mmdc_calib, > &mt41k128m16jt_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
Re: [U-Boot] [PULL] Please pull u-boot-imx
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(&mem_qdl, &mx6dl_1g_mmdc_calib, &mt41k128m16jt_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__GPIO
Re: [U-Boot] [PULL] Please pull u-boot-imx
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
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
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
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
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
> -Original Message- > From: Fabio Estevam [mailto:feste...@gmail.com] > Sent: Monday, December 18, 2017 8:04 AM > To: Tom Rini ; Peng Fan > Cc: Stefano Babic ; 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 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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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 + > arch/arm/dts/M