From mboxrd@z Thu Jan  1 00:00:00 1970
From: Steven A. Falco <sfalco@harris.com>
Date: Tue, 09 Dec 2008 14:07:36 -0500
Subject: [U-Boot] [PATCH v3 2/2] ppc4xx: Add PPC4xx SPI helpers to
	Sequoia
In-Reply-To: <20081209185341.71DAD834B020@gemini.denx.de>
References: <493E9C46.8000101@harris.com> <493EA876.7090008@gmail.com>
	<20081209185341.71DAD834B020@gemini.denx.de>
Message-ID: <493EC1F8.7080107@harris.com>
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

Wolfgang Denk wrote:
> Dear Ben Warren,
> 
> In message <493EA876.7090008@gmail.com> you wrote:
>> Why not enable this feature on Sequoia?  Wolfgang's argument for keeping 
>> the patch out then goes away.  IMHO, eval boards should have as many 
>> options enabled by default as possible, and the user then has the option 
>> to opt out.
> 
> But there is not a single SPI device on the Sequoia board, and if you
> attach one, you have to write driver code for it that implements  the
> chip  select handling and the specific device protocol. We would have
> a driver included, without any  "users"  (code  that  actually  calls
> these functions).
> 
> In other words, this driver is a prerequisite for  other  SPI  device
> drivers  that  might  follow  later,  but as is, it's just a waste of
> memory.
> 
> It would just waste memory to enable it.
Makes sense.
> 
> Best regards,
> 
> Wolfgang Denk
> 
The problem is that if it isn't accepted now, nobody is likely to remember
that it is available, and so they will have to re-write it.  Also, it will
suffer from bit-rot, if just left as a loose patch.

	Steve