Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752899AbbFVOQc (ORCPT ); Mon, 22 Jun 2015 10:16:32 -0400 Received: from mail-wi0-f173.google.com ([209.85.212.173]:36674 "EHLO mail-wi0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752076AbbFVOQW (ORCPT ); Mon, 22 Jun 2015 10:16:22 -0400 Date: Mon, 22 Jun 2015 16:19:08 +0200 From: Daniel Vetter To: Varka Bhadram Cc: Shobhit Kumar , linux-pwm , intel-gfx , linux-kernel , dri-devel , linux-gpio , Alexandre Courbot , Paul Bolle , Samuel Ortiz , Povilas Staniulis , Jani Nikula , Linus Walleij , Paul Gortmaker , bloften80@gmail.com, David Airlie , Chih-Wei Huang , Thierry Reding , Daniel Vetter , Lee Jones Subject: Re: [Intel-gfx] [v2 4/7] mfd: intel_soc_pmic_core: ADD PWM lookup table for CRC PMIC based PWM Message-ID: <20150622141908.GM25769@phenom.ffwll.local> Mail-Followup-To: Varka Bhadram , Shobhit Kumar , linux-pwm , intel-gfx , linux-kernel , dri-devel , linux-gpio , Alexandre Courbot , Paul Bolle , Samuel Ortiz , Povilas Staniulis , Jani Nikula , Linus Walleij , Paul Gortmaker , bloften80@gmail.com, David Airlie , Chih-Wei Huang , Thierry Reding , Daniel Vetter , Lee Jones References: <1434970465-12687-1-git-send-email-shobhit.kumar@intel.com> <1434970465-12687-5-git-send-email-shobhit.kumar@intel.com> <5587EB7A.5090402@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5587EB7A.5090402@gmail.com> X-Operating-System: Linux phenom 4.0.0-rc3+ User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1620 Lines: 54 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 > --- > > >CC: Samuel Ortiz > >Cc: Linus Walleij > >Cc: Alexandre Courbot > >Cc: Thierry Reding > >Acked-by: Lee Jones > >Signed-off-by: Shobhit Kumar > >--- > > 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. -Daniel > > > -- > Best regards, > Varka Bhadram. > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/