Re: [PATCH v3 0/5] staging: et131x: patches for et131x driver

[Mark Einon]

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.

- 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.

Cheers,

Mark
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/