1 From af3f381a59c10f6bd49d86a5ff2325b6ebeb79e9 Mon Sep 17 00:00:00 2001
2 From: popcornmix <popcornmix@gmail.com>
3 Date: Mon, 27 Apr 2020 19:07:50 +0100
4 Subject: [PATCH] vc4_hdmi: BCM2835 requires a fixed hsm clock for CEC
7 Signed-off-by: popcornmix <popcornmix@gmail.com>
9 drivers/gpu/drm/vc4/vc4_hdmi.c | 32 ++++++++++++++++++++++++++------
10 drivers/gpu/drm/vc4/vc4_hdmi.h | 3 +++
11 2 files changed, 29 insertions(+), 6 deletions(-)
13 --- a/drivers/gpu/drm/vc4/vc4_hdmi.c
14 +++ b/drivers/gpu/drm/vc4/vc4_hdmi.c
15 @@ -580,12 +580,7 @@ static void vc4_hdmi_encoder_enable(stru
20 - * The HSM rate needs to be slightly greater than the pixel clock, with
21 - * a minimum of 108MHz.
22 - * Use 101% as this is what the firmware uses.
24 - hsm_rate = max_t(unsigned long, 108000000, (pixel_rate / 100) * 101);
25 + hsm_rate = vc4_hdmi->variant->calc_hsm_clock(vc4_hdmi, pixel_rate);
26 ret = clk_set_rate(vc4_hdmi->hsm_clock, hsm_rate);
28 DRM_ERROR("Failed to set HSM clock rate: %d\n", ret);
29 @@ -753,6 +748,28 @@ static u32 vc5_hdmi_get_hsm_clock(struct
33 +static u32 vc4_hdmi_calc_hsm_clock(struct vc4_hdmi *vc4_hdmi, unsigned long pixel_rate)
36 + * This is the rate that is set by the firmware. The number
37 + * needs to be a bit higher than the pixel clock rate
38 + * (generally 148.5Mhz).
44 +static u32 vc5_hdmi_calc_hsm_clock(struct vc4_hdmi *vc4_hdmi, unsigned long pixel_rate)
47 + * The HSM rate needs to be slightly greater than the pixel clock, with
48 + * a minimum of 108MHz.
49 + * Use 101% as this is what the firmware uses.
52 + return max_t(unsigned long, 108000000, (pixel_rate / 100) * 101);
55 static u32 vc4_hdmi_channel_map(struct vc4_hdmi *vc4_hdmi, u32 channel_mask)
58 @@ -1748,6 +1765,7 @@ static const struct vc4_hdmi_variant bcm
59 .phy_rng_enable = vc4_hdmi_phy_rng_enable,
60 .phy_rng_disable = vc4_hdmi_phy_rng_disable,
61 .get_hsm_clock = vc4_hdmi_get_hsm_clock,
62 + .calc_hsm_clock = vc4_hdmi_calc_hsm_clock,
63 .channel_map = vc4_hdmi_channel_map,
66 @@ -1772,6 +1790,7 @@ static const struct vc4_hdmi_variant bcm
67 .phy_rng_enable = vc5_hdmi_phy_rng_enable,
68 .phy_rng_disable = vc5_hdmi_phy_rng_disable,
69 .get_hsm_clock = vc5_hdmi_get_hsm_clock,
70 + .calc_hsm_clock = vc5_hdmi_calc_hsm_clock,
71 .channel_map = vc5_hdmi_channel_map,
74 @@ -1796,6 +1815,7 @@ static const struct vc4_hdmi_variant bcm
75 .phy_rng_enable = vc5_hdmi_phy_rng_enable,
76 .phy_rng_disable = vc5_hdmi_phy_rng_disable,
77 .get_hsm_clock = vc5_hdmi_get_hsm_clock,
78 + .calc_hsm_clock = vc5_hdmi_calc_hsm_clock,
79 .channel_map = vc5_hdmi_channel_map,
82 --- a/drivers/gpu/drm/vc4/vc4_hdmi.h
83 +++ b/drivers/gpu/drm/vc4/vc4_hdmi.h
84 @@ -92,6 +92,9 @@ struct vc4_hdmi_variant {
85 /* Callback to get hsm clock */
86 u32 (*get_hsm_clock)(struct vc4_hdmi *vc4_hdmi);
88 + /* Callback to get hsm clock */
89 + u32 (*calc_hsm_clock)(struct vc4_hdmi *vc4_hdmi, unsigned long pixel_rate);
91 /* Callback to get channel map */
92 u32 (*channel_map)(struct vc4_hdmi *vc4_hdmi, u32 channel_mask);