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;
Date:   Thu, 15 Apr 2021 11:22:09 -0500
From:   Rob Herring <robh@kernel.org>
To:     Badhri Jagan Sridharan <badhri@google.com>
Cc:     Guenter Roeck <linux@roeck-us.net>, linux-kernel@vger.kernel.org,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-usb@vger.kernel.org, Kyle Tso <kyletso@google.com>,
        Heikki Krogerus <heikki.krogerus@linux.intel.com>,
        Adam Thomson <Adam.Thomson.Opensource@diasemi.com>,
        Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org
Subject: Re: [PATCH v4 3/3] dt-bindings: connector: Add slow-charger-loop
 definition
Message-ID: <20210415162209.GA1510914@robh.at.kernel.org>
References: <20210414142656.63749-1-badhri@google.com>
 <20210414142656.63749-3-badhri@google.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210414142656.63749-3-badhri@google.com>
Precedence: bulk

On Wed, 14 Apr 2021 07:26:56 -0700, Badhri Jagan Sridharan wrote:
> Allows PMIC charger loops which are slow(i.e. cannot meet the
> 15ms deadline) to still comply to pSnkStby i.e Maximum power
> that can be consumed by sink while in Sink Standby state as defined
> in 7.4.2 Sink Electrical Parameters of USB Power Delivery Specification
> Revision 3.0, Version 1.2.
> 
> This patch introduces slow-charger-loop which when set makes
> the port request PD_P_SNK_STDBY_MW(2.5W i.e 500mA@5V) upon entering
> SNK_DISCOVERY (instead of 3A or the 1.5A during SNK_DISCOVERY) and the
> actual currrent limit after RX of PD_CTRL_PSRDY for PD link or during
> SNK_READY for non-pd link.
> 
> Signed-off-by: Badhri Jagan Sridharan <badhri@google.com>
> ---
> Changes since v3:
> * Removed additional semi-colon in the subject line
> Changes since V2:
> To address Rob's comments
> * Updated commit and the binding description.
> * Updated subject as well.
> ---
>  .../devicetree/bindings/connector/usb-connector.yaml   | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 

Acked-by: Rob Herring <robh@kernel.org>