On Thu, Sep 9, 2010 at 09:25, Steve deRosier <steve@xxxxxxxxxxx> wrote: > > Signed-off-by: Steve deRosier <steve@xxxxxxxxxxx> It'd be nice if the subject line of this patch was a bit more descriptive. Something like "libertas_tf: Update beacon code" would be descriptive. Also, it appears that this patch does two things, update the beacon code and add some debugging. You may want to separate this into two patches. Thanks, -- Julian Calaby Email: julian.calaby@xxxxxxxxx Profile: http://www.google.com/profiles/julian.calaby/ .Plan: http://sites.google.com/site/juliancalaby/ -- 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