<div dir="ltr">Hi Boris,<div><br></div><div>Thanks for your prompt reply,</div><div><br></div><div>We will start to work on it soon.</div><div><br></div><div>Regards,</div><div><br></div><div>Tomer</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 3 Dec 2018 at 16:30, Boris Brezillon <<a href="mailto:boris.brezillon@bootlin.com">boris.brezillon@bootlin.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Tomer;<br>
<br>
On Mon, 3 Dec 2018 16:09:19 +0200<br>
Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>> wrote:<br>
<br>
<br>
> A few comments/input:<br>
> <br>
> <br>
>    1. We have been working on this driver for quite a long time to port it<br>
>    to the latest Linux conventions, polish the code, run tests and reach high<br>
>    quality.<br>
>    Our partners and customers are waiting to get this driver upstream so<br>
>    they can freely use it.<br>
<br>
Your patch prefix says "v1", so I'm assuming this is the first public<br>
version. I'm sure you spent a lot of time developing this driver<br>
internally, but no matter how long it took, it's a first version for<br>
us, and since we are moving away from the spi_nor controller interface,<br>
I'm not willing to accept new drivers using this interface.<br>
<br>
If you want more background about the spi_nor controller interface<br>
deprecation, you can read [1].<br>
<br>
>    Since this driver is already in final stages and is in very good shape<br>
>    we will appreciate if you can review this specific driver/interface and<br>
>    help us to upstream it.<br>
<br>
Actually, I'd like to do it the other way around: let you rework the<br>
driver to implement the spi-mem interface and review this version.<br>
Otherwise I'll be reviewing things I don't intend to merge anyway.<br>
<br>
>    2.  As for the new interface, we are open for any discussion and for<br>
>    porting the driver as required.<br>
>    We are unsure what is this specific interface and weather it really fits<br>
>    a driver for a Flash Interface Controller module (rather than a SPI flash<br>
>    device).<br>
<br>
Didn't go through the code in details, but at first glance, it looks<br>
like it would fit pretty well.<br>
<br>
>    Is it possible to get a sample driver from another Flash Interface<br>
>    Controller module that was ported to this new interface ?<br>
<br>
We recently converted the atmel QSPI driver [2].<br>
<br>
Regards,<br>
<br>
Boris<br>
<br>
[1]<a href="https://bootlin.com/blog/spi-mem-bringing-some-consistency-to-the-spi-memory-ecosystem/" rel="noreferrer" target="_blank">https://bootlin.com/blog/spi-mem-bringing-some-consistency-to-the-spi-memory-ecosystem/</a><br>
[2]<a href="https://patchwork.kernel.org/project/spi-devel-general/list/?series=38347&state=*" rel="noreferrer" target="_blank">https://patchwork.kernel.org/project/spi-devel-general/list/?series=38347&state=*</a><br>
</blockquote></div>