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:ja...@amarulasolutions.com]
> Sent: 2018年11月26日 1:28
> To: Adam Ford <aford...@gmail.com>; Vitaly Andrianov <vita...@ti.com>;
> Stefano Babic <sba...@denx.de>; Peter Howard <phow...@gme.net.au>;
> Tom Rini <tr...@konsulko.com>; Walter Schweizer
> <s...@users.sourceforge.net>; Valentin Longchamp
> <valentin.longch...@keymile.com>; Phil Sutter <p...@nwl.cc>; Stefan
> Roese <s...@denx.de>; Simon Glass <s...@chromium.org>; Tuomas Tynkkynen
> <tuo...@tuxera.com>; Sam Protsenko <semen.protse...@linaro.org>; Dirk
> Eibach <dirk.eib...@gdsys.cc>; Mario Six <mario....@gdsys.cc>; Simon
> Guinot <simon.gui...@sequanux.org>; Konstantin Porotchkin
> <kos...@marvell.com>; Jason Cooper <u-b...@lakedaemon.net>; Marek
> Behún <marek.be...@nic.cz>; Lokesh Vutla <lokeshvu...@ti.com>; Andrew
> F . Davis <a...@ti.com>; Uri Mashiach <uri.mashi...@compulab.co.il>; Heiko
> Schocher <h...@denx.de>; Tc Liew <tsi-chung.l...@nxp.com>; Joe
> Hershberger <joe.hershber...@ni.com>; Angelo Dureghello
> <ang...@sysam.it>; Vladimir Zapolskiy <v...@mleia.com>; Albert ARIBAUD
> <albert.arib...@3adev.fr>; Marek Vasut <ma...@denx.de>; Fabio Estevam
> <fabio.este...@nxp.com>; Nikita Kiryanov <nik...@compulab.co.il>; Lukasz
> Majewski <lu...@denx.de>; Markus Niebel
> <markus.nie...@tq-group.com>; Troy Kisky
> <troy.ki...@boundarydevices.com>; Akshay Bhat
> <akshayb...@timesys.com>; Ken Lin <ken....@advantech.com.tw>; Peng
> Fan <peng....@nxp.com>; Magnus Lilja <lilja.mag...@gmail.com>; Christian
> Gmeiner <christian.gmei...@gmail.com>; Eric Bénard <e...@eukrea.com>;
> Ian Ray <ian....@ge.com>; Otavio Salvador <ota...@ossystems.com.br>;
> Andreas Geisreiter <ageisrei...@dh-electronics.de>; Ludwig Zenz
> <lz...@dh-electronics.de>; Dmitry Lifshitz <lifsh...@compulab.co.il>; Roger
> Meier <r.me...@siemens.com>; Gilles Gameiro <gil...@gigadevices.com>;
> Lothar Felten <lothar.fel...@gmail.com>; Tapani Utriainen
> <linux...@technexion.com>; Steve Sakoman <sako...@gmail.com>; Pali
> Rohár <pali.ro...@gmail.com>; Marcin Niestroj
> <m.niest...@grinn-global.com>; Nishanth Menon <n...@ti.com>; Lars
> Poeschel <poesc...@lemonage.de>; Enric Balletbo i Serra
> <eballe...@gmail.com>; Yegor Yefremov <yegorsli...@googlemail.com>; Ash
> Charles <a...@gumstix.com>; Grazvydas Ignotas <nota...@gmail.com>; Igor
> Grinberg <grinb...@compulab.co.il>; Derald D . Woods
> <woods.techni...@gmail.com>; Hannes Schmelzer
> <hannes.schmel...@br-automation.com>; Eugen Hristev
> <eugen.hris...@microchip.com>; Ludovic Desroches
> <ludovic.desroc...@microchip.com>; Stelian Pop <stel...@popies.net>;
> Mateusz Kulikowski <mateusz.kulikow...@gmail.com>; Ilko Iliev
> <il...@ronetix.at>; Daniel Gorsulowski <daniel.gorsulow...@esd.eu>; Ben
> Whitten <ben.whit...@lairdtech.com>; Andreas Bießmann
> <andr...@biessmann.org>; Erik van Luijk <evanlu...@interact.nl>; Gregory
> CLEMENT <gregory.clem...@free-electrons.com>; Prabhakar Kushwaha
> <prabhakar.kushw...@nxp.com>; Sumit Garg <sumit.g...@nxp.com>; York
> Sun <york....@nxp.com>; Yinbo Zhu <yinbo....@nxp.com>; Priyanka Jain
> <priyanka.j...@nxp.com>; Mingkai Hu <mingkai...@nxp.com>; Alison Wang
> <alison.w...@nxp.com>; Ashish Kumar <ashish.ku...@nxp.com>; Udit
> Agarwal <udit.agar...@nxp.com>; VINITHA V PILLAI
> <vinitha.pil...@nxp.com>; Feng Li <feng.l...@nxp.com>; Qiang Zhao
> <qiang.z...@nxp.com>; Naveen Burmi <naveen.bu...@nxp.com>;
> Shengzhou Liu <shengzhou....@nxp.com>; Ruchika Gupta
> <ruchika.gu...@nxp.com>; Poonam Aggrwal <poonam.aggr...@nxp.com>;
> Timur Tabi <ti...@tabi.org>; Xiaobo Xie <xiaobo....@nxp.com>; PO LIU
> <po....@nxp.com>
> Cc: u-boot@lists.denx.de; Jagan Teki <ja...@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&amp;data=02%7C01%7Cpeng.fa
> n%40nxp.com%7C575d95c25be74fc237bb08d652fb8b34%7C686ea1d3bc2b4
> c6fa92cd99c5c301635%7C0%7C0%7C636787637664212094&amp;sdata=Wy
> H54iAVB%2BRPIVvgpQsw1T3cG5UUtNRKbFR08wibgOg%3D&amp;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

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

Reply via email to