public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: vikasm <vikas.manocha@st.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH RESEND 0/7] spi: cadence_qspi: optimize & fix indirect rd-writes
Date: Wed, 12 Aug 2015 10:52:28 -0700	[thread overview]
Message-ID: <55CB87DC.2030607@st.com> (raw)
In-Reply-To: <CAD6G_RS8F1OyS1KHJMTXKWjU2AG=EG55o_n2yWd47T0m16zBTA@mail.gmail.com>

Thanks Jagan,

On 08/12/2015 05:01 AM, Jagan Teki wrote:
> Vikas,
> 
> Did you verified on board, can you just verified with 'sf update' before
> and after.. I just wanted to see if you get any performance
> improvement with these
> optimization fixes.

The main point is someone needs to debug the patchset on socfpga platform. It works on stv0991 platform.
But which patch is really creating issue on socfpga & debug that patch.

Rgds,
Vikas

> 
> Will get back again for my comments.
> 
> On 12 August 2015 at 17:06, Stefan Roese <sr@denx.de> wrote:
>> Hi Vikas,
>>
>> (added Marek to Cc)
>>
>>
>> On 11.08.2015 23:19, vikasm wrote:
>>>
>>> On 07/23/2015 05:22 AM, Stefan Roese wrote:
>>>>
>>>> Hi Vikas,
>>>>
>>>> On 16.07.2015 08:46, Stefan Roese wrote:
>>>>>>>>
>>>>>>>> In addition can you please check the patch causing this instability
>>>>>>>> on
>>>>>>>> socfpga. I don't like to bug you but to close this patchset, this
>>>>>>>> info
>>>>>>>> & tests mentioned above seems to be required.
>>>>>>>
>>>>>>>
>>>>>>> Okay. I'll try to find some time this week to do some testing here.
>>>>>>> It seems
>>>>>>> that the other cadence patchset from you ([v4 00/10] spi:
>>>>>>> cadence_qspi: sram depth from DT & fix for FIFO width) is not pulled
>>>>>>> into
>>>>>>> mainline yet. To make it easier for me, could you perhaps publish a
>>>>>>> git
>>>>>>> repository that is based on current mainline. And has the mentioned
>>>>>>> above
>>>>>>> patch series included. And all the patches in the latest version that
>>>>>>> are
>>>>>>> currently causing these problems on SoCFPGA?
>>>>>>
>>>>>>
>>>>>> The patchset was in u-boot-spi repository, yesterday pulled by Tom in
>>>>>> mainline. I will rebase the patchset in discussion
>>>>>> (spi: cadence_qspi: optimize & fix indirect rd-writes) on mainline
>>>>>> master & send the V2.
>>>>>>
>>>>>> Let me know if it is ok.
>>>>>
>>>>>
>>>>> Okay, I'll try to find some time later this week or next week for some
>>>>> tests.
>>>>
>>>>
>>>> I couldn't find the required time to  test these patches again yet.
>>>> Sorry. And I'll leave for a short vacation tomorrow. So I won't be able
>>>> to get to this issue before week 32.
>>>>
>>>> Perhaps somebody else finds some time to look at this QSPI driver issue
>>>> in the meantime
>>>
>>>
>>> Any update ?
>>
>>
>> Not from me, sorry. And I really won't be able to dig into this in the next
>> weeks.
>>
>> Again, perhaps someone else finds the time to look at this QSPI driver issue
>> in the meantime?
> 
> thanks!
> 

  reply	other threads:[~2015-08-12 17:52 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-17  2:14 [U-Boot] [PATCH RESEND 0/7] spi: cadence_qspi: optimize & fix indirect rd-writes Vikas Manocha
2015-06-17  2:14 ` [U-Boot] [PATCH RESEND 1/7] spi: cadence_qspi: remove sram polling from flash read Vikas Manocha
2015-06-17  2:14 ` [U-Boot] [PATCH RESEND 2/7] spi: cadence_qspi: read can be independent of fifo width Vikas Manocha
2015-06-17  2:14 ` [U-Boot] [PATCH RESEND 3/7] spi: cadence_qspi: remove sram polling from flash write Vikas Manocha
2015-06-17  2:14 ` [U-Boot] [PATCH RESEND 4/7] spi: cadence_qspi: move trigger base configuration in init Vikas Manocha
2015-06-17  2:14 ` [U-Boot] [PATCH RESEND 5/7] spi: cadence_qspi: fix indirect read/write start address Vikas Manocha
2015-06-17  2:14 ` [U-Boot] [PATCH RESEND 6/7] spi: cadence_qspi: fix base trigger address & transfer " Vikas Manocha
2015-06-17  2:14 ` [U-Boot] [PATCH RESEND 7/7] spi: cadence_qspi: get fifo width from device tree Vikas Manocha
2015-06-18 12:02 ` [U-Boot] [PATCH RESEND 0/7] spi: cadence_qspi: optimize & fix indirect rd-writes Stefan Roese
2015-06-18 18:05   ` Vikas MANOCHA
2015-06-19  6:16     ` Stefan Roese
2015-06-19 21:38       ` Vikas MANOCHA
2015-06-22  8:34         ` Stefan Roese
2015-06-22 23:31           ` Vikas MANOCHA
2015-06-23 14:36             ` Graham Moore
2015-06-23 14:51               ` Vikas MANOCHA
2015-07-02 17:50               ` Vikas MANOCHA
2015-07-06 17:56                 ` Graham Moore
2015-07-06 18:19                   ` Vikas MANOCHA
2015-07-01 16:24           ` Vikas MANOCHA
2015-07-09  1:29           ` Vikas MANOCHA
2015-07-13  9:00             ` Stefan Roese
2015-07-15 21:14               ` Vikas MANOCHA
2015-07-16  6:46                 ` Stefan Roese
2015-07-23 12:22                   ` Stefan Roese
2015-08-11 21:19                     ` vikasm
2015-08-12 11:36                       ` Stefan Roese
2015-08-12 12:01                         ` Jagan Teki
2015-08-12 17:52                           ` vikasm [this message]
2015-08-12 20:22                             ` Marek Vasut
2015-08-13  0:16                               ` vikasm
2015-08-13  0:26                                 ` Marek Vasut
2015-08-13  0:36                                   ` vikasm
2015-08-13  2:15                                     ` Marek Vasut

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55CB87DC.2030607@st.com \
    --to=vikas.manocha@st.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox