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

Kumar Gala galak at kernel.crashing.org
Wed Jan 16 01:13:56 EST 2008


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



More information about the Linuxppc-dev mailing list