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

