Re: [PATCH 2/2] pinctrl: sh-pfc: r8a77990: Add voltage switch operations for SDHI
On Mon, Nov 5, 2018 at 10:40 PM Marek Vasut wrote: > From: Takeshi Kihara > > This patch supports the {get,set}_io_voltage operations of SDHI. > > This operates the IOCTRL30 register on the R8A77990 SoC and makes > 1.8V/3.3V signal voltage switch possible. > > Signed-off-by: Takeshi Kihara > Signed-off-by: Marek Vasut Reviewed-by: Geert Uytterhoeven i.e. will queue in sh-pfc-for-v4.21. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org 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
Re: [PATCH 2/2] pinctrl: sh-pfc: r8a77990: Add voltage switch operations for SDHI
On 11/06/2018 12:15 PM, Wolfram Sang wrote: > On Mon, Nov 05, 2018 at 10:40:12PM +0100, Marek Vasut wrote: >> From: Takeshi Kihara >> >> This patch supports the {get,set}_io_voltage operations of SDHI. >> >> This operates the IOCTRL30 register on the R8A77990 SoC and makes >> 1.8V/3.3V signal voltage switch possible. >> >> Signed-off-by: Takeshi Kihara >> Signed-off-by: Marek Vasut >> Cc: Geert Uytterhoeven >> Cc: Simon Horman >> Cc: Wolfram Sang >> Cc: Yoshihiro Shimoda >> Cc: linux-renesas-soc@vger.kernel.org > > Looks good from a high-level point of view. Didn't check the actual > values. I trust Shimoda-san and Geert for that: > > Acked-by: Wolfram Sang > I also checked them in U-Boot :) -- Best regards, Marek Vasut
Re: [PATCH 2/2] pinctrl: sh-pfc: r8a77990: Add voltage switch operations for SDHI
On Mon, Nov 05, 2018 at 10:40:12PM +0100, Marek Vasut wrote: > From: Takeshi Kihara > > This patch supports the {get,set}_io_voltage operations of SDHI. > > This operates the IOCTRL30 register on the R8A77990 SoC and makes > 1.8V/3.3V signal voltage switch possible. > > Signed-off-by: Takeshi Kihara > Signed-off-by: Marek Vasut > Cc: Geert Uytterhoeven > Cc: Simon Horman > Cc: Wolfram Sang > Cc: Yoshihiro Shimoda > Cc: linux-renesas-soc@vger.kernel.org Looks good from a high-level point of view. Didn't check the actual values. I trust Shimoda-san and Geert for that: Acked-by: Wolfram Sang signature.asc Description: PGP signature
RE: [PATCH 2/2] pinctrl: sh-pfc: r8a77990: Add voltage switch operations for SDHI
Hello Marek-san, > From: Marek Vasut, Sent: Tuesday, November 6, 2018 6:40 AM > > From: Takeshi Kihara > > This patch supports the {get,set}_io_voltage operations of SDHI. > > This operates the IOCTRL30 register on the R8A77990 SoC and makes > 1.8V/3.3V signal voltage switch possible. > > Signed-off-by: Takeshi Kihara > Signed-off-by: Marek Vasut > Cc: Geert Uytterhoeven > Cc: Simon Horman > Cc: Wolfram Sang > Cc: Yoshihiro Shimoda > Cc: linux-renesas-soc@vger.kernel.org > --- Thank you for the patch! Reviewed-by: Yoshihiro Shimoda Best regards, Yoshihiro Shimoda