All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lee Jones <lee.jones@linaro.org>
To: Fabien Parent <fparent@baylibre.com>
Cc: Sebastian Reichel <sre@kernel.org>,
	linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
	Enric Balletbo Serra <eballetbo@gmail.com>,
	Guillaume Pain <gpain@baylibre.com>
Subject: Re: [PATCH 1/3] mfd: cros: add charger port count command definition
Date: Tue, 5 Jun 2018 08:11:20 +0100	[thread overview]
Message-ID: <20180605071120.GI21163@dell> (raw)
In-Reply-To: <CAOwMV_xyqj027+iRAQtZ5Hv8NDUc6s9OfS0vAkumUyyc4LUjzw@mail.gmail.com>

On Mon, 04 Jun 2018, Fabien Parent wrote:

> On Mon, Jun 4, 2018 at 1:59 AM, Lee Jones <lee.jones@linaro.org> wrote:
> > On Tue, 29 May 2018, Fabien Parent wrote:
> >
> >> A new more command has been added to the ChromeOS embedded controller
> >> that allows to get the number of charger port count. Unlike
> >> EC_CMD_USB_PD_PORTS, this new command also includes the dedicated
> >> port if present.
> >>
> >> This command will be used to expose the dedicated charger port
> >> in the ChromeOS charger driver.
> >>
> >> Signed-off-by: Fabien Parent <fparent@baylibre.com>
> >> ---
> >>  include/linux/mfd/cros_ec_commands.h | 10 ++++++++++
> >>  1 file changed, 10 insertions(+)
> >
> > Does not want to apply.  I didn't investigate why.
> >
> > Please rebase and resend?
> 
> Sorry, I forgot to add you as "to:" to the patch series.
> This patch applies on top of https://lkml.org/lkml/2018/5/2/590
> ([PATCH v4 1/3] mfd: cros_ec: Add USBPD charger commands and struct
> definitions)

Okay.  Please re-submit after that set has landed then.

Please, also add my Ack to the patch:

For my own reference:
  Acked-for-MFD-by: Lee Jones <lee.jones@linaro.org>

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

  reply	other threads:[~2018-06-05  7:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-30  3:17 [PATCH 0/3] power: supply: cros: add support for dedicated port and expose connected ports Fabien Parent
2018-05-30  3:17 ` [PATCH 1/3] mfd: cros: add charger port count command definition Fabien Parent
2018-06-01 16:40   ` Enric Balletbo Serra
2018-06-04  8:59   ` Lee Jones
2018-06-04 17:12     ` Fabien Parent
2018-06-05  7:11       ` Lee Jones [this message]
2018-05-30  3:17 ` [PATCH 2/3] power: supply: cros: add support for dedicated port Fabien Parent
2018-06-13 10:57   ` Fabien Parent
2018-05-30  3:17 ` [PATCH 3/3] power: supply: cros: add property to detect connected ports Fabien Parent
2018-06-01 16:30   ` Enric Balletbo Serra
2018-06-04 17:13 ` [PATCH 0/3] power: supply: cros: add support for dedicated port and expose " Fabien Parent

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=20180605071120.GI21163@dell \
    --to=lee.jones@linaro.org \
    --cc=eballetbo@gmail.com \
    --cc=fparent@baylibre.com \
    --cc=gpain@baylibre.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=sre@kernel.org \
    /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.