On Fri, Jun 14, 2013 at 6:54 PM, Richard Retanubun richardretanubun@ruggedcom.com wrote:
On 14/06/13 09:09 AM, Axel Lin wrote:
2013/6/14 Jagan Tekijagannadh.teki@gmail.com:
Hi,
IMHO: Can you please use the proper commit header prefix.
I am just sharing my thoughts, ignore this if you know it already. I followed below syntax. "<main_module>:<sub_module>:<sub_sub_module>:<COMMIT_HEADER>"
Ex: for this commit (seems like this commit changes two drivers) spi: cf_qspi | mxc_spi: Use DIV_ROUND_UP at appropriate places
This does not scale. What if a (trivial) patch touches 10 drivers?
Depends how trivial it is I guess. I can see benefits to both approaches.
This one is simple enough that I am okay if it is grouped.
Just for my own education, how do I correctly ACK just the part I know (mcf_qspi in this case) in a grouped patch? Is there a way to automate my "Signed-off-by:" via e-mail?
Usually normal mail server like gmail or outlook doesn' update these s-o-b or a-b use thunderbread mail logic, that will automate when u did these changes.
-- Thanks, Jagan.