usb: typec: displayport: Export probe and remove functions
Greg Kroah-Hartman
maintainerPATCH v2 6/7 usb: typec: displayport: Export probe and remove functions
Quoted 42 lines · Heikki Krogerus
Quoted 35 lines · Heikki Krogerus
Quoted 34 lines · Ajay Gupta
Hi Heikki,
Quoted 32 lines
-----Original Message-----
From: [email protected] <[email protected]> On Behalf Of Heikki Krogerus
Sent: Monday, April 15, 2019 5:10 AM
To: Greg Kroah-Hartman <[email protected]>
Cc: Ajay Gupta <[email protected]>; [email protected]
Subject: [PATCH v2 6/7] usb: typec: displayport: Export probe and remove functionsFrom: Ajay Gupta <[email protected]>
VirtualLink standard extends the DisplayPort Alt Mode by utilizing also the USB 2 pins on the USB Type-C connector.
It uses the same messages as DisplayPort, but not the DP SVID. At the time of writing, USB IF has not assigned a Standard ID (SID) for VirtualLink, so the manufacturers of VirtualLink adapters use their Vendor IDs as the SVID.Since the SVID specific communication is exactly the same as with DisplayPort alternate mode, there is no need to implement separate driver for VirtualLink. We'll handle the current VirtualLink adapters with probe drivers, and once there is SVID assigned for it, we add it to the displayport alt mode driver.
To support probing drivers, exporting the probe and remove functions, and also changing the
DP_HEADERhelper macro to use the SVID of the alternate mode device instead of the DisplayPort alt mode SVID.Suggested-by Heikki Krogerus <[email protected]>Signed-off-by Ajay Gupta <[email protected]>Signed-off-by Heikki Krogerus <[email protected]>--- drivers/usb/typec/altmodes/displayport.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-)Quoted 5 lines
Quoted 2 lines
Looks like you missed adding
displayport.hfile. It is available in original patch below,
https://marc.info/?l=linux-usb&m=155492587224379&w=2That's the one I picked, and I did compile the driver as well.. I'm not sure what I've done, but I'll resend the series.
Quoted 6 lines
It looks like I broke the patch after I did a small modification to it. I had changed your "From" address so it matches your singed-of-by address. My compile test passed because the header was an untracked file :-)
Sorry again. Let's do one more test round, and resend.
Ok, dropping this whole series from my queue now, thanks.
greg k-h