From mboxrd@z Thu Jan  1 00:00:00 1970
From: Hannes Petermaier <hannes@petermaier.org>
Date: Sat, 08 Mar 2014 20:46:06 +0100
Subject: [U-Boot] [PATCH] lcd: Add support for CONFIG_LCD_NOSTDOUT
In-Reply-To: <20140306194905.GJ3327@book.gsilab.sittig.org>
References: <1394115971-22079-1-git-send-email-oe5hpm@oevsv.at>
	<20140306194905.GJ3327@book.gsilab.sittig.org>
Message-ID: <531B737E.1050300@petermaier.org>
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

Hi Gerhard,

On 2014-03-06 20:49, Gerhard Sittig wrote:
> In general U-Boot tries to get away from the multitude of ifdefs
> where possible.  I'm afraid adding a new one needs a very good
> reason to get perceived as welcome.
By the way i've found a passage within README, which tells to do so as 
id id:

'* If you modify existing code, make sure that your new code does not
   add to the memory footprint of the code ;-) Small is beautiful!

   When adding new features, these should compile conditionally only
   (using #ifdef), and the resulting code with the new feature
   disabled must not need more memory than the old code without your
   modification.
'
whats your opinion about this ?
> virtually yours
> Gerhard Sittig
best regards,
Hannes