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:   Wed, 20 Jan 2021 18:17:35 +0100
From:   Greg KH <gregkh@linuxfoundation.org>
To:     Hans de Goede <hdegoede@redhat.com>
Cc:     Kyle Tso <kyletso@google.com>, linux@roeck-us.net,
        heikki.krogerus@linux.intel.com, badhri@google.com,
        linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v6 0/3] AMS, Collision Avoidance, and Protocol Error
Message-ID: <YAhlrxRdURu0djbf@kroah.com>
References: <20210114145053.1952756-1-kyletso@google.com>
 <2211ad43-b76c-fbe9-2cc8-bb40c4ee4e89@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <2211ad43-b76c-fbe9-2cc8-bb40c4ee4e89@redhat.com>
Precedence: bulk

On Wed, Jan 20, 2021 at 02:45:02PM +0100, Hans de Goede wrote:
> Hi,
> 
> On 1/14/21 3:50 PM, Kyle Tso wrote:
> > v5 https://lore.kernel.org/r/20210105163927.1376770-1-kyletso@google.com
> > 
> > "usb: typec: tcpm: AMS and Collision Avoidance"
> >  - removed the signed-off
> >  - modified the coding style suggested from Heikki
> >  - added FR_SWAP AMS handling
> > 
> > "usb: typec: tcpm: Protocol Error handling"
> >  - removed the signed-off
> >  - modified the coding style suggested from Heikki
> >  - modified more coding style problems (line wrapping limit)
> > 
> > "usb: typec: tcpm: Respond Wait if VDM state machine is running"
> >  - no change
> 
> I've finally gotten around to testing this. I'm happy to
> report that the power-role swapping regression seen in one
> of the older versions of this patch-set is gone.
> 
> So the entire series is:
> Tested-by: Hans de Goede <hdegoede@redhat.com>

Thanks for testing and the review.

Heikki, any thoughts?

thanks,

greg k-h