On Fri, 2011-02-18 at 00:08 +0200, George Kashperko wrote: > const struct ssb_bus_ops *ops; > + const struct ssb_bus_helpers *helpers; Uhm, well. "helpers"... . Seems like a pretty generic term. I think this is still violating abstraction layering. Not as much as your previous attempt, but still... You're putting the ops structure that is supposed to abstract the ssb bus into the ssb_device data structure. That should ring a bell. -- Greetings Michael. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html