[U-Boot] [PATCH v1 06/15] dm: clk: imx: Add support for controlling imx6q clocks via Driver Model
Lukasz Majewski
lukma at denx.de
Mon Jan 21 14:19:26 UTC 2019
Hi Fabio,
> Hi Lukasz,
>
> On Sat, Jan 19, 2019 at 7:15 AM Lukasz Majewski <lukma at denx.de> wrote:
>
> > +static ulong imx6q_clk_get_rate(struct clk *clk)
> > +{
> > + ulong rate = 0;
> > +
> > + debug("%s(#%lu)\n", __func__, clk->id);
> > +
> > + switch (clk->id) {
> > + case IMX6QDL_CLK_ECSPI1:
> > + case IMX6QDL_CLK_ECSPI2:
> > + case IMX6QDL_CLK_ECSPI3:
> > + case IMX6QDL_CLK_ECSPI4:
> > + return imx6_get_cspi_clk();
> > +
> > + case IMX6QDL_CLK_USDHC1:
> > + case IMX6QDL_CLK_USDHC2:
> > + case IMX6QDL_CLK_USDHC3:
> > + case IMX6QDL_CLK_USDHC4:
> > + return imx6_get_usdhc_clk(clk->id -
> > IMX6QDL_CLK_USDHC1);
>
> I don't think this scales well as this needs to grow for all other
> peripherals and for each port instance.
The rationale regarding this approach:
1. Reuse the clock.c code for iMX6Q as much as possible.
2, This code is based on the clk-imx8q.c file - hence the question
why the Linux clock API was not ported for this new SoC?.
>
> If we are adding a clock driver for mx6, why don't we add it just like
> the kernel one?
I can try to port the Linux code, but IMHO it would be feasible to port
only relevant (ECSPI, USDHC) parts of it (not all as I cannot test it
all properly).
>
> Barebox imports the clock driver from the kernel and it is much
> cleaner:
> https://git.pengutronix.de/cgit/barebox/tree/drivers/clk/imx/clk-imx6.c
Yes, it has been trimmed (...a bit...) when compared to original
v4.20 :-) .
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190121/5d208d98/attachment.sig>
More information about the U-Boot
mailing list