[patch 1/1] updated version, fixed the compiler warning
Arnd Bergmann
arnd at arndb.de
Fri Jan 26 15:23:02 EST 2007
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 <><
More information about the Linuxppc-dev
mailing list