All of lore.kernel.org
 help / color / mirror / Atom feed
From: Detlev Zundel <dzu@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] cmd_onenand.c: moved to standard subcommand handling
Date: Mon, 29 Mar 2010 11:39:54 +0200	[thread overview]
Message-ID: <m2iq8feaad.fsf@ohwell.denx.de> (raw)
In-Reply-To: <u2sac9c93b11003290228y25015507hfb45a63b93c45c9f@mail.gmail.com> (Frans Meulenbroeks's message of "Mon, 29 Mar 2010 11:28:31 +0200")

Hi Frans,

> 2010/3/29 Detlev Zundel <dzu@denx.de>:
>> Hi Frans,
>>
>>> Forgot this disclaimer, thought I added it, sry.
>>>
>>> NOTE:
>>> This code is untested as I have no board with onenand support.
>>> However, it compiles properly (tested by building opap3_evm, which uses this).
>>> The changes I did are purely splitting do_onenand in separate
>>> functions and add the subcommand parsing (which itself was taken from
>>> i2c, on which I tested it before).
>>> So I do not expect any serious issues, but I would suggest testing
>>> this before integrating it.
>>
>> Why don't you just add the maintainer of omap3_evm board (as I did) on
>> CC to your mail, so he knows that his testing would be greatly
>> appreciated?
>>
> Will try to do so next time :-)
> Actually there are more boards which utilize onenand. I just picked
> this one as I have an omap3 board (and actually only discovered when I
> wanted to start testing that it did not have onenand, but just regular
> nand)

Hm ok, then maybe add someone else from the list of boards using it, as
the e-mail of the omap3_evm maintainer "Manikandan Pillai
<mani.pillai@ti.com>" bounces.

I added Nishanth Menon from ti on CC, maybe he can tell us the
whereabouts of Manikanden.  Will someone else from ti be maintaining
this board?  If so, then a patch to MAINTAINERS is welcome :)

Cheers
  Detlev

-- 
Those who would trade safety for freedom deserve neither.
           -- Thomas Jefferson
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de

  parent reply	other threads:[~2010-03-29  9:39 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-27 16:14 [U-Boot] [PATCH] cmd_onenand.c: moved to standard subcommand handling Frans Meulenbroeks
2010-03-27 16:19 ` Frans Meulenbroeks
2010-03-29  9:15   ` Detlev Zundel
     [not found]     ` <u2sac9c93b11003290228y25015507hfb45a63b93c45c9f@mail.gmail.com>
2010-03-29  9:39       ` Detlev Zundel [this message]
2010-05-05 21:57 ` Wolfgang Denk

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=m2iq8feaad.fsf@ohwell.denx.de \
    --to=dzu@denx.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.