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 pandora: update pin mux for rev3 boards
Date: Sun, 28 Jun 2009 11:13:57 +0200	[thread overview]
Message-ID: <20090628091357.GU8587@game.jcrosoft.org> (raw)
In-Reply-To: <4A470262.6050207@googlemail.com>

On 07:40 Sun 28 Jun     , Dirk Behme wrote:
> Dear Jean-Christophe,
> 
> Jean-Christophe PLAGNIOL-VILLARD wrote:
> >On 19:12 Thu 25 Jun     , Jason Kridner wrote:
> >>   On Thu, Jun 25, 2009 at 4:59 PM, Jean-Christophe PLAGNIOL-VILLARD
> >>   <plagnioj@jcrosoft.com> wrote:
> >>
> >>     On 14:57 Mon 08 Jun     , Grazvydas Ignotas wrote:
> >>     > The update consists of following changes:
> >>     > - remove configuration of not connected pins, effectively
> >>     >   leaving them in safe mode.
> >>     > - remove unused GPIOs, setup newly added ones.
> >>     > - setup pulls for various GPIOs. Disable pulls for game
> >>     >   buttons, as they have external pulls.
> >>     > - SDRC CS change based on recent patch for
> >>     >   Beagle and Overo.
> >>     >
> >>     > Old boards are no longer supported, but there was only
> >>     > small number of test boards made. Updated configuration
> >>     > is expected to be used for mass production.
> >>     If user have old version in possession NACK
> >>
> >>   I believe no users who would possibly object have the old version (or any
> >>   version) in possession.  Only the core developers ever got
> >>these boards.    Is the expectation to create #ifdef or some
> >>sort of auto-detection
> >>   (unlikely possible)?
> >untill the hardware will be really not anymore use yes please
> 
> If two or three people (from the board manufacturer?) which are more
> familiar with the development board situation than you say "we don't
> need it" then this should be accepted. If nobody uses the older
> boards any more (and this is what I understood they said: "There
> were only few older boards, we know where they are and they are
> replaced by new ones") then there is absolutely no reason to pollute
> U-Boot with support for it. There is no need to add dead code to
> U-Boot.
No, it's different no people have a board and some people have a board
> 
> We should trust the board maintainers somehow.
It's not me who tell that some people have the board

when you will be in possession of the old version of a board and just because
few people have the board is remove from the mainline you will not be happy.
So no we will support the both
> 
> >>     this kind of huge update is non bisectable so we do need to use a true
> >>     mux api
> >>     as the kernel lot's of other arch in u-boot
> >>
> >>   Why is it not bisectable?
> >because your mix cleanup, fixup and new board support
> >>   Do you have a "true mux api" to suggest?
> >the same as the kernel one is the best for code sharing
> 
> OMAP3 pin mux in kernel is totally broken.
do you really think it's a good reason to have copy & paster everywhere and
not a simple api as at91, davinic and others?
I do not

Best Regards,
J.

  reply	other threads:[~2009-06-28  9:13 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-08 11:57 [U-Boot] [PATCH] OMAP3 pandora: update pin mux for rev3 boards Grazvydas Ignotas
2009-06-25 21:59 ` Jean-Christophe PLAGNIOL-VILLARD
2009-06-26  0:12   ` Jason Kridner
2009-06-27 21:58     ` Jean-Christophe PLAGNIOL-VILLARD
2009-06-28  5:40       ` Dirk Behme
2009-06-28  9:13         ` Jean-Christophe PLAGNIOL-VILLARD [this message]
2009-06-28 11:11           ` Dirk Behme
2009-06-28 11:50             ` Jean-Christophe PLAGNIOL-VILLARD
2009-06-28 12:17               ` Dirk Behme
2009-06-28 20:07           ` Grazvydas Ignotas
2009-06-28 21:21             ` Jean-Christophe PLAGNIOL-VILLARD
2009-06-26  8:43   ` Grazvydas Ignotas
2009-06-27 21:55     ` Jean-Christophe PLAGNIOL-VILLARD

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=20090628091357.GU8587@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