[PATCH 6/8] mpc83xx: timer driver for PM wakeup

Stephen Rothwell sfr at canb.auug.org.au
Thu Dec 13 11:14:39 EST 2007


Hi Scott,

On Wed, 12 Dec 2007 11:36:04 -0600 Scott Wood <scottwood at freescale.com> wrote:
>
> +static int __devinit gtm_probe(struct of_device *dev,
> +                               const struct of_device_id *match)
> +{
	.
	.
	.
> +out:

Just for consistency, you probably want
	dev_set_drvdata(&dev->dev, NULL);
here

> +	kfree(priv);
> +	return ret;
> +}

-- 
Cheers,
Stephen Rothwell                    sfr at canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20071213/49ac52f0/attachment.pgp>


More information about the Linuxppc-dev mailing list