public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 7/9] Tegra30: Add config files (common and Cardhu)
Date: Fri, 14 Sep 2012 01:38:57 +0200	[thread overview]
Message-ID: <201209140138.57317.marex@denx.de> (raw)
In-Reply-To: <505261FA.3070909@ti.com>

Dear Tom Rini,

> On 09/13/2012 03:33 PM, Stephen Warren wrote:
> > On 09/12/2012 04:10 PM, Tom Warren wrote:
> >> diff --git a/include/configs/tegra30-common.h
> >> b/include/configs/tegra30-common.h
> >> 
> >> +/*
> >> + * QUOTE(m) will evaluate to a string version of the value of the macro
> >> m + * passed in.  The extra level of indirection here is to first
> >> evaluate the + * macro m before applying the quoting operator.
> >> + */
> >> +#define QUOTE_(m)       #m
> >> +#define QUOTE(m)        QUOTE_(m)
> > 
> > Seems like a good candidate for a common header (not Tegra-common, but
> > across all of U-Boot).
> 
> This is __stringify() that Marek needs to address Wolfgang's comment on
> still, iirc.

I need to roll out the stringify patches ... the issues should be fixed.

> >> +#define CONFIG_SYS_BAUDRATE_TABLE	{4800, 9600, 19200, 38400, 57600,\
> >> +					115200}
> > 
> > IIRC, that's the default now and can be removed. There may be other
> > variables the comment applies to - Tom Rini would know better since he
> > pointed them out in my RPi patches.
> 
> That and the hush prompt one if it's there need to go.

Best regards,
Marek Vasut

  reply	other threads:[~2012-09-13 23:38 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-12 22:10 [U-Boot] [PATCH 0/9] Add basic Tegra30 (T30) support Tom Warren
2012-09-12 22:10 ` [U-Boot] [PATCH 1/9] Tegra: T30: Add include files Tom Warren
2012-09-13 18:06   ` Tom Rini
2012-09-13 21:10     ` Tom Warren
2012-09-18 19:29       ` Simon Glass
2012-09-18 21:07         ` Tom Warren
2012-09-13 19:35   ` Stephen Warren
2012-09-13 20:51     ` Tom Warren
2012-09-12 22:10 ` [U-Boot] [PATCH 2/9] Tegra: T30: Add AVP (arm720t) files Tom Warren
2012-09-13 20:02   ` Stephen Warren
2012-09-13 21:00     ` Tom Warren
2012-09-13 21:47       ` Lucas Stach
2012-09-13 22:06         ` Tom Warren
2012-09-18 19:37   ` Simon Glass
2012-09-18 21:19     ` Tom Warren
2012-09-12 22:10 ` [U-Boot] [PATCH 3/9] Tegra: T30: Add CPU (armv7) files Tom Warren
2012-09-13 20:03   ` Stephen Warren
2012-09-13 20:16     ` Tom Warren
2012-09-13 20:30       ` Stephen Warren
2012-09-13 20:33         ` Tom Rini
2012-09-13 21:21           ` Tom Warren
2012-09-13 22:04             ` Tom Rini
2012-09-13 22:16               ` Tom Warren
2012-09-13 22:28                 ` Tom Rini
2012-09-12 22:10 ` [U-Boot] [PATCH 4/9] Tegra: T30: Add common Tegra30 CPU files Tom Warren
2012-09-13 22:08   ` Stephen Warren
2012-09-18 19:40   ` Simon Glass
2012-09-12 22:10 ` [U-Boot] [PATCH 5/9] Tegra: DT: Add preliminary device tree files for T30 Cardhu Tom Warren
2012-09-13 22:14   ` Stephen Warren
2012-09-12 22:10 ` [U-Boot] [PATCH 6/9] Tegra30: Add Cardhu board files Tom Warren
2012-09-13 22:23   ` Stephen Warren
2012-09-18 19:44   ` Simon Glass
2012-09-12 22:10 ` [U-Boot] [PATCH 7/9] Tegra30: Add config files (common and Cardhu) Tom Warren
2012-09-13 22:33   ` Stephen Warren
2012-09-13 22:45     ` Tom Rini
2012-09-13 23:38       ` Marek Vasut [this message]
2012-09-18 19:46   ` Simon Glass
2012-09-18 21:21     ` Tom Warren
2012-09-12 22:10 ` [U-Boot] [PATCH 8/9] Tegra30: Add common pinmux config in board_early_init_f Tom Warren
2012-09-13 22:37   ` Stephen Warren
2012-09-18 19:53     ` Simon Glass
2012-09-18 21:32       ` Tom Warren
2012-09-18 21:53         ` Stephen Warren
2012-09-12 22:10 ` [U-Boot] [PATCH 9/9] Tegra30: Enable Cardhu build (SPL) Tom Warren
2012-09-13 18:00   ` Tom Rini
2012-09-13 20:02     ` Tom Warren
2012-09-13 22:47   ` Stephen Warren
2012-09-13 18:36 ` [U-Boot] [PATCH 0/9] Add basic Tegra30 (T30) support Stephen Warren
2012-09-13 21:04 ` Tom Rini
2012-09-13 21:25   ` Tom Warren

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=201209140138.57317.marex@denx.de \
    --to=marex@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox