Re: [PATCH] clk: imx: Add imx8dxl clk driver
From: Abel Vesa
Date: Tue Jan 18 2022 - 05:23:50 EST
On 22-01-18 12:18:18, Abel Vesa wrote:
> On 21-12-16 10:50:23, Stephen Boyd wrote:
> > Quoting Abel Vesa (2021-11-10 09:56:40)
> > > diff --git a/drivers/clk/imx/clk-imx8qxp.c b/drivers/clk/imx/clk-imx8qxp.c
> > > index c53a688d8ccc..7f4893706cc2 100644
> > > --- a/drivers/clk/imx/clk-imx8qxp.c
> > > +++ b/drivers/clk/imx/clk-imx8qxp.c
> > > @@ -297,6 +297,7 @@ static const struct of_device_id imx8qxp_match[] = {
> > > { .compatible = "fsl,scu-clk", },
> > > { .compatible = "fsl,imx8qxp-clk", &imx_clk_scu_rsrc_imx8qxp, },
> > > { .compatible = "fsl,imx8qm-clk", &imx_clk_scu_rsrc_imx8qm, },
> > > + { .compatible = "fsl,imx8dxl-clk", &imx_clk_scu_rsrc_imx8dxl, },
> >
> > Can you keep this sorted on compatible string? Then we don't have to
> > worry about conflicts as much.
> >
>
> Yep. Will resend.
Actually, v2 is right here:
https://lore.kernel.org/linux-arm-kernel/1639747533-9778-1-git-send-email-abel.vesa@xxxxxxx/
You ACK'ed it already. And it adds the dxl compatible in the right spot.
>
> > > { /* sentinel */ }
> > > };
> > >