On Tue, 2012-08-07 at 20:27 +0530, Jeffrin Jose wrote: > / Is putting "rc = put_user(amount, (int __user *)argp);" on the same > / indentation level as "amount = skb->len - sizeof(struct ddpehdr);" > / really what you want to do?
> No. I was trying to place those lines at one > indentation back from their current position. If you are using checkpatch as a guide, run "./scripts/checkpatch.pl -f --strict net/appletalk/ddp.c" after you patch it and also correct any problems found in the area you want to update too. A second option is to use emacs with the proper linux style indentation. (See: Documentation/CodingStyle, Chapter 9: You've made a mess of it) Another option is not to patch the file at all. Also, don't submit multiple patches, resubmit 1 single patch with all the appropriate corrections. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
