[patch 1/1] updated version, fixed the compiler warning

[Arnd Bergmann]

On Friday 26 January 2007 03:54, Christoph Hellwig wrote:
> >  
> > +#ifdef CONFIG_PPC_OF
> > +#include <asm/of_device.h>
> > +#include <asm/of_platform.h>
> > +#endif
> 
> Adding this OF-specific code to the generic file doesn't look exactly
> nice.  Is it possible to separate the code out to a separate ipmi_of.c
> file?

This file already contains the same code for a number of other bus
layers, it might be possible to split out the of part, but then we
should also have separate files for ACPI, PCI and all the others.

I think that reworking the ipmi layer is outside of the scope of
what we want right now, that can be another patch if there is
a consensus that it would be a good idea.

> > +static int __devexit ipmi_of_remove(struct of_device *dev)
> > +{
> > +     /* should call
> > +      * cleanup_one_si(dev->dev.driver_data); */
> 
> Wo why doesn't it do that currently? 

This mimics the pci driver that also doesn't do it. The comment
is a hint that when it the ipmi driver gets cleaned up so
that _pci gets it right, _of should do the same. That should
also be a separate patch.

	Arnd <><