[U-Boot] [PATCH 4/5] toradex: configblock: fix apalis imx8 target

[Peng Fan]

Hi,

> Subject: Re: [U-Boot] [PATCH 4/5] toradex: configblock: fix apalis imx8 target
> 
> Hi Igor
> 
> On Tue, 2019-07-09 at 10:27 +0000, Igor Opaniuk wrote:
> > HI Marcel,
> >
> > On Tue, Jul 9, 2019 at 12:43 PM Marcel Ziswiler <marcel at ziswiler.com>
> > wrote:
> > > From: Marcel Ziswiler <marcel.ziswiler at toradex.com>
> > >
> > > The Apalis iMX8 was missing the interactive part should a customer
> > > have bricked his module and want to re-create the configuration
> > > block.
> > > Fix
> > > this.
> > >
> > > Signed-off-by: Marcel Ziswiler <marcel.ziswiler at toradex.com>
> > >
> > > ---
> > >
> > >  board/toradex/common/tdx-cfg-block.c | 15 ++++++++++++++-
> > >  1 file changed, 14 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/board/toradex/common/tdx-cfg-block.c
> > > b/board/toradex/common/tdx-cfg-block.c
> > > index 90df5bb2ff..a81b0f637c 100644
> > > --- a/board/toradex/common/tdx-cfg-block.c
> > > +++ b/board/toradex/common/tdx-cfg-block.c
> > > @@ -7,6 +7,7 @@
> > >  #include "tdx-cfg-block.h"
> > >
> > >  #if defined(CONFIG_TARGET_APALIS_IMX6) || \
> > > +       defined(CONFIG_TARGET_APALIS_IMX8) || \
> > >         defined(CONFIG_TARGET_COLIBRI_IMX6) || \
> > >         defined(CONFIG_TARGET_COLIBRI_IMX8X)
> > >  #include <asm/arch/sys_proto.h>
> > > @@ -356,7 +357,19 @@ static int get_cfgblock_interactive(void)
> > >                 tdx_hw_tag.prodid = COLIBRI_IMX7D;
> > >         else if (!strcmp("imx7s", soc))
> > >                 tdx_hw_tag.prodid = COLIBRI_IMX7S;
> > > -       else if (is_cpu_type(MXC_CPU_IMX8QXP)) {
> > > +       else if (is_cpu_type(MXC_CPU_IMX8QM)) {
> > > +               if (it == 'y' || it == 'Y') {
> > > +                       if (wb == 'y' || wb == 'Y')
> > > +                               tdx_hw_tag.prodid =
> > > APALIS_IMX8QM_WIFI_BT_IT;
> > > +                       else
> > > +                               tdx_hw_tag.prodid =
> > > APALIS_IMX8QM_IT;
> > > +               } else {
> > > +                       if (wb == 'y' || wb == 'Y')
> > > +                               tdx_hw_tag.prodid =
> > > APALIS_IMX8QP_WIFI_BT;
> >
> > Just curious why NXP guys decided to call this CPU type as
> > MXC_CPU_IMX8QM, and why not to drop the last M, or even both QM
> > (taking into account that marketing name is simple IMX8 for these SoC
> > series [1]. and iMX8M and iMX8X respectively for M and X series),
> > because it's not obvious for me that APALIS_IMX8QP has any relation to
> > MXC_CPU_IMX8QM.
> 
> Good questions. Maybe some of the NXP folks dare answering?

Thanks for adding me, please also add uboot-imx at nxp.com for i.MX patches
for patch review. I am not able catch every thread in community.

Back to the question,
MXC_CPU_IMX8QM is just follow i.MX8QM. no special reason.

So you are using i.MX8QP? or i.MX8QM?

Thanks,
Peng.

> 
> Cheers
> 
> Marcel
> 
> > > +                       else
> > > +                               tdx_hw_tag.prodid =
> APALIS_IMX8QP;
> > > +               }
> > > +       } else if (is_cpu_type(MXC_CPU_IMX8QXP)) {
> > >                 if (it == 'y' || it == 'Y') {
> > >                         if (wb == 'y' || wb == 'Y')
> > >                                 tdx_hw_tag.prodid =
> > > COLIBRI_IMX8QXP_WIFI_BT_IT;
> > > --
> > > 2.21.0
> > >
> > > _______________________________________________
> > > U-Boot mailing list
> > > U-Boot at lists.denx.de
> > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fli
> > >
> sts.denx.de%2Flistinfo%2Fu-boot&data=02%7C01%7Cpeng.fan%40nxp.c
> o
> > >
> m%7C6f1126063c684a60ae7008d7050115ca%7C686ea1d3bc2b4c6fa92cd99
> c5c301
> > >
> 635%7C0%7C0%7C636983374525754073&sdata=heJJTIdWibdA9594lJu
> FXZ6AS
> > > bWx%2FKrjL160ahWX0dM%3D&reserved=0
> >
> > Reviewed-by: Igor Opaniuk <igor.opaniuk at toradex.com>
> >
> > [1]
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fen.w
> >
> ikipedia.org%2Fwiki%2FI.MX%23i.MX_8_series&data=02%7C01%7Cpen
> g.fan
> > %40nxp.com%7C6f1126063c684a60ae7008d7050115ca%7C686ea1d3bc2b
> 4c6fa92cd9
> >
> 9c5c301635%7C0%7C0%7C636983374525754073&sdata=x7uth7RukJ1J
> x46Dzkav
> > 39TtPidNmXrggBc7a4xi2Kk%3D&reserved=0
> >
> > --
> > Best regards - Freundliche Grüsse - Meilleures salutations
> >
> > Senior Development Engineer,
> > Igor Opaniuk
> >
> > Toradex AG
> > Altsagenstrasse 5 | 6048 Horw/Luzern | Switzerland | T: +41 41 500 48
> > 00 (main line)