On Friday 18 October 2013 06:38 PM, Mark Brown wrote: > On Fri, Oct 18, 2013 at 05:18:15PM +0530, Sourav Poddar wrote: >> On Friday 18 October 2013 04:01 PM, Mark Brown wrote: >>> On Fri, Oct 18, 2013 at 12:57:51PM +0530, Sourav Poddar wrote: >>>> 3. For filling memory mapped register in qspi controller, we can >>>> pass that information >>>> from dt rather than hardcoding as macros. >>> Or from the flash driver at runtime... >> No, as for now, its better to keep mtd framework independent of this >> spi controller configuration and pass the information via dt. >> Since we dont have a common protocol, If we start putting this information >> in mtd, it will clutter the mtd space and it may happen that we may end up >> with differnet if defs for all the kind of flash devices. > I thought this was the information for the flash commands that the > driver would need to know anyway for use with SPI controllers without > explicit flash support? But perhaps I'm misunderstanding. Flash driver need to know them, but as of now, everything is done in a api mode stuff in flash driver. As in, we have write/erase/read, quad_read (proposed through this series) where we provide the commands directly. And to handle different flash types, we use macros to decide which command set is used.
May be, this can come as a part of a seperate discussion to modify flash driver also to take data(commands, other parameters) through dt. Though, for the case in hand, better to set up spi mem map specific registers with dts from spi controller node itself. There might be different varieties of flashes, with different commands and cluttering mtd flash to configure a controller specific register does not look to be a good idea. ------------------------------------------------------------------------------ October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register > http://pubads.g.doubleclick.net/gampad/clk?id=60135031&iu=/4140/ostg.clktrk _______________________________________________ spi-devel-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/spi-devel-general
