From mboxrd@z Thu Jan  1 00:00:00 1970
From: Singh Sandeep-B37400 <B37400@freescale.com>
Date: Tue, 2 Jul 2013 10:01:46 +0000
Subject: [U-Boot] [PATCH] powerpc/mpc85xx: Enabling CPC conditionally
 based on hwconfig options
In-Reply-To: <1371238766.2996.6@snotra>
References: <3F1D9DCAAB49B94D88DBE05911FA4E6E070AF829@039-SN1MPN1-006.039d.mgd.msft.net>
	(from B37400@freescale.com on Fri Jun 14 00:26:18 2013)
	<1371238766.2996.6@snotra>
Message-ID: <3F1D9DCAAB49B94D88DBE05911FA4E6E070D5878@039-SN1MPN1-001.039d.mgd.msft.net>
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

Sorry somehow I missed this earlier. Please find my reply inline.

Regards,
Sandeep


> -----Original Message-----
> From: Wood Scott-B07421
> Sent: 15 June 2013 01:09
> To: Singh Sandeep-B37400
> Cc: Wood Scott-B07421; u-boot at lists.denx.de; afleming at gmail.com
> Subject: Re: [U-Boot] [PATCH] powerpc/mpc85xx: Enabling CPC conditionally
> based on hwconfig options
> 
> On 06/14/2013 12:26:18 AM, Singh Sandeep-B37400 wrote:
> > This was done with the intension of providing greater
> > configurability. When
> > en_cpc is defined then it's entirely up to user to decide which cpcs
> > are to
> > be enabled. Hence we do following:
> >
> > if_defined("en_cpc"){
> > 	only_then
> > 		check_for_cpc_options;
> > } else {
> > 	enable_all_cpc;
> > }
> 
> OK, I see.
> 
> What is the use case for enabling one cpc but not another?  What is the
> use case for disabling cpc at all?  The answer should go in the commit
> message.

In B4860, Starcore owns one CPC and second CPC is owned by PowerPC. So we need to enable only one CPC at u-boot.

> 
> -Scott