* [PATCH] video: da8xx-fb: reset LCDC only if functional clock changes with DVFS
@ 2011-12-21 7:39 Manjunathappa, Prakash
2011-12-21 10:59 ` [PATCH] video: da8xx-fb: reset LCDC only if functional clock Madhvapathi Sriram
2011-12-21 11:11 ` Manjunathappa, Prakash
0 siblings, 2 replies; 3+ messages in thread
From: Manjunathappa, Prakash @ 2011-12-21 7:39 UTC (permalink / raw)
To: linux-fbdev
LCDC functional clock may or may not be derived from CPU/MPU DPLL,
For example,
AM335x => Separate independent DPLL for LCDC
Davinci => Same DPLL as MPU
So, on platforms where LCDC functional clock is not derived from CPU/MPU
PLL it is not required to reset LCDC module, if CPU frequency does not
change with DVFS.
This patch adds check to do reset only if functional clock changes
between pre and post notifier callbacks with DVFS.
Signed-off-by: Manjunathappa, Prakash <prakash.pm@ti.com>
---
drivers/video/da8xx-fb.c | 13 ++++++++-----
1 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c
index 29577bf..1334a68 100644
--- a/drivers/video/da8xx-fb.c
+++ b/drivers/video/da8xx-fb.c
@@ -161,6 +161,7 @@ struct da8xx_fb_par {
int vsync_timeout;
#ifdef CONFIG_CPU_FREQ
struct notifier_block freq_transition;
+ unsigned int lcd_fck_rate;
#endif
void (*panel_power_ctrl)(int);
};
@@ -840,11 +841,12 @@ static int lcd_da8xx_cpufreq_transition(struct notifier_block *nb,
struct da8xx_fb_par *par;
par = container_of(nb, struct da8xx_fb_par, freq_transition);
- if (val = CPUFREQ_PRECHANGE) {
- lcd_disable_raster();
- } else if (val = CPUFREQ_POSTCHANGE) {
- lcd_calc_clk_divider(par);
- lcd_enable_raster();
+ if (val = CPUFREQ_POSTCHANGE) {
+ if (par->lcd_fck_rate != clk_get_rate(par->lcdc_clk)) {
+ lcd_disable_raster();
+ lcd_calc_clk_divider(par);
+ lcd_enable_raster();
+ }
}
return 0;
@@ -1137,6 +1139,7 @@ static int __devinit fb_probe(struct platform_device *device)
par = da8xx_fb_info->par;
par->lcdc_clk = fb_clk;
+ par->lcd_fck_rate = clk_get_rate(fb_clk);
par->pxl_clk = lcdc_info->pxl_clk;
if (fb_pdata->panel_power_ctrl) {
par->panel_power_ctrl = fb_pdata->panel_power_ctrl;
--
1.7.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] video: da8xx-fb: reset LCDC only if functional clock
2011-12-21 7:39 [PATCH] video: da8xx-fb: reset LCDC only if functional clock changes with DVFS Manjunathappa, Prakash
@ 2011-12-21 10:59 ` Madhvapathi Sriram
2011-12-21 11:11 ` Manjunathappa, Prakash
1 sibling, 0 replies; 3+ messages in thread
From: Madhvapathi Sriram @ 2011-12-21 10:59 UTC (permalink / raw)
To: linux-fbdev
Prakash,
On Wed, Dec 21, 2011 at 12:57 PM, Manjunathappa, Prakash
<prakash.pm@ti.com> wrote:
>
> LCDC functional clock may or may not be derived from CPU/MPU DPLL,
> For example,
> AM335x => Separate independent DPLL for LCDC
> Davinci => Same DPLL as MPU
>
> So, on platforms where LCDC functional clock is not derived from CPU/MPU
> PLL it is not required to reset LCDC module, if CPU frequency does not
> change with DVFS.
Is it not the case that the CPU frequency always changes during DVFS?
This sentence does not strike me straight.
Did you mean that if the frequency of the PLL (from which LCDC
derives) does not change with DVFS, then LCDC module is not required
to be reset?
>
> This patch adds check to do reset only if functional clock changes
> between pre and post notifier callbacks with DVFS.
>
> Signed-off-by: Manjunathappa, Prakash <prakash.pm@ti.com>
> ---
> drivers/video/da8xx-fb.c | 13 ++++++++-----
> 1 files changed, 8 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c
> index 29577bf..1334a68 100644
> --- a/drivers/video/da8xx-fb.c
> +++ b/drivers/video/da8xx-fb.c
> @@ -161,6 +161,7 @@ struct da8xx_fb_par {
> int vsync_timeout;
> #ifdef CONFIG_CPU_FREQ
> struct notifier_block freq_transition;
> + unsigned int lcd_fck_rate;
> #endif
> void (*panel_power_ctrl)(int);
> };
> @@ -840,11 +841,12 @@ static int lcd_da8xx_cpufreq_transition(struct notifier_block *nb,
> struct da8xx_fb_par *par;
>
> par = container_of(nb, struct da8xx_fb_par, freq_transition);
> - if (val = CPUFREQ_PRECHANGE) {
> - lcd_disable_raster();
> - } else if (val = CPUFREQ_POSTCHANGE) {
> - lcd_calc_clk_divider(par);
> - lcd_enable_raster();
> + if (val = CPUFREQ_POSTCHANGE) {
> + if (par->lcd_fck_rate != clk_get_rate(par->lcdc_clk)) {
> + lcd_disable_raster();
> + lcd_calc_clk_divider(par);
> + lcd_enable_raster();
> + }
> }
>
> return 0;
> @@ -1137,6 +1139,7 @@ static int __devinit fb_probe(struct platform_device *device)
>
> par = da8xx_fb_info->par;
> par->lcdc_clk = fb_clk;
> + par->lcd_fck_rate = clk_get_rate(fb_clk);
> par->pxl_clk = lcdc_info->pxl_clk;
> if (fb_pdata->panel_power_ctrl) {
> par->panel_power_ctrl = fb_pdata->panel_power_ctrl;
> --
> 1.7.1
>
> _______________________________________________
> Davinci-linux-open-source mailing list
> Davinci-linux-open-source@linux.davincidsp.com
> http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [PATCH] video: da8xx-fb: reset LCDC only if functional clock
2011-12-21 7:39 [PATCH] video: da8xx-fb: reset LCDC only if functional clock changes with DVFS Manjunathappa, Prakash
2011-12-21 10:59 ` [PATCH] video: da8xx-fb: reset LCDC only if functional clock Madhvapathi Sriram
@ 2011-12-21 11:11 ` Manjunathappa, Prakash
1 sibling, 0 replies; 3+ messages in thread
From: Manjunathappa, Prakash @ 2011-12-21 11:11 UTC (permalink / raw)
To: linux-fbdev
Hi Sriram,
On Wed, Dec 21, 2011 at 16:17:01, Madhvapathi Sriram wrote:
> Prakash,
>
> On Wed, Dec 21, 2011 at 12:57 PM, Manjunathappa, Prakash
> <prakash.pm@ti.com> wrote:
> >
> > LCDC functional clock may or may not be derived from CPU/MPU DPLL,
> > For example,
> > AM335x => Separate independent DPLL for LCDC
> > Davinci => Same DPLL as MPU
> >
> > So, on platforms where LCDC functional clock is not derived from CPU/MPU
> > PLL it is not required to reset LCDC module, if CPU frequency does not
> > change with DVFS.
> Is it not the case that the CPU frequency always changes during DVFS?
> This sentence does not strike me straight.
> Did you mean that if the frequency of the PLL (from which LCDC
> derives) does not change with DVFS, then LCDC module is not required
> to be reset?
Ok I will re-phrase it as below:
So, on platforms where LCDC functional clock is not derived from CPU/MPU
PLL it is not required to reset LCDC module as its functional clock does not change with DVFS.
> >
> > This patch adds check to do reset only if functional clock changes
> > between pre and post notifier callbacks with DVFS.
> >
> > Signed-off-by: Manjunathappa, Prakash <prakash.pm@ti.com>
> > ---
> > drivers/video/da8xx-fb.c | 13 ++++++++-----
> > 1 files changed, 8 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c
> > index 29577bf..1334a68 100644
> > --- a/drivers/video/da8xx-fb.c
> > +++ b/drivers/video/da8xx-fb.c
> > @@ -161,6 +161,7 @@ struct da8xx_fb_par {
> > int vsync_timeout;
> > #ifdef CONFIG_CPU_FREQ
> > struct notifier_block freq_transition;
> > + unsigned int lcd_fck_rate;
> > #endif
> > void (*panel_power_ctrl)(int);
> > };
> > @@ -840,11 +841,12 @@ static int lcd_da8xx_cpufreq_transition(struct notifier_block *nb,
> > struct da8xx_fb_par *par;
> >
> > par = container_of(nb, struct da8xx_fb_par, freq_transition);
> > - if (val = CPUFREQ_PRECHANGE) {
> > - lcd_disable_raster();
> > - } else if (val = CPUFREQ_POSTCHANGE) {
> > - lcd_calc_clk_divider(par);
> > - lcd_enable_raster();
> > + if (val = CPUFREQ_POSTCHANGE) {
> > + if (par->lcd_fck_rate != clk_get_rate(par->lcdc_clk)) {
> > + lcd_disable_raster();
> > + lcd_calc_clk_divider(par);
> > + lcd_enable_raster();
> > + }
> > }
> >
> > return 0;
> > @@ -1137,6 +1139,7 @@ static int __devinit fb_probe(struct platform_device *device)
> >
> > par = da8xx_fb_info->par;
> > par->lcdc_clk = fb_clk;
> > + par->lcd_fck_rate = clk_get_rate(fb_clk);
> > par->pxl_clk = lcdc_info->pxl_clk;
> > if (fb_pdata->panel_power_ctrl) {
> > par->panel_power_ctrl = fb_pdata->panel_power_ctrl;
> > --
> > 1.7.1
> >
> > _______________________________________________
> > Davinci-linux-open-source mailing list
> > Davinci-linux-open-source@linux.davincidsp.com
> > http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-12-21 11:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-21 7:39 [PATCH] video: da8xx-fb: reset LCDC only if functional clock changes with DVFS Manjunathappa, Prakash
2011-12-21 10:59 ` [PATCH] video: da8xx-fb: reset LCDC only if functional clock Madhvapathi Sriram
2011-12-21 11:11 ` Manjunathappa, Prakash
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).