[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