On Wed, Mar 23 2011, Daniel Walker wrote:

Should let everyone know exactly what your doing, even breaking up the
changes into specific patches.

Rohit, but other MSM developers as well.  In general, the commit text
needs to be a lot better.  A good test would be to look at each part of
the patch, and ask how that is described by the commit text.  If it
feels like the commit text is describing separate things, the patch
probably needs to be split up.

Please spend some time running 'git log' on other parts of the kernel to
get a feel for what is expected.

Thanks,
David

-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.