public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Aneesh V <aneesh@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/3] omap4: splitting padconfs into common, 4430 and 4460
Date: Thu, 22 Sep 2011 12:12:53 +0530	[thread overview]
Message-ID: <4E7AD8ED.2030507@ti.com> (raw)
In-Reply-To: <1316636251-23961-4-git-send-email-ricardo.salveti@linaro.org>

Hi Ricardo,

On Thursday 22 September 2011 01:47 AM, Ricardo Salveti de Araujo wrote:
> Not all padconfs are the same between 4430 and 4460, so instead of
> working around this with an if, we should have an specific padconf
> structure for both chips (like handling the differences between the LEDs
> GPIOs and TPS).

I was actually wondering whether I was right in keeping some mux data
in the SoC directory and was thinking of moving it back to the board
dir. Strictly speaking mux data is board specific and I have already
encountered a customer board that uses a different UART for traces. So
essential, non_essential etc completely depends on the boards not SoCs.
So, I am inclined to move it back to the board directories, because
that will be more maintainable when more and more boards get added with
conflicting mux definitions.

So, the arrays should be rather padconf_panda_common,
padconf_4460panda, padconf_4430panda etc. and they should be all in
their respective board directories.

What do you think?

br,
Aneesh

  reply	other threads:[~2011-09-22  6:42 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-21 20:17 [U-Boot] [PATCH 0/3] omap4: changes for 4460 ES1.1 Ricardo Salveti de Araujo
2011-09-21 20:17 ` [U-Boot] [PATCH 1/3] omap4: replacing OMAP4_CONTROL with OMAP4430_CONTROL Ricardo Salveti de Araujo
2011-09-21 20:17 ` [U-Boot] [PATCH 2/3] omap4: adding revision detection for 4460 ES1.1 Ricardo Salveti de Araujo
2011-09-21 20:17 ` [U-Boot] [PATCH 3/3] omap4: splitting padconfs into common, 4430 and 4460 Ricardo Salveti de Araujo
2011-09-22  6:42   ` Aneesh V [this message]
2011-09-22  7:00     ` Ricardo Salveti

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=4E7AD8ED.2030507@ti.com \
    --to=aneesh@ti.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