On Mon, 18 Nov 2013, Mark Brown wrote: > On Mon, Nov 18, 2013 at 03:31:10PM +0000, Lee Jones wrote: > > On Mon, 18 Nov 2013, Mark Brown wrote: > > > > This doesn't seem realistic, you're assuming that system integrators > > > won't go and use chips you've not heard of and at least in the case of > > > things like quad read my understanding is that the commands aren't > > > standardised so the host just has to know what to write. > > > I'm not following? What are you suggesting? > > Like I say I'm suggesting that the bit of the code that understands the > flash chip is separate to the bit of code that knows the mechanics of > sending commands and data to the chip. > > > After some analysis we have come to the conclusion that using m25p80 > > is not feasible. It makes more sense for this to be an > > orthogonal/stand-alone driver. > > That seems plausible for the controller side but it seems surprising for > the flash chip side.
The issue is that almost the entire driver is controller side. The only bits that are the same (and not in all cases) are the OPCODEs, but they are one liners (21 lines out of 1153). Most of the controllers which use this stuff could reuse quite a bit of the m25p80 driver as they just write the message containing the OPCODE as the m25p80 driver sets it up, but that's simply not the case with our controller. We would have to pull the OPCODE out and based on which one it is, we'd have to build our own message. Put it this way, if we tried to use the m25p80 our controller driver would most likely be twice as large and twice as complex as it is currently, which is exactly the inverse of what we're trying to achieve here. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
