Re: [PATCH 01/18] staging: rtl8188eu: Remove "silentreset_mutex" from struct sreset_priv

From: Dan Carpenter
Date: Thu May 29 2014 - 17:50:23 EST


I have reviewed this patchset.

Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Btw, I wish you would fold some of these patches together next time.
For example, patches 3-6 are very closely related and they should have
just been one thing.

Or the times where the change log says, "I am going to delete function
frob_frob_frob_whatever() in the next patch." Just delete it in that
patch.

But anyway, you're doing a great job. Very nice clean ups.

regards,
dan carpenter

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