Quoting Peter Chen (2016-06-28 23:26:00)

On Sun, Jun 26, 2016 at 12:28:26AM -0700, Stephen Boyd wrote:

quoted

@@ -187,6 +190,7 @@ struct hw_bank {
  * @test_mode: the selected test mode
  * @platdata: platform specific information supplied by parent device
  * @vbus_active: is VBUS active
+ * @ulpi: pointer to ULPI device, if any

One more kernel-doc

Done.

quoted

  * @phy: pointer to PHY, if any
  * @usb_phy: pointer to USB PHY, if any and if using the USB PHY framework
  * @hcd: pointer to usb_hcd for ehci host driver

@@ -236,6 +240,10 @@ struct ci_hdrc {
 
      struct ci_hdrc_platform_data    *platdata;
      int                             vbus_active;
+#ifdef CONFIG_USB_CHIPIDEA_ULPI
+     struct ulpi                     *ulpi;
+     struct ulpi_ops                 ulpi_ops;
+#endif
      struct phy                      *phy;
      /* old usb_phy interface */
      struct usb_phy                  *usb_phy;

@@ -418,6 +426,17 @@ static inline bool ci_otg_is_fsm_mode(struct ci_hdrc *ci)
 #endif
 }
 

Others are ok, but I can't accept you change current PHY initialization
at your previous patch, so you may need to refine this patch a little.

Ok. No problem.