[U-Boot] [EXT] Re: [PATCH v4 1/5] spl: dm: disable SPI DM flash for non-DM SPL
Lukasz Majewski
lukma at denx.de
Thu Aug 15 08:12:20 UTC 2019
On Thu, 15 Aug 2019 07:57:00 +0000
Chuanhua Han <chuanhua.han at nxp.com> wrote:
> > -----Original Message-----
> > From: Lukasz Majewski <lukma at denx.de>
> > Sent: 2019年8月15日 15:54
> > To: Chuanhua Han <chuanhua.han at nxp.com>
> > Cc: Jagan Teki <jagan at amarulasolutions.com>; Xiaowei Bao
> > <xiaowei.bao at nxp.com>; U-Boot-Denx <u-boot at lists.denx.de>; Pan
> > Jiafei <Jiafei.Pan at freescale.com>; Jagan Teki <jagan at openedev.com>;
> > Ruchika Gupta <ruchika.gupta at nxp.com>
> > Subject: Re: [U-Boot] [EXT] Re: [PATCH v4 1/5] spl: dm: disable SPI
> > DM flash for non-DM SPL
> >
> > On Thu, 15 Aug 2019 07:39:45 +0000
> > Chuanhua Han <chuanhua.han at nxp.com> wrote:
> >
> > > > -----Original Message-----
> > > > From: Lukasz Majewski <lukma at denx.de>
> > > > Sent: 2019年8月15日 15:15
> > > > To: Chuanhua Han <chuanhua.han at nxp.com>
> > > > Cc: Jagan Teki <jagan at amarulasolutions.com>; Xiaowei Bao
> > > > <xiaowei.bao at nxp.com>; U-Boot-Denx <u-boot at lists.denx.de>; Pan
> > > > Jiafei <Jiafei.Pan at freescale.com>; Jagan Teki
> > > > <jagan at openedev.com>; Ruchika Gupta <ruchika.gupta at nxp.com>
> > > > Subject: Re: [U-Boot] [EXT] Re: [PATCH v4 1/5] spl: dm: disable
> > > > SPI DM flash for non-DM SPL
> > > >
> > > > On Thu, 15 Aug 2019 03:35:28 +0000
> > > > Chuanhua Han <chuanhua.han at nxp.com> wrote:
> > > >
> > > > > > -----Original Message-----
> > > > > > From: Lukasz Majewski <lukma at denx.de>
> > > > > > Sent: 2019年8月14日 19:39
> > > > > > To: Chuanhua Han <chuanhua.han at nxp.com>
> > > > > > Cc: Jagan Teki <jagan at amarulasolutions.com>; Xiaowei Bao
> > > > > > <xiaowei.bao at nxp.com>; U-Boot-Denx <u-boot at lists.denx.de>;
> > > > > > Pan Jiafei <Jiafei.Pan at freescale.com>; Jagan Teki
> > > > > > <jagan at openedev.com>; Ruchika Gupta <ruchika.gupta at nxp.com>
> > > > > > Subject: Re: [U-Boot] [EXT] Re: [PATCH v4 1/5] spl: dm:
> > > > > > disable SPI DM flash for non-DM SPL
> > > > > >
> > > > > > On Wed, 14 Aug 2019 11:10:46 +0000 Chuanhua Han
> > > > > > <chuanhua.han at nxp.com> wrote:
> > > > > >
> > > > > > > > -----Original Message-----
> > > > > > > > From: Lukasz Majewski <lukma at denx.de>
> > > > > > > > Sent: 2019年8月14日 18:53
> > > > > > > > To: Chuanhua Han <chuanhua.han at nxp.com>
> > > > > > > > Cc: Jagan Teki <jagan at amarulasolutions.com>; Xiaowei Bao
> > > > > > > > <xiaowei.bao at nxp.com>; U-Boot-Denx
> > > > > > > > <u-boot at lists.denx.de>; Pan Jiafei
> > > > > > > > <Jiafei.Pan at freescale.com>; Jagan Teki
> > > > > > > > <jagan at openedev.com>; Ruchika Gupta
> > > > > > > > <ruchika.gupta at nxp.com> Subject: Re: [U-Boot] [EXT] Re:
> > > > > > > > [PATCH v4 1/5] spl: dm: disable SPI DM flash for non-DM
> > > > > > > > SPL
> > > > > > > >
> > > > > > > > On Wed, 14 Aug 2019 10:41:32 +0000 Chuanhua Han
> > > > > > > > <chuanhua.han at nxp.com> wrote:
> > > > > > > >
> > > > > > > > > > -----Original Message-----
> > > > > > > > > > From: Lukasz Majewski <lukma at denx.de>
> > > > > > > > > > Sent: 2019年8月14日 16:57
> > > > > > > > > > To: Chuanhua Han <chuanhua.han at nxp.com>
> > > > > > > > > > Cc: Jagan Teki <jagan at amarulasolutions.com>;
> > > > > > > > > > Xiaowei Bao <xiaowei.bao at nxp.com>; U-Boot-Denx
> > > > > > > > > > <u-boot at lists.denx.de>; Pan Jiafei
> > > > > > > > > > <Jiafei.Pan at freescale.com>; Jagan Teki
> > > > > > > > > > <jagan at openedev.com>; Ruchika Gupta
> > > > > > > > > > <ruchika.gupta at nxp.com> Subject: Re: [U-Boot] [EXT]
> > > > > > > > > > Re: [PATCH v4 1/5] spl: dm: disable SPI DM flash
> > > > > > > > > > for non-DM SPL
> > > > > > > > > >
> > > > > > > > > > Hi Chuanhua,
> > > > > > > > > >
> > > > > > > > > > > > -----Original Message-----
> > > > > > > > > > > > From: Jagan Teki <jagan at amarulasolutions.com>
> > > > > > > > > > > > Sent: 2019年8月14日 15:45
> > > > > > > > > > > > To: Chuanhua Han <chuanhua.han at nxp.com>
> > > > > > > > > > > > Cc: Wolfgang Denk <wd at denx.de>; Shengzhou Liu
> > > > > > > > > > > > <shengzhou.liu at nxp.com>; Ruchika Gupta
> > > > > > > > > > > > <ruchika.gupta at nxp.com>; Jagan Teki
> > > > > > > > > > > > <jagan at openedev.com>; Simon Glass
> > > > > > > > > > > > <sjg at chromium.org>; Prabhakar Kushwaha
> > > > > > > > > > > > <prabhakar.kushwaha at nxp.com>; U-Boot-Denx
> > > > > > > > > > > > <u-boot at lists.denx.de>; Pan Jiafei
> > > > > > > > > > > > <Jiafei.Pan at freescale.com> Subject: [EXT] Re:
> > > > > > > > > > > > [U-Boot] [PATCH v4 1/5] spl: dm: disable SPI DM
> > > > > > > > > > > > flash for non-DM SPL
> > > > > > > > > > > >
> > > > > > > > > > > > Caution: EXT Email
> > > > > > > > > > > >
> > > > > > > > > > > > On Fri, Jun 21, 2019 at 7:51 AM Chuanhua Han
> > > > > > > > > > > > <chuanhua.han at nxp.com> wrote:
> > > > > > > > > > > > >
> > > > > > > > > > > > > This patch solves the problem that spiboot
> > > > > > > > > > > > > cannot be performed in non-DM SPL.
> > > > > > > > > > > > >
> > > > > > > > > > > > > Signed-off-by: Pan Jiafei
> > > > > > > > > > > > > <Jiafei.Pan at freescale.com> Signed-off-by:
> > > > > > > > > > > > > Chuanhua Han <chuanhua.han at nxp.com> ---
> > > > > > > > > > > > > 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
> > > > > > > > > > > > > c2f9735ce7..da94b3d9df 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
> > > > > > > > > > > >
> > > > > > > > > > > > Have you made any Travis-CI run?
> > > > > > > > > > > I tested it with a local board
> > > > > > > > > >
> > > > > > > > > > Could you be so kind and also test following
> > > > > > > > > > patchset (parts from Layerscape also needs testing):
> > > > > > > > > > https://patchwork.ozlabs.org/cover/1146494/
> > > > > > > > > I don't quite understand this patch. How can I test
> > > > > > > > > it?
> > > > > > > >
> > > > > > > > You need to apply it first to -master branch.
> > > > > > > >
> > > > > > > > Then apply your board code (or if it is already in
> > > > > > > > mainline just remove the #undef(s) from
> > > > > > > > include/config_uncmd_spl.h)
> > > > > > > >
> > > > > > > > As a last step - run make menuconfig and enable
> > > > > > > > SPL_DM_SPI* flags.
> > > > > > > Currently, it is in the master, and SPI* in the #undef(s)
> > > > > > > from include/ uncmd_spl.h has been deleted, but an error
> > > > > > > was reported: CC spl/lib/display_options.o
> > > > > > > drivers/spi/spi.c: In function 'spi_do_alloc_slave':
> > > > > > > drivers/spi/spi.c:34:8: error: 'struct spi_slave' has no
> > > > > > > member named 'bus' slave->bus = bus; ^~
> > > > > > > drivers/spi/spi.c:35:8: error: 'struct spi_slave' has no
> > > > > > > member named 'cs' slave->cs = cs;
> > > > > > > ^~
> > > > > > > make[3]: *** [spl/drivers/spi/spi.o] Error 1
> > > > > > > make[3]: *** Waiting for unfinished jobs....
> > > > > > > make[2]: *** [spl/drivers/spi] Error 2
> > > > > > > make[1]: *** [spl/drivers] Error 2
> > > > > > > make[1]: *** Waiting for unfinished jobs....
> > > > > > > LD spl/lib/built-in.o
> > > > > > > make: *** [spl/u-boot-spl] Error 2
> > > > > >
> > > > > > If your board is in mainline (master branch) it shall build
> > > > > > without issues:
> > > > > >
> > > > > > Applied on top of -master branch:
> > > > > > 'commit feb5a02f869d ("Merge branch 'master' of
> > > > > > git://git.denx.de/u-boot-sh")' Travis-CI (clean build):
> > > > > > https://travis-ci.org/lmajewski/u-boot-dfu/builds/570594278
> > > > > >
> > > > > >
> > > > > > The patch itself:
> > > > > > https://patchwork.ozlabs.org/patch/1146377/
> > > > > 1. I git reset to this commit: feb5a02f869d 2. Your patch has
> > > > > been applied(https://patchwork.ozlabs.org/patch/1146377/)
> > > >
> > > > Could you check if your board (the code which is now in
> > > > mainline) compiles without issues at this step?
> > > Yes,.no issue!(because this commit not support espi dm mode)
> >
> > Ok.
> >
> > > >
> > > > >3. Apply my
> > > > > current patch set 4. Include/uncmd_uncmd_spl.h file is
> > > > > deleted as follows:
> > > > > - # undef CONFIG_DM_SPI
> > > > > - # undef CONFIG_DM_SPI_FLASH
> > > > > 5. In include/configs/T208xQDS.h file:
> > > > > +/* SPI Flash Configs */
> > > > > + # if defined (CONFIG_SPL_BUILD)
> > > > > + # undef CONFIG_DM_SPI
> > > > > + # undef CONFIG_DM_SPI_FLASH
> > > > > + # undef CONFIG_SPI_FLASH_MTD
> >
> > Setting #undef in config file causes !CONFIG_IS_ENABLED(FOO), to
> > malfunction as the FOO is not set in .config.
> So I need to put these #undef's in a Kconfig file, right?
The patchset which I've posted does exactly that. Just try to adjust
your code on top of it.
> >
> > > > > + # endif
> > > > > 6. Make menuconfig:
> > > > > Configuration
> > > > > CONFIG_SPL_DM_SPI = y
> > > > > CONFIG_SPL_DM_SPI_FLASH = y
> > > > > 7. Compilation errors still occur:
> > > > > LD spl/lib/built-in.o
> > > > > LD spl/u-boot-spl
> > > > > drivers/built-in.o: In function `ofnode_read_u32':
> > > > > /home/hanch/uboot/uboot-upstream/u-boot/drivers/core/ofnode.c:29:
> > > > > undefined reference to `fdt_getprop' drivers/built-in.o: In
> > > > > function `ofnode_get_property':
> > > > > /home/hanch/uboot/uboot-upstream/u-boot/drivers/core/ofnode.c:528:
> > > > > undefined reference to `fdt_getprop' drivers/built-in.o: In
> > > > > function `blk_post_probe':
> > > > >
> > /home/hanch/uboot/uboot-upstream/u-boot/drivers/block/blk-uclass.c:655:
> >
> > > > > undefined reference to `part_init'
> > > > > drivers/built-in.o:(.u_boot_list_2_uclass_2_spi+0x8):
> > > > > undefined reference to `dm_scan_fdt_dev' make[1]: ***
> > > > > [spl/u-boot-spl] Error 1 make: *** [spl/u-boot-spl] Error 2
> > > > > >
> > > > > > >
> > > > > > > Do I need to modify a lot of files?
> > > > > > > >
> > > > > > > > > >
> > > > > > > > > > Which allows removal of
> > > > > > > > > > #undef CONFIG_DM_SPI
> > > > > > > > > > #undef CONFIG_DM_SPI_FLASH
> > > > > > > > > >
> > > > > > > > > > and have the same configuration done via Kconfig ?
> > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > > _______________________________________________
> > > > > > > > > > > U-Boot mailing list
> > > > > > > > > > > U-Boot at lists.denx.de
> > > > > > > > > > > https://lists.denx.de/listinfo/u-boot
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > 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:
> > > > > > > > > > lukma at 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: lukma at 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: lukma at 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: lukma at 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:
> > lukma at 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: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190815/150b6fbf/attachment.sig>
More information about the U-Boot
mailing list