[PATCH 10/13] powerpc: Add arch/powerpc mv64x60 PCI setup

[Dale Farnsworth]

On Thu, May 03, 2007 at 12:13:58PM +1000, Stephen Rothwell wrote:
> On Wed, 2 May 2007 14:46:09 -0700 "Dale Farnsworth" <dale at farnsworth.org> wrote:
> >
> > +DECLARE_MUTEX(mv64x60_hs_lock);
> 
> Does this need to be global (it is not declared in a header file)?

Nope.  Fixed.

> > +	np = of_find_compatible_node(NULL, NULL, "mv64x60");
> > +	if (!np)
> > +		return 0;
> > +
> > +	pdev = platform_device_register_simple("mv64x60", 0, NULL, 0);
> > +	if (IS_ERR(pdev))
> > +		return PTR_ERR(pdev);
> 
> This error leaks a reference count on np.

Good find.  Fixed.

Stephen, thanks for taking the time to review and comment.
I'll wait to see what other comments I get before reposting.

-Dale