Hi Greg,

> -----Original Message-----
> From: Greg Kroah-Hartman <[email protected]>
> Sent: Thursday, July 16, 2020 12:05 AM
> To: Mani, Rajmohan <[email protected]>
> Cc: Darren Hart <[email protected]>; Andy Shevchenko
> <[email protected]>; Mika Westerberg
> <[email protected]>; Dmitry Torokhov
> <[email protected]>; Lee Jones <[email protected]>; Ayman
> Bagabas <[email protected]>; Masahiro Yamada
> <[email protected]>; Joseph, Jithu <[email protected]>; Blaž
> Hrastnik <[email protected]>; Srinivas Pandruvada
> <[email protected]>; [email protected];
> [email protected]; Heikki Krogerus
> <[email protected]>; [email protected];
> [email protected]; [email protected]
> Subject: Re: [PATCH 2/2] usb: typec: intel_pmc_mux: Check the port status
> before connect
> 
> On Wed, Jul 15, 2020 at 05:33:10PM -0700, Rajmohan Mani wrote:
> > From: Heikki Krogerus <[email protected]>
> >
> > The PMC microcontroller that we use for configuration, does not supply
> > any status information back. For port status we need to talk to
> > another controller on the board called IOM (I/O manager).
> >
> > By checking the port status before configuring the muxes, we can make
> > sure that we do not reconfigure the port after bootup when the system
> > firmware (for example BIOS) has already configured it.
> >
> > Using the status information also to check if DisplayPort HPD is still
> > asserted when the cable plug is disconnected, and clearing it if it
> > is.
> >
> > Signed-off-by: Heikki Krogerus <[email protected]>
> 
> You can't just forward on patches from others without also adding your
> signed-off-by to them, right?
> 

Sorry I missed this.

> Please fix up this series and try again.
> 

Ack. Will fix this with v2.

> thanks,
> 
> greg k-h

Reply via email to