Re: [PATCH 3/6 v2] staging/rtl8192u: indent with tabs, not spaces

From: devendra.aaru
Date: Thu Dec 06 2012 - 07:55:17 EST


Hi,

>From the next time onward while you put a patch with updated code
change as mentioned by reviewers, you tell what changed in the next
interation.

like the below,

otherthan that your patch is good,

On Thu, Dec 6, 2012 at 6:23 AM, Sebastian Hahn <snsehahn@xxxxxxxxxxxxx> wrote:
> Converted staging/rtl8187se to use tabs instead of spaces for
> indentation to fix the checkpatch error "code indent should use tabs
> where possible".
>
> Signed-off-by: Sebastian Hahn <snsehahn@xxxxxxxxxxxxx>
> ---
V2:
Removed .c.bak file as suggested by Dan.

> drivers/staging/rtl8192u/ieee80211/ieee80211.h | 14 +-
> drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c | 370 ++++++++++----------
> .../staging/rtl8192u/ieee80211/ieee80211_softmac.c | 4 +-
> drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c | 14 +-
> drivers/staging/rtl8192u/ieee80211/rtl_crypto.h | 84 ++---
> drivers/staging/rtl8192u/r8180_93cx6.c | 2 +-
> drivers/staging/rtl8192u/r8180_pm.h | 2 +-
> drivers/staging/rtl8192u/r8192U.h | 198 +++++------
> drivers/staging/rtl8192u/r8192U_core.c | 42 +--
> 9 files changed, 365 insertions(+), 365 deletions(-)
>

> devel mailing list
> devel@xxxxxxxxxxxxxxxxxxxxxx
> http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
--
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/