[PATCH 04/27] imx: imx8ulp: clock: Add clock support for i3c controller

[Fabio Estevam]

On Mon, Feb 14, 2022 at 9:08 AM Peng Fan (OSS) <peng.fan at oss.nxp.com> wrote:
>
> From: Clark Wang <xiaoning.wang at nxp.com>
>
> Add i3c controller clock enable/disable function for imx8ulp.
>
> Reviewed-by: Ye Li <ye.li at nxp.com>
> Signed-off-by: Clark Wang <xiaoning.wang at nxp.com>
> Signed-off-by: Peng Fan <peng.fan at nxp.com>
> ---
>  arch/arm/mach-imx/imx8ulp/clock.c | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
>
> diff --git a/arch/arm/mach-imx/imx8ulp/clock.c b/arch/arm/mach-imx/imx8ulp/clock.c
> index 91580b2c29..6f52ccee34 100644
> --- a/arch/arm/mach-imx/imx8ulp/clock.c
> +++ b/arch/arm/mach-imx/imx8ulp/clock.c
> @@ -237,6 +237,32 @@ u32 imx_get_i2cclk(u32 i2c_num)
>  }
>  #endif
>
> +#if IS_ENABLED(CONFIG_SYS_I2C_IMX_I3C)
> +int enable_i3c_clk(unsigned char enable, u32 i3c_num)
> +{
> +       if (i3c_num != 8)

What does this magic 8 mean? Please replace with some more meaningful define.

> +u32 imx_get_i3cclk(u32 i3c_num)
> +{
> +       if (i3c_num != 8)

Same here.