Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On Sat, Feb 9, 2019 at 8:48 PM Abel Vesa wrote: > So, do I need to resend this with all the R-bs ? Just realized that you missed to Cc Stefano Babic on this series. Stefano, Should Abel resend this series with you on Cc or can you get review it from patchwork? Thanks ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On 19-02-04 08:19:23, Tom Rini wrote: > On Mon, Feb 04, 2019 at 09:03:43AM -0200, Fabio Estevam wrote: > > On Mon, Feb 4, 2019 at 7:55 AM Abel Vesa wrote: > > > > > If the SPL size (without the dtb appended) is larger then yes, the build > > > fails. > > > Trouble is if the SPL (without the dtb appended) is, lets say, 63kB and > > > then the dtb is larger than 1kB. Then there is no mechanism in place to > > > check that > > > and it will just fail to boot without giving any clues why. But this is a > > > totally > > > unrelated problem from this patchset's point of view and I think it > > > impacts all > > > the platforms that support SPL with DM. > > > > Yes, it is unrelated to this series and should be treated separately. > > I also agree. But, don't we have a mechanism for that? It sounds like > some additional targets need to call $(BOARD_SIZE_CHECK) at the end. Or > am I missing something? > So, do I need to resend this with all the R-bs ? > -- > Tom ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On Mon, Feb 04, 2019 at 04:21:37PM +0100, Simon Goldschmidt wrote: > Am Mo., 4. Feb. 2019, 16:16 hat Abel Vesa geschrieben: > > > On 19-02-04 08:19:23, Tom Rini wrote: > > > On Mon, Feb 04, 2019 at 09:03:43AM -0200, Fabio Estevam wrote: > > > > On Mon, Feb 4, 2019 at 7:55 AM Abel Vesa wrote: > > > > > > > > > If the SPL size (without the dtb appended) is larger then yes, the > > build fails. > > > > > Trouble is if the SPL (without the dtb appended) is, lets say, 63kB > > and > > > > > then the dtb is larger than 1kB. Then there is no mechanism in place > > to check that > > > > > and it will just fail to boot without giving any clues why. But this > > is a totally > > > > > unrelated problem from this patchset's point of view and I think it > > impacts all > > > > > the platforms that support SPL with DM. > > > > > > > > Yes, it is unrelated to this series and should be treated separately. > > > > > > I also agree. But, don't we have a mechanism for that? It sounds like > > > some additional targets need to call $(BOARD_SIZE_CHECK) at the end. Or > > > am I missing something? > > > > > > > Hmm, I believe that is true. I haven't looked deeper into it but it seems > > that's the thing the SPL is missing. > > > Haven't we talked about that just some days or weeks ago? That's exactly > what I am missing for socfpga. I keep getting broken SPL images when adding > functionality. And you don't even get an error message because the dtb is > broken and cannot provide the serial output mode... Yes, I believe it is. I guess what I'm waiting / hoping for is someone to patch the top-level Makefile to call $(BOARD_SIZE_CHECK) on a number of make targets that look reasonable, throw it at travis and see what if anything fails (and if stuff fails, ask for help to see if it's right or investigate a bit more). More or less, if it's a final target for U-Boot, we should probably give BOARD_SIZE_CHECK a try. Most of the rules start with u-boot, but I bet the %.imx target should also do it, and then we can drop all of that from arch/arm/mach-imx/Makefile. -- 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] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
Am Mo., 4. Feb. 2019, 16:16 hat Abel Vesa geschrieben: > On 19-02-04 08:19:23, Tom Rini wrote: > > On Mon, Feb 04, 2019 at 09:03:43AM -0200, Fabio Estevam wrote: > > > On Mon, Feb 4, 2019 at 7:55 AM Abel Vesa wrote: > > > > > > > If the SPL size (without the dtb appended) is larger then yes, the > build fails. > > > > Trouble is if the SPL (without the dtb appended) is, lets say, 63kB > and > > > > then the dtb is larger than 1kB. Then there is no mechanism in place > to check that > > > > and it will just fail to boot without giving any clues why. But this > is a totally > > > > unrelated problem from this patchset's point of view and I think it > impacts all > > > > the platforms that support SPL with DM. > > > > > > Yes, it is unrelated to this series and should be treated separately. > > > > I also agree. But, don't we have a mechanism for that? It sounds like > > some additional targets need to call $(BOARD_SIZE_CHECK) at the end. Or > > am I missing something? > > > > Hmm, I believe that is true. I haven't looked deeper into it but it seems > that's the thing the SPL is missing. Haven't we talked about that just some days or weeks ago? That's exactly what I am missing for socfpga. I keep getting broken SPL images when adding functionality. And you don't even get an error message because the dtb is broken and cannot provide the serial output mode... Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On 19-02-04 08:19:23, Tom Rini wrote: > On Mon, Feb 04, 2019 at 09:03:43AM -0200, Fabio Estevam wrote: > > On Mon, Feb 4, 2019 at 7:55 AM Abel Vesa wrote: > > > > > If the SPL size (without the dtb appended) is larger then yes, the build > > > fails. > > > Trouble is if the SPL (without the dtb appended) is, lets say, 63kB and > > > then the dtb is larger than 1kB. Then there is no mechanism in place to > > > check that > > > and it will just fail to boot without giving any clues why. But this is a > > > totally > > > unrelated problem from this patchset's point of view and I think it > > > impacts all > > > the platforms that support SPL with DM. > > > > Yes, it is unrelated to this series and should be treated separately. > > I also agree. But, don't we have a mechanism for that? It sounds like > some additional targets need to call $(BOARD_SIZE_CHECK) at the end. Or > am I missing something? > Hmm, I believe that is true. I haven't looked deeper into it but it seems that's the thing the SPL is missing. > -- > Tom ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On Mon, Feb 04, 2019 at 09:03:43AM -0200, Fabio Estevam wrote: > On Mon, Feb 4, 2019 at 7:55 AM Abel Vesa wrote: > > > If the SPL size (without the dtb appended) is larger then yes, the build > > fails. > > Trouble is if the SPL (without the dtb appended) is, lets say, 63kB and > > then the dtb is larger than 1kB. Then there is no mechanism in place to > > check that > > and it will just fail to boot without giving any clues why. But this is a > > totally > > unrelated problem from this patchset's point of view and I think it impacts > > all > > the platforms that support SPL with DM. > > Yes, it is unrelated to this series and should be treated separately. I also agree. But, don't we have a mechanism for that? It sounds like some additional targets need to call $(BOARD_SIZE_CHECK) at the end. Or am I missing something? -- 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] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On Mon, Feb 4, 2019 at 7:55 AM Abel Vesa wrote: > If the SPL size (without the dtb appended) is larger then yes, the build > fails. > Trouble is if the SPL (without the dtb appended) is, lets say, 63kB and > then the dtb is larger than 1kB. Then there is no mechanism in place to check > that > and it will just fail to boot without giving any clues why. But this is a > totally > unrelated problem from this patchset's point of view and I think it impacts > all > the platforms that support SPL with DM. Yes, it is unrelated to this series and should be treated separately. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On 19-02-02 07:50:28, Fabio Estevam wrote: > Hi Abel, > > On Fri, Feb 1, 2019 at 2:43 PM Abel Vesa wrote: > > > > The third version is here: > > https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.denx.de%2Fpipermail%2Fu-boot%2F2019-January%2F356903.htmldata=02%7C01%7Cabel.vesa%40nxp.com%7Ca93c33315b5b492ca15308d688f3e275%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636846978387908349sdata=wQrrLoMyouM5vdUlfXI325wqZYt4tTa5WTRJsgwMEz0%3Dreserved=0 > > > > So, this time I hope I got it right. Before, I was stupidly trying > > to put a fit in another fit without a really good reason. To my > > excuse, that was working even with the spl_image->os set to 0, > > bug which I (hope) I fixed in the first patch (a new one) > > of this series. > > I am happy with the entire series. > > One more question: do we get build error when the SPL gets larger than 64kB? If the SPL size (without the dtb appended) is larger then yes, the build fails. Trouble is if the SPL (without the dtb appended) is, lets say, 63kB and then the dtb is larger than 1kB. Then there is no mechanism in place to check that and it will just fail to boot without giving any clues why. But this is a totally unrelated problem from this patchset's point of view and I think it impacts all the platforms that support SPL with DM. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
Hi Abel, On Fri, Feb 1, 2019 at 2:43 PM Abel Vesa wrote: > > The third version is here: > https://lists.denx.de/pipermail/u-boot/2019-January/356903.html > > So, this time I hope I got it right. Before, I was stupidly trying > to put a fit in another fit without a really good reason. To my > excuse, that was working even with the spl_image->os set to 0, > bug which I (hope) I fixed in the first patch (a new one) > of this series. I am happy with the entire series. One more question: do we get build error when the SPL gets larger than 64kB? ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On Fri, Feb 01, 2019 at 04:40:05PM +, Abel Vesa wrote: > The third version is here: > https://lists.denx.de/pipermail/u-boot/2019-January/356903.html > > So, this time I hope I got it right. Before, I was stupidly trying > to put a fit in another fit without a really good reason. To my > excuse, that was working even with the spl_image->os set to 0, > bug which I (hope) I fixed in the first patch (a new one) > of this series. > > Changes since v3: > * Reduced the SPL size by disabling EXT_SUPPORT, otherwise the >6qp-sabresd wouldn't boot due to having its dtb last in the fit. >I forgot to mention in the last version that EFI and DOS SPL support >also had to be disabled to get the size right. > * Dropped the fit_spl.its file > * Dropped the (now) unnecessary updates to the README files. We're >sticking to the u-boot.img since it's exactly what is needed. > * Fixed a bug related to FIT_IMAGE_TINY when CONFIG_SPL_OS_BOOT >is enabled. The short story here is that spl_image->os was >remaining set to 0. (see first patch in the series for details) Thanks for being persistent and figuring all of this out! -- 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] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On Fri, Feb 1, 2019 at 3:59 PM Abel Vesa wrote: > > Is Ethernet functional? > > I haven't tested the ethernet on all of them, just one of each > (sabresd/sabreauto). > > I'll test them all when I get back at the office tomorrow. Don't worry about that. If you tested in one of each then it is fine. Thanks ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
On 19-02-01 15:13:32, Fabio Estevam wrote: > Hi Abel, > > On Fri, Feb 1, 2019 at 2:43 PM Abel Vesa wrote: > > > > The third version is here: > > https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.denx.de%2Fpipermail%2Fu-boot%2F2019-January%2F356903.htmldata=02%7C01%7Cabel.vesa%40nxp.com%7Cc7db2be273394c06098308d688689e41%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636846380248922351sdata=fhDqNxkE1dq0fGNroGRR%2F0OkmBPRlVBqCefS6GM8kns%3Dreserved=0 > > > > So, this time I hope I got it right. Before, I was stupidly trying > > to put a fit in another fit without a really good reason. To my > > excuse, that was working even with the spl_image->os set to 0, > > bug which I (hope) I fixed in the first patch (a new one) > > of this series. > > Thanks for your work on this! > > Just to confirm: with this series all sabresd variants (imx6q, imx6dl, > imx6qp) boot with the same binary, right? > That is correct. For mx6sabresd_defconfig, all imx6q_sabresd, imx6qp_sabresd and imx6dl_sabresd have been tested with the same binary. And I've tested exactly the same the mx6sabreauto_defconfig with all three SoC combinations. > Is there anything known to be not working that used to work before due > to missing driver model conversion? > I don't know about anything that might not work. As specified in the cover letter I had to get rid (disable) of EFI, EXT and DOS support in SPL. > Is Ethernet functional? I haven't tested the ethernet on all of them, just one of each (sabresd/sabreauto). I'll test them all when I get back at the office tomorrow. > > Thanks ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support
Hi Abel, On Fri, Feb 1, 2019 at 2:43 PM Abel Vesa wrote: > > The third version is here: > https://lists.denx.de/pipermail/u-boot/2019-January/356903.html > > So, this time I hope I got it right. Before, I was stupidly trying > to put a fit in another fit without a really good reason. To my > excuse, that was working even with the spl_image->os set to 0, > bug which I (hope) I fixed in the first patch (a new one) > of this series. Thanks for your work on this! Just to confirm: with this series all sabresd variants (imx6q, imx6dl, imx6qp) boot with the same binary, right? Is there anything known to be not working that used to work before due to missing driver model conversion? Is Ethernet functional? Thanks ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot