Hi Wolfram, On Mon, Aug 6, 2018 at 12:43 PM Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > From: Masaharu Hayakawa <masaharu.hayakawa.ry@xxxxxxxxxxx> > > According to documentation, setting of PHY registers is unnecessary with > R-Car Gen3. The registers are not even described. So, don't initialize > them. > > Signed-off-by: Masaharu Hayakawa <masaharu.hayakawa.ry@xxxxxxxxxxx> > [wsa: updated commit message] > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Thanks for your patch! > Tested with a Renesas R-Car M3N Salvator-XS board. I did a platform suspend to > check if the controller and PHY work properly after resume. "platform suspend"? Is that s2ram aka a full PSCI system suspend? I guess not, as it is supposed to fail without PCA9654 suspend/resume support... So more information is needed to convince me ;-) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html