public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Chin Liang See <clsee@altera.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] nand/denali: Adding Denali NAND driver support
Date: Fri, 21 Feb 2014 14:49:10 -0600	[thread overview]
Message-ID: <1393015750.3602.13.camel@clsee-VirtualBox> (raw)
In-Reply-To: <20140221195711.40B7.AA925319@jp.panasonic.com>

Hi Masahiro,

On Fri, 2014-02-21 at 19:57 +0900, Masahiro Yamada wrote:
> Hi Chin,
> (adding Albert to Cc:)
> 
> > Hi Scott,
> > 
> > As there are no further comments, would need your help to apply this
> > patch. Thanks and appreciate for your support.
> > 
> > Chin Liang
> 
> 
> Panasonic SoCs are using Denali NAND controller.
> I guess I can test your patch.
> Please wait a week.
> 
> 
> As Albert mentioned, the problem is
> there must be a board which uses your driver code.
> 

Thanks for the help. Actually this driver is used in socfpga_cyclone5
board. Just that I misunderstood they would need separate patches. I
will fix this in next commit.

> I'll review and test your code (it looks like most parts are ported from
> Linux) and if it works fine, then I will post Panasonic SoCs (based on
> ARM coretexA9) and boards series with CONFIG_NAND_DENALI enabled.
> 

Yup, the code base are from Linux. But I need to modify it to certain
extend to get it working at dev kit. Anyway, it would be excited to
figure whether it work for other platform too.


> As for the initial run test on my board, it is not working so far.
> So I am diving into the code...

If you want to run under SPL, there are some patches for that. Let me
know if you need that. While for U-Boot, they are working fine. Probably
it might due to parameter setting. I will attach my configuration in my
new patches. Maybe worth to note, this driver was tested against 3
different devices but they all are ONFI parts.

Thanks and appreciate for your offers to help

Chin Liang

> 
> 
> Best Regards
> Masahiro Yamada
> 
> 

  reply	other threads:[~2014-02-21 20:49 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-18 21:18 [U-Boot] [PATCH] nand/denali: Adding Denali NAND driver support Chin Liang See
2014-01-09 14:47 ` Chin Liang See
2014-02-04 19:07 ` Chin Liang See
2014-02-21 10:57   ` Masahiro Yamada
2014-02-21 20:49     ` Chin Liang See [this message]
     [not found] <D32F37106BA492429BA67AF5061FA1A266515302@NACLMBX01.optos.eye>
2014-06-02  5:14 ` Chin Liang See
  -- strict thread matches above, loose matches on Subject: below --
2014-08-19  9:47 Chin Liang See
2014-08-20  3:47 ` Masahiro Yamada
2014-08-28  2:13 ` Masahiro Yamada
2014-08-30 11:45   ` Tom Rini
2014-09-01  3:54     ` Masahiro Yamada
2014-09-03  2:07       ` Scott Wood
2014-09-03  2:15     ` Scott Wood
2014-09-05  3:01       ` Masahiro Yamada
2014-09-08  9:36         ` Chin Liang See
2014-09-12  5:20       ` Chin Liang See

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=1393015750.3602.13.camel@clsee-VirtualBox \
    --to=clsee@altera.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