Re: [PATCH] leds: Add BD2802GU LED driver

From: Kim Kyuwon
Date: Thu Feb 19 2009 - 23:50:55 EST


Hi,

>
> Your email client has wordwrapped the patch and has replaced all tabs
> with eight spaces. Please fix that up when resending.
> Documentation/email-clients.txt has some gmail tips.
>
> I had a go at cleaning it up, and found lots of code whcih was indented
> with seven-spaces. Perhaps this file was origininaly edited with
> space-space-space-space indenting. If so, please don't do that - use
> hard tab characters.
>
> Anyway, I ended up with soemthing reasonable-looking which actually
> compiles. Please check the result.
>

To make sure that this patch is well-formated, I sent this patch to myself
first, and then forwarded it to the LKML. I think this 'forwarding' makes the
problem worse. Sorry for the problem :)

And thank you for reviewing this driver and giving me valuable comments.

I will resend the patch as following Gmail tips in 'email-clients.txt'
and yours.

Regards,

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