On Thu, Nov 17, 2016 at 11:53 AM, Siva Durga Prasad Paladugu siva.durga.paladugu@xilinx.com wrote:
Hi,
-----Original Message----- From: Jagan Teki [mailto:jagan@openedev.com] Sent: Wednesday, November 16, 2016 7:23 PM To: Siva Durga Prasad Paladugu sivadur@xilinx.com Cc: u-boot@lists.denx.de; Michal Simek michal.simek@xilinx.com Subject: Re: [U-Boot] [PATCH v6 16/21] spi: Remove dual flash options/flags
On Wed, Nov 16, 2016 at 12:27 PM, Siva Durga Prasad Paladugu siva.durga.paladugu@xilinx.com wrote:
Hi,
-----Original Message----- From: Jagan Teki [mailto:jagan@openedev.com] Sent: Wednesday, November 16, 2016 9:33 AM To: u-boot@lists.denx.de Cc: Jagan Teki jagan@openedev.com; Bin Meng
York Sun york.sun@nxp.com; Vignesh R vigneshr@ti.com;
Mugunthan V
N mugunthanvnm@ti.com; Michal Simek michal.simek@xilinx.com;
Siva
Durga Prasad Paladugu sivadur@xilinx.com Subject: [PATCH v6 16/21] spi: Remove dual flash options/flags
Dual flash code in spi are usually take the spi controller to work with dual connected flash devices. Usually these dual connection operation's are referred to flash controller protocol rather with spi controller protocol, these are still present in flash side for the usage of
spi-nor controllers.
So, this patch remove the dual_flash options or flags in sf which are triggered from spi controller side.
How are you going to handle this? any followup patches on this? This will break for some who are using dual flash till now. Isnt it?
Yes, but I've been mentioned many times about this as we need to re-write the driver on mtd side since many flash futures are dealt with spi area, which is bad and inconvenient to add new futures. and also there is no driver using this dual flash so far, SoI am going with this patch either way.
If I get it correctly, you would like to move all the spi flash related drivers to drivers/mtd/spi ? If yes, then how do you handle a driver which supports both flash and other devices as well.
Sorry, look like the old story begin again, take a look at this [1]
[1] http://git.denx.de/?p=u-boot-spi.git;a=commitdiff;h=cfd285e4c5727d406fdd62b9...
thanks!