The enable,disable,suspend and resume entry in the struct mfd_cell do not seem to be necessary as they should be handled by the platform driver that the device created has bound to. Signed-off-by: Ben Dooks Index: linux-2.6.26-rc9-next20080709/include/linux/mfd/core.h =================================================================== --- linux-2.6.26-rc9-next20080709.orig/include/linux/mfd/core.h 2008-07-09 10:44:45.000000000 +0100 +++ linux-2.6.26-rc9-next20080709/include/linux/mfd/core.h 2008-07-09 10:45:06.000000000 +0100 @@ -23,12 +26,6 @@ */ struct mfd_cell { const char *name; - - int (*enable)(struct platform_device *dev); - int (*disable)(struct platform_device *dev); - int (*suspend)(struct platform_device *dev); - int (*resume)(struct platform_device *dev); - void *driver_data; /* driver-specific data */ /* -- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/