Hi Jagan,

> -----Original Message-----
> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> Sent: Tuesday, October 31, 2017 2:40 PM
> To: Siva Durga Prasad Paladugu <siva...@xilinx.com>
> Cc: u-boot@lists.denx.de; Liam Beguin <liambeg...@gmail.com>; Siva
> Durga Prasad Paladugu <siva...@xilinx.com>
> Subject: Re: [UBOOT PATCH 1/2] spi: zynqmp_qspi: Add support for
> ZynqMP qspi driver
> 
> On Tue, Oct 24, 2017 at 3:33 PM, Siva Durga Prasad Paladugu
> <siva.durga.palad...@xilinx.com> wrote:
> > This patch adds qspi driver support for ZynqMP SoC. This driver is
> > responsible for communicating with qspi flash devices.
> 
> Legacy question, what is your approach for dual memory setup? Did you
> write another flash driver?
No

> 
> I see this driver use dual flash slave 'option' which doesn't live on spi side
> anymore. better to have a discussion on approach and will review further.

I can see that spi_flash.c(driver/mtd/spi/spi_flash.c) has the option for dual 
flash under CONFIG_SF_DUAL_FLASH.
I thought of using the same. Isn't it takes care of dual flash case? 
Please let me know if you any further thoughts on how it has to be handled.

Thanks,
Siva

> 
> thanks!
> --
> Jagan Teki
> Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream
> Maintainer Hyderabad, India.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to