[PATCH] Add support for Analogue & Micro ASP837E board

Bryan O'Donoghue bodonoghue at codehermit.ie
Tue May 6 20:30:55 EST 2008


On Tue, 6 May 2008 16:11:10 +1000
Stephen Rothwell <sfr at canb.auug.org.au> wrote:

> Hi Bryan,
> 
> On Tue, 6 May 2008 03:28:13 +0100 Bryan O'Donoghue <bodonoghue at codehermit.ie>
> wrote:
> >
> > +static void __init asp834x_init_IRQ(void)
> > +{
> > +	struct device_node *np;
> > +
> > +	np = of_find_node_by_type(NULL, "ipic");
> > +	if (!np)
> > +		return;
> > +
> > +	ipic_init(np, 0);
> 
> You need an "of_node_put(np)" here to drop the reference gained in
> "of_find_node_by_type".
> 
> > +static struct of_device_id asp8347_ids[] = {
> 
> Please make this __initdata.
> 

Hey Stephen.

Thanks for spotting that. Will do.


Cheers,
Bryan



More information about the Linuxppc-dev mailing list