* mkimage_fit_atf.sh: not found @ 2022-01-04 22:48 Tim Harvey 2022-01-04 22:54 ` Fabio Estevam 2022-01-05 11:34 ` ZHIZHIKIN Andrey 0 siblings, 2 replies; 17+ messages in thread From: Tim Harvey @ 2022-01-04 22:48 UTC (permalink / raw) To: u-boot, Stefano Babic, Fabio Estevam Cc: Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki Stefano and Fabio, I'm seeing the imx8mm_venice_defconfig target failing to build on master due to mkimage_fit_atf.sh not found: ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb arch/arm/dts/imx8mm-venice-gw7901.dtb arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found As far as I can tell the other boards that are still using SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). What is the state of the binman conversion? I submitted a series to convert my boards to binman and it has just been sitting without any response for months now [1]. I'm not sure when the above breakage occurred but the conversion to binman resolves it and other things. Please let me know what you expect me to do to resolve this as there is a release pending. Best regards, Tim [1] https://patchwork.ozlabs.org/project/uboot/patch/20211006201700.3018-1-tharvey@gateworks.com/ ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-04 22:48 mkimage_fit_atf.sh: not found Tim Harvey @ 2022-01-04 22:54 ` Fabio Estevam 2022-01-05 13:16 ` Tom Rini 2022-01-05 11:34 ` ZHIZHIKIN Andrey 1 sibling, 1 reply; 17+ messages in thread From: Fabio Estevam @ 2022-01-04 22:54 UTC (permalink / raw) To: Tim Harvey, Tom Rini Cc: u-boot, Stefano Babic, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki HI Tim, On Tue, Jan 4, 2022 at 7:48 PM Tim Harvey <tharvey@gateworks.com> wrote: > > Stefano and Fabio, > > I'm seeing the imx8mm_venice_defconfig target failing to build on > master due to mkimage_fit_atf.sh not found: > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > arch/arm/dts/imx8mm-venice-gw7901.dtb > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > As far as I can tell the other boards that are still using > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > What is the state of the binman conversion? I submitted a series to > convert my boards to binman and it has just been sitting without any > response for months now [1]. > > I'm not sure when the above breakage occurred but the conversion to > binman resolves it and other things. > > Please let me know what you expect me to do to resolve this as there > is a release pending. > > Best regards, > > Tim > [1] https://patchwork.ozlabs.org/project/uboot/patch/20211006201700.3018-1-tharvey@gateworks.com/ Stefano is on vacation. Tom, would you mind picking Tim's series? Thanks ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-04 22:54 ` Fabio Estevam @ 2022-01-05 13:16 ` Tom Rini 0 siblings, 0 replies; 17+ messages in thread From: Tom Rini @ 2022-01-05 13:16 UTC (permalink / raw) To: Fabio Estevam Cc: Tim Harvey, u-boot, Stefano Babic, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki [-- Attachment #1: Type: text/plain, Size: 1666 bytes --] On Tue, Jan 04, 2022 at 07:54:38PM -0300, Fabio Estevam wrote: > HI Tim, > > On Tue, Jan 4, 2022 at 7:48 PM Tim Harvey <tharvey@gateworks.com> wrote: > > > > Stefano and Fabio, > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > master due to mkimage_fit_atf.sh not found: > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > As far as I can tell the other boards that are still using > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > What is the state of the binman conversion? I submitted a series to > > convert my boards to binman and it has just been sitting without any > > response for months now [1]. > > > > I'm not sure when the above breakage occurred but the conversion to > > binman resolves it and other things. > > > > Please let me know what you expect me to do to resolve this as there > > is a release pending. > > > > Best regards, > > > > Tim > > [1] https://patchwork.ozlabs.org/project/uboot/patch/20211006201700.3018-1-tharvey@gateworks.com/ > > Stefano is on vacation. Tom, would you mind picking Tim's series? Looking at the thread, there's a few pre-requisite patches? And is this also one of the "switch to binman" patches that then fails in CI? Thanks. -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* RE: mkimage_fit_atf.sh: not found 2022-01-04 22:48 mkimage_fit_atf.sh: not found Tim Harvey 2022-01-04 22:54 ` Fabio Estevam @ 2022-01-05 11:34 ` ZHIZHIKIN Andrey 2022-01-05 19:08 ` Tim Harvey 1 sibling, 1 reply; 17+ messages in thread From: ZHIZHIKIN Andrey @ 2022-01-05 11:34 UTC (permalink / raw) To: Tim Harvey, u-boot, Stefano Babic, Fabio Estevam Cc: Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki Hello Tim, > -----Original Message----- > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > Sent: Tuesday, January 4, 2022 11:48 PM > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > <festevam@gmail.com> > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > <jagan@amarulasolutions.com> > Subject: mkimage_fit_atf.sh: not found > > Stefano and Fabio, > > I'm seeing the imx8mm_venice_defconfig target failing to build on > master due to mkimage_fit_atf.sh not found: > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > arch/arm/dts/imx8mm-venice-gw7901.dtb > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator script") > As far as I can tell the other boards that are still using > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > What is the state of the binman conversion? I submitted a series to > convert my boards to binman and it has just been sitting without any > response for months now [1]. I believe that the reason for your series sitting in the queue is the same as for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > I'm not sure when the above breakage occurred but the conversion to > binman resolves it and other things. > > Please let me know what you expect me to do to resolve this as there > is a release pending. > > Best regards, > > Tim > [1] https://patchwork.ozlabs.org/project/uboot/patch/20211006201700.3018-1- > tharvey@gateworks.com/ -- andrey Link: [1]: http://patchwork.ozlabs.org/project/uboot/patch/20211203161802.12699-1-andrey.zhizhikin@leica-geosystems.com/ ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-05 11:34 ` ZHIZHIKIN Andrey @ 2022-01-05 19:08 ` Tim Harvey 2022-01-05 21:51 ` Marcel Ziswiler 2022-01-06 10:07 ` ZHIZHIKIN Andrey 0 siblings, 2 replies; 17+ messages in thread From: Tim Harvey @ 2022-01-05 19:08 UTC (permalink / raw) To: ZHIZHIKIN Andrey Cc: u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com> wrote: > > Hello Tim, > > > -----Original Message----- > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > Sent: Tuesday, January 4, 2022 11:48 PM > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > > <festevam@gmail.com> > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > <jagan@amarulasolutions.com> > > Subject: mkimage_fit_atf.sh: not found > > > > Stefano and Fabio, > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > master due to mkimage_fit_atf.sh not found: > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator script") So why was that merged when it breaks several boards that are not switched to binman because of the CI issue? > > > As far as I can tell the other boards that are still using > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > What is the state of the binman conversion? I submitted a series to > > convert my boards to binman and it has just been sitting without any > > response for months now [1]. > > I believe that the reason for your series sitting in the queue is the same as > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > Right, so imx8mq_evk (and others) are completely broken for the pending release correct? Sounds like we need to revert d9a6f0eed6 ("tree: imx: remove old fit generator script") Tim > > > > I'm not sure when the above breakage occurred but the conversion to > > binman resolves it and other things. > > > > Please let me know what you expect me to do to resolve this as there > > is a release pending. > > > > Best regards, > > > > Tim > > [1] https://patchwork.ozlabs.org/project/uboot/patch/20211006201700.3018-1- > > tharvey@gateworks.com/ > > -- andrey > > Link: [1]: http://patchwork.ozlabs.org/project/uboot/patch/20211203161802.12699-1-andrey.zhizhikin@leica-geosystems.com/ ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-05 19:08 ` Tim Harvey @ 2022-01-05 21:51 ` Marcel Ziswiler 2022-01-05 22:04 ` Tom Rini 2022-01-06 10:07 ` ZHIZHIKIN Andrey 1 sibling, 1 reply; 17+ messages in thread From: Marcel Ziswiler @ 2022-01-05 21:51 UTC (permalink / raw) To: Tim Harvey, ZHIZHIKIN Andrey Cc: u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Jagan Teki Hi Tim et al. On Wed, 2022-01-05 at 11:08 -0800, Tim Harvey wrote: > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > Hello Tim, > > > > > -----Original Message----- > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > > > <festevam@gmail.com> > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > <jagan@amarulasolutions.com> > > > Subject: mkimage_fit_atf.sh: not found > > > > > > Stefano and Fabio, > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > master due to mkimage_fit_atf.sh not found: > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator script") > > So why was that merged when it breaks several boards that are not > switched to binman because of the CI issue? I have to admit that I did not closely follow that discussion lately. But it seems to me that this should be solvable, not? Anyway, at least in my local buildman use case just touching resp. binary blob file names helped me getting thought this: ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_1d_imem.bin ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_1d_dmem.bin ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_2d_imem.bin ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_2d_dmem.bin ⬢[zim@toolbox u-boot.git]$ touch bl31.bin Couldn't that somehow also be done for CI? > > > As far as I can tell the other boards that are still using > > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > > > > What is the state of the binman conversion? I submitted a series to > > > convert my boards to binman and it has just been sitting without any > > > response for months now [1]. > > > > I believe that the reason for your series sitting in the queue is the same as > > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > > > Right, so imx8mq_evk (and others) are completely broken for the > pending release correct? > > Sounds like we need to revert d9a6f0eed6 ("tree: imx: remove old fit > generator script") I kind of agree. However, much smarter would be to finish that binman conversion which I also still have one in flight [1] plus the follow on patch series [2] (plus an unrelated i.MX 8M Plus board addition [3]) all still sitting there idle since October with acks resp. reviewed-bys in place! Cheers Marcel > Tim > > > > > > > I'm not sure when the above breakage occurred but the conversion to > > > binman resolves it and other things. > > > > > > Please let me know what you expect me to do to resolve this as there > > > is a release pending. > > > > > > Best regards, > > > > > > Tim > > > [1] https://patchwork.ozlabs.org/project/uboot/patch/20211006201700.3018-1- > > > tharvey@gateworks.com/ > > > > -- andrey > > > > Link: [1]: > > http://patchwork.ozlabs.org/project/uboot/patch/20211203161802.12699-1-andrey.zhizhikin@leica-geosystems.com/ [1] http://patchwork.ozlabs.org/project/uboot/patch/20211022214340.1214793-1-marcel@ziswiler.com/ [2] http://patchwork.ozlabs.org/project/uboot/list/?series=268510 [3] http://patchwork.ozlabs.org/project/uboot/patch/20211022073104.1105628-1-marcel@ziswiler.com/ ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-05 21:51 ` Marcel Ziswiler @ 2022-01-05 22:04 ` Tom Rini 2022-01-05 22:07 ` Marcel Ziswiler 0 siblings, 1 reply; 17+ messages in thread From: Tom Rini @ 2022-01-05 22:04 UTC (permalink / raw) To: Marcel Ziswiler Cc: Tim Harvey, ZHIZHIKIN Andrey, u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Jagan Teki [-- Attachment #1: Type: text/plain, Size: 3782 bytes --] On Wed, Jan 05, 2022 at 10:51:23PM +0100, Marcel Ziswiler wrote: > Hi Tim et al. > > On Wed, 2022-01-05 at 11:08 -0800, Tim Harvey wrote: > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > > > Hello Tim, > > > > > > > -----Original Message----- > > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > > > > <festevam@gmail.com> > > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > > <jagan@amarulasolutions.com> > > > > Subject: mkimage_fit_atf.sh: not found > > > > > > > > Stefano and Fabio, > > > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > > master due to mkimage_fit_atf.sh not found: > > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator script") > > > > So why was that merged when it breaks several boards that are not > > switched to binman because of the CI issue? > > I have to admit that I did not closely follow that discussion lately. But it seems to me that this should be > solvable, not? > > Anyway, at least in my local buildman use case just touching resp. binary blob file names helped me getting > thought this: > > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_1d_imem.bin > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_1d_dmem.bin > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_2d_imem.bin > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_2d_dmem.bin > ⬢[zim@toolbox u-boot.git]$ touch bl31.bin > > Couldn't that somehow also be done for CI? There's a whole thread on making binman be able to fake things out in this case, so that it doesn't require too much special casing within CI itself. > > > > As far as I can tell the other boards that are still using > > > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > > > > > > > What is the state of the binman conversion? I submitted a series to > > > > convert my boards to binman and it has just been sitting without any > > > > response for months now [1]. > > > > > > I believe that the reason for your series sitting in the queue is the same as > > > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > > > > > > Right, so imx8mq_evk (and others) are completely broken for the > > pending release correct? > > > > Sounds like we need to revert d9a6f0eed6 ("tree: imx: remove old fit > > generator script") > > I kind of agree. However, much smarter would be to finish that binman conversion which I also still have one in > flight [1] plus the follow on patch series [2] (plus an unrelated i.MX 8M Plus board addition [3]) all still > sitting there idle since October with acks resp. reviewed-bys in place! Yes, I am hopeful many outstanding things in this area will get picked up after v2022.01 is released. Might it make sense to revert what Tim suggested for now tho? -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-05 22:04 ` Tom Rini @ 2022-01-05 22:07 ` Marcel Ziswiler 2022-01-05 22:09 ` Tom Rini 0 siblings, 1 reply; 17+ messages in thread From: Marcel Ziswiler @ 2022-01-05 22:07 UTC (permalink / raw) To: Tom Rini Cc: Tim Harvey, ZHIZHIKIN Andrey, u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Jagan Teki On Wed, 2022-01-05 at 17:04 -0500, Tom Rini wrote: > On Wed, Jan 05, 2022 at 10:51:23PM +0100, Marcel Ziswiler wrote: > > Hi Tim et al. > > > > On Wed, 2022-01-05 at 11:08 -0800, Tim Harvey wrote: > > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > > > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > > > > > Hello Tim, > > > > > > > > > -----Original Message----- > > > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > > > > > <festevam@gmail.com> > > > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > > > <jagan@amarulasolutions.com> > > > > > Subject: mkimage_fit_atf.sh: not found > > > > > > > > > > Stefano and Fabio, > > > > > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > > > master due to mkimage_fit_atf.sh not found: > > > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator script") > > > > > > So why was that merged when it breaks several boards that are not > > > switched to binman because of the CI issue? > > > > I have to admit that I did not closely follow that discussion lately. But it seems to me that this should > > be > > solvable, not? > > > > Anyway, at least in my local buildman use case just touching resp. binary blob file names helped me getting > > thought this: > > > > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_1d_imem.bin > > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_1d_dmem.bin > > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_2d_imem.bin > > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_2d_dmem.bin > > ⬢[zim@toolbox u-boot.git]$ touch bl31.bin > > > > Couldn't that somehow also be done for CI? > > There's a whole thread on making binman be able to fake things out in > this case, so that it doesn't require too much special casing within CI > itself. > > > > > > As far as I can tell the other boards that are still using > > > > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > > > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > > > > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > > > > > > > > > > What is the state of the binman conversion? I submitted a series to > > > > > convert my boards to binman and it has just been sitting without any > > > > > response for months now [1]. > > > > > > > > I believe that the reason for your series sitting in the queue is the same as > > > > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > > > > > > > > > Right, so imx8mq_evk (and others) are completely broken for the > > > pending release correct? > > > > > > Sounds like we need to revert d9a6f0eed6 ("tree: imx: remove old fit > > > generator script") > > > > I kind of agree. However, much smarter would be to finish that binman conversion which I also still have > > one in > > flight [1] plus the follow on patch series [2] (plus an unrelated i.MX 8M Plus board addition [3]) all > > still > > sitting there idle since October with acks resp. reviewed-bys in place! > > Yes, I am hopeful many outstanding things in this area will get picked > up after v2022.01 is released. Might it make sense to revert what Tim > suggested for now tho? Sure, fair enough. And what about them pending patch series? ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-05 22:07 ` Marcel Ziswiler @ 2022-01-05 22:09 ` Tom Rini 0 siblings, 0 replies; 17+ messages in thread From: Tom Rini @ 2022-01-05 22:09 UTC (permalink / raw) To: Marcel Ziswiler Cc: Tim Harvey, ZHIZHIKIN Andrey, u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Jagan Teki [-- Attachment #1: Type: text/plain, Size: 4534 bytes --] On Wed, Jan 05, 2022 at 11:07:03PM +0100, Marcel Ziswiler wrote: > On Wed, 2022-01-05 at 17:04 -0500, Tom Rini wrote: > > On Wed, Jan 05, 2022 at 10:51:23PM +0100, Marcel Ziswiler wrote: > > > Hi Tim et al. > > > > > > On Wed, 2022-01-05 at 11:08 -0800, Tim Harvey wrote: > > > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > > > > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > > > > > > > Hello Tim, > > > > > > > > > > > -----Original Message----- > > > > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > > > > > > <festevam@gmail.com> > > > > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > > > > <jagan@amarulasolutions.com> > > > > > > Subject: mkimage_fit_atf.sh: not found > > > > > > > > > > > > Stefano and Fabio, > > > > > > > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > > > > master due to mkimage_fit_atf.sh not found: > > > > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > > > > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator script") > > > > > > > > So why was that merged when it breaks several boards that are not > > > > switched to binman because of the CI issue? > > > > > > I have to admit that I did not closely follow that discussion lately. But it seems to me that this should > > > be > > > solvable, not? > > > > > > Anyway, at least in my local buildman use case just touching resp. binary blob file names helped me getting > > > thought this: > > > > > > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_1d_imem.bin > > > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_1d_dmem.bin > > > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_2d_imem.bin > > > ⬢[zim@toolbox u-boot.git]$ touch lpddr4_pmu_train_2d_dmem.bin > > > ⬢[zim@toolbox u-boot.git]$ touch bl31.bin > > > > > > Couldn't that somehow also be done for CI? > > > > There's a whole thread on making binman be able to fake things out in > > this case, so that it doesn't require too much special casing within CI > > itself. > > > > > > > > As far as I can tell the other boards that are still using > > > > > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > > > > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > > > > > > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > > > > > > > > > > > > > What is the state of the binman conversion? I submitted a series to > > > > > > convert my boards to binman and it has just been sitting without any > > > > > > response for months now [1]. > > > > > > > > > > I believe that the reason for your series sitting in the queue is the same as > > > > > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > > > > > > > > > > > > Right, so imx8mq_evk (and others) are completely broken for the > > > > pending release correct? > > > > > > > > Sounds like we need to revert d9a6f0eed6 ("tree: imx: remove old fit > > > > generator script") > > > > > > I kind of agree. However, much smarter would be to finish that binman conversion which I also still have > > > one in > > > flight [1] plus the follow on patch series [2] (plus an unrelated i.MX 8M Plus board addition [3]) all > > > still > > > sitting there idle since October with acks resp. reviewed-bys in place! > > > > Yes, I am hopeful many outstanding things in this area will get picked > > up after v2022.01 is released. Might it make sense to revert what Tim > > suggested for now tho? > > Sure, fair enough. > > And what about them pending patch series? Fabio noted elsewhere that Stefano is on vacation still right now, but yes, I very much want to see these outstanding items picked up, and I'm unsure what additional help Stefano would find useful. -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* RE: mkimage_fit_atf.sh: not found 2022-01-05 19:08 ` Tim Harvey 2022-01-05 21:51 ` Marcel Ziswiler @ 2022-01-06 10:07 ` ZHIZHIKIN Andrey 2022-01-06 17:10 ` Tim Harvey 1 sibling, 1 reply; 17+ messages in thread From: ZHIZHIKIN Andrey @ 2022-01-06 10:07 UTC (permalink / raw) To: Tim Harvey Cc: u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki Hello Tim, > -----Original Message----- > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > Sent: Wednesday, January 5, 2022 8:08 PM > To: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com> > Cc: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > <festevam@gmail.com>; Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > <jagan@amarulasolutions.com> > Subject: Re: mkimage_fit_atf.sh: not found > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > Hello Tim, > > > > > -----Original Message----- > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio > Estevam > > > <festevam@gmail.com> > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > <jagan@amarulasolutions.com> > > > Subject: mkimage_fit_atf.sh: not found > > > > > > Stefano and Fabio, > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > master due to mkimage_fit_atf.sh not found: > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator > script") > > So why was that merged when it breaks several boards that are not > switched to binman because of the CI issue? Because the FIT generator script has been broken after commit 3f04db891a ("image: Check for unit addresses in FITs"), which has covered CVE-2021-27138. You can see the reasoning of merging d9a6f0eed6 ("tree: imx: remove old fit generator script") in the commit message. In addition, d9a6f0eed6 ("tree: imx: remove old fit generator script") has been introduced as a part of discussion stemmed from [1], where it has been pointed out that certain boards are still using old FIT generator which is not working, and I've listed those board config files for maintainers to react. Binman CI missing binaries verification came later I believe, that is the reason we are seeing those conversions pending. But this is unrelated to the FIT generator script removal, which was broken anyways. > > > > > > As far as I can tell the other boards that are still using > > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > > > > What is the state of the binman conversion? I submitted a series to > > > convert my boards to binman and it has just been sitting without any > > > response for months now [1]. > > > > I believe that the reason for your series sitting in the queue is the same as > > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > > > Right, so imx8mq_evk (and others) are completely broken for the > pending release correct? That is correct, and there are patches to address this. There is also a patch series from Heiko to address the "missing binary" CI issue, see [2]. > > Sounds like we need to revert d9a6f0eed6 ("tree: imx: remove old fit > generator script") No, because reverting this patch would bring the FIT generator which does not produce correct images due to reasons stated above. If there are plans to revert it - then it should also be complemented with the patch that fixes the FIT generator. I'm not sure if this is a desired behavior to bring back old functionality and fix it for the sake of removing it at the later stage. > > Tim > > > > > > > I'm not sure when the above breakage occurred but the conversion to > > > binman resolves it and other things. > > > > > > Please let me know what you expect me to do to resolve this as there > > > is a release pending. > > > > > > Best regards, > > > > > > Tim > > > [1] https://patchwork.ozlabs.org/project/uboot/patch/20211006201700.3018-1- > > > tharvey@gateworks.com/ > > > > -- andrey > > > > Link: [1]: > http://patchwork.ozlabs.org/project/uboot/patch/20211203161802.12699-1- > andrey.zhizhikin@leica-geosystems.com/ -- andrey Link: [1]: https://lore.kernel.org/u-boot/AM6PR06MB46912C898E0D6EA797D1EB5EA6C49@AM6PR06MB4691.eurprd06.prod.outlook.com/ Link: [2]: https://lore.kernel.org/u-boot/20220105125816.197045-1-heiko.thiery@gmail.com/ ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-06 10:07 ` ZHIZHIKIN Andrey @ 2022-01-06 17:10 ` Tim Harvey 2022-01-06 18:59 ` Tom Rini ` (3 more replies) 0 siblings, 4 replies; 17+ messages in thread From: Tim Harvey @ 2022-01-06 17:10 UTC (permalink / raw) To: ZHIZHIKIN Andrey Cc: u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki, Oliver Graute On Thu, Jan 6, 2022 at 2:07 AM ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com> wrote: > > Hello Tim, > > > -----Original Message----- > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > Sent: Wednesday, January 5, 2022 8:08 PM > > To: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com> > > Cc: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > > <festevam@gmail.com>; Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > <jagan@amarulasolutions.com> > > Subject: Re: mkimage_fit_atf.sh: not found > > > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > > > Hello Tim, > > > > > > > -----Original Message----- > > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio > > Estevam > > > > <festevam@gmail.com> > > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > > <jagan@amarulasolutions.com> > > > > Subject: mkimage_fit_atf.sh: not found > > > > > > > > Stefano and Fabio, > > > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > > master due to mkimage_fit_atf.sh not found: > > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator > > script") > > > > So why was that merged when it breaks several boards that are not > > switched to binman because of the CI issue? > > Because the FIT generator script has been broken after commit 3f04db891a > ("image: Check for unit addresses in FITs"), which has covered CVE-2021-27138. > You can see the reasoning of merging d9a6f0eed6 ("tree: imx: remove old fit > generator script") in the commit message. > > In addition, d9a6f0eed6 ("tree: imx: remove old fit generator script") has > been introduced as a part of discussion stemmed from [1], where it has been > pointed out that certain boards are still using old FIT generator which is > not working, and I've listed those board config files for maintainers to react. > Andrey, I agree removing the old fit generator is the right path but I am very surprised it was merged when it broke boards that had not moved to binman yet. There has been a warning to migrate to binman but there was never a deadline. > Binman CI missing binaries verification came later I believe, that is the > reason we are seeing those conversions pending. But this is unrelated to the > FIT generator script removal, which was broken anyways. > I have not followed the discussion about what was wrong with the FIT generator or what was specifically 'broken' but boards at least built and booted with it and now they do not build. I do know there was a time where I needed a patch that dealt with '@' symbols and I'm not sure if that ever got merged before the FIT generator was simply removed. Again, I hadn't thought much of it because I had binman conversion patches in flight not realizing they would get stuck because of CI. I don't really know anything about U-Boot CI but I'm also surprised it doesn't point out the fact that several boards won't currently build. > > > > > > > > > As far as I can tell the other boards that are still using > > > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > > > > > > > What is the state of the binman conversion? I submitted a series to > > > > convert my boards to binman and it has just been sitting without any > > > > response for months now [1]. > > > > > > I believe that the reason for your series sitting in the queue is the same as > > > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > > > > > > Right, so imx8mq_evk (and others) are completely broken for the > > pending release correct? > > That is correct, and there are patches to address this. > > There is also a patch series from Heiko to address the "missing binary" CI issue, see [2]. > Again personally I did not pay attention to d9a6f0eed6 ("tree: imx: remove old fit generator script") because I already had submitted a series to move my board to binman (other boards still had not done this however and would be broken without a change). I see that Stefano applied d9a6f0eed6 ("tree: imx: remove old fit generator script") on Oct 7 and he must not have realized it broke boards that had not moved to binman. Several of us have submitted patches to move to binman and thus also likely did not notice but those patches never got merged because of this CI issue. I see Heiko has a v4 of his patch here https://patchwork.ozlabs.org/project/uboot/list/?series=279595. Can that get merged? It looks to me like the following boards would be currently broken as they all use CONFIG_SPL_FIT_GENERATOR="arch/arm/mach-imx/mkimage_fit_atf.sh": cgtqmx8_defconfig imx8mm-icore-mx8mm-ctouch2_defconfig imx8mm-icore-mx8mm-edimm2.2_defconfig imx8mm_beacon_defconfig imx8mm_venice_defconfig imx8mn_beacon_2g_defconfig imx8mn_beacon_defconfig imx8mq_evk_defconfig imx8mq_phanbell_defconfig imx8qm_rom7720_a1_4G_defconfig pico-imx8mq_defconfig I don't know exactly how the U-Boot release schedule works and when 2021.01 is released but in order to save these boards from being broken the following would have to occur: apply Heiko's [PATCH v4] binman: add support for creating dummy files for external blobs (https://patchwork.ozlabs.org/project/uboot/list/?series=279595) apply the conversion to binman series that has been pending because of the CI breakage for these boards: - Adam Ford: imx8mm_beacon* boards - Tim Harvey: imx8mm_venice boards - Andrey Zhizhikin: imx8mq_evk - Peng Fan: imx8mq_phanbell and pico-imx8mq That would still leave the following boards broken as nobody has submitted binman patches for them. These are both maintained by Oliver Graute (added to cc): imx8qm_rom7720_a1_4G_defconfig cgtqmx8_defconfig I believe Stefano is not currently available and I'm not sure when he is back to merge the pending binman conversion series. Additionally some of these series may need to be rebased (I know I had to do that twice because of things being moved to Kconfig) Best regards, Tim > > > > Sounds like we need to revert d9a6f0eed6 ("tree: imx: remove old fit > > generator script") > > No, because reverting this patch would bring the FIT generator which > does not produce correct images due to reasons stated above. > > If there are plans to revert it - then it should also be complemented with > the patch that fixes the FIT generator. I'm not sure if this is a desired > behavior to bring back old functionality and fix it for the sake of removing > it at the later stage. > > > > > Tim > > > > > > > > > > I'm not sure when the above breakage occurred but the conversion to > > > > binman resolves it and other things. > > > > > > > > Please let me know what you expect me to do to resolve this as there > > > > is a release pending. > > > > > > > > Best regards, > > > > > > > > Tim > > > > [1] https://patchwork.ozlabs.org/project/uboot/patch/20211006201700.3018-1- > > > > tharvey@gateworks.com/ > > > > > > -- andrey > > > > > > Link: [1]: > > http://patchwork.ozlabs.org/project/uboot/patch/20211203161802.12699-1- > > andrey.zhizhikin@leica-geosystems.com/ > > -- andrey > > Link: [1]: https://lore.kernel.org/u-boot/AM6PR06MB46912C898E0D6EA797D1EB5EA6C49@AM6PR06MB4691.eurprd06.prod.outlook.com/ > Link: [2]: https://lore.kernel.org/u-boot/20220105125816.197045-1-heiko.thiery@gmail.com/ > ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-06 17:10 ` Tim Harvey @ 2022-01-06 18:59 ` Tom Rini 2022-01-06 19:05 ` ZHIZHIKIN Andrey ` (2 subsequent siblings) 3 siblings, 0 replies; 17+ messages in thread From: Tom Rini @ 2022-01-06 18:59 UTC (permalink / raw) To: Tim Harvey Cc: ZHIZHIKIN Andrey, u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki, Oliver Graute [-- Attachment #1: Type: text/plain, Size: 5661 bytes --] On Thu, Jan 06, 2022 at 09:10:00AM -0800, Tim Harvey wrote: > On Thu, Jan 6, 2022 at 2:07 AM ZHIZHIKIN Andrey > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > Hello Tim, > > > > > -----Original Message----- > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > Sent: Wednesday, January 5, 2022 8:08 PM > > > To: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com> > > > Cc: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > > > <festevam@gmail.com>; Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > <jagan@amarulasolutions.com> > > > Subject: Re: mkimage_fit_atf.sh: not found > > > > > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > > > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > > > > > Hello Tim, > > > > > > > > > -----Original Message----- > > > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio > > > Estevam > > > > > <festevam@gmail.com> > > > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > > > <jagan@amarulasolutions.com> > > > > > Subject: mkimage_fit_atf.sh: not found > > > > > > > > > > Stefano and Fabio, > > > > > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > > > master due to mkimage_fit_atf.sh not found: > > > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator > > > script") > > > > > > So why was that merged when it breaks several boards that are not > > > switched to binman because of the CI issue? > > > > Because the FIT generator script has been broken after commit 3f04db891a > > ("image: Check for unit addresses in FITs"), which has covered CVE-2021-27138. > > You can see the reasoning of merging d9a6f0eed6 ("tree: imx: remove old fit > > generator script") in the commit message. > > > > In addition, d9a6f0eed6 ("tree: imx: remove old fit generator script") has > > been introduced as a part of discussion stemmed from [1], where it has been > > pointed out that certain boards are still using old FIT generator which is > > not working, and I've listed those board config files for maintainers to react. > > > > Andrey, > > I agree removing the old fit generator is the right path but I am very > surprised it was merged when it broke boards that had not moved to > binman yet. There has been a warning to migrate to binman but there > was never a deadline. > > > Binman CI missing binaries verification came later I believe, that is the > > reason we are seeing those conversions pending. But this is unrelated to the > > FIT generator script removal, which was broken anyways. > > > > I have not followed the discussion about what was wrong with the FIT > generator or what was specifically 'broken' but boards at least built > and booted with it and now they do not build. I do know there was a > time where I needed a patch that dealt with '@' symbols and I'm not > sure if that ever got merged before the FIT generator was simply > removed. Again, I hadn't thought much of it because I had binman > conversion patches in flight not realizing they would get stuck > because of CI. > > I don't really know anything about U-Boot CI but I'm also surprised it > doesn't point out the fact that several boards won't currently build. Note that all boards currently _do_ build because whatever is missing today isn't part of the "all" target. This is likely because there's a few build targets that rely on some CONFIG options that in turn are not part of Kconfig and so don't always work. [snip] > I don't know exactly how the U-Boot release schedule works and when > 2021.01 is released but in order to save these boards from being > broken the following would have to occur: I guess https://www.denx.de/wiki/U-Boot/ReleaseCycle needs to be more widely publicized? > apply Heiko's [PATCH v4] binman: add support for creating dummy files > for external blobs > (https://patchwork.ozlabs.org/project/uboot/list/?series=279595) > apply the conversion to binman series that has been pending because of > the CI breakage for these boards: > - Adam Ford: imx8mm_beacon* boards > - Tim Harvey: imx8mm_venice boards > - Andrey Zhizhikin: imx8mq_evk > - Peng Fan: imx8mq_phanbell and pico-imx8mq > > That would still leave the following boards broken as nobody has > submitted binman patches for them. These are both maintained by Oliver > Graute (added to cc): > imx8qm_rom7720_a1_4G_defconfig > cgtqmx8_defconfig > > I believe Stefano is not currently available and I'm not sure when he > is back to merge the pending binman conversion series. Additionally > some of these series may need to be rebased (I know I had to do that > twice because of things being moved to Kconfig) I'm doing one last test of reverting d9a6f0eed6 which should also restore functionality for now, yes? -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* RE: mkimage_fit_atf.sh: not found 2022-01-06 17:10 ` Tim Harvey 2022-01-06 18:59 ` Tom Rini @ 2022-01-06 19:05 ` ZHIZHIKIN Andrey 2022-01-06 20:43 ` Heiko Thiery 2022-01-07 12:22 ` Oliver Graute 3 siblings, 0 replies; 17+ messages in thread From: ZHIZHIKIN Andrey @ 2022-01-06 19:05 UTC (permalink / raw) To: Tim Harvey Cc: u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki, Oliver Graute Hello Tim, > -----Original Message----- > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > Sent: Thursday, January 6, 2022 6:10 PM > To: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com> > Cc: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > <festevam@gmail.com>; Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > <jagan@amarulasolutions.com>; Oliver Graute <oliver.graute@kococonnector.com> > Subject: Re: mkimage_fit_atf.sh: not found > > On Thu, Jan 6, 2022 at 2:07 AM ZHIZHIKIN Andrey > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > Hello Tim, > > > > > -----Original Message----- > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > Sent: Wednesday, January 5, 2022 8:08 PM > > > To: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com> > > > Cc: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio > Estevam > > > <festevam@gmail.com>; Schrempf Frieder <frieder.schrempf@kontron.de>; Adam > Ford > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > <jagan@amarulasolutions.com> > > > Subject: Re: mkimage_fit_atf.sh: not found > > > > > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > > > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > > > > > Hello Tim, > > > > > > > > > -----Original Message----- > > > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio > > > Estevam > > > > > <festevam@gmail.com> > > > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > > > <jagan@amarulasolutions.com> > > > > > Subject: mkimage_fit_atf.sh: not found > > > > > > > > > > Stefano and Fabio, > > > > > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > > > master due to mkimage_fit_atf.sh not found: > > > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator > > > script") > > > > > > So why was that merged when it breaks several boards that are not > > > switched to binman because of the CI issue? > > > > Because the FIT generator script has been broken after commit 3f04db891a > > ("image: Check for unit addresses in FITs"), which has covered CVE-2021-27138. > > You can see the reasoning of merging d9a6f0eed6 ("tree: imx: remove old fit > > generator script") in the commit message. > > > > In addition, d9a6f0eed6 ("tree: imx: remove old fit generator script") has > > been introduced as a part of discussion stemmed from [1], where it has been > > pointed out that certain boards are still using old FIT generator which is > > not working, and I've listed those board config files for maintainers to react. > > > > Andrey, > > I agree removing the old fit generator is the right path but I am very > surprised it was merged when it broke boards that had not moved to > binman yet. There has been a warning to migrate to binman but there > was never a deadline. I totally agree that there was no deadline set for this, hence I submitted the patch as RFC in a good faith that board maintainers (which were all included in the patch submission mail) would take a look and comment on this removal. Since nobody complained about that removal at the submission time - the patch has been accepted. If you take a look at the discussion that originated this removal patch - you would see more background and insights on why it has been introduced, and I saw no objections there either. I've listed those board configs there which would be affected by the removal, and even included Oliver as I did not see any binman conversion series from him at that time. Even that did not trigger any red flag at the time... This all may come from the fact that board maintainers (like you) had already patches prepared to move to binman, and I do believe that at the time of submission there has been no checks in CI about those missing binaries, hence the submission came clean. > > > Binman CI missing binaries verification came later I believe, that is the > > reason we are seeing those conversions pending. But this is unrelated to the > > FIT generator script removal, which was broken anyways. > > > > I have not followed the discussion about what was wrong with the FIT > generator or what was specifically 'broken' but boards at least built > and booted with it and now they do not build. I do know there was a > time where I needed a patch that dealt with '@' symbols and I'm not > sure if that ever got merged before the FIT generator was simply > removed. Again, I hadn't thought much of it because I had binman > conversion patches in flight not realizing they would get stuck > because of CI. This is exactly the point: those board builds were broken *way before* this removal due to fact that FIT generator has been producing ITS with "@" symbols which were not accepted by updated mkimage. > > I don't really know anything about U-Boot CI but I'm also surprised it > doesn't point out the fact that several boards won't currently build. Honestly, I'm in the same position here with the imx8mq_evk conversion patch: I know nothing about CI, and only deducted from the comments on conversion patch that this might be an issue, with Heiko's patch solving it... > > > > > > > > > > > > > As far as I can tell the other boards that are still using > > > > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > > > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > > > > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > > > > > > > > > > What is the state of the binman conversion? I submitted a series to > > > > > convert my boards to binman and it has just been sitting without any > > > > > response for months now [1]. > > > > > > > > I believe that the reason for your series sitting in the queue is the same > as > > > > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > > > > > > > > > Right, so imx8mq_evk (and others) are completely broken for the > > > pending release correct? > > > > That is correct, and there are patches to address this. > > > > There is also a patch series from Heiko to address the "missing binary" CI > issue, see [2]. > > > > Again personally I did not pay attention to d9a6f0eed6 ("tree: imx: > remove old fit generator script") because I already had submitted a > series to move my board to binman (other boards still had not done > this however and would be broken without a change). I see that Stefano > applied d9a6f0eed6 ("tree: imx: remove old fit generator script") on > Oct 7 and he must not have realized it broke boards that had not moved > to binman. Several of us have submitted patches to move to binman and > thus also likely did not notice but those patches never got merged > because of this CI issue. > > I see Heiko has a v4 of his patch here > https://patchwork.ozlabs.org/project/uboot/list/?series=279595. Can > that get merged? > > It looks to me like the following boards would be currently broken as > they all use CONFIG_SPL_FIT_GENERATOR="arch/arm/mach-imx/mkimage_fit_atf.sh": > cgtqmx8_defconfig > imx8mm-icore-mx8mm-ctouch2_defconfig > imx8mm-icore-mx8mm-edimm2.2_defconfig > imx8mm_beacon_defconfig > imx8mm_venice_defconfig > imx8mn_beacon_2g_defconfig > imx8mn_beacon_defconfig > imx8mq_evk_defconfig > imx8mq_phanbell_defconfig > imx8qm_rom7720_a1_4G_defconfig > pico-imx8mq_defconfig > > I don't know exactly how the U-Boot release schedule works and when > 2021.01 is released but in order to save these boards from being > broken the following would have to occur: > apply Heiko's [PATCH v4] binman: add support for creating dummy files > for external blobs > (https://patchwork.ozlabs.org/project/uboot/list/?series=279595) > apply the conversion to binman series that has been pending because of > the CI breakage for these boards: > - Adam Ford: imx8mm_beacon* boards > - Tim Harvey: imx8mm_venice boards > - Andrey Zhizhikin: imx8mq_evk > - Peng Fan: imx8mq_phanbell and pico-imx8mq That sounds like a plan to me, and this is what I was expecting to happen. Hopefully, this would make it into 2022.01... > > That would still leave the following boards broken as nobody has > submitted binman patches for them. These are both maintained by Oliver > Graute (added to cc): > imx8qm_rom7720_a1_4G_defconfig > cgtqmx8_defconfig I guess Oliver would need to step up here to have those boards converted. > > I believe Stefano is not currently available and I'm not sure when he > is back to merge the pending binman conversion series. Additionally > some of these series may need to be rebased (I know I had to do that > twice because of things being moved to Kconfig) > > Best regards, > > Tim > > > > > > > Sounds like we need to revert d9a6f0eed6 ("tree: imx: remove old fit > > > generator script") > > > > No, because reverting this patch would bring the FIT generator which > > does not produce correct images due to reasons stated above. > > > > If there are plans to revert it - then it should also be complemented with > > the patch that fixes the FIT generator. I'm not sure if this is a desired > > behavior to bring back old functionality and fix it for the sake of removing > > it at the later stage. > > > > > > > > Tim > > > > > > > > > > > > > I'm not sure when the above breakage occurred but the conversion to > > > > > binman resolves it and other things. > > > > > > > > > > Please let me know what you expect me to do to resolve this as there > > > > > is a release pending. > > > > > > > > > > Best regards, > > > > > > > > > > Tim > > > > > [1] https://patchwork.ozlabs.org/project/uboot/patch/20211006201700.3018- > 1- > > > > > tharvey@gateworks.com/ > > > > > > > > -- andrey > > > > > > > > Link: [1]: > > > http://patchwork.ozlabs.org/project/uboot/patch/20211203161802.12699-1- > > > andrey.zhizhikin@leica-geosystems.com/ > > > > -- andrey > > > > Link: [1]: https://lore.kernel.org/u- > boot/AM6PR06MB46912C898E0D6EA797D1EB5EA6C49@AM6PR06MB4691.eurprd06.prod.outlook.c > om/ > > Link: [2]: https://lore.kernel.org/u-boot/20220105125816.197045-1- > heiko.thiery@gmail.com/ > > Cheers, Andrey ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-06 17:10 ` Tim Harvey 2022-01-06 18:59 ` Tom Rini 2022-01-06 19:05 ` ZHIZHIKIN Andrey @ 2022-01-06 20:43 ` Heiko Thiery 2022-01-07 1:27 ` Simon Glass 2022-01-07 12:22 ` Oliver Graute 3 siblings, 1 reply; 17+ messages in thread From: Heiko Thiery @ 2022-01-06 20:43 UTC (permalink / raw) To: Tim Harvey Cc: ZHIZHIKIN Andrey, u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki, Oliver Graute, Simon Glass Hi Tim and all, Am Do., 6. Jan. 2022 um 18:10 Uhr schrieb Tim Harvey <tharvey@gateworks.com>: > > On Thu, Jan 6, 2022 at 2:07 AM ZHIZHIKIN Andrey > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > Hello Tim, > > > > > -----Original Message----- > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > Sent: Wednesday, January 5, 2022 8:08 PM > > > To: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com> > > > Cc: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > > > <festevam@gmail.com>; Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > <jagan@amarulasolutions.com> > > > Subject: Re: mkimage_fit_atf.sh: not found > > > > > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > > > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > > > > > Hello Tim, > > > > > > > > > -----Original Message----- > > > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio > > > Estevam > > > > > <festevam@gmail.com> > > > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > > > <jagan@amarulasolutions.com> > > > > > Subject: mkimage_fit_atf.sh: not found > > > > > > > > > > Stefano and Fabio, > > > > > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > > > master due to mkimage_fit_atf.sh not found: > > > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator > > > script") > > > > > > So why was that merged when it breaks several boards that are not > > > switched to binman because of the CI issue? > > > > Because the FIT generator script has been broken after commit 3f04db891a > > ("image: Check for unit addresses in FITs"), which has covered CVE-2021-27138. > > You can see the reasoning of merging d9a6f0eed6 ("tree: imx: remove old fit > > generator script") in the commit message. > > > > In addition, d9a6f0eed6 ("tree: imx: remove old fit generator script") has > > been introduced as a part of discussion stemmed from [1], where it has been > > pointed out that certain boards are still using old FIT generator which is > > not working, and I've listed those board config files for maintainers to react. > > > > Andrey, > > I agree removing the old fit generator is the right path but I am very > surprised it was merged when it broke boards that had not moved to > binman yet. There has been a warning to migrate to binman but there > was never a deadline. > > > Binman CI missing binaries verification came later I believe, that is the > > reason we are seeing those conversions pending. But this is unrelated to the > > FIT generator script removal, which was broken anyways. > > > > I have not followed the discussion about what was wrong with the FIT > generator or what was specifically 'broken' but boards at least built > and booted with it and now they do not build. I do know there was a > time where I needed a patch that dealt with '@' symbols and I'm not > sure if that ever got merged before the FIT generator was simply > removed. Again, I hadn't thought much of it because I had binman > conversion patches in flight not realizing they would get stuck > because of CI. > > I don't really know anything about U-Boot CI but I'm also surprised it > doesn't point out the fact that several boards won't currently build. > > > > > > > > > > > > > As far as I can tell the other boards that are still using > > > > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > > > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > > > > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > > > > > > > > > > What is the state of the binman conversion? I submitted a series to > > > > > convert my boards to binman and it has just been sitting without any > > > > > response for months now [1]. > > > > > > > > I believe that the reason for your series sitting in the queue is the same as > > > > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > > > > > > > > > Right, so imx8mq_evk (and others) are completely broken for the > > > pending release correct? > > > > That is correct, and there are patches to address this. > > > > There is also a patch series from Heiko to address the "missing binary" CI issue, see [2]. > > > > Again personally I did not pay attention to d9a6f0eed6 ("tree: imx: > remove old fit generator script") because I already had submitted a > series to move my board to binman (other boards still had not done > this however and would be broken without a change). I see that Stefano > applied d9a6f0eed6 ("tree: imx: remove old fit generator script") on > Oct 7 and he must not have realized it broke boards that had not moved > to binman. Several of us have submitted patches to move to binman and > thus also likely did not notice but those patches never got merged > because of this CI issue. > > I see Heiko has a v4 of his patch here > https://patchwork.ozlabs.org/project/uboot/list/?series=279595. Can > that get merged? I think applying this patch will not solve the issue finally. This change will add the ability to fake the needed files (lpddr fw, hdmi fw ...) by binman with adding the introduced u-boot make option. But this option has to be set by buildman somehow for the CI. Or we need to change the default behavior of binman as Simon already proposed. -- Heiko ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-06 20:43 ` Heiko Thiery @ 2022-01-07 1:27 ` Simon Glass 0 siblings, 0 replies; 17+ messages in thread From: Simon Glass @ 2022-01-07 1:27 UTC (permalink / raw) To: Heiko Thiery Cc: Tim Harvey, ZHIZHIKIN Andrey, u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki, Oliver Graute Hi Heiko, On Thu, 6 Jan 2022 at 13:44, Heiko Thiery <heiko.thiery@gmail.com> wrote: > > Hi Tim and all, > > Am Do., 6. Jan. 2022 um 18:10 Uhr schrieb Tim Harvey <tharvey@gateworks.com>: > > > > On Thu, Jan 6, 2022 at 2:07 AM ZHIZHIKIN Andrey > > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > > > Hello Tim, > > > > > > > -----Original Message----- > > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > > Sent: Wednesday, January 5, 2022 8:08 PM > > > > To: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com> > > > > Cc: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio Estevam > > > > <festevam@gmail.com>; Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > > <jagan@amarulasolutions.com> > > > > Subject: Re: mkimage_fit_atf.sh: not found > > > > > > > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > > > > <andrey.zhizhikin@leica-geosystems.com> wrote: > > > > > > > > > > Hello Tim, > > > > > > > > > > > -----Original Message----- > > > > > > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tim Harvey > > > > > > Sent: Tuesday, January 4, 2022 11:48 PM > > > > > > To: u-boot <u-boot@lists.denx.de>; Stefano Babic <sbabic@denx.de>; Fabio > > > > Estevam > > > > > > <festevam@gmail.com> > > > > > > Cc: Schrempf Frieder <frieder.schrempf@kontron.de>; Adam Ford > > > > > > <aford173@gmail.com>; Marcel Ziswiler <marcel@ziswiler.com>; Jagan Teki > > > > > > <jagan@amarulasolutions.com> > > > > > > Subject: mkimage_fit_atf.sh: not found > > > > > > > > > > > > Stefano and Fabio, > > > > > > > > > > > > I'm seeing the imx8mm_venice_defconfig target failing to build on > > > > > > master due to mkimage_fit_atf.sh not found: > > > > > > ./"arch/arm/mach-imx/mkimage_fit_atf.sh" \ > > > > > > arch/arm/dts/imx8mm-venice-gw71xx-0x.dtb > > > > > > arch/arm/dts/imx8mm-venice-gw72xx-0x.dtb > > > > > > arch/arm/dts/imx8mm-venice-gw73xx-0x.dtb > > > > > > arch/arm/dts/imx8mm-venice-gw7901.dtb > > > > > > arch/arm/dts/imx8mm-venice-gw7902.dtb > u-boot.its > > > > > > /bin/sh: 1: ./arch/arm/mach-imx/mkimage_fit_atf.sh: not found > > > > > > > > > > > > > > > > This has been dropped in d9a6f0eed6 ("tree: imx: remove old fit generator > > > > script") > > > > > > > > So why was that merged when it breaks several boards that are not > > > > switched to binman because of the CI issue? > > > > > > Because the FIT generator script has been broken after commit 3f04db891a > > > ("image: Check for unit addresses in FITs"), which has covered CVE-2021-27138. > > > You can see the reasoning of merging d9a6f0eed6 ("tree: imx: remove old fit > > > generator script") in the commit message. > > > > > > In addition, d9a6f0eed6 ("tree: imx: remove old fit generator script") has > > > been introduced as a part of discussion stemmed from [1], where it has been > > > pointed out that certain boards are still using old FIT generator which is > > > not working, and I've listed those board config files for maintainers to react. > > > > > > > Andrey, > > > > I agree removing the old fit generator is the right path but I am very > > surprised it was merged when it broke boards that had not moved to > > binman yet. There has been a warning to migrate to binman but there > > was never a deadline. > > > > > Binman CI missing binaries verification came later I believe, that is the > > > reason we are seeing those conversions pending. But this is unrelated to the > > > FIT generator script removal, which was broken anyways. > > > > > > > I have not followed the discussion about what was wrong with the FIT > > generator or what was specifically 'broken' but boards at least built > > and booted with it and now they do not build. I do know there was a > > time where I needed a patch that dealt with '@' symbols and I'm not > > sure if that ever got merged before the FIT generator was simply > > removed. Again, I hadn't thought much of it because I had binman > > conversion patches in flight not realizing they would get stuck > > because of CI. > > > > I don't really know anything about U-Boot CI but I'm also surprised it > > doesn't point out the fact that several boards won't currently build. > > > > > > > > > > > > > > > > > As far as I can tell the other boards that are still using > > > > > > SPL_FIT_GENERATOR also fail due to this (ie imx8mm_beacon_defconfig, > > > > > > imx8mq_evk_defconfig, imx8mm-icore-mx8mm-edimm2.2_defconfig, etc). > > > > > > > > > > imx8mq_evk is already converted and I've sent a patch for it, see [1]. > > > > > > > > > > > > > > > > > What is the state of the binman conversion? I submitted a series to > > > > > > convert my boards to binman and it has just been sitting without any > > > > > > response for months now [1]. > > > > > > > > > > I believe that the reason for your series sitting in the queue is the same as > > > > > for imx8mq_evk: missing binary blobs (ATF and DDR) are failing CI builds. > > > > > > > > > > > > > Right, so imx8mq_evk (and others) are completely broken for the > > > > pending release correct? > > > > > > That is correct, and there are patches to address this. > > > > > > There is also a patch series from Heiko to address the "missing binary" CI issue, see [2]. > > > > > > > Again personally I did not pay attention to d9a6f0eed6 ("tree: imx: > > remove old fit generator script") because I already had submitted a > > series to move my board to binman (other boards still had not done > > this however and would be broken without a change). I see that Stefano > > applied d9a6f0eed6 ("tree: imx: remove old fit generator script") on > > Oct 7 and he must not have realized it broke boards that had not moved > > to binman. Several of us have submitted patches to move to binman and > > thus also likely did not notice but those patches never got merged > > because of this CI issue. > > > > I see Heiko has a v4 of his patch here > > https://patchwork.ozlabs.org/project/uboot/list/?series=279595. Can > > that get merged? > > I think applying this patch will not solve the issue finally. This > change will add the ability to fake the needed files (lpddr fw, hdmi > fw ...) by binman with adding the introduced u-boot make option. But > this option has to be set by buildman somehow for the CI. Or we need > to change the default behavior of binman as Simon already proposed. Yes and I will add a patch for that in the series I expect to send soon (which deals with missing binary tools). We do the same for missing binaries, so I think it makes sense. Regards, Simon ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-06 17:10 ` Tim Harvey ` (2 preceding siblings ...) 2022-01-06 20:43 ` Heiko Thiery @ 2022-01-07 12:22 ` Oliver Graute 2022-01-07 12:57 ` Adam Ford 3 siblings, 1 reply; 17+ messages in thread From: Oliver Graute @ 2022-01-07 12:22 UTC (permalink / raw) To: Tim Harvey Cc: ZHIZHIKIN Andrey, u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki > That would still leave the following boards broken as nobody has > submitted binman patches for them. These are both maintained by Oliver > Graute (added to cc): > imx8qm_rom7720_a1_4G_defconfig > cgtqmx8_defconfig I will look at this. So far I don't know very much about binman. So Can you point me to similar patches as examples? I can work on this in the next few days, perhaps I need some hints for this. > I believe Stefano is not currently available and I'm not sure when he > is back to merge the pending binman conversion series. Additionally > some of these series may need to be rebased (I know I had to do that > twice because of things being moved to Kconfig) Best regards, Oliver ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: mkimage_fit_atf.sh: not found 2022-01-07 12:22 ` Oliver Graute @ 2022-01-07 12:57 ` Adam Ford 0 siblings, 0 replies; 17+ messages in thread From: Adam Ford @ 2022-01-07 12:57 UTC (permalink / raw) To: Tim Harvey, ZHIZHIKIN Andrey, u-boot, Stefano Babic, Fabio Estevam, Schrempf Frieder, Adam Ford, Marcel Ziswiler, Jagan Teki On Fri, Jan 7, 2022 at 6:22 AM Oliver Graute <oliver.graute@gmail.com> wrote: > > That would still leave the following boards broken as nobody has > > submitted binman patches for them. These are both maintained by Oliver > > Graute (added to cc): > > imx8qm_rom7720_a1_4G_defconfig > > cgtqmx8_defconfig > > I will look at this. So far I don't know very much about binman. So Can > you point me to similar patches as examples? I can work on this in the > next few days, perhaps I need some hints for this. > > > I believe Stefano is not currently available and I'm not sure when he > > is back to merge the pending binman conversion series. Additionally > > some of these series may need to be rebased (I know I had to do that > > twice because of things being moved to Kconfig) > The series to switch the imx8mq was submitted here. https://patchwork.ozlabs.org/project/uboot/list/?series=268380 I would expect the imx8qm_rom7720_a1_4G_defconfig to be similar. There is another series to unify the binman file generation so it's common to mulitple devices with a siilar SoC: https://patchwork.ozlabs.org/project/uboot/list/?series=268510 adam > > Best regards, > > Oliver > ^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2022-01-07 12:58 UTC | newest] Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-01-04 22:48 mkimage_fit_atf.sh: not found Tim Harvey 2022-01-04 22:54 ` Fabio Estevam 2022-01-05 13:16 ` Tom Rini 2022-01-05 11:34 ` ZHIZHIKIN Andrey 2022-01-05 19:08 ` Tim Harvey 2022-01-05 21:51 ` Marcel Ziswiler 2022-01-05 22:04 ` Tom Rini 2022-01-05 22:07 ` Marcel Ziswiler 2022-01-05 22:09 ` Tom Rini 2022-01-06 10:07 ` ZHIZHIKIN Andrey 2022-01-06 17:10 ` Tim Harvey 2022-01-06 18:59 ` Tom Rini 2022-01-06 19:05 ` ZHIZHIKIN Andrey 2022-01-06 20:43 ` Heiko Thiery 2022-01-07 1:27 ` Simon Glass 2022-01-07 12:22 ` Oliver Graute 2022-01-07 12:57 ` Adam Ford
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.