[U-Boot] [PATCH v6 16/21] spi: Remove dual flash options/flags

[Jagan Teki]

On Thu, Nov 17, 2016 at 11:53 AM, Siva Durga Prasad Paladugu
<siva.durga.paladugu at xilinx.com> wrote:
> Hi,
>
>> -----Original Message-----
>> From: Jagan Teki [mailto:jagan at openedev.com]
>> Sent: Wednesday, November 16, 2016 7:23 PM
>> To: Siva Durga Prasad Paladugu <sivadur at xilinx.com>
>> Cc: u-boot at lists.denx.de; Michal Simek <michal.simek at 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 at xilinx.com> wrote:
>> > Hi,
>> >
>> >> -----Original Message-----
>> >> From: Jagan Teki [mailto:jagan at openedev.com]
>> >> Sent: Wednesday, November 16, 2016 9:33 AM
>> >> To: u-boot at lists.denx.de
>> >> Cc: Jagan Teki <jagan at openedev.com>; Bin Meng
>> <bmeng.cn at gmail.com>;
>> >> York Sun <york.sun at nxp.com>; Vignesh R <vigneshr at ti.com>;
>> Mugunthan V
>> >> N <mugunthanvnm at ti.com>; Michal Simek <michal.simek at xilinx.com>;
>> Siva
>> >> Durga Prasad Paladugu <sivadur at 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=cfd285e4c5727d406fdd62b9befb1b528edd49fb

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.