All of lore.kernel.org
 help / color / mirror / Atom feed
* [8/8] plarform: x86: intel_cht_int33fe: Remove the old connections for the muxes
@ 2018-08-31 14:20 Heikki Krogerus
  0 siblings, 0 replies; only message in thread
From: Heikki Krogerus @ 2018-08-31 14:20 UTC (permalink / raw)
  To: Hans de Goede, Greg Kroah-Hartman
  Cc: Andy Shevchenko, Darren Hart, platform-driver-x86, linux-usb

USB Type-C class driver now expects the muxes to be always
assigned to the ports and not controllers, so the
connections for the mux and fusb302 can be removed.

Signed-off-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
---
 drivers/platform/x86/intel_cht_int33fe.c | 19 ++++++-------------
 1 file changed, 6 insertions(+), 13 deletions(-)

diff --git a/drivers/platform/x86/intel_cht_int33fe.c b/drivers/platform/x86/intel_cht_int33fe.c
index 4a41c546ce2f..f4449d546459 100644
--- a/drivers/platform/x86/intel_cht_int33fe.c
+++ b/drivers/platform/x86/intel_cht_int33fe.c
@@ -35,7 +35,7 @@ struct cht_int33fe_data {
 	struct i2c_client *fusb302;
 	struct i2c_client *pi3usb30532;
 	/* Contain a list-head must be per device */
-	struct device_connection connections[6];
+	struct device_connection connections[4];
 };
 
 /*
@@ -176,27 +176,20 @@ static int cht_int33fe_probe(struct i2c_client *client)
 			return -EPROBE_DEFER; /* Wait for i2c-adapter to load */
 	}
 
-	data->connections[0].endpoint[0] = "i2c-fusb302";
+	data->connections[0].endpoint[0] = "port0";
 	data->connections[0].endpoint[1] = "i2c-pi3usb30532";
 	data->connections[0].id = "typec-switch";
-	data->connections[1].endpoint[0] = "i2c-fusb302";
+	data->connections[1].endpoint[0] = "port0";
 	data->connections[1].endpoint[1] = "i2c-pi3usb30532";
 	data->connections[1].id = "idff01m01";
 
-	data->connections[2].endpoint[0] = "port0";
-	data->connections[2].endpoint[1] = "i2c-pi3usb30532";
-	data->connections[2].id = "typec-switch";
-	data->connections[3].endpoint[0] = "port0";
-	data->connections[3].endpoint[1] = "i2c-pi3usb30532";
-	data->connections[3].id = "idff01m01";
-
 	/* Only adding connection for role switch if UDC exists */
 	udc = pci_get_class(PCI_CLASS_SERIAL_USB_DEVICE, NULL);
 	if (udc) {
 		pci_dev_put(udc);
-		data->connections[4].endpoint[0] = "i2c-fusb302";
-		data->connections[4].endpoint[1] = "intel_xhci_usb_sw-role-switch";
-		data->connections[4].id = "usb-role-switch";
+		data->connections[2].endpoint[0] = "i2c-fusb302";
+		data->connections[2].endpoint[1] = "intel_xhci_usb_sw-role-switch";
+		data->connections[2].id = "usb-role-switch";
 	}
 
 	device_connections_add(data->connections);

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2018-08-31 14:20 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-31 14:20 [8/8] plarform: x86: intel_cht_int33fe: Remove the old connections for the muxes Heikki Krogerus

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.