All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thierry Reding <thierry.reding@gmail.com>
To: "Zha, Qipeng" <qipeng.zha@intel.com>
Cc: "linux-pwm@vger.kernel.org" <linux-pwm@vger.kernel.org>,
	"Westerberg, Mika" <mika.westerberg@intel.com>
Subject: Re: [PATCH] pwm: update lpss pwm driver to support Broxton platform
Date: Fri, 12 Jun 2015 12:47:23 +0200	[thread overview]
Message-ID: <20150612104722.GJ19400@ulmo.nvidia.com> (raw)
In-Reply-To: <C6287702A945AC47BE5DB5DFD4B5C6DD0185D774@SHSMSX104.ccr.corp.intel.com>

[-- Attachment #1: Type: text/plain, Size: 1602 bytes --]

Looks like I never got around to applying this. Done now.

Thierry

On Fri, Mar 13, 2015 at 12:51:35AM +0000, Zha, Qipeng wrote:
> Thierry
> 
> Thank you very much:)
> 
> 
> Best wishes
> Qipeng
> 
> -----Original Message-----
> From: linux-pwm-owner@vger.kernel.org [mailto:linux-pwm-owner@vger.kernel.org] On Behalf Of Thierry Reding
> Sent: Thursday, March 12, 2015 3:59 PM
> To: Zha, Qipeng
> Cc: linux-pwm@vger.kernel.org; Westerberg, Mika
> Subject: Re: [PATCH] pwm: update lpss pwm driver to support Broxton platform
> 
> On Thu, Mar 12, 2015 at 10:28:28PM +0800, qipeng.zha wrote:
> > From: "qipeng.zha" <qipeng.zha@intel.com>
> > 
> > Add pci device id for Broxton platform.
> 
> For reference, this should be "PCI" and "ID", but I can fix those up while applying.
> 
> Also, it's customary to include a changelog in subsequent versions of a patch, as well as change the subject prefix to include the patch version number. That makes it easier for other people to track things.
> 
> You can do so using:
> 
> 	$ git format-patch --subject-prefix='PATCH v2' ...
> 
> The changelog typically goes below the --- marker (the one below the Signed-off-by line) and would look something like this:
> 
> ---
> Changes in v2:
> - keep board information table sorted by device ID
> - reuse Braswell board information
> 
> Again, no need to resend this time, but something to keep in mind for future patches.
> 
> > Signed-off-by: qipeng.zha <qipeng.zha@intel.com>
> 
> I'd prefer a proper spelling of your name. Would "Qipeng Zha" be correct?
> 
> Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2015-06-12 10:48 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-12 14:28 [PATCH] pwm: update lpss pwm driver to support Broxton platform qipeng.zha
2015-03-12  7:58 ` Thierry Reding
2015-03-13  0:51   ` Zha, Qipeng
2015-06-12 10:47     ` Thierry Reding [this message]
2015-03-12  9:23 ` Mika Westerberg
  -- strict thread matches above, loose matches on Subject: below --
2015-03-12 14:27 qipeng.zha
2015-03-11 17:17 qipeng.zha
2015-03-11 10:00 ` Mika Westerberg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150612104722.GJ19400@ulmo.nvidia.com \
    --to=thierry.reding@gmail.com \
    --cc=linux-pwm@vger.kernel.org \
    --cc=mika.westerberg@intel.com \
    --cc=qipeng.zha@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.