From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Thu, 27 Aug 2015 10:30:06 +0200
Subject: [U-Boot] [PATCH] spi: cadence_qspi: Enable quad mode for read
	and programming
In-Reply-To: <1440652447.1929.2.camel@clsee-VirtualBox>
References: <1440547795-2077-1-git-send-email-clsee@altera.com>
 <201508261558.44818.marex@denx.de> <1440652447.1929.2.camel@clsee-VirtualBox>
Message-ID: <201508271030.06677.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Thursday, August 27, 2015 at 07:14:07 AM, Chin Liang See wrote:
> On Wed, 2015-08-26 at 15:58 +0200, marex at denx.de wrote:
> > On Wednesday, August 26, 2015 at 03:42:36 PM, Chin Liang See wrote:
> > > On Wed, 2015-08-26 at 15:37 +0200, marex at denx.de wrote:
> > > > On Wednesday, August 26, 2015 at 03:13:15 PM, Chin Liang See wrote:
> > > > 
> > > > Hi,
> > > > 
> > > > [...]
> > > > 
> > > > > > > Yah, that is a nice enhancement in order to keep up with
> > > > > > > controller enhancement. We definitely want to explore and
> > > > > > > enable that at U-Boot in the future.
> > > > > > 
> > > > > > You mean you'll implement this functionality and then make your
> > > > > > change to the QSPI driver to use it, in order to implement things
> > > > > > properly ? :) In that case, I agree.
> > > > > 
> > > > > Sure, something I can look at in 1-2 months time.
> > > > 
> > > > Sure, no problem, that should be in time for the next merge window.
> > > > 
> > > > > I still have laundry
> > > > > lists on ensuring the mainline SOCFPGA U-Boot has all the essential
> > > > > features as we have in github. Prior that happen, can I get an ACK
> > > > > from you? :)
> > > > 
> > > > No, sorry, you can not get ack on a patch which is hacky. You cannot
> > > > push hacky code into mainline just for the sake of getting something
> > > > somehow working, that's not how it works. We either do things proper
> > > > or not at all.
> > > 
> > > In this case, probably its something we can work together on enabling
> > > the spi-nor?
> > 
> > Hi!
> > 
> > But the Cadence QSPI works in U-Boot, I'm using it on a board here.

Hi!

> Yup, the basic function works. I am looking at 2 areas below
> 
> 1. Quad mode
> Current driver is running at single IO mode. We yet to utilize the quad
> mode that is supported by the controller. Its for performance.

OK

> 2. SCLK at 80MHz and beyond
> The driver fail to work at higher SCLK frequency. The github version
> work well at 100MHz. I am troubleshooting on this now

Cool, thanks!

Best regards,
Marek Vasut