[U-Boot] [PATCH v6 00/76] mtd: Add SPI-NOR core support
Jagan Teki
jagannadh.teki at gmail.com
Mon Mar 21 10:20:44 CET 2016
Hi Mugunthan,
On 16 March 2016 at 19:50, Jagan Teki <jagannadh.teki at gmail.com> wrote:
> On 16 March 2016 at 09:14, Qianyu Gong <qianyu.gong at nxp.com> wrote:
>>
>>> -----Original Message-----
>>> From: Jagan Teki [mailto:jagannadh.teki at gmail.com]
>>> Sent: Tuesday, March 15, 2016 7:18 PM
>>> To: Qianyu Gong <qianyu.gong at nxp.com>; Huan Wang <alison.wang at nxp.com>
>>> Cc: york sun <york.sun at nxp.com>; Tom Rini <trini at konsulko.com>; Siva Durga
>>> Prasad Paladugu <sivadur at xilinx.com>; Michal Simek <michal.simek at xilinx.com>;
>>> u-boot at lists.denx.de; Stefan Roese <sr at denx.de>; Mingkai Hu
>>> <mingkai.hu at nxp.com>
>>> Subject: Re: [U-Boot] [PATCH v6 00/76] mtd: Add SPI-NOR core support
>>>
>>> On 9 March 2016 at 13:37, Qianyu Gong <qianyu.gong at nxp.com> wrote:
>>> > Hi Jagan,
>>> >
>>> >> -----Original Message-----
>>> >> From: york sun
>>> >> Sent: Tuesday, March 08, 2016 12:46 AM
>>> >> To: Jagan Teki <jteki at openedev.com>; Huan Wang <alison.wang at nxp.com>;
>>> >> Qianyu Gong <qianyu.gong at nxp.com>
>>> >> Cc: u-boot at lists.denx.de; Siva Durga Prasad Paladugu
>>> >> <sivadur at xilinx.com>; Stefan Roese <sr at denx.de>; Michal Simek
>>> >> <michal.simek at xilinx.com>; Tom Rini <trini at konsulko.com>
>>> >> Subject: Re: [U-Boot] [PATCH v6 00/76] mtd: Add SPI-NOR core support
>>> >>
>>> >> On 03/03/2016 01:06 PM, york sun wrote:
>>> >> > On 02/29/2016 04:26 AM, Jagan Teki wrote:
>>> >> >> Hi York,
>>> >> >>
>>> >> >> On 27 February 2016 at 02:14, york sun <york.sun at nxp.com> wrote:
>>> >> >>> On 02/22/2016 10:18 AM, Jagan Teki wrote:
>>> >>
>>> >> <snip>
>>> >>
>>> >> >>>>
>>> >> >>>> Can you pls- test the dataflash changes? use u-boot-spi/spi-nor
>>> >> >>>>
>>> >> >>> Jagan,
>>> >> >>>
>>> >> >>> I am getting there. Will test sf probe/read/write and probably
>>> >> >>> boot on selected platforms. Is there any specific platform/test in your mind?
>>> >> >>
>>> >> >> Yes, these tests OK. and if possible please verify 'sf protect' as well.
>>> >> >>
>>> >> >
>>> >> > Jagan,
>>> >> >
>>> >> > Sorry for the delay. I am having issue with both ls1021aqds and
>>> >> > ls1043aqds. Need to work with internal team to sort it out.
>>> >> >
>>> >> > My understanding the tests you need are for spi-nor, in my case,
>>> >> > fsl-qspi, right? This is not for fsl-dspi or fsl-espi driver. I
>>> >> > only have ls1021aqds and ls1043aqds with such support.
>>> >> >
>>> >>
>>> >> Jagan,
>>> >>
>>> >> Alison and Qianyu have tested on LS1021AQDS and LS1043AQDS. There
>>> >> were some issues. They will post their findings (and possible fix) to this thread.
>>> >>
>>> >> York
>>> >
>>> > I tested on LS1043AQDS board based on your spi-nor branch. The
>>> 'spi_flash_probe'
>>> > failed during QSPI boot. Then I found that CONFIG_DM_MTD_SPI_NOR is
>>> > not defined for the board but it could be selected if CONFIG_MTD &&
>>> CONFIG_DM_SPI, correct?
>>> >
>>> > So after I moved CONFIG_MTD to the defconfig file, the probing could work now.
>>> >
>>> > And there is another small effect on our Fman driver. It would like to
>>> > read ucode blob from the QSPI flash. As the 'spi_flash_read' interface
>>> > is changed, I've modified the address value to an offset value in flash.
>>>
>>> Qianyu,
>>>
>>> > Here are my patches:
>>> > http://patchwork.ozlabs.org/patch/594872/
>>> > http://patchwork.ozlabs.org/patch/594873/
>>> > http://patchwork.ozlabs.org/patch/594874/
>>>
>>> I have applied 3rd patch on my test branch, and remaining two will go in another
>>> time like once spi-nor got merged.
>>>
>>> Qianyu/Alison Wang,
>>>
>>> Please test this[1] branch for your new changes and let me know for any issues.
>>>
>>> [1] http://git.denx.de/?p=u-boot/u-boot-spi.git;a=shortlog;h=refs/heads/spi-nor-
>>> next
>>>
>>> --
>>> Jagan.
>>
>> Hi Jagan,
>>
>> Both DSPI and QSPI work well on LS1043ARDB/LS1043AQDS board.
>> Thanks!
>
> thanks!
>
> Mugunthan/Alison Wang,
>
> Please use u-boot-spi/spi-nor-test for your testing.
Please test this?
thanks!
--
Jagan.
More information about the U-Boot
mailing list