Hey, On Sun, Jan 19, 2014 at 07:47:06PM +0100, Hans de Goede wrote: > As I see it either doing clks, regulator and sata-core things in a common > place makes sense, and then it goes for suspend and resume too, or we > opt for always following the complete override model, and which point > it becomes more sensible to just do a separate platform driver per > ahci implementation. It makes sense in light of those specific cases, but there are gonna be cases where the placement of the callback is slightly wrong and we end up with ->XXX_ops_pre() and then ->XXX_ops_post() and so on. Please make the whole op overridable and then export the default op and use it as library. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html