i2c: sh_mobile: Use new clock calculation formulas for RZ/G2E

Message ID 2a1f0271cc00bf2821aa8b6609c452a9823980f6.1620299581.git.geert+renesas@glider.be
State New
Headers show
Series
  • i2c: sh_mobile: Use new clock calculation formulas for RZ/G2E
Related show

Commit Message

Geert Uytterhoeven May 6, 2021, 11:15 a.m.
When switching the Gen3 SoCs to the new clock calculation formulas, the
match entry for RZ/G2E added in commit 51243b73455f2d12 ("i2c:
sh_mobile: Add support for r8a774c0 (RZ/G2E)") was forgotten.

Fixes: e8a27567509b2439 ("i2c: sh_mobile: use new clock calculation formulas for Gen3")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/i2c/busses/i2c-sh_mobile.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Fabrizio Castro May 6, 2021, 2:45 p.m. | #1
Hi Geert,

Thank you for your patch!

> From: Geert Uytterhoeven <geert+renesas@glider.be>
> Sent: 06 May 2021 12:16
> Uytterhoeven <geert+renesas@glider.be>
> Subject: [PATCH] i2c: sh_mobile: Use new clock calculation formulas for
> RZ/G2E
> 
> When switching the Gen3 SoCs to the new clock calculation formulas, the
> match entry for RZ/G2E added in commit 51243b73455f2d12 ("i2c:
> sh_mobile: Add support for r8a774c0 (RZ/G2E)") was forgotten.
> 
> Fixes: e8a27567509b2439 ("i2c: sh_mobile: use new clock calculation
> formulas for Gen3")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Reviewed-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>

Cheers,
Fab

> ---
>  drivers/i2c/busses/i2c-sh_mobile.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/i2c/busses/i2c-sh_mobile.c b/drivers/i2c/busses/i2c-
> sh_mobile.c
> index 3ae6ca21a02c6b9f..2d2e630fd438712f 100644
> --- a/drivers/i2c/busses/i2c-sh_mobile.c
> +++ b/drivers/i2c/busses/i2c-sh_mobile.c
> @@ -807,7 +807,7 @@ static const struct sh_mobile_dt_config
> r8a7740_dt_config = {
>  static const struct of_device_id sh_mobile_i2c_dt_ids[] = {
>  	{ .compatible = "renesas,iic-r8a73a4", .data = &fast_clock_dt_config
> },
>  	{ .compatible = "renesas,iic-r8a7740", .data = &r8a7740_dt_config },
> -	{ .compatible = "renesas,iic-r8a774c0", .data =
> &fast_clock_dt_config },
> +	{ .compatible = "renesas,iic-r8a774c0", .data =
> &v2_freq_calc_dt_config },
>  	{ .compatible = "renesas,iic-r8a7790", .data =
> &v2_freq_calc_dt_config },
>  	{ .compatible = "renesas,iic-r8a7791", .data =
> &v2_freq_calc_dt_config },
>  	{ .compatible = "renesas,iic-r8a7792", .data =
> &v2_freq_calc_dt_config },
> --
> 2.25.1
Wolfram Sang May 25, 2021, 7:55 p.m. | #2
On Thu, May 06, 2021 at 01:15:40PM +0200, Geert Uytterhoeven wrote:
> When switching the Gen3 SoCs to the new clock calculation formulas, the

> match entry for RZ/G2E added in commit 51243b73455f2d12 ("i2c:

> sh_mobile: Add support for r8a774c0 (RZ/G2E)") was forgotten.

> 

> Fixes: e8a27567509b2439 ("i2c: sh_mobile: use new clock calculation formulas for Gen3")

> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>


Applied to for-next, thanks!
Wolfram Sang May 25, 2021, 7:56 p.m. | #3
On Tue, May 25, 2021 at 09:55:09PM +0200, Wolfram Sang wrote:
> On Thu, May 06, 2021 at 01:15:40PM +0200, Geert Uytterhoeven wrote:

> > When switching the Gen3 SoCs to the new clock calculation formulas, the

> > match entry for RZ/G2E added in commit 51243b73455f2d12 ("i2c:

> > sh_mobile: Add support for r8a774c0 (RZ/G2E)") was forgotten.

> > 

> > Fixes: e8a27567509b2439 ("i2c: sh_mobile: use new clock calculation formulas for Gen3")

> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

> 

> Applied to for-next, thanks!


Geez, too fast again; applied to for-current.

Patch

diff --git a/drivers/i2c/busses/i2c-sh_mobile.c b/drivers/i2c/busses/i2c-sh_mobile.c
index 3ae6ca21a02c6b9f..2d2e630fd438712f 100644
--- a/drivers/i2c/busses/i2c-sh_mobile.c
+++ b/drivers/i2c/busses/i2c-sh_mobile.c
@@ -807,7 +807,7 @@  static const struct sh_mobile_dt_config r8a7740_dt_config = {
 static const struct of_device_id sh_mobile_i2c_dt_ids[] = {
 	{ .compatible = "renesas,iic-r8a73a4", .data = &fast_clock_dt_config },
 	{ .compatible = "renesas,iic-r8a7740", .data = &r8a7740_dt_config },
-	{ .compatible = "renesas,iic-r8a774c0", .data = &fast_clock_dt_config },
+	{ .compatible = "renesas,iic-r8a774c0", .data = &v2_freq_calc_dt_config },
 	{ .compatible = "renesas,iic-r8a7790", .data = &v2_freq_calc_dt_config },
 	{ .compatible = "renesas,iic-r8a7791", .data = &v2_freq_calc_dt_config },
 	{ .compatible = "renesas,iic-r8a7792", .data = &v2_freq_calc_dt_config },