[PATCH v2 6/8] omapdss: Features: Fix some parameter ranges

2013-03-27 Thread Archit Taneja 
On Wednesday 27 March 2013 01:03 PM, Tomi Valkeinen wrote:
> On 2013-03-26 15:45, Archit Taneja wrote:
>> Increase the DSS_FCLK and DSI_FCLK max supported frequencies, these come 
>> because
>> some frequencies were increased from OMAP5 ES1 to OMAP5 ES2. We support only
>> OMAP5 ES2 in the kernel, so replace the ES1 values with ES2 values. Increase 
>> the
>> DSI PLL Fint range, this was previously just copied from the OMAP4 param 
>> range
>> struct.
>>
>> Fix the maximum DSS_FCLK on OMAP2, it's 133 Mhz according to the TRM.
>>
>> Signed-off-by: Archit Taneja 
>> ---
>>   drivers/video/omap2/dss/dss_features.c |8 
>>   1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/video/omap2/dss/dss_features.c 
>> b/drivers/video/omap2/dss/dss_features.c
>> index 7f791ae..77dbe0c 100644
>> --- a/drivers/video/omap2/dss/dss_features.c
>> +++ b/drivers/video/omap2/dss/dss_features.c
>> @@ -414,7 +414,7 @@ static const char * const omap5_dss_clk_source_names[] = 
>> {
>>   };
>>
>>   static const struct dss_param_range omap2_dss_param_range[] = {
>> -[FEAT_PARAM_DSS_FCK]= { 0, 17300 },
>> +[FEAT_PARAM_DSS_FCK]= { 0, 13300 },
>>  [FEAT_PARAM_DSS_PCD]= { 2, 255 },
>>  [FEAT_PARAM_DSIPLL_REGN]= { 0, 0 },
>>  [FEAT_PARAM_DSIPLL_REGM]= { 0, 0 },
>> @@ -459,15 +459,15 @@ static const struct dss_param_range 
>> omap4_dss_param_range[] = {
>>   };
>>
>>   static const struct dss_param_range omap5_dss_param_range[] = {
>> -[FEAT_PARAM_DSS_FCK]= { 0, 2 },
>> +[FEAT_PARAM_DSS_FCK]= { 0, 20925 },
>>  [FEAT_PARAM_DSS_PCD]= { 1, 255 },
>>  [FEAT_PARAM_DSIPLL_REGN]= { 0, (1 << 8) - 1 },
>>  [FEAT_PARAM_DSIPLL_REGM]= { 0, (1 << 12) - 1 },
>>  [FEAT_PARAM_DSIPLL_REGM_DISPC]  = { 0, (1 << 5) - 1 },
>>  [FEAT_PARAM_DSIPLL_REGM_DSI]= { 0, (1 << 5) - 1 },
>> -[FEAT_PARAM_DSIPLL_FINT]= { 50, 250 },
>> +[FEAT_PARAM_DSIPLL_FINT]= { 15, 5200 },
>
> Just a note, I think the PLL FINT range for OMAP3/4 may be wrong also.
> Some TRMs mention the FINT range being up to 52MHz or so. I don't think
> it's ever very clearly stated, though...

I'll drop the FINT range modification for now. It's not that critical 
anyway since we manage to lock the PLL for most frequencies anyway. I'll 
do an update later after reading more detailed specs of the PLL.

Archit

[PATCH v2 6/8] omapdss: Features: Fix some parameter ranges

2013-03-27 Thread Tomi Valkeinen 
On 2013-03-26 15:45, Archit Taneja wrote:
> Increase the DSS_FCLK and DSI_FCLK max supported frequencies, these come 
> because
> some frequencies were increased from OMAP5 ES1 to OMAP5 ES2. We support only
> OMAP5 ES2 in the kernel, so replace the ES1 values with ES2 values. Increase 
> the
> DSI PLL Fint range, this was previously just copied from the OMAP4 param range
> struct.
> 
> Fix the maximum DSS_FCLK on OMAP2, it's 133 Mhz according to the TRM.
> 
> Signed-off-by: Archit Taneja 
> ---
>  drivers/video/omap2/dss/dss_features.c |8 
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/video/omap2/dss/dss_features.c 
> b/drivers/video/omap2/dss/dss_features.c
> index 7f791ae..77dbe0c 100644
> --- a/drivers/video/omap2/dss/dss_features.c
> +++ b/drivers/video/omap2/dss/dss_features.c
> @@ -414,7 +414,7 @@ static const char * const omap5_dss_clk_source_names[] = {
>  };
>  
>  static const struct dss_param_range omap2_dss_param_range[] = {
> - [FEAT_PARAM_DSS_FCK]= { 0, 17300 },
> + [FEAT_PARAM_DSS_FCK]= { 0, 13300 },
>   [FEAT_PARAM_DSS_PCD]= { 2, 255 },
>   [FEAT_PARAM_DSIPLL_REGN]= { 0, 0 },
>   [FEAT_PARAM_DSIPLL_REGM]= { 0, 0 },
> @@ -459,15 +459,15 @@ static const struct dss_param_range 
> omap4_dss_param_range[] = {
>  };
>  
>  static const struct dss_param_range omap5_dss_param_range[] = {
> - [FEAT_PARAM_DSS_FCK]= { 0, 2 },
> + [FEAT_PARAM_DSS_FCK]= { 0, 20925 },
>   [FEAT_PARAM_DSS_PCD]= { 1, 255 },
>   [FEAT_PARAM_DSIPLL_REGN]= { 0, (1 << 8) - 1 },
>   [FEAT_PARAM_DSIPLL_REGM]= { 0, (1 << 12) - 1 },
>   [FEAT_PARAM_DSIPLL_REGM_DISPC]  = { 0, (1 << 5) - 1 },
>   [FEAT_PARAM_DSIPLL_REGM_DSI]= { 0, (1 << 5) - 1 },
> - [FEAT_PARAM_DSIPLL_FINT]= { 50, 250 },
> + [FEAT_PARAM_DSIPLL_FINT]= { 15, 5200 },

Just a note, I think the PLL FINT range for OMAP3/4 may be wrong also.
Some TRMs mention the FINT range being up to 52MHz or so. I don't think
it's ever very clearly stated, though...

 Tomi


-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 899 bytes
Desc: OpenPGP digital signature
URL:

Re: [PATCH v2 6/8] omapdss: Features: Fix some parameter ranges

2013-03-27 Thread Tomi Valkeinen 
On 2013-03-26 15:45, Archit Taneja wrote:
> Increase the DSS_FCLK and DSI_FCLK max supported frequencies, these come 
> because
> some frequencies were increased from OMAP5 ES1 to OMAP5 ES2. We support only
> OMAP5 ES2 in the kernel, so replace the ES1 values with ES2 values. Increase 
> the
> DSI PLL Fint range, this was previously just copied from the OMAP4 param range
> struct.
> 
> Fix the maximum DSS_FCLK on OMAP2, it's 133 Mhz according to the TRM.
> 
> Signed-off-by: Archit Taneja 
> ---
>  drivers/video/omap2/dss/dss_features.c |8 
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/video/omap2/dss/dss_features.c 
> b/drivers/video/omap2/dss/dss_features.c
> index 7f791ae..77dbe0c 100644
> --- a/drivers/video/omap2/dss/dss_features.c
> +++ b/drivers/video/omap2/dss/dss_features.c
> @@ -414,7 +414,7 @@ static const char * const omap5_dss_clk_source_names[] = {
>  };
>  
>  static const struct dss_param_range omap2_dss_param_range[] = {
> - [FEAT_PARAM_DSS_FCK]= { 0, 17300 },
> + [FEAT_PARAM_DSS_FCK]= { 0, 13300 },
>   [FEAT_PARAM_DSS_PCD]= { 2, 255 },
>   [FEAT_PARAM_DSIPLL_REGN]= { 0, 0 },
>   [FEAT_PARAM_DSIPLL_REGM]= { 0, 0 },
> @@ -459,15 +459,15 @@ static const struct dss_param_range 
> omap4_dss_param_range[] = {
>  };
>  
>  static const struct dss_param_range omap5_dss_param_range[] = {
> - [FEAT_PARAM_DSS_FCK]= { 0, 2 },
> + [FEAT_PARAM_DSS_FCK]= { 0, 20925 },
>   [FEAT_PARAM_DSS_PCD]= { 1, 255 },
>   [FEAT_PARAM_DSIPLL_REGN]= { 0, (1 << 8) - 1 },
>   [FEAT_PARAM_DSIPLL_REGM]= { 0, (1 << 12) - 1 },
>   [FEAT_PARAM_DSIPLL_REGM_DISPC]  = { 0, (1 << 5) - 1 },
>   [FEAT_PARAM_DSIPLL_REGM_DSI]= { 0, (1 << 5) - 1 },
> - [FEAT_PARAM_DSIPLL_FINT]= { 50, 250 },
> + [FEAT_PARAM_DSIPLL_FINT]= { 15, 5200 },

Just a note, I think the PLL FINT range for OMAP3/4 may be wrong also.
Some TRMs mention the FINT range being up to 52MHz or so. I don't think
it's ever very clearly stated, though...

 Tomi




signature.asc
Description: OpenPGP digital signature
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH v2 6/8] omapdss: Features: Fix some parameter ranges

2013-03-27 Thread Archit Taneja 

On Wednesday 27 March 2013 01:03 PM, Tomi Valkeinen wrote:

On 2013-03-26 15:45, Archit Taneja wrote:

Increase the DSS_FCLK and DSI_FCLK max supported frequencies, these come because
some frequencies were increased from OMAP5 ES1 to OMAP5 ES2. We support only
OMAP5 ES2 in the kernel, so replace the ES1 values with ES2 values. Increase the
DSI PLL Fint range, this was previously just copied from the OMAP4 param range
struct.

Fix the maximum DSS_FCLK on OMAP2, it's 133 Mhz according to the TRM.

Signed-off-by: Archit Taneja 
---
  drivers/video/omap2/dss/dss_features.c |8 
  1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/video/omap2/dss/dss_features.c 
b/drivers/video/omap2/dss/dss_features.c
index 7f791ae..77dbe0c 100644
--- a/drivers/video/omap2/dss/dss_features.c
+++ b/drivers/video/omap2/dss/dss_features.c
@@ -414,7 +414,7 @@ static const char * const omap5_dss_clk_source_names[] = {
  };

  static const struct dss_param_range omap2_dss_param_range[] = {
-   [FEAT_PARAM_DSS_FCK]= { 0, 17300 },
+   [FEAT_PARAM_DSS_FCK]= { 0, 13300 },
[FEAT_PARAM_DSS_PCD]= { 2, 255 },
[FEAT_PARAM_DSIPLL_REGN]= { 0, 0 },
[FEAT_PARAM_DSIPLL_REGM]= { 0, 0 },
@@ -459,15 +459,15 @@ static const struct dss_param_range 
omap4_dss_param_range[] = {
  };

  static const struct dss_param_range omap5_dss_param_range[] = {
-   [FEAT_PARAM_DSS_FCK]= { 0, 2 },
+   [FEAT_PARAM_DSS_FCK]= { 0, 20925 },
[FEAT_PARAM_DSS_PCD]= { 1, 255 },
[FEAT_PARAM_DSIPLL_REGN]= { 0, (1 << 8) - 1 },
[FEAT_PARAM_DSIPLL_REGM]= { 0, (1 << 12) - 1 },
[FEAT_PARAM_DSIPLL_REGM_DISPC]  = { 0, (1 << 5) - 1 },
[FEAT_PARAM_DSIPLL_REGM_DSI]= { 0, (1 << 5) - 1 },
-   [FEAT_PARAM_DSIPLL_FINT]= { 50, 250 },
+   [FEAT_PARAM_DSIPLL_FINT]= { 15, 5200 },


Just a note, I think the PLL FINT range for OMAP3/4 may be wrong also.
Some TRMs mention the FINT range being up to 52MHz or so. I don't think
it's ever very clearly stated, though...


I'll drop the FINT range modification for now. It's not that critical 
anyway since we manage to lock the PLL for most frequencies anyway. I'll 
do an update later after reading more detailed specs of the PLL.


Archit

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH v2 6/8] omapdss: Features: Fix some parameter ranges

2013-03-26 Thread Archit Taneja 
Increase the DSS_FCLK and DSI_FCLK max supported frequencies, these come because
some frequencies were increased from OMAP5 ES1 to OMAP5 ES2. We support only
OMAP5 ES2 in the kernel, so replace the ES1 values with ES2 values. Increase the
DSI PLL Fint range, this was previously just copied from the OMAP4 param range
struct.

Fix the maximum DSS_FCLK on OMAP2, it's 133 Mhz according to the TRM.

Signed-off-by: Archit Taneja 
---
 drivers/video/omap2/dss/dss_features.c |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/video/omap2/dss/dss_features.c 
b/drivers/video/omap2/dss/dss_features.c
index 7f791ae..77dbe0c 100644
--- a/drivers/video/omap2/dss/dss_features.c
+++ b/drivers/video/omap2/dss/dss_features.c
@@ -414,7 +414,7 @@ static const char * const omap5_dss_clk_source_names[] = {
 };

 static const struct dss_param_range omap2_dss_param_range[] = {
-   [FEAT_PARAM_DSS_FCK]= { 0, 17300 },
+   [FEAT_PARAM_DSS_FCK]= { 0, 13300 },
[FEAT_PARAM_DSS_PCD]= { 2, 255 },
[FEAT_PARAM_DSIPLL_REGN]= { 0, 0 },
[FEAT_PARAM_DSIPLL_REGM]= { 0, 0 },
@@ -459,15 +459,15 @@ static const struct dss_param_range 
omap4_dss_param_range[] = {
 };

 static const struct dss_param_range omap5_dss_param_range[] = {
-   [FEAT_PARAM_DSS_FCK]= { 0, 2 },
+   [FEAT_PARAM_DSS_FCK]= { 0, 20925 },
[FEAT_PARAM_DSS_PCD]= { 1, 255 },
[FEAT_PARAM_DSIPLL_REGN]= { 0, (1 << 8) - 1 },
[FEAT_PARAM_DSIPLL_REGM]= { 0, (1 << 12) - 1 },
[FEAT_PARAM_DSIPLL_REGM_DISPC]  = { 0, (1 << 5) - 1 },
[FEAT_PARAM_DSIPLL_REGM_DSI]= { 0, (1 << 5) - 1 },
-   [FEAT_PARAM_DSIPLL_FINT]= { 50, 250 },
+   [FEAT_PARAM_DSIPLL_FINT]= { 15, 5200 },
[FEAT_PARAM_DSIPLL_LPDIV]   = { 0, (1 << 13) - 1 },
-   [FEAT_PARAM_DSI_FCK]= { 0, 17000 },
+   [FEAT_PARAM_DSI_FCK]= { 0, 20925 },
[FEAT_PARAM_DOWNSCALE]  = { 1, 4 },
[FEAT_PARAM_LINEWIDTH]  = { 1, 2048 },
 };
-- 
1.7.10.4

[PATCH v2 6/8] omapdss: Features: Fix some parameter ranges

2013-03-26 Thread Archit Taneja 
Increase the DSS_FCLK and DSI_FCLK max supported frequencies, these come because
some frequencies were increased from OMAP5 ES1 to OMAP5 ES2. We support only
OMAP5 ES2 in the kernel, so replace the ES1 values with ES2 values. Increase the
DSI PLL Fint range, this was previously just copied from the OMAP4 param range
struct.

Fix the maximum DSS_FCLK on OMAP2, it's 133 Mhz according to the TRM.

Signed-off-by: Archit Taneja 
---
 drivers/video/omap2/dss/dss_features.c |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/video/omap2/dss/dss_features.c 
b/drivers/video/omap2/dss/dss_features.c
index 7f791ae..77dbe0c 100644
--- a/drivers/video/omap2/dss/dss_features.c
+++ b/drivers/video/omap2/dss/dss_features.c
@@ -414,7 +414,7 @@ static const char * const omap5_dss_clk_source_names[] = {
 };
 
 static const struct dss_param_range omap2_dss_param_range[] = {
-   [FEAT_PARAM_DSS_FCK]= { 0, 17300 },
+   [FEAT_PARAM_DSS_FCK]= { 0, 13300 },
[FEAT_PARAM_DSS_PCD]= { 2, 255 },
[FEAT_PARAM_DSIPLL_REGN]= { 0, 0 },
[FEAT_PARAM_DSIPLL_REGM]= { 0, 0 },
@@ -459,15 +459,15 @@ static const struct dss_param_range 
omap4_dss_param_range[] = {
 };
 
 static const struct dss_param_range omap5_dss_param_range[] = {
-   [FEAT_PARAM_DSS_FCK]= { 0, 2 },
+   [FEAT_PARAM_DSS_FCK]= { 0, 20925 },
[FEAT_PARAM_DSS_PCD]= { 1, 255 },
[FEAT_PARAM_DSIPLL_REGN]= { 0, (1 << 8) - 1 },
[FEAT_PARAM_DSIPLL_REGM]= { 0, (1 << 12) - 1 },
[FEAT_PARAM_DSIPLL_REGM_DISPC]  = { 0, (1 << 5) - 1 },
[FEAT_PARAM_DSIPLL_REGM_DSI]= { 0, (1 << 5) - 1 },
-   [FEAT_PARAM_DSIPLL_FINT]= { 50, 250 },
+   [FEAT_PARAM_DSIPLL_FINT]= { 15, 5200 },
[FEAT_PARAM_DSIPLL_LPDIV]   = { 0, (1 << 13) - 1 },
-   [FEAT_PARAM_DSI_FCK]= { 0, 17000 },
+   [FEAT_PARAM_DSI_FCK]= { 0, 20925 },
[FEAT_PARAM_DOWNSCALE]  = { 1, 4 },
[FEAT_PARAM_LINEWIDTH]  = { 1, 2048 },
 };
-- 
1.7.10.4

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel