Hi Ulrich Thank you for your patch > In preparation for DT conversion to reduce reliance on platform device > callbacks. > > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> > --- (snip) > @@ -186,6 +192,8 @@ struct renesas_usbhs_platform_info { > * driver use these param for some register > */ > struct renesas_usbhs_driver_param driver_param; > + > + struct usb_phy *phy; > }; Using renesas_usbhs_platform_info for *phy is not good idea from my point of view. Because its memory is located on platform side. (this is the reason why usbhs_probe() using memcpy() for getting info->xxx) I guess struct usbhs_priv is good place, and you can use usbhs_pdev_to_priv() for this purpose ? Best regards --- Kuninori Morimoto -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html