On 07/13/2011 03:27 AM, Dan Carpenter wrote:
On Tue, Jul 12, 2011 at 08:24:08PM -0500, Larry Finger wrote:
On 07/12/2011 04:26 PM, Dan Carpenter wrote:
@@ -1712,25 +1602,6 @@ void rtl8192_query_rxphystatus(
{
u8 report;
-#ifdef RTL8190P
- u8 tmp_pwdb;
- char cck_adc_pwdb[4];
-#endif
- priv->stats.numqry_phystatusCCK++;
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Did you intend to remove that?
No, I fat-fingered that line. I'll fix with V2.
I've sort of decided that I hate re-reviewing version 2 of
patchsets. If you send a v2 does that mean you have to redo a lot
of other patches or is it just this one? The bug here is not a very
serious one, and I would be fine with just a fixup patch applied on
top of the set.
As this is the only one of the patches with a negative comment, I will do as you
suggest.
Thanks,
Larry
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
