[PATCH 1/3] sbc834x: Add support for Wind River SBC834x boards

[Kumar Gala]

On Jan 9, 2008, at 1:01 AM, Stephen Rothwell wrote:

> Hi Paul,
>
> On Wed,  9 Jan 2008 00:49:30 -0500 Paul Gortmaker <paul.gortmaker at windriver.com 
> > wrote:
>>
>> +static void __init sbc834x_init_IRQ(void)
>> +{
>> +	struct device_node *np;
>> +
>> +	np = of_find_node_by_type(NULL, "ipic");
>> +	if (!np)
>> +		return;
>> +
>> +	ipic_init(np, 0);
>> +
>> +	/* Initialize the default interrupt mapping priorities,
>> +	 * in case the boot rom changed something on us.
>> +	 */
>> +	ipic_set_default_priority();
>> +}
>
> This needs an of_node_put(np) somewhere.
>
>> +static struct of_device_id sbc834x_ids[] = {
>
> __initdata
>
>> +static int __init sbc834x_declare_of_platform_devices(void)
>> +{
>> +	if (!machine_is(sbc834x))
>> +		return 0;
>> +
>> +	of_platform_bus_probe(NULL, sbc834x_ids, NULL);
>> +	return 0;
>> +}
>> +device_initcall(sbc834x_declare_of_platform_devices);
>
> machine_device_initcall(sbc834x, ...

Any updates here?  While David's .dts comments are probably valid I'm  
not going to make you have to deal with them to get into the tree.  We  
need to fix all the other 83xx platforms w/respect to those things.

- k