On Mon, 22 Jun 2015, Daniel Vetter wrote:
On Mon, Jun 22, 2015 at 04:33:22PM +0530, Varka Bhadram wrote:
Hi Shobhit Kumar,
On 06/22/2015 04:24 PM, Shobhit Kumar wrote:
On some BYT PLatform the PWM is controlled using CRC PMIC. Add a lookup entry for the same to be used by the consumer (Intel GFX)
v2: Remove the lookup table on driver unload (Thierry)
v3: Correct the subject line (Lee jones)
This part should only describe what this is about..
Don't put this patch change history over here. Include this change history after ... Signed-off-by: Author <email>
CC: Samuel Ortiz sameo@linux.intel.com Cc: Linus Walleij linus.walleij@linaro.org Cc: Alexandre Courbot gnurou@gmail.com Cc: Thierry Reding thierry.reding@gmail.com Acked-by: Lee Jones lee.jones@linaro.org Signed-off-by: Shobhit Kumar shobhit.kumar@intel.com
Here you add this change history so that after applying this will not be the part of your commit description.
This comment is applicable for all of your patches.
It's honestly a per-maintainer thing and hard to tell who wants what ... Personally I do want to include the patch changelog in the commit message.
The patch change-log should go below the '---'. There are very few (weird ;) ) Maintainers who like to see them in the commit log.