From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Fri, 8 Mar 2013 15:08:49 +0100
Subject: [U-Boot] [PATCH v9 15/30] autoconfig.mk: Make it possible to
	define configs from other configs
In-Reply-To: <20130308052519.190E4200643@gemini.denx.de>
References: <1362596377-5827-1-git-send-email-benoit.thebaudeau@advansee.com>
	<201303080429.00780.marex@denx.de>
	<20130308052519.190E4200643@gemini.denx.de>
Message-ID: <201303081508.49239.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Wolfgang Denk,

> Dear Marek Vasut,
> 
> In message <201303080429.00780.marex@denx.de> you wrote:
> > > > +	# ... and from configs defined from other configs
> > > > +	s/="\(CONFIG_[A-Za-z0-9_][A-Za-z0-9_]*\)"/=$(\1)/;
> > > 
> > > Should we not remove the lower case letters here?  Such are not
> > > supposed to be used in macro names.
> > 
> > btw. CONFIG_[[:alnum:]_]\+ is shorter ;-)
> 
> ...and has the same issue of including the unwanted lower case
> letters.

What about the CONFIG_8xx_* variables then ? :)

Best regards,
Marek Vasut