Hi Chris,

[ ... ]

+       ret = extcon_register_notifier(tcphy->pd_extcon, EXTCON_USB,
+                                      &tcphy->event_nb);
+       if (ret) {
+               dev_err(dev, "regitster EXTCON_USB notifer failed\n");
+               return ret;
+       }
+
+       ret = extcon_register_notifier(tcphy->pd_extcon, EXTCON_USB_HOST,
+                                      &tcphy->event_nb);
+       if (ret) {
+               dev_err(dev, "regitster EXTCON_USB_HOST notifer failed\n");
+               return ret;
+       }
+
+       ret = extcon_register_notifier(tcphy->pd_extcon, EXTCON_DISP_DP,
+                                      &tcphy->event_nb);
+       if (ret) {
+               dev_err(dev, "regitster EXTCON_DISP_DP notifer failed\n");
+               return ret;
+       }
+

I don't think you can register multiple notifiers with the same
notifier block. It may work by chance, but at least the 'next' object
in notifier_clock is set in notifier_chain_register(). Best case it
may work, but worst case it might cause a loop in the list of
notifiers.

Guenter