i.MX ProcessorsのトピックRe: imx23. Increase PIXCLOCK in linux 4.x

imx23. Increase PIXCLOCK in linux 4.x

anatolivanov — Fri, 02 Oct 2015 07:36:45 GMT

HI!

When i use fbset to set pixclock value 21000 picosecond and it fails: real clock does not exceed 24 MHz. That is because bit 1 of HW_CLKCTRL_CLKSEQ register is set to 0 so ref_xtal is selected to clock lcdif. I need to select ref_pix and enable PIX fractional divider in HW_CLKCTRL_FRAC with setting it`s value. I think its possible to do this by modify .dts file because driver code file drivers/clk/mxs/clk-imx23.c contains string

clks[lcdif_sel] = mxs_clk_mux("lcdif_sel", CLKSEQ, 1, 1, sel_pix, ARRAY_SIZE(sel_pix));

So can anybody show me way how to do this or advice something else?

Thanks,

Anatol

Re: imx23. Increase PIXCLOCK in linux 4.x

lategoodbye — Sat, 03 Oct 2015 09:41:55 GMT

Hi Anatol,

how do you come to the conclusion to change the dts file instead of the clock driver?

Re: imx23. Increase PIXCLOCK in linux 4.x

anatolivanov — Sat, 03 Oct 2015 10:07:50 GMT

Hi Stefan,

Unfortunately i do not understand well how linux clock framework works and how it communicates with dts blob. I saw in the clk-imx23.c implemented something about multiplexing clock which used to clock lcdif. I thought as its possible to select clock in dts file, so its possible to set mux parametrs.

Am i wrong? Do you know what is "lcdif_sel" in the clocks list?

Re: imx23. Increase PIXCLOCK in linux 4.x

lategoodbye — Sat, 03 Oct 2015 10:49:55 GMT

Hi Anatol,

the device tree source (dts) usually describe the static hardware not the configuration. The clock muxing must be dynamic and should be done by clock driver and the clock framework.

I hope this little gist answer your last question.

Stefan

Edit: Maybe this reference is helpful.

Re: imx23. Increase PIXCLOCK in linux 4.x

anatolivanov — Sat, 03 Oct 2015 11:04:38 GMT

I understand about the gist. I dont understand how to select ref_pix as clock source. I tried to set "clocks = <&clks 4>;" to bind lcdif with ref_pix. But then i got CLK_PIX gated off by bit 31 (CLKGATE == 1) of register HW_CLKCTRL_PIX.

May be i should to use lcdif_sel somehow?

Re: imx23. Increase PIXCLOCK in linux 4.x

lategoodbye — Sat, 03 Oct 2015 11:48:32 GMT

Please do not change dts. I create a patch for the clock driver, which should do the trick. But i can't test it.

Re: imx23. Increase PIXCLOCK in linux 4.x

anatolivanov — Mon, 05 Oct 2015 04:17:12 GMT

Stefan, thank you!

Your method works fine.

Also exist alternative method suggested by Marek Vasut at [PATCH] clk: mxs: source LCD clock from PLL instead of from ref_xtal.

Re: imx23. Increase PIXCLOCK in linux 4.x

fabio_estevam — Mon, 05 Oct 2015 04:25:33 GMT

It would be nice if someone could resend Hector's patch.

Thanks

Re: imx23. Increase PIXCLOCK in linux 4.x

lategoodbye — Mon, 05 Oct 2015 16:47:15 GMT

Is there a reason why the patch was never applied?

Re: imx23. Increase PIXCLOCK in linux 4.x

fabio_estevam — Mon, 05 Oct 2015 16:49:59 GMT

Probably because the clock maintainer was not Cc'ed. It got an Acked-by from Shawn too.

Re: imx23. Increase PIXCLOCK in linux 4.x

fabio_estevam — Mon, 05 Oct 2015 19:58:49 GMT

Yes, that was the case:

[PATCH] clk: mxs: source LCD clock from PLL instead of from ref_xtal

Stefan, do you plan to resend it?

Regards,

Fabio Estevam

Re: imx23. Increase PIXCLOCK in linux 4.x

lategoodbye — Tue, 06 Oct 2015 06:22:45 GMT

Currently i don't have a test environment for i.MX23 or i.MX28.

Do you know any drawbacks of the this patch (higher power consumption etc)?

Is this patch compatible with i.MX280 since it has no lcdif interface?

Re: imx23. Increase PIXCLOCK in linux 4.x

fabio_estevam — Tue, 06 Oct 2015 12:44:44 GMT

Not aware of any drawbacks. Don't have access to mx280 here.