On Friday 26 January 2007 03:54, Christoph Hellwig wrote:

quoted

=A0
+#ifdef CONFIG_PPC_OF
+#include <asm/of_device.h>
+#include <asm/of_platform.h>
+#endif

=20
Adding this OF-specific code to the generic file doesn't look exactly
nice. =A0Is 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.

quoted

+static int __devexit ipmi_of_remove(struct of_device *dev)
+{
+=A0=A0=A0=A0=A0/* should call
+=A0=A0=A0=A0=A0 * cleanup_one_si(dev->dev.driver_data); */

=20
Wo why doesn't it do that currently?=20

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 <><