Re: Cpufreq for opteron

[Pavel Machek]

Hi!

> > +/* driver entry point for term */
> > +static void __exit
> > +drv_exit(void)
> > +{
> > +	dprintk(KERN_INFO PFX "drv_exit\n");
> > +
> > +	cpufreq_unregister_driver(&cpufreq_amd64_driver);
> > +	if (ppst) {
> > +		kfree(ppst);
> 
> kfree(NULL) is fine.
> 
> > +		ppst = 0;
> 
> this should be ppst = NULL but in fact is completly superflous as
> the module is gone afterwards.

Ok.
								Pavel
-- 
When do you have a heart between your knees?
[Johanka's followup: and *two* hearts?]
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/