On Fri, Nov 22, 2013 at 5:18 AM, Mark Einon <mark.einon@xxxxxxxxx> wrote: > On Wed, Nov 20, 2013 at 03:53:50PM +0800, ZHAO Gang wrote: >> This patch set should apply to current staging-next tree >> >> ZHAO Gang (5): >> staging: et131x: clean up code >> staging: et131x: drop packet when error occurs in et131x_tx >> staging: et131x: stop read when hit max delay in et131x_phy_mii_read >> staging: et131x: remove spinlock adapter->lock >> staging: et131x: update TODO list >> >> drivers/staging/et131x/README | 5 - >> drivers/staging/et131x/et131x.c | 301 +++++++++++++++++----------------------- >> 2 files changed, 127 insertions(+), 179 deletions(-) >> >> -- > > Hi Zhao, > > Thanks for the patches. A few pointers for the future: > > - Use scripts/get_maintainer.pl to discover to which mailing lists / > maintainers patches should be sent. I've cc'd devel@xxxxxxxxxxxxxxxxxxxx > which was missing from your original submission. I will add it to cc list next time. > > - Please don't roll up patches you've previously sent into newer > patchsets, even if they haven't been applied by the tree maintainer > yet. This just means that the same lines of code get reviewed twice, > wasting the reviewer's time. Greg also told me this problem, I will not do this any more. > > Cheers, > > Mark _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel