On Wed, 2011-02-09 at 22:55 +0100, RafaÅ MiÅecki wrote: > I aksed about reading, you gave me examples of writing. I want to > avoid such a non-readable disasters: > u32 tmp; > ssb_core_ctl_flags(dev, 0, 0, &tmp); A good function name consists of: "WHAT is done and WHERE is it done" ssb_core_ctl_flags() violates both of them. It doesn't specify what is done at all, except that it might be something random with ctl flags. And it lies about where it does it. It implies that it operates on SSB. However, it might operate on SSB or AI. -- 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
