drm/vc4: hdmi: Move clock validation to its own function
Our code is doing the same clock rate validation in multiple instances. Let's create a helper to share the rate validation. Signed-off-by: Maxime Ripard <maxime@cerno.tech>
This commit is contained in:
committed by
Dom Cobley
parent
fcfff78534
commit
4abb3e17be
@@ -1250,6 +1250,19 @@ static void vc4_hdmi_encoder_atomic_mode_set(struct drm_encoder *encoder,
|
|||||||
mutex_unlock(&vc4_hdmi->mutex);
|
mutex_unlock(&vc4_hdmi->mutex);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static enum drm_mode_status
|
||||||
|
vc4_hdmi_encoder_clock_valid(const struct vc4_hdmi *vc4_hdmi,
|
||||||
|
unsigned long long clock)
|
||||||
|
{
|
||||||
|
if (clock > vc4_hdmi->variant->max_pixel_clock)
|
||||||
|
return MODE_CLOCK_HIGH;
|
||||||
|
|
||||||
|
if (vc4_hdmi->disable_4kp60 && clock > HDMI_14_MAX_TMDS_CLK)
|
||||||
|
return MODE_CLOCK_HIGH;
|
||||||
|
|
||||||
|
return MODE_OK;
|
||||||
|
}
|
||||||
|
|
||||||
#define WIFI_2_4GHz_CH1_MIN_FREQ 2400000000ULL
|
#define WIFI_2_4GHz_CH1_MIN_FREQ 2400000000ULL
|
||||||
#define WIFI_2_4GHz_CH1_MAX_FREQ 2422000000ULL
|
#define WIFI_2_4GHz_CH1_MAX_FREQ 2422000000ULL
|
||||||
|
|
||||||
@@ -1294,10 +1307,7 @@ static int vc4_hdmi_encoder_atomic_check(struct drm_encoder *encoder,
|
|||||||
if (mode->flags & DRM_MODE_FLAG_DBLCLK)
|
if (mode->flags & DRM_MODE_FLAG_DBLCLK)
|
||||||
pixel_rate = pixel_rate * 2;
|
pixel_rate = pixel_rate * 2;
|
||||||
|
|
||||||
if (pixel_rate > vc4_hdmi->variant->max_pixel_clock)
|
if (vc4_hdmi_encoder_clock_valid(vc4_hdmi, pixel_rate) != MODE_OK)
|
||||||
return -EINVAL;
|
|
||||||
|
|
||||||
if (vc4_hdmi->disable_4kp60 && (pixel_rate > HDMI_14_MAX_TMDS_CLK))
|
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
vc4_state->pixel_rate = pixel_rate;
|
vc4_state->pixel_rate = pixel_rate;
|
||||||
@@ -1317,13 +1327,7 @@ vc4_hdmi_encoder_mode_valid(struct drm_encoder *encoder,
|
|||||||
(mode->hsync_end % 2) || (mode->htotal % 2)))
|
(mode->hsync_end % 2) || (mode->htotal % 2)))
|
||||||
return MODE_H_ILLEGAL;
|
return MODE_H_ILLEGAL;
|
||||||
|
|
||||||
if ((mode->clock * 1000) > vc4_hdmi->variant->max_pixel_clock)
|
return vc4_hdmi_encoder_clock_valid(vc4_hdmi, mode->clock * 1000);
|
||||||
return MODE_CLOCK_HIGH;
|
|
||||||
|
|
||||||
if (vc4_hdmi->disable_4kp60 && vc4_hdmi_mode_needs_scrambling(mode))
|
|
||||||
return MODE_CLOCK_HIGH;
|
|
||||||
|
|
||||||
return MODE_OK;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static const struct drm_encoder_helper_funcs vc4_hdmi_encoder_helper_funcs = {
|
static const struct drm_encoder_helper_funcs vc4_hdmi_encoder_helper_funcs = {
|
||||||
|
|||||||
Reference in New Issue
Block a user