public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH-OMAP3] OMAP3: Clean up EVM board file
Date: Fri, 21 Nov 2008 00:27:03 +0100	[thread overview]
Message-ID: <20081120232703.GA10892@game.jcrosoft.org> (raw)
In-Reply-To: <20081120223802.GK20436@game.jcrosoft.org>

On 23:38 Thu 20 Nov     , Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 21:29 Sat 15 Nov     , dirk.behme at googlemail.com wrote:
> > - Use readx/writex instead of direct pointer access
> > - Replace hardcoded values by macros
> > - Minor space to tab conversion
> > 
> > Notes:
> > 
> > - Instead of read/modify/write GPIO_DATAOUT register use SET/CLEARDATAOUT
> >   register for GPIO64
> > - GPIO 1-6 CM_FCLKEN_PER and CM_ICLKEN_PER are already configured in
> >   per_clocks_enable(), remove it here
> > - Remove pin mux. Is done in set_muxconf_regs.
> > 
> > No functional change.
> > 
> > Signed-off-by: Dirk Behme <dirk.behme@gmail.com>
> > 
> > ---
> >  board/omap3/evm/evm.c              |  143 +++++++++----------------------------
> >  board/omap3/evm/evm.h              |   64 ++++++++--------
> >  include/asm-arm/arch-omap3/cpu.h   |    2 
> >  include/asm-arm/arch-omap3/mem.h   |    8 ++
> >  include/asm-arm/arch-omap3/omap3.h |    1 
> >  5 files changed, 78 insertions(+), 140 deletions(-)
> apply to u-boot-arm/omap3

please not that I've to remove the modification of
include/asm-arm/arch-omap3/omap3.h which does not apply

please send an incremental patch and please number them to known the order to
apply them

Best Regards,
J.

      reply	other threads:[~2008-11-20 23:27 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-15 20:29 [U-Boot] [PATCH-OMAP3] OMAP3: Clean up EVM board file dirk.behme at googlemail.com
2008-11-20 22:38 ` Jean-Christophe PLAGNIOL-VILLARD
2008-11-20 23:27   ` Jean-Christophe PLAGNIOL-VILLARD [this message]

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=20081120232703.GA10892@game.jcrosoft.org \
    --to=plagnioj@jcrosoft.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