Message ID | 031c373b-e03a-0f0e-bfc1-f335e9939fea@denx.de |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
Series | [GIT,PULL] Pull request: u-boot-imx u-boot-imx-20200310 | expand |
Hi Stefano, On Tue, Mar 10, 2020 at 12:43 PM Stefano Babic <sbabic@denx.de> wrote: > > Hi Tom, > > please pull (fixes) from u-boot-imx, thanks ! It seems that Anatolij's fix is still missing: https://patchwork.ozlabs.org/patch/1239043/ It is needed for booting i.MX8/8X. Thanks
On Tue, Mar 10, 2020 at 01:37:33PM -0300, Fabio Estevam wrote: > Hi Stefano, > > On Tue, Mar 10, 2020 at 12:43 PM Stefano Babic <sbabic@denx.de> wrote: > > > > Hi Tom, > > > > please pull (fixes) from u-boot-imx, thanks ! > > It seems that Anatolij's fix is still missing: > https://patchwork.ozlabs.org/patch/1239043/ > > It is needed for booting i.MX8/8X. I probably messed things up here then. With https://patchwork.ozlabs.org/patch/1239143/ and https://patchwork.ozlabs.org/patch/1239144/ is there not a different fix needed for i.MX8 now, or were there two issues here that I confused?
On Tue, Mar 10, 2020 at 2:15 PM Tom Rini <trini@konsulko.com> wrote: > I probably messed things up here then. With > https://patchwork.ozlabs.org/patch/1239143/ and > https://patchwork.ozlabs.org/patch/1239144/ is there not a different fix > needed for i.MX8 now, or were there two issues here that I confused? My understanding is that these two issues are meson related and we still need the one for i.MX8QXP to boot. Regards, Fabio Estevam
On Tue, Mar 10, 2020 at 02:34:26PM -0300, Fabio Estevam wrote: > On Tue, Mar 10, 2020 at 2:15 PM Tom Rini <trini@konsulko.com> wrote: > > > I probably messed things up here then. With > > https://patchwork.ozlabs.org/patch/1239143/ and > > https://patchwork.ozlabs.org/patch/1239144/ is there not a different fix > > needed for i.MX8 now, or were there two issues here that I confused? > > My understanding is that these two issues are meson related and we > still need the one for i.MX8QXP to boot. To be clear, we don't just need a flag added to some i.MX8 drivers like in the Meson case?
On Tue, 10 Mar 2020 13:15:52 -0400 Tom Rini trini@konsulko.com wrote: > On Tue, Mar 10, 2020 at 01:37:33PM -0300, Fabio Estevam wrote: > > Hi Stefano, > > > > On Tue, Mar 10, 2020 at 12:43 PM Stefano Babic <sbabic@denx.de> wrote: > > > > > > Hi Tom, > > > > > > please pull (fixes) from u-boot-imx, thanks ! > > > > It seems that Anatolij's fix is still missing: > > https://patchwork.ozlabs.org/patch/1239043/ > > > > It is needed for booting i.MX8/8X. > > I probably messed things up here then. With > https://patchwork.ozlabs.org/patch/1239143/ and > https://patchwork.ozlabs.org/patch/1239144/ is there not a different fix > needed for i.MX8 now, or were there two issues here that I confused? there were two different issues, so patch 1239043 is still needed to boot on i.MX8X. -- Anatolij
On Tue, 10 Mar 2020 13:37:32 -0400 Tom Rini trini@konsulko.com wrote: > On Tue, Mar 10, 2020 at 02:34:26PM -0300, Fabio Estevam wrote: > > On Tue, Mar 10, 2020 at 2:15 PM Tom Rini <trini@konsulko.com> wrote: > > > > > I probably messed things up here then. With > > > https://patchwork.ozlabs.org/patch/1239143/ and > > > https://patchwork.ozlabs.org/patch/1239144/ is there not a different fix > > > needed for i.MX8 now, or were there two issues here that I confused? > > > > My understanding is that these two issues are meson related and we > > still need the one for i.MX8QXP to boot. > > To be clear, we don't just need a flag added to some i.MX8 drivers like > in the Meson case? no, currently a flag is not required in i.MX8 drivers. -- Anatolij
On Tue, Mar 10, 2020 at 09:26:06PM +0100, Anatolij Gustschin wrote: > On Tue, 10 Mar 2020 13:37:32 -0400 > Tom Rini trini@konsulko.com wrote: > > > On Tue, Mar 10, 2020 at 02:34:26PM -0300, Fabio Estevam wrote: > > > On Tue, Mar 10, 2020 at 2:15 PM Tom Rini <trini@konsulko.com> wrote: > > > > > > > I probably messed things up here then. With > > > > https://patchwork.ozlabs.org/patch/1239143/ and > > > > https://patchwork.ozlabs.org/patch/1239144/ is there not a different fix > > > > needed for i.MX8 now, or were there two issues here that I confused? > > > > > > My understanding is that these two issues are meson related and we > > > still need the one for i.MX8QXP to boot. > > > > To be clear, we don't just need a flag added to some i.MX8 drivers like > > in the Meson case? > > no, currently a flag is not required in i.MX8 drivers. OK, my fault. I see the patch already has Simon's RB so I'll pick it up directly. Sorry for the confusion!
On Tue, Mar 10, 2020 at 04:43:20PM +0100, Stefano Babic wrote: > Hi Tom, > > please pull (fixes) from u-boot-imx, thanks ! > > The following changes since commit 9f976bac2b4a81a13de8124a896a198cbeabb634: > > Merge https://gitlab.denx.de/u-boot/custodians/u-boot-x86 (2020-03-05 > 07:51:12 -0500) > > are available in the Git repository at: > > https://gitlab.denx.de/u-boot/custodians/u-boot-imx.git > tags/u-boot-imx-20200310 > > for you to fetch changes up to 9f656fbee3b433ae1e9daf5d92d46bbdcc9551eb: > > mx6slevk: Convert to DM_ETH (2020-03-10 10:59:08 +0100) > Applied to u-boot/master, thanks!