Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210415021456.1805797-1-pmalani@chromium.org> <CAFqH_53KBrjeZG5k1npbboLOGcbPv3_8_dTQ-ZsJcUwvKAY1nA@mail.gmail.com>
In-Reply-To: <CAFqH_53KBrjeZG5k1npbboLOGcbPv3_8_dTQ-ZsJcUwvKAY1nA@mail.gmail.com>
From:   Prashant Malani <pmalani@chromium.org>
Date:   Thu, 15 Apr 2021 23:25:00 -0700
Message-ID: <CACeCKadCASMrAY4RPn4mZ8L0+nv2BopiRb-S2=Lb4ijWXr-z3w@mail.gmail.com>
Subject: Re: [PATCH] platform/chrome: cros_ec_typec: Handle hard reset
To:     Enric Balletbo Serra <eballetbo@gmail.com>
Cc:     linux-kernel <linux-kernel@vger.kernel.org>,
        Benson Leung <bleung@chromium.org>,
        Enric Balletbo i Serra <enric.balletbo@collabora.com>,
        Guenter Roeck <groeck@chromium.org>,
        "Gustavo A. R. Silva" <gustavoars@kernel.org>,
        Mark Brown <broonie@kernel.org>,
        Pi-Hsun Shih <pihsun@chromium.org>,
        Utkarsh Patel <utkarsh.h.patel@intel.com>,
        Yu-Hsuan Hsu <yuhsuan@chromium.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Enric,

Thanks for taking a look.

On Thu, Apr 15, 2021 at 10:39 PM Enric Balletbo Serra
<eballetbo@gmail.com> wrote:
>
> Hi Prashant,
>
> Thank you for your patch.
>
> Missatge de Prashant Malani <pmalani@chromium.org> del dia dj., 15
> d=E2=80=99abr. 2021 a les 4:15:
> >
> > The Chrome Embedded Controller (EC) generates a hard reset type C event
> > when a USB Power Delivery (PD) hard reset is encountered. Handle this
> > event by unregistering the partner and cable on the associated port and
> > clearing the event flag.
> >
> > Also update the EC command header to include the new event bit. This bi=
t
> > is included in the latest version of the Chrome EC headers[1].
> >
> > [1] https://chromium.googlesource.com/chromiumos/platform/ec/+/refs/hea=
ds/main/include/ec_commands.h
> >
> > Cc: Benson Leung <bleung@chromium.org>
> > Signed-off-by: Prashant Malani <pmalani@chromium.org>
> > ---
> >  drivers/platform/chrome/cros_ec_typec.c        | 13 +++++++++++++
> >  include/linux/platform_data/cros_ec_commands.h |  1 +
>
> Could this be a separate patch?

Sure. I'll send a v2 with them split up.