On Mon, Mar 2, 2015 at 6:29 PM, Alexandre Courbot <gnurou@xxxxxxxxx> wrote: >>> These functions are identical, excepted for the callback they are >>> invoking. Could you merge the common part into a function that returns >>> the right client to call the callback on, or ERR_PTR(-EINVAL) in case >>> of failure? >> >> I couldn't think of a clever way to do this. Any ideas? :) > > How about something like this (warning: might now be that great, untested): > > /* Have this in your .h and use it in your tegra_mc_ops struct */ > typedef int (*mc_op)(struct tegra_mc *mc, > const struct tegra_mc_hotreset *hotreset) This type should be named tegra_mc_op, since the header that defines it is in include/linux. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html