On 20 July 2011 17:07, Dan Williams <dcbw@xxxxxxxxxx> wrote: > Storing it is fine; I was just trying to keep the functions that built > firmware commands from poking priv->xxx stuff. Hence why there was a > 'chan' parameter to the function, to push the actual decision about what > channel to change to up to the thing that actually decided it was > necessary to change the channel at all. In this case it's not as > relevant as other cases, so in the end I don't really care as much. Ah, now I understand. I'll redo the patch so that it passes it as a parameter, for consistency with the other functions that you have designed in this way. Thanks, Daniel -- 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