On 12.11.20 11:51, Oliver Graute wrote:
> This patch fixes this clk issue on I2C on imx8qm
>
> => i2c bus
> Bus 3: i2c@5a830000
> => i2c dev 3
> Setting bus to 3
> Failed to enable ipg clk
> Failure changing bus number (-524)
>
> Signed-off-by: Oliver Graute <oliver.gra...@kococonnector.com>
> Cc: Stefano Babic <sba...@denx.de>
> Cc: Peng Fan <peng....@nxp.com>
> Cc: uboot-imx <uboot-...@nxp.com>
> ---
> drivers/clk/imx/clk-imx8qm.c | 24 ++++++++++++++++++++++++
> 1 file changed, 24 insertions(+)
>
> diff --git a/drivers/clk/imx/clk-imx8qm.c b/drivers/clk/imx/clk-imx8qm.c
> index 54fb09fda4..7e466d630a 100644
> --- a/drivers/clk/imx/clk-imx8qm.c
> +++ b/drivers/clk/imx/clk-imx8qm.c
> @@ -53,19 +53,27 @@ ulong imx8_clk_get_rate(struct clk *clk)
> resource = SC_R_A53;
> pm_clk = SC_PM_CLK_CPU;
> break;
> + case IMX8QM_I2C0_IPG_CLK:
> case IMX8QM_I2C0_CLK:
> + case IMX8QM_I2C0_DIV:
> resource = SC_R_I2C_0;
> pm_clk = SC_PM_CLK_PER;
> break;
> + case IMX8QM_I2C1_IPG_CLK:
> case IMX8QM_I2C1_CLK:
> + case IMX8QM_I2C1_DIV:
> resource = SC_R_I2C_1;
> pm_clk = SC_PM_CLK_PER;
> break;
> + case IMX8QM_I2C2_IPG_CLK:
> case IMX8QM_I2C2_CLK:
> + case IMX8QM_I2C2_DIV:
> resource = SC_R_I2C_2;
> pm_clk = SC_PM_CLK_PER;
> break;
> + case IMX8QM_I2C3_IPG_CLK:
> case IMX8QM_I2C3_CLK:
> + case IMX8QM_I2C3_DIV:
> resource = SC_R_I2C_3;
> pm_clk = SC_PM_CLK_PER;
> break;
> @@ -148,19 +156,27 @@ ulong imx8_clk_set_rate(struct clk *clk, unsigned long
> rate)
> debug("%s(#%lu), rate: %lu\n", __func__, clk->id, rate);
>
> switch (clk->id) {
> + case IMX8QM_I2C0_IPG_CLK:
> case IMX8QM_I2C0_CLK:
> + case IMX8QM_I2C0_DIV:
> resource = SC_R_I2C_0;
> pm_clk = SC_PM_CLK_PER;
> break;
> + case IMX8QM_I2C1_IPG_CLK:
> case IMX8QM_I2C1_CLK:
> + case IMX8QM_I2C1_DIV:
> resource = SC_R_I2C_1;
> pm_clk = SC_PM_CLK_PER;
> break;
> + case IMX8QM_I2C2_IPG_CLK:
> case IMX8QM_I2C2_CLK:
> + case IMX8QM_I2C2_DIV:
> resource = SC_R_I2C_2;
> pm_clk = SC_PM_CLK_PER;
> break;
> + case IMX8QM_I2C3_IPG_CLK:
> case IMX8QM_I2C3_CLK:
> + case IMX8QM_I2C3_DIV:
> resource = SC_R_I2C_3;
> pm_clk = SC_PM_CLK_PER;
> break;
> @@ -242,19 +258,27 @@ int __imx8_clk_enable(struct clk *clk, bool enable)
> debug("%s(#%lu)\n", __func__, clk->id);
>
> switch (clk->id) {
> + case IMX8QM_I2C0_IPG_CLK:
> case IMX8QM_I2C0_CLK:
> + case IMX8QM_I2C0_DIV:
> resource = SC_R_I2C_0;
> pm_clk = SC_PM_CLK_PER;
> break;
> + case IMX8QM_I2C1_IPG_CLK:
> case IMX8QM_I2C1_CLK:
> + case IMX8QM_I2C1_DIV:
> resource = SC_R_I2C_1;
> pm_clk = SC_PM_CLK_PER;
> break;
> + case IMX8QM_I2C2_IPG_CLK:
> case IMX8QM_I2C2_CLK:
> + case IMX8QM_I2C2_DIV:
> resource = SC_R_I2C_2;
> pm_clk = SC_PM_CLK_PER;
> break;
> + case IMX8QM_I2C3_IPG_CLK:
> case IMX8QM_I2C3_CLK:
> + case IMX8QM_I2C3_DIV:
> resource = SC_R_I2C_3;
> pm_clk = SC_PM_CLK_PER;
> break;
>
Reviewed-by: Stefano Babic <sba...@denx.de>
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=====================================================================
