[U-Boot] [PATCH v2 00/32] spi: DM_SPI migration timeout, remainder(2)
Peng Fan
peng.fan at nxp.com
Mon Nov 26 07:18:52 UTC 2018
Hi Jagan,
Just have a quick question here.
After dropping non-DM code, for SPL use non-DM code should switch to SPL_DM and use SPL OF CONTROL?
Thanks,
Peng.
> -----Original Message-----
> From: Jagan Teki [mailto:jagan at amarulasolutions.com]
> Sent: 2018年11月26日 1:28
> To: Adam Ford <aford173 at gmail.com>; Vitaly Andrianov <vitalya at ti.com>;
> Stefano Babic <sbabic at denx.de>; Peter Howard <phoward at gme.net.au>;
> Tom Rini <trini at konsulko.com>; Walter Schweizer
> <swwa at users.sourceforge.net>; Valentin Longchamp
> <valentin.longchamp at keymile.com>; Phil Sutter <phil at nwl.cc>; Stefan
> Roese <sr at denx.de>; Simon Glass <sjg at chromium.org>; Tuomas Tynkkynen
> <tuomas at tuxera.com>; Sam Protsenko <semen.protsenko at linaro.org>; Dirk
> Eibach <dirk.eibach at gdsys.cc>; Mario Six <mario.six at gdsys.cc>; Simon
> Guinot <simon.guinot at sequanux.org>; Konstantin Porotchkin
> <kostap at marvell.com>; Jason Cooper <u-boot at lakedaemon.net>; Marek
> Behún <marek.behun at nic.cz>; Lokesh Vutla <lokeshvutla at ti.com>; Andrew
> F . Davis <afd at ti.com>; Uri Mashiach <uri.mashiach at compulab.co.il>; Heiko
> Schocher <hs at denx.de>; Tc Liew <tsi-chung.liew at nxp.com>; Joe
> Hershberger <joe.hershberger at ni.com>; Angelo Dureghello
> <angelo at sysam.it>; Vladimir Zapolskiy <vz at mleia.com>; Albert ARIBAUD
> <albert.aribaud at 3adev.fr>; Marek Vasut <marex at denx.de>; Fabio Estevam
> <fabio.estevam at nxp.com>; Nikita Kiryanov <nikita at compulab.co.il>; Lukasz
> Majewski <lukma at denx.de>; Markus Niebel
> <Markus.Niebel at tq-group.com>; Troy Kisky
> <troy.kisky at boundarydevices.com>; Akshay Bhat
> <akshaybhat at timesys.com>; Ken Lin <Ken.Lin at advantech.com.tw>; Peng
> Fan <peng.fan at nxp.com>; Magnus Lilja <lilja.magnus at gmail.com>; Christian
> Gmeiner <christian.gmeiner at gmail.com>; Eric Bénard <eric at eukrea.com>;
> Ian Ray <ian.ray at ge.com>; Otavio Salvador <otavio at ossystems.com.br>;
> Andreas Geisreiter <ageisreiter at dh-electronics.de>; Ludwig Zenz
> <lzenz at dh-electronics.de>; Dmitry Lifshitz <lifshitz at compulab.co.il>; Roger
> Meier <r.meier at siemens.com>; Gilles Gameiro <gilles at gigadevices.com>;
> Lothar Felten <lothar.felten at gmail.com>; Tapani Utriainen
> <linuxfae at technexion.com>; Steve Sakoman <sakoman at gmail.com>; Pali
> Rohár <pali.rohar at gmail.com>; Marcin Niestroj
> <m.niestroj at grinn-global.com>; Nishanth Menon <nm at ti.com>; Lars
> Poeschel <poeschel at lemonage.de>; Enric Balletbo i Serra
> <eballetbo at gmail.com>; Yegor Yefremov <yegorslists at googlemail.com>; Ash
> Charles <ash at gumstix.com>; Grazvydas Ignotas <notasas at gmail.com>; Igor
> Grinberg <grinberg at compulab.co.il>; Derald D . Woods
> <woods.technical at gmail.com>; Hannes Schmelzer
> <hannes.schmelzer at br-automation.com>; Eugen Hristev
> <eugen.hristev at microchip.com>; Ludovic Desroches
> <ludovic.desroches at microchip.com>; Stelian Pop <stelian at popies.net>;
> Mateusz Kulikowski <mateusz.kulikowski at gmail.com>; Ilko Iliev
> <iliev at ronetix.at>; Daniel Gorsulowski <daniel.gorsulowski at esd.eu>; Ben
> Whitten <ben.whitten at lairdtech.com>; Andreas Bießmann
> <andreas at biessmann.org>; Erik van Luijk <evanluijk at interact.nl>; Gregory
> CLEMENT <gregory.clement at free-electrons.com>; Prabhakar Kushwaha
> <prabhakar.kushwaha at nxp.com>; Sumit Garg <sumit.garg at nxp.com>; York
> Sun <york.sun at nxp.com>; Yinbo Zhu <yinbo.zhu at nxp.com>; Priyanka Jain
> <priyanka.jain at nxp.com>; Mingkai Hu <mingkai.hu at nxp.com>; Alison Wang
> <alison.wang at nxp.com>; Ashish Kumar <ashish.kumar at nxp.com>; Udit
> Agarwal <udit.agarwal at nxp.com>; VINITHA V PILLAI
> <vinitha.pillai at nxp.com>; Feng Li <feng.li_2 at nxp.com>; Qiang Zhao
> <qiang.zhao at nxp.com>; Naveen Burmi <naveen.burmi at nxp.com>;
> Shengzhou Liu <shengzhou.liu at nxp.com>; Ruchika Gupta
> <ruchika.gupta at nxp.com>; Poonam Aggrwal <poonam.aggrwal at nxp.com>;
> Timur Tabi <timur at tabi.org>; Xiaobo Xie <xiaobo.xie at nxp.com>; PO LIU
> <po.liu at nxp.com>
> Cc: u-boot at lists.denx.de; Jagan Teki <jagan at amarulasolutions.com>
> Subject: [PATCH v2 00/32] spi: DM_SPI migration timeout, remainder(2)
>
> This series fix, build issues and comments from previous series [1].
>
> We have not heard any positive feed back about dm-conversion on spi drivers
> along with no response for the work done by Akash and me.
>
> Plan is to send the same series remainder recursively based on the dm
> conversion updates and finally merged into v2019.04 MW.
>
> So expecting inputs from relevant developers to move their board/drivers to
> support DM_SPI.
>
> - cf_spi, lpc32xx_ssp, mxs_spi, sh_spi, soft_spi_legacy, fsl_espi drivers are
> drooped with their respective CONFIG items.
> the spi/spi-flash usage commands still build but the used spi core function
> calls return 0.
> - non-dm code in davinci, kirkwood, ti_qspi, mpc8xxx, mxc, omap3, atmel,
> fsl_dspi drivers were drooped and u-boot build system will trigger warning for
> the same.
>
> Note:
> - Let's discuss if any foot-print issues?
>
> Any inputs,
> Jagan.
>
> [1]
> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> hwork.ozlabs.org%2Fcover%2F1000444%2F&data=02%7C01%7Cpeng.fa
> n%40nxp.com%7C575d95c25be74fc237bb08d652fb8b34%7C686ea1d3bc2b4
> c6fa92cd99c5c301635%7C0%7C0%7C636787637664212094&sdata=Wy
> H54iAVB%2BRPIVvgpQsw1T3cG5UUtNRKbFR08wibgOg%3D&reserved=
> 0
>
> Jagan Teki (15):
> spi: davinci: Full dm conversion
> spi: kirkwood: Full dm conversion
> spi: ti_qspi: Full dm conversion
> spi: mpc8xxx: Convert to DM
> spi: Zap cf_spi driver-related code
> spi: Zap lpc32xx_ssp driver-related code
> spi: Zap mxs_spi driver-related code
> spi: Zap sh_spi driver-related code
> spi: Zap soft_spi_legacy driver-related code
> spi: mxc: Drop non-dm code
> spi: omap3: Drop non-dm code
> spi: atmel: Drop non-dm code
> spi: Zap fsl_espi driver-related code
> spi: fsl_dspi: Drop non-dm code
> dm: MIGRATION: spi: Update SPI driver status
>
> Mario Six (17):
> spi: mpc8xxx: Use short type names
> spi: mpc8xxx: Fix comments
> spi: mpc8xxx: Rename camel-case variables
> spi: mpc8xxx: Fix space after cast
> spi: mpc8xxx: Fix function names in strings
> spi: mpc8xxx: Replace defines with enums
> spi: mpc8xxx: Use IO accessors
> spi: mpc8xxx: Simplify if
> spi: mpc8xxx: Get rid of is_read
> spi: mpc8xxx: Simplify logic a bit
> spi: mpc8xxx: Reduce scope of loop variables
> spi: mpc8xxx: Make code more readable
> spi: mpc8xxx: Rename variable
> spi: mpc8xxx: Document LEN setting better
> spi: mpc8xxx: Re-order transfer setup
> spi: mpc8xxx: Fix if check
> spi: mpc8xxx: Use get_timer
>
> Makefile | 11 +
> arch/m68k/cpu/mcf5227x/cpu_init.c | 54 --
> arch/m68k/cpu/mcf5445x/cpu_init.c | 113 ----
> board/Arcturus/ucp1020/ucp1020.c | 6 +
> configs/M52277EVB_defconfig | 1 -
> configs/M52277EVB_stmicro_defconfig | 1 -
> configs/M54418TWR_defconfig | 1 -
> configs/M54418TWR_nand_mii_defconfig | 1 -
> configs/M54418TWR_nand_rmii_defconfig | 1 -
> configs/M54418TWR_nand_rmii_lowfreq_defconfig | 1 -
> configs/M54418TWR_serial_mii_defconfig | 1 -
> configs/M54418TWR_serial_rmii_defconfig | 1 -
> configs/M54451EVB_defconfig | 1 -
> configs/M54451EVB_stmicro_defconfig | 1 -
> configs/M54455EVB_a66_defconfig | 1 -
> configs/M54455EVB_defconfig | 1 -
> configs/M54455EVB_i66_defconfig | 1 -
> configs/M54455EVB_intel_defconfig | 1 -
> configs/M54455EVB_stm33_defconfig | 1 -
> configs/bg0900_defconfig | 1 -
> configs/devkit3250_defconfig | 1 -
> configs/mx28evk_auart_console_defconfig | 1 -
> configs/mx28evk_defconfig | 1 -
> configs/mx28evk_nand_defconfig | 1 -
> configs/mx28evk_spi_defconfig | 1 -
> configs/sh7752evb_defconfig | 1 -
> configs/sh7753evb_defconfig | 1 -
> configs/sh7757lcr_defconfig | 1 -
> configs/stmark2_defconfig | 1 -
> configs/work_92105_defconfig | 1 -
> doc/driver-model/MIGRATION.txt | 18 +-
> drivers/spi/Kconfig | 42 +-
> drivers/spi/Makefile | 22 +-
> drivers/spi/atmel_spi.c | 198 -------
> drivers/spi/cf_spi.c | 335 ------------
> drivers/spi/davinci_spi.c | 157 ------
> drivers/spi/fsl_dspi.c | 132 -----
> drivers/spi/fsl_espi.c | 383 --------------
> drivers/spi/kirkwood_spi.c | 218 ++------
> drivers/spi/lpc32xx_ssp.c | 134 -----
> drivers/spi/mpc8xxx_spi.c | 279 ++++++----
> drivers/spi/mxc_spi.c | 123 +----
> drivers/spi/mxs_spi.c | 358 -------------
> drivers/spi/omap3_spi.c | 126 -----
> drivers/spi/sh_spi.c | 249 ---------
> drivers/spi/sh_spi.h | 67 ---
> drivers/spi/soft_spi_legacy.c | 168 ------
> drivers/spi/spi.c | 38 ++
> drivers/spi/ti_qspi.c | 490 +++++++-----------
> include/configs/M52277EVB.h | 1 -
> include/configs/M54418TWR.h | 1 -
> include/configs/M54451EVB.h | 1 -
> include/configs/M54455EVB.h | 1 -
> include/configs/MPC8349EMDS.h | 3 -
> include/configs/devkit3250.h | 5 -
> include/configs/sh7752evb.h | 3 -
> include/configs/sh7753evb.h | 3 -
> include/configs/sh7757lcr.h | 3 -
> include/configs/stmark2.h | 1 -
> include/configs/work_92105.h | 5 -
> include/configs/zipitz2.h | 1 -
> include/dm/platform_data/spi_kirkwood.h | 15 +
> include/dm/platform_data/spi_ti_qspi.h | 19 +
> scripts/config_whitelist.txt | 3 -
> 64 files changed, 502 insertions(+), 3310 deletions(-) delete mode 100644
> drivers/spi/cf_spi.c delete mode 100644 drivers/spi/fsl_espi.c delete
> mode 100644 drivers/spi/lpc32xx_ssp.c delete mode 100644
> drivers/spi/mxs_spi.c delete mode 100644 drivers/spi/sh_spi.c delete
> mode 100644 drivers/spi/sh_spi.h delete mode 100644
> drivers/spi/soft_spi_legacy.c create mode 100644
> include/dm/platform_data/spi_kirkwood.h
> create mode 100644 include/dm/platform_data/spi_ti_qspi.h
>
> --
> 2.18.0.321.gffc6fa0e3
More information about the U-Boot
mailing list