1 From 99a13ce3a12303dfb54815637972627a7d207086 Mon Sep 17 00:00:00 2001
2 From: Maxime Ripard <maxime@cerno.tech>
3 Date: Fri, 17 Feb 2023 15:14:55 +0100
4 Subject: [PATCH] drm/vc4: hvs: Create cob_init function
6 Just like the HVS itself, the COB parameters will be fairly different in
9 Let's move the COB parameters computation and its initialisation to a
10 separate function that will be easier to extend in the future.
12 Signed-off-by: Maxime Ripard <maxime@cerno.tech>
14 drivers/gpu/drm/vc4/vc4_hvs.c | 128 ++++++++++++++++++++--------------
15 1 file changed, 74 insertions(+), 54 deletions(-)
17 --- a/drivers/gpu/drm/vc4/vc4_hvs.c
18 +++ b/drivers/gpu/drm/vc4/vc4_hvs.c
19 @@ -1379,6 +1379,77 @@ static int vc4_hvs_hw_init(struct vc4_hv
23 +static int vc4_hvs_cob_init(struct vc4_hvs *hvs)
25 + struct vc4_dev *vc4 = hvs->vc4;
29 + * Recompute Composite Output Buffer (COB) allocations for the
34 + /* The COB is 20736 pixels, or just over 10 lines at 2048 wide.
35 + * The bottom 2048 pixels are full 32bpp RGBA (intended for the
36 + * TXP composing RGBA to memory), whilst the remainder are only
39 + * Assign 3 lines to channels 1 & 2, and just over 4 lines to
42 + #define VC4_COB_SIZE 20736
43 + #define VC4_COB_LINE_WIDTH 2048
44 + #define VC4_COB_NUM_LINES 3
46 + top = VC4_COB_LINE_WIDTH * VC4_COB_NUM_LINES;
47 + reg |= (top - 1) << 16;
48 + HVS_WRITE(SCALER_DISPBASE2, reg);
50 + top += VC4_COB_LINE_WIDTH * VC4_COB_NUM_LINES;
51 + reg |= (top - 1) << 16;
52 + HVS_WRITE(SCALER_DISPBASE1, reg);
55 + reg |= (top - 1) << 16;
56 + HVS_WRITE(SCALER_DISPBASE0, reg);
60 + /* The COB is 44416 pixels, or 10.8 lines at 4096 wide.
61 + * The bottom 4096 pixels are full RGBA (intended for the TXP
62 + * composing RGBA to memory), whilst the remainder are only
63 + * RGB. Addressing is always pixel wide.
65 + * Assign 3 lines of 4096 to channels 1 & 2, and just over 4
66 + * lines. to channel 0.
68 + #define VC5_COB_SIZE 44416
69 + #define VC5_COB_LINE_WIDTH 4096
70 + #define VC5_COB_NUM_LINES 3
72 + top = VC5_COB_LINE_WIDTH * VC5_COB_NUM_LINES;
74 + HVS_WRITE(SCALER_DISPBASE2, reg);
77 + top += VC5_COB_LINE_WIDTH * VC5_COB_NUM_LINES;
79 + HVS_WRITE(SCALER_DISPBASE1, reg);
84 + HVS_WRITE(SCALER_DISPBASE0, reg);
94 static int vc4_hvs_bind(struct device *dev, struct device *master, void *data)
96 struct platform_device *pdev = to_platform_device(dev);
97 @@ -1386,7 +1457,6 @@ static int vc4_hvs_bind(struct device *d
98 struct vc4_dev *vc4 = to_vc4_dev(drm);
99 struct vc4_hvs *hvs = NULL;
103 hvs = __vc4_hvs_alloc(vc4, NULL);
105 @@ -1456,59 +1526,9 @@ static int vc4_hvs_bind(struct device *d
109 - /* Recompute Composite Output Buffer (COB) allocations for the displays
111 - if (vc4->gen == VC4_GEN_4) {
112 - /* The COB is 20736 pixels, or just over 10 lines at 2048 wide.
113 - * The bottom 2048 pixels are full 32bpp RGBA (intended for the
114 - * TXP composing RGBA to memory), whilst the remainder are only
117 - * Assign 3 lines to channels 1 & 2, and just over 4 lines to
120 - #define VC4_COB_SIZE 20736
121 - #define VC4_COB_LINE_WIDTH 2048
122 - #define VC4_COB_NUM_LINES 3
124 - top = VC4_COB_LINE_WIDTH * VC4_COB_NUM_LINES;
125 - reg |= (top - 1) << 16;
126 - HVS_WRITE(SCALER_DISPBASE2, reg);
128 - top += VC4_COB_LINE_WIDTH * VC4_COB_NUM_LINES;
129 - reg |= (top - 1) << 16;
130 - HVS_WRITE(SCALER_DISPBASE1, reg);
132 - top = VC4_COB_SIZE;
133 - reg |= (top - 1) << 16;
134 - HVS_WRITE(SCALER_DISPBASE0, reg);
136 - /* The COB is 44416 pixels, or 10.8 lines at 4096 wide.
137 - * The bottom 4096 pixels are full RGBA (intended for the TXP
138 - * composing RGBA to memory), whilst the remainder are only
139 - * RGB. Addressing is always pixel wide.
141 - * Assign 3 lines of 4096 to channels 1 & 2, and just over 4
142 - * lines. to channel 0.
144 - #define VC5_COB_SIZE 44416
145 - #define VC5_COB_LINE_WIDTH 4096
146 - #define VC5_COB_NUM_LINES 3
148 - top = VC5_COB_LINE_WIDTH * VC5_COB_NUM_LINES;
150 - HVS_WRITE(SCALER_DISPBASE2, reg);
153 - top += VC5_COB_LINE_WIDTH * VC5_COB_NUM_LINES;
155 - HVS_WRITE(SCALER_DISPBASE1, reg);
158 - top = VC5_COB_SIZE;
160 - HVS_WRITE(SCALER_DISPBASE0, reg);
162 + ret = vc4_hvs_cob_init(hvs);
166 ret = devm_request_irq(dev, platform_get_irq(pdev, 0),
167 vc4_hvs_irq_handler, 0, "vc4 hvs", drm);