From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <hs@denx.de>
Received: from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9])
	by ozlabs.org (Postfix) with ESMTP id 53F64DDD0B
	for <linuxppc-dev@ozlabs.org>; Tue, 18 Mar 2008 18:05:27 +1100 (EST)
Message-ID: <47DF6B82.4030404@denx.de>
Date: Tue, 18 Mar 2008 08:13:06 +0100
From: Heiko Schocher <hs@denx.de>
MIME-Version: 1.0
To: Stephen Rothwell <sfr@canb.auug.org.au>
Subject: Re: [PATCH v2] 8xx: Add support for the MPC852 based board from
	keymile.
References: <47DA444E.9050507@denx.de>
	<20080315102508.bc698a90.sfr@canb.auug.org.au>
In-Reply-To: <20080315102508.bc698a90.sfr@canb.auug.org.au>
Content-Type: text/plain; charset=ISO-8859-15
Cc: linuxppc-dev@ozlabs.org
Reply-To: hs@denx.de
List-Id: Linux on PowerPC Developers Mail List <linuxppc-dev.ozlabs.org>
List-Unsubscribe: <https://ozlabs.org/mailman/listinfo/linuxppc-dev>,
	<mailto:linuxppc-dev-request@ozlabs.org?subject=unsubscribe>
List-Archive: <http://ozlabs.org/pipermail/linuxppc-dev>
List-Post: <mailto:linuxppc-dev@ozlabs.org>
List-Help: <mailto:linuxppc-dev-request@ozlabs.org?subject=help>
List-Subscribe: <https://ozlabs.org/mailman/listinfo/linuxppc-dev>,
	<mailto:linuxppc-dev-request@ozlabs.org?subject=subscribe>

Hello Stephen,

Stephen Rothwell wrote:
> On Fri, 14 Mar 2008 10:24:30 +0100 Heiko Schocher <hs@denx.de> wrote:
[...]
>> +struct cpm_pin {
>> +		int port, pin, flags;
>> +};
> 
> I wish someone would consolidate all these definitions of cpm_pin.

Hmm... do you mean something like, moving this struct
in cpm1.h and ...

>> +static void __init init_ioports(void)
>> +{
>> +	int i;
>> +
>> +	for (i = 0; i < ARRAY_SIZE(mgsuvd_pins); i++) {
>> +		struct cpm_pin *pin = &mgsuvd_pins[i];
>> +		cpm1_set_pin(pin->port, pin->pin, pin->flags);
>> +	}
> 
> And the code that uses them ...

making in arch/powerpc/sysdev/cpm1.c a function
cpm1_setup_pins (struct cpm_pin *pins) ?

>> +static void __init mgsuvd_setup_arch(void)
>> +{
>> +	struct device_node *cpu;
>> +
>> +	cpu = of_find_node_by_type(NULL, "cpu");
>> +	if (cpu != 0) {
> 
> "cpu" is a pointer, so "cpu != NULL" or just "cpu"
> 
>> +		const unsigned int *fp;
>> +
>> +		fp = of_get_property(cpu, "clock-frequency", NULL);
>> +		if (fp != 0)
> 
> Ditto for "fp" test

Its no longer necessary, thanks to Scott

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany