On Mon, May 14, 2018 at 2:42 PM, Ashish Kumar ashish.kumar@nxp.com wrote:
-----Original Message----- From: Jagan Teki [mailto:jagannadh.teki@gmail.com] Sent: Friday, May 11, 2018 11:31 AM To: Prabhakar Kushwaha prabhakar.kushwaha@nxp.com Cc: Jagan Teki jagan@amarulasolutions.com; York Sun york.sun@nxp.com; Yogesh Narayan Gaur yogeshnarayan.gaur@nxp.com; Poonam Aggrwal poonam.aggrwal@nxp.com; Ashish Kumar ashish.kumar@nxp.com; u- boot@lists.denx.de Subject: Re: mtd: spi-nor: new NXP FlexSPI driver location & framework to use ?
On Fri, May 11, 2018 at 11:08 AM, Prabhakar Kushwaha prabhakar.kushwaha@nxp.com wrote:
Dear Jagan,
NXP is coming up with new FlexSPI controller. It is similar to existing QSPI with
enhanced feature-set.
We have the driver ready as per existing framework i.e. driver/spi.
From recend discussion, we go to know about framework change. Migration of qspi drivers in u-boot-spi/drivers/mtd/spi-nor/
git://git.denx.de/u-boot-spi.git branch mtd-spinor-working.
We are in dilemma for sending FlexSPI driver upstream. Do we follow existing framework i.e. driver/spi or new proposed framework
i.e. u-boot-spi/drivers/mtd/spi-nor/
Also, do we have any timeline of u-boot-spi/drivers/mtd/spi-nor/ to become
default.
Idea is to move spi-nor, mtd-spinor-working is paused because of non-dm drivers accessing. We are clear that we can't create another legacy layer to access spi-nor for the sake of non-dm driver to work which eventually ended-up another mess and also no one take care of non-dm conversion to dm if we give such feasibility. So we exported a deadline for full SPI/SPI_FLASH DM conversion till v2018.09. Once all these relevant conversion done, will try to move.
Hello Jagan,
You have mentioned here http://git.denx.de/?p=u-boot-spi.git;a=commitdiff;h=c4e68d3aa8178f6aa63a79c4...
Snip from above link: +Partially converted:
drivers/spi/atcspi200_spi.cdrivers/spi/davinci_spi.cdrivers/spi/fsl_dspi.cdrivers/spi/fsl_qspi.cThat fsl_qspi.c is partial converted to DM model, is there any guide line to follow for missing stuff ?
We should fully convert the driver into dm, no #ifdef CONFIG_DM_SPI