Message ID | 4672a67c-c1fd-471f-adb9-d54a4f737be8@denx.de |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
Series | Pull request: u-boot-imx u-boot-imx-20220412 | expand |
On Wed, Apr 13, 2022 at 11:41:45AM +0200, Stefano Babic wrote: > Hi Tom, > > please pull from u-boot-imx, thanks ! > > Note: I picked up Heinrich's patch for Renesas : > ARM: renesas: reduce rcar3_salvator-x image size > > due to CI failure. Breakage is in master, too. I don't see breakage in master for this platform, in CI, fwiw. Other toolchains may have problems as it is indeed close to the limit. > > The following changes since commit 33ae8c5bebba0874fbc432914406e63fbc219080: > > Merge tag 'efi-2022-07-rc1' of > https://source.denx.de/u-boot/custodians/u-boot-efi (2022-04-10 11:21:39 > -0400) > > are available in the Git repository at: > > https://gitlab.denx.de/u-boot/custodians/u-boot-imx.git > tags/u-boot-imx-20220413 > > for you to fetch changes up to a1c711046b0d5478a702b27c6773ea6231eba057: > > board: gateworks: venice: add imx8mm-gw7903 support (2022-04-13 09:55:26 > +0200) > So, I've applied this to master and pushed, and thanks for clearing out much of the backlog. Is there still things left that aren't recently newly posted? But also, the size problem on rcar3_salvator-x is because every platform with DM_REGULATOR_FIXED grew because: commit f3b5100aff3a7edd53fd5d15bd6db92d294865e2 Author: Philippe Schenker <philippe.schenker@toradex.com> Date: Fri Apr 8 10:07:10 2022 +0200 regulator: fixed: add possibility to enable by clock adds 300-400 bytes on 64bit platforms. I'm mainly just noting this as I don't see a nice way to handle this that doesn't then make the CONFIG side of things troublesome / annoying.
On Wed, Apr 13, 2022 at 9:24 AM Tom Rini <trini@konsulko.com> wrote: > > On Wed, Apr 13, 2022 at 11:41:45AM +0200, Stefano Babic wrote: > > > Hi Tom, > > > > please pull from u-boot-imx, thanks ! > > > > Note: I picked up Heinrich's patch for Renesas : > > ARM: renesas: reduce rcar3_salvator-x image size > > > > due to CI failure. Breakage is in master, too. > > I don't see breakage in master for this platform, in CI, fwiw. Other > toolchains may have problems as it is indeed close to the limit. > > > > > The following changes since commit 33ae8c5bebba0874fbc432914406e63fbc219080: > > > > Merge tag 'efi-2022-07-rc1' of > > https://source.denx.de/u-boot/custodians/u-boot-efi (2022-04-10 11:21:39 > > -0400) > > > > are available in the Git repository at: > > > > https://gitlab.denx.de/u-boot/custodians/u-boot-imx.git > > tags/u-boot-imx-20220413 > > > > for you to fetch changes up to a1c711046b0d5478a702b27c6773ea6231eba057: > > > > board: gateworks: venice: add imx8mm-gw7903 support (2022-04-13 09:55:26 > > +0200) > > > > So, I've applied this to master and pushed, and thanks for clearing out > much of the backlog. Is there still things left that aren't recently Tom, Yes, there are still patches posted some time ago that were not picked up yet due to merge conflicts. I'm not sure how many but I know I had about 9 that I just rebased and re-submitted for Stefano. Best Regards, Tim > newly posted? But also, the size problem on rcar3_salvator-x is because > every platform with DM_REGULATOR_FIXED grew because: > commit f3b5100aff3a7edd53fd5d15bd6db92d294865e2 > Author: Philippe Schenker <philippe.schenker@toradex.com> > Date: Fri Apr 8 10:07:10 2022 +0200 > > regulator: fixed: add possibility to enable by clock > > adds 300-400 bytes on 64bit platforms. I'm mainly just noting this as I > don't see a nice way to handle this that doesn't then make the CONFIG > side of things troublesome / annoying. > > -- > Tom
Hi Tom, On 13.04.22 18:24, Tom Rini wrote: > On Wed, Apr 13, 2022 at 11:41:45AM +0200, Stefano Babic wrote: > >> Hi Tom, >> >> please pull from u-boot-imx, thanks ! >> >> Note: I picked up Heinrich's patch for Renesas : >> ARM: renesas: reduce rcar3_salvator-x image size >> >> due to CI failure. Breakage is in master, too. > > I don't see breakage in master for this platform, in CI, fwiw. Other > toolchains may have problems as it is indeed close to the limit. I get the error when I run buildman locally on my host, so I guess it was near to the limit if you have not seen. > >> >> The following changes since commit 33ae8c5bebba0874fbc432914406e63fbc219080: >> >> Merge tag 'efi-2022-07-rc1' of >> https://source.denx.de/u-boot/custodians/u-boot-efi (2022-04-10 11:21:39 >> -0400) >> >> are available in the Git repository at: >> >> https://gitlab.denx.de/u-boot/custodians/u-boot-imx.git >> tags/u-boot-imx-20220413 >> >> for you to fetch changes up to a1c711046b0d5478a702b27c6773ea6231eba057: >> >> board: gateworks: venice: add imx8mm-gw7903 support (2022-04-13 09:55:26 >> +0200) >> > > So, I've applied this to master and pushed, and thanks for clearing out > much of the backlog. Is there still things left that aren't recently > newly posted? There are still 3 big series that should flow in: - Marek's series about Power Domain - Gateworks series (Tim) - Toradex series These series conflict with the rest, so they needed to be rebased. But also, the size problem on rcar3_salvator-x is because > every platform with DM_REGULATOR_FIXED grew because: > commit f3b5100aff3a7edd53fd5d15bd6db92d294865e2 > Author: Philippe Schenker <philippe.schenker@toradex.com> > Date: Fri Apr 8 10:07:10 2022 +0200 > > regulator: fixed: add possibility to enable by clock > > adds 300-400 bytes on 64bit platforms. I'm mainly just noting this as I > don't see a nice way to handle this that doesn't then make the CONFIG > side of things troublesome / annoying. > Regards, Stefano
On 4/13/22 22:21, Stefano Babic wrote: > Hi Tom, Hi, [...] >> So, I've applied this to master and pushed, and thanks for clearing out >> much of the backlog. Is there still things left that aren't recently >> newly posted? > > There are still 3 big series that should flow in: > > - Marek's series about Power Domain These are the two series that are left from my side, already rebased: https://patchwork.ozlabs.org/project/uboot/list/?series=294845 https://patchwork.ozlabs.org/project/uboot/list/?series=294849