On Fri, Jul 31, 2009 at 11:39 AM, Nick Kossifidis<mickflemm@xxxxxxxxx> wrote: > 2009/7/31 Luis R. Rodriguez <mcgrof@xxxxxxxxx>: >> On Fri, Jul 31, 2009 at 11:08 AM, Nick >> Kossifidis<mick@xxxxxxxxxxxxxxxxxxx> wrote: >>> * Don't put chip to full sleep because there are problems during >>> wakeup. Instead hold MAC/Baseband on warm reset state via a new >>> function ath5k_hw_on_hold. >>> >>> * Durring attach preserve pcicfg bits when enabling pci core >>> sw retry fix. >>> >>> * Minor cleanups >> >> Can you address these changes separately? I see you tend to itemize >> the things you change, even when you just make one change. Please >> consider addressing one change per commit and just ensure the why is >> crystal clear. >> >> Luis >> > > Cleanup doesn't count as a change, the only real change is the first one. > I 'm trying not to pollute the logs. The purpose of the patch review cycle is to make it very easy to review code changes, the more you split your work the easier it is to review, that is not polluting the logs in any way. > This patch fixes an issue reported when > card doesn't wake up. This is not mentioned in the commit log, please add that to the commit log and refer to a URL if possible. Luis -- 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
