Re: [PATCH 1/4] Emerson ATCA-C125 bootwrapper

[David Gibson <david@gibson.dropbear.id.au>]

On Mon, Mar 17, 2008 at 03:19:45PM +0300, Alexandr Smirnov wrote:
> David,
>>> +	/* Unfortunately, the specific model number is encoded in the
>>> +	 * soc node name in existing dts files -- once that is fixed,
>>> +	 * this can do a simple path lookup.
>>> +	 */
>>>     
>>
>> Since this is a new board, couldn't you name the soc node /soc and
>> dispense with this more complicated way of doing things?  Or would
>> that break other versions of u-boot that are in use?
>>   
> As I understand you would like to rename /soc8548 node to /soc and make a 
> simple path lookup like following:
> - soc = find_node_by_devtype(NULL, "soc");
> + soc = finddevice("/soc");
> Is this correct?

That's right.

> But fsl_soc code still uses of_find_node_by_type so we can't remove 
> device_type property from node /soc and therefor simplify device
> tree.

It does?  Grrr...  I thought that had been fixed and the only reason
the device_type was still needed was because some u-boot versions
needed it.  But you're right, that's not the case.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson