On Tue, 20 Aug 2019 07:51:18 +0000
Xiaowei Bao <xiaowei....@nxp.com> wrote:

> > -----Original Message-----
> > From: Lukasz Majewski <lu...@denx.de>
> > Sent: 2019年8月20日 15:43
> > To: Xiaowei Bao <xiaowei....@nxp.com>
> > Cc: w...@denx.de; Ruchika Gupta <ruchika.gu...@nxp.com>; Shengzhou Liu
> > <shengzhou....@nxp.com>; ja...@amarulasolutions.com; Prabhakar
> > Kushwaha <prabhakar.kushw...@nxp.com>; u-boot@lists.denx.de; Jiafei
> > Pan <jiafei....@nxp.com>; Pan Jiafei <jiafei....@freescale.com>;
> > Chuanhua Han <chuanhua....@nxp.com>
> > Subject: Re: [U-Boot] [PATCH v5 1/5] spl: dm: disable SPI DM flash
> > for non-DM SPL
> > 
> > On Tue, 20 Aug 2019 07:27:39 +0000
> > Xiaowei Bao <xiaowei....@nxp.com> wrote:
> >   
> > > > -----Original Message-----
> > > > From: Lukasz Majewski <lu...@denx.de>
> > > > Sent: 2019年8月20日 15:14
> > > > To: Xiaowei Bao <xiaowei....@nxp.com>
> > > > Cc: w...@denx.de; Ruchika Gupta <ruchika.gu...@nxp.com>;
> > > > Shengzhou  
> > Liu  
> > > > <shengzhou....@nxp.com>; ja...@amarulasolutions.com; Prabhakar
> > > > Kushwaha <prabhakar.kushw...@nxp.com>; u-boot@lists.denx.de;
> > > > Jiafei Pan <jiafei....@nxp.com>; Pan Jiafei
> > > > <jiafei....@freescale.com>; Chuanhua Han <chuanhua....@nxp.com>
> > > > Subject: Re: [U-Boot] [PATCH v5 1/5] spl: dm: disable SPI DM
> > > > flash for non-DM SPL
> > > >
> > > > Hi Xiaowei,
> > > >  
> > > > > From: Chuanhua Han <chuanhua....@nxp.com>
> > > > >
> > > > > This patch solves the problem that spiboot cannot be
> > > > > performed in non-DM SPL.  
> > > >
> > > > I've posted recently a set of patches to move setting of
> > > > DM_SPI_FLASH to Kconfig:
> > > > https://patchwork.ozlabs.org/cover/1146494/
> > > >
> > > > Please rebase your work on top of it.  
> > > Did this set of patches merge to the u-boot mainline?  
> > 
> > Those patches were not yet accepted to mainline.
> >   
> > > Do I need to
> > > depend on your patches?  
> > 
> > It would be best to avoid duplicating the effort.
> >   
> > > If yes, please provide the full patches link, I can't see the
> > > contents of this set of patches.  
> > 
> > When you open the link:
> > https://patchwork.ozlabs.org/cover/1146494/
> > 
> > On top left you have:
> > "Related" -> "show"
> > 
> > After clicking on it you will see the list of patches.
> > 
> > You may also want to use "Series" on top right and download the
> > whole patch series.  
> I have check these set of patches, it don't impact my patches, my
> patches implemented the ESPI DM driver of POWERPC.

But you add:

+#undef CONFIG_DM_SPI_FLASH

for SPL, Which is problematic as explained in the patch set.

> >   
> > > >  
> > > > >
> > > > > Signed-off-by: Pan Jiafei <jiafei....@freescale.com>
> > > > > Signed-off-by: Chuanhua Han <chuanhua....@nxp.com>
> > > > > ---
> > > > > Changes in v5:
> > > > >       - No change.
> > > > > Changes in v4:
> > > > >       - No change.
> > > > > Changes in v3:
> > > > >       - Add a cover-letter for this patch set.
> > > > > Changes in v2:
> > > > >       - No change.
> > > > >
> > > > >  include/config_uncmd_spl.h | 1 +
> > > > >  1 file changed, 1 insertion(+)
> > > > >
> > > > > diff --git a/include/config_uncmd_spl.h
> > > > > b/include/config_uncmd_spl.h index c2f9735..da94b3d 100644
> > > > > --- a/include/config_uncmd_spl.h
> > > > > +++ b/include/config_uncmd_spl.h
> > > > > @@ -15,6 +15,7 @@
> > > > >  #undef CONFIG_DM_GPIO
> > > > >  #undef CONFIG_DM_I2C
> > > > >  #undef CONFIG_DM_SPI
> > > > > +#undef CONFIG_DM_SPI_FLASH
> > > > >  #endif
> > > > >
> > > > >  #undef CONFIG_DM_WARN  
> > > >
> > > >
> > > >
> > > > Best regards,
> > > >
> > > > Lukasz Majewski
> > > >
> > > > --
> > > >
> > > > DENX Software Engineering GmbH,      Managing Director: Wolfgang
> > > > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194
> > > > Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax:
> > > > (+49)-8142-66989-80 Email: lu...@denx.de  
> > 
> > 
> > 
> > Best regards,
> > 
> > Lukasz Majewski
> > 
> > --
> > 
> > DENX Software Engineering GmbH,      Managing Director: Wolfgang
> > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell,
> > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email:
> > lu...@denx.de  



Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de

Attachment: pgpV3rTsICBaF.pgp
Description: OpenPGP digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to