Hi Wolfram-san, > From: Wolfram Sang, Sent: Friday, October 26, 2018 7:58 AM <snip> > > static const struct dev_pm_ops usbhsc_pm_ops = { > > .suspend = usbhsc_suspend, > > .resume = usbhsc_resume, > > Unrelated to this patch, but I wonder right now: is there a reason not > to use SET_SYSTEM_SLEEP_PM_OPS here? Shimoda-san? I don't know why because this code is contributed from Morimoto-san. I'm guessing this code seems to come from sh_eth.c or i2c-sh_mobile.c and we don't have the SET_SYSTEM_SLEEP_PM_OPS macro at 2009. Morimoto-san contributed this code at 2010, but it seems not to realize we have such macro. Anyway, I'll try to use SET_SYSTEM_SLEEP_PM_OPS on the renesas_usbhs driver. Best regards, Yoshihiro Shimoda