On Fri, May 22, 2015 at 4:56 AM, Greg KH <gre...@linuxfoundation.org> wrote: > On Thu, May 21, 2015 at 05:30:40PM -0700, Kevin Tsai wrote: >> Signed-off-by: Kevin Tsai <kt...@capellamicro.com> >> --- > > Your whole changelog ended up in the Subject line :(
Hi Kevin, Few comments: 1) Please also add a cover letter. Use --cover-letter parameter of git format-patch. 2) Please create a patch for each logical change. Your current 1/6 patch should be split in several small patches: * 1/6 iio: light: cm32181: Update company name * 2/6 iio: light: cm32181: Remove unused include files * etc. 3) Please use specific wordings for commit subject. A subject like this: "Updated Vishay Capella cm32181 ambient light sensor driver" doesn't say much. thanks, Daniel. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
