On Mon, Mar 25, 2019 at 10:16:36PM +0000, Ajay Gupta wrote:
> Hi Heikki,
> 
> > -----Original Message-----
> > From: [email protected] <[email protected]> On
> > Behalf Of Heikki Krogerus
> > Sent: Wednesday, March 20, 2019 4:45 AM
> > To: Ajay Gupta <[email protected]>; Michael Hsu <[email protected]>
> > Cc: [email protected]
> > Subject: [PATCH v2 0/2] usb: typec: ucsi: Support for DP alt mode
> > 
> > Hi,
> > 
> > This is the second version of my proposal to add alt mode handling to ucsi 
> > driver.
> > The problem in the first version was that I was not using the port alt mode 
> > that
> > was associated with the partner alt mode (and vise versa), but I was instead
> > searching the alt mode objects in every case separately.
> > 
> > So in this version I'm now always using the already associated alternate 
> > mode
> > partner in every case, and as far as the UCSI driver (or any other USB 
> > Type-C
> > driver) is concerned, that is all that it needs to do. The problem of 
> > actually
> > associating the correct alt mode partner object belongs to the Type-C 
> > subsystem
> > core code, and any improvements to that code are out side of the scope of 
> > this
> > series.
> > 
> > I've prepared a patch that allows the alt mode drivers to find the correct 
> > port alt
> > mode for the partner alt modes. That should cover SVIDs that define multiple
> > modes (note, DisplayPort Alt mode is not one of those). But as I mentioned
> > above, that is not related to this series.
> Thanks for the new set. It looks good to me.
> I tested it with UCSI CCG on NVIDIA GPU and it works.
> 
> Tested-By: Ajay Gupta <[email protected]>

Thank you!

Br,

-- 
heikki

Reply via email to