On Wed, Sep 19, 2012 at 09:32:49AM +0800, Richard Zhao wrote:
On Wed, Sep 19, 2012 at 03:30:41AM +0200, Marek Vasut wrote:
quoted
Dear Richard Zhao,
You know, commit message would be nice to have.
Yes, but for this one, the title shows obviously what it changed,
doesn't it?
Alex has the the final word, but I'm also very picky about commit
messages. IMHO, all patches should have commit message, even if it
sounds obvious.
For example, you could let us know the reasoning behind converting to
devm_request_irq().
--
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120919/cd4cf30e/attachment.sig>