* [PATCH] drm/stm: ltdc: filter mode pixel clock vs pad constraint
@ 2018-06-29 11:22 Yannick Fertre
2018-07-03 11:01 ` Philippe CORNU
0 siblings, 1 reply; 3+ messages in thread
From: Yannick Fertre @ 2018-06-29 11:22 UTC (permalink / raw)
To: Yannick Fertre, Philippe Cornu, Benjamin Gaignard,
Vincent Abriou, David Airlie, dri-devel, linux-kernel
Filter the requested mode pixel clock frequency according
to the pad maximum supported frequency.
Signed-off-by: Yannick Fertre <yannick.fertre@st.com>
---
drivers/gpu/drm/stm/ltdc.c | 16 ++++++++++++----
drivers/gpu/drm/stm/ltdc.h | 1 +
2 files changed, 13 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
index 00e3262..96d20c2 100644
--- a/drivers/gpu/drm/stm/ltdc.c
+++ b/drivers/gpu/drm/stm/ltdc.c
@@ -491,6 +491,14 @@ static void ltdc_crtc_atomic_disable(struct drm_crtc *crtc,
int target_max = target + CLK_TOLERANCE_HZ;
int result;
+ result = clk_round_rate(ldev->pixel_clk, target);
+
+ DRM_DEBUG_DRIVER("clk rate target %d, available %d\n", target, result);
+
+ /* Filter modes according to the max frequency supported by the pads */
+ if (result > ldev->caps.pad_max_freq_hz)
+ return MODE_CLOCK_HIGH;
+
/*
* Accept all "preferred" modes:
* - this is important for panels because panel clock tolerances are
@@ -502,10 +510,6 @@ static void ltdc_crtc_atomic_disable(struct drm_crtc *crtc,
if (mode->type & DRM_MODE_TYPE_PREFERRED)
return MODE_OK;
- result = clk_round_rate(ldev->pixel_clk, target);
-
- DRM_DEBUG_DRIVER("clk rate target %d, available %d\n", target, result);
-
/*
* Filter modes according to the clock value, particularly useful for
* hdmi modes that require precise pixel clocks.
@@ -1039,11 +1043,15 @@ static int ltdc_get_caps(struct drm_device *ddev)
* does not work on 2nd layer.
*/
ldev->caps.non_alpha_only_l1 = true;
+ ldev->caps.pad_max_freq_hz = 90000000;
+ if (ldev->caps.hw_version == HWVER_10200)
+ ldev->caps.pad_max_freq_hz = 65000000;
break;
case HWVER_20101:
ldev->caps.reg_ofs = REG_OFS_4;
ldev->caps.pix_fmt_hw = ltdc_pix_fmt_a1;
ldev->caps.non_alpha_only_l1 = false;
+ ldev->caps.pad_max_freq_hz = 150000000;
break;
default:
return -ENODEV;
diff --git a/drivers/gpu/drm/stm/ltdc.h b/drivers/gpu/drm/stm/ltdc.h
index c14e4db..0294d38 100644
--- a/drivers/gpu/drm/stm/ltdc.h
+++ b/drivers/gpu/drm/stm/ltdc.h
@@ -20,6 +20,7 @@ struct ltdc_caps {
u32 bus_width; /* bus width (32 or 64 bits) */
const u32 *pix_fmt_hw; /* supported pixel formats */
bool non_alpha_only_l1; /* non-native no-alpha formats on layer 1 */
+ int pad_max_freq_hz; /* max frequency supported by pad */
};
#define LTDC_MAX_LAYER 4
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/stm: ltdc: filter mode pixel clock vs pad constraint
2018-06-29 11:22 [PATCH] drm/stm: ltdc: filter mode pixel clock vs pad constraint Yannick Fertre
@ 2018-07-03 11:01 ` Philippe CORNU
2018-07-06 8:22 ` Benjamin Gaignard
0 siblings, 1 reply; 3+ messages in thread
From: Philippe CORNU @ 2018-07-03 11:01 UTC (permalink / raw)
To: Yannick FERTRE, Benjamin Gaignard, Vincent ABRIOU, David Airlie,
dri-devel, linux-kernel
Hi Yannick,
On 06/29/2018 01:22 PM, Yannick Fertre wrote:
> Filter the requested mode pixel clock frequency according
> to the pad maximum supported frequency.
>
> Signed-off-by: Yannick Fertre <yannick.fertre@st.com>
> ---
> drivers/gpu/drm/stm/ltdc.c | 16 ++++++++++++----
> drivers/gpu/drm/stm/ltdc.h | 1 +
> 2 files changed, 13 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
> index 00e3262..96d20c2 100644
> --- a/drivers/gpu/drm/stm/ltdc.c
> +++ b/drivers/gpu/drm/stm/ltdc.c
> @@ -491,6 +491,14 @@ static void ltdc_crtc_atomic_disable(struct drm_crtc *crtc,
> int target_max = target + CLK_TOLERANCE_HZ;
> int result;
>
> + result = clk_round_rate(ldev->pixel_clk, target);
> +
> + DRM_DEBUG_DRIVER("clk rate target %d, available %d\n", target, result);
> +
> + /* Filter modes according to the max frequency supported by the pads */
> + if (result > ldev->caps.pad_max_freq_hz)
> + return MODE_CLOCK_HIGH;
> +
> /*
> * Accept all "preferred" modes:
> * - this is important for panels because panel clock tolerances are
> @@ -502,10 +510,6 @@ static void ltdc_crtc_atomic_disable(struct drm_crtc *crtc,
> if (mode->type & DRM_MODE_TYPE_PREFERRED)
> return MODE_OK;
>
> - result = clk_round_rate(ldev->pixel_clk, target);
> -
> - DRM_DEBUG_DRIVER("clk rate target %d, available %d\n", target, result);
> -
> /*
> * Filter modes according to the clock value, particularly useful for
> * hdmi modes that require precise pixel clocks.
> @@ -1039,11 +1043,15 @@ static int ltdc_get_caps(struct drm_device *ddev)
> * does not work on 2nd layer.
> */
> ldev->caps.non_alpha_only_l1 = true;
> + ldev->caps.pad_max_freq_hz = 90000000;
> + if (ldev->caps.hw_version == HWVER_10200)
> + ldev->caps.pad_max_freq_hz = 65000000;
> break;
> case HWVER_20101:
> ldev->caps.reg_ofs = REG_OFS_4;
> ldev->caps.pix_fmt_hw = ltdc_pix_fmt_a1;
> ldev->caps.non_alpha_only_l1 = false;
> + ldev->caps.pad_max_freq_hz = 150000000;
> break;
> default:
> return -ENODEV;
> diff --git a/drivers/gpu/drm/stm/ltdc.h b/drivers/gpu/drm/stm/ltdc.h
> index c14e4db..0294d38 100644
> --- a/drivers/gpu/drm/stm/ltdc.h
> +++ b/drivers/gpu/drm/stm/ltdc.h
> @@ -20,6 +20,7 @@ struct ltdc_caps {
> u32 bus_width; /* bus width (32 or 64 bits) */
> const u32 *pix_fmt_hw; /* supported pixel formats */
> bool non_alpha_only_l1; /* non-native no-alpha formats on layer 1 */
> + int pad_max_freq_hz; /* max frequency supported by pad */
Thank you for your patch,
Reviewed-by: Philippe Cornu <philippe.cornu@st.com>
Tested-by: Philippe Cornu <philippe.cornu@st.com>
Philippe :-)
> };
>
> #define LTDC_MAX_LAYER 4
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/stm: ltdc: filter mode pixel clock vs pad constraint
2018-07-03 11:01 ` Philippe CORNU
@ 2018-07-06 8:22 ` Benjamin Gaignard
0 siblings, 0 replies; 3+ messages in thread
From: Benjamin Gaignard @ 2018-07-06 8:22 UTC (permalink / raw)
To: Philippe CORNU
Cc: Yannick FERTRE, Vincent ABRIOU, David Airlie, dri-devel, linux-kernel
2018-07-03 13:01 GMT+02:00 Philippe CORNU <philippe.cornu@st.com>:
> Hi Yannick,
>
> On 06/29/2018 01:22 PM, Yannick Fertre wrote:
>> Filter the requested mode pixel clock frequency according
>> to the pad maximum supported frequency.
>>
>> Signed-off-by: Yannick Fertre <yannick.fertre@st.com>
Applied on drm-misc-next.
Thansk,
Benjamin
>> ---
>> drivers/gpu/drm/stm/ltdc.c | 16 ++++++++++++----
>> drivers/gpu/drm/stm/ltdc.h | 1 +
>> 2 files changed, 13 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
>> index 00e3262..96d20c2 100644
>> --- a/drivers/gpu/drm/stm/ltdc.c
>> +++ b/drivers/gpu/drm/stm/ltdc.c
>> @@ -491,6 +491,14 @@ static void ltdc_crtc_atomic_disable(struct drm_crtc *crtc,
>> int target_max = target + CLK_TOLERANCE_HZ;
>> int result;
>>
>> + result = clk_round_rate(ldev->pixel_clk, target);
>> +
>> + DRM_DEBUG_DRIVER("clk rate target %d, available %d\n", target, result);
>> +
>> + /* Filter modes according to the max frequency supported by the pads */
>> + if (result > ldev->caps.pad_max_freq_hz)
>> + return MODE_CLOCK_HIGH;
>> +
>> /*
>> * Accept all "preferred" modes:
>> * - this is important for panels because panel clock tolerances are
>> @@ -502,10 +510,6 @@ static void ltdc_crtc_atomic_disable(struct drm_crtc *crtc,
>> if (mode->type & DRM_MODE_TYPE_PREFERRED)
>> return MODE_OK;
>>
>> - result = clk_round_rate(ldev->pixel_clk, target);
>> -
>> - DRM_DEBUG_DRIVER("clk rate target %d, available %d\n", target, result);
>> -
>> /*
>> * Filter modes according to the clock value, particularly useful for
>> * hdmi modes that require precise pixel clocks.
>> @@ -1039,11 +1043,15 @@ static int ltdc_get_caps(struct drm_device *ddev)
>> * does not work on 2nd layer.
>> */
>> ldev->caps.non_alpha_only_l1 = true;
>> + ldev->caps.pad_max_freq_hz = 90000000;
>> + if (ldev->caps.hw_version == HWVER_10200)
>> + ldev->caps.pad_max_freq_hz = 65000000;
>> break;
>> case HWVER_20101:
>> ldev->caps.reg_ofs = REG_OFS_4;
>> ldev->caps.pix_fmt_hw = ltdc_pix_fmt_a1;
>> ldev->caps.non_alpha_only_l1 = false;
>> + ldev->caps.pad_max_freq_hz = 150000000;
>> break;
>> default:
>> return -ENODEV;
>> diff --git a/drivers/gpu/drm/stm/ltdc.h b/drivers/gpu/drm/stm/ltdc.h
>> index c14e4db..0294d38 100644
>> --- a/drivers/gpu/drm/stm/ltdc.h
>> +++ b/drivers/gpu/drm/stm/ltdc.h
>> @@ -20,6 +20,7 @@ struct ltdc_caps {
>> u32 bus_width; /* bus width (32 or 64 bits) */
>> const u32 *pix_fmt_hw; /* supported pixel formats */
>> bool non_alpha_only_l1; /* non-native no-alpha formats on layer 1 */
>> + int pad_max_freq_hz; /* max frequency supported by pad */
>
> Thank you for your patch,
>
> Reviewed-by: Philippe Cornu <philippe.cornu@st.com>
> Tested-by: Philippe Cornu <philippe.cornu@st.com>
>
> Philippe :-)
>
>> };
>>
>> #define LTDC_MAX_LAYER 4
>>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-07-06 8:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-29 11:22 [PATCH] drm/stm: ltdc: filter mode pixel clock vs pad constraint Yannick Fertre
2018-07-03 11:01 ` Philippe CORNU
2018-07-06 8:22 ` Benjamin Gaignard
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).