Am 13.02.2017 16:43, schrieb Timur Tabi: > walter harms wrote: >> The question is: why is a simple calculation const*const >> separated into a function ? > > This is a callback function. That's just how it's defined. > > It's rare, but there are drivers that use the parameter, like this one: > > http://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/tree/drivers/net/ethernet/qlogic/qlge/qlge_ethtool.c#n591 > > this is the problem: I used: http://lxr.free-electrons.com/source/drivers/net/ibm_emac/ibm_emac_core.c?v=2.6.24#L1778 here is the patched function emac_get_regs_len used the calculate the length in emac_ethtool_get_regs_len what is used as callback. in http://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/tree/drivers/net/ethernet/qualcomm/emac/emac-ethtool.c is it now used directly. sorry for the noise, re, wh -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html