Commit | Line | Data |
---|---|---|
79e53945 JB |
1 | /* |
2 | * Copyright 2006 Dave Airlie <airlied@linux.ie> | |
3 | * Copyright © 2006-2007 Intel Corporation | |
4 | * | |
5 | * Permission is hereby granted, free of charge, to any person obtaining a | |
6 | * copy of this software and associated documentation files (the "Software"), | |
7 | * to deal in the Software without restriction, including without limitation | |
8 | * the rights to use, copy, modify, merge, publish, distribute, sublicense, | |
9 | * and/or sell copies of the Software, and to permit persons to whom the | |
10 | * Software is furnished to do so, subject to the following conditions: | |
11 | * | |
12 | * The above copyright notice and this permission notice (including the next | |
13 | * paragraph) shall be included in all copies or substantial portions of the | |
14 | * Software. | |
15 | * | |
16 | * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR | |
17 | * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, | |
18 | * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL | |
19 | * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER | |
20 | * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING | |
21 | * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER | |
22 | * DEALINGS IN THE SOFTWARE. | |
23 | * | |
24 | * Authors: | |
25 | * Eric Anholt <eric@anholt.net> | |
26 | */ | |
27 | #include <linux/i2c.h> | |
5a0e3ad6 | 28 | #include <linux/slab.h> |
79e53945 JB |
29 | #include "drmP.h" |
30 | #include "drm.h" | |
31 | #include "drm_crtc.h" | |
32 | #include "intel_drv.h" | |
33 | #include "i915_drm.h" | |
34 | #include "i915_drv.h" | |
35 | #include "dvo.h" | |
36 | ||
37 | #define SIL164_ADDR 0x38 | |
38 | #define CH7xxx_ADDR 0x76 | |
39 | #define TFP410_ADDR 0x38 | |
40 | ||
ea5b213a | 41 | static const struct intel_dvo_device intel_dvo_devices[] = { |
79e53945 JB |
42 | { |
43 | .type = INTEL_DVO_CHIP_TMDS, | |
44 | .name = "sil164", | |
45 | .dvo_reg = DVOC, | |
46 | .slave_addr = SIL164_ADDR, | |
47 | .dev_ops = &sil164_ops, | |
48 | }, | |
49 | { | |
50 | .type = INTEL_DVO_CHIP_TMDS, | |
51 | .name = "ch7xxx", | |
52 | .dvo_reg = DVOC, | |
53 | .slave_addr = CH7xxx_ADDR, | |
54 | .dev_ops = &ch7xxx_ops, | |
55 | }, | |
56 | { | |
57 | .type = INTEL_DVO_CHIP_LVDS, | |
58 | .name = "ivch", | |
59 | .dvo_reg = DVOA, | |
60 | .slave_addr = 0x02, /* Might also be 0x44, 0x84, 0xc4 */ | |
61 | .dev_ops = &ivch_ops, | |
62 | }, | |
63 | { | |
64 | .type = INTEL_DVO_CHIP_TMDS, | |
65 | .name = "tfp410", | |
66 | .dvo_reg = DVOC, | |
67 | .slave_addr = TFP410_ADDR, | |
68 | .dev_ops = &tfp410_ops, | |
69 | }, | |
70 | { | |
71 | .type = INTEL_DVO_CHIP_LVDS, | |
72 | .name = "ch7017", | |
73 | .dvo_reg = DVOC, | |
74 | .slave_addr = 0x75, | |
75 | .gpio = GPIOE, | |
76 | .dev_ops = &ch7017_ops, | |
77 | } | |
78 | }; | |
79 | ||
ea5b213a CW |
80 | struct intel_dvo { |
81 | struct intel_encoder base; | |
82 | ||
83 | struct intel_dvo_device dev; | |
84 | ||
85 | struct drm_display_mode *panel_fixed_mode; | |
86 | bool panel_wants_dither; | |
87 | }; | |
88 | ||
89 | static struct intel_dvo *enc_to_intel_dvo(struct drm_encoder *encoder) | |
90 | { | |
4ef69c7a | 91 | return container_of(encoder, struct intel_dvo, base.base); |
ea5b213a CW |
92 | } |
93 | ||
df0e9248 CW |
94 | static struct intel_dvo *intel_attached_dvo(struct drm_connector *connector) |
95 | { | |
96 | return container_of(intel_attached_encoder(connector), | |
97 | struct intel_dvo, base); | |
98 | } | |
99 | ||
79e53945 JB |
100 | static void intel_dvo_dpms(struct drm_encoder *encoder, int mode) |
101 | { | |
102 | struct drm_i915_private *dev_priv = encoder->dev->dev_private; | |
ea5b213a CW |
103 | struct intel_dvo *intel_dvo = enc_to_intel_dvo(encoder); |
104 | u32 dvo_reg = intel_dvo->dev.dvo_reg; | |
79e53945 JB |
105 | u32 temp = I915_READ(dvo_reg); |
106 | ||
107 | if (mode == DRM_MODE_DPMS_ON) { | |
108 | I915_WRITE(dvo_reg, temp | DVO_ENABLE); | |
109 | I915_READ(dvo_reg); | |
ea5b213a | 110 | intel_dvo->dev.dev_ops->dpms(&intel_dvo->dev, mode); |
79e53945 | 111 | } else { |
ea5b213a | 112 | intel_dvo->dev.dev_ops->dpms(&intel_dvo->dev, mode); |
79e53945 JB |
113 | I915_WRITE(dvo_reg, temp & ~DVO_ENABLE); |
114 | I915_READ(dvo_reg); | |
115 | } | |
116 | } | |
117 | ||
79e53945 JB |
118 | static int intel_dvo_mode_valid(struct drm_connector *connector, |
119 | struct drm_display_mode *mode) | |
120 | { | |
df0e9248 | 121 | struct intel_dvo *intel_dvo = intel_attached_dvo(connector); |
79e53945 JB |
122 | |
123 | if (mode->flags & DRM_MODE_FLAG_DBLSCAN) | |
124 | return MODE_NO_DBLESCAN; | |
125 | ||
126 | /* XXX: Validate clock range */ | |
127 | ||
ea5b213a CW |
128 | if (intel_dvo->panel_fixed_mode) { |
129 | if (mode->hdisplay > intel_dvo->panel_fixed_mode->hdisplay) | |
79e53945 | 130 | return MODE_PANEL; |
ea5b213a | 131 | if (mode->vdisplay > intel_dvo->panel_fixed_mode->vdisplay) |
79e53945 JB |
132 | return MODE_PANEL; |
133 | } | |
134 | ||
ea5b213a | 135 | return intel_dvo->dev.dev_ops->mode_valid(&intel_dvo->dev, mode); |
79e53945 JB |
136 | } |
137 | ||
138 | static bool intel_dvo_mode_fixup(struct drm_encoder *encoder, | |
139 | struct drm_display_mode *mode, | |
140 | struct drm_display_mode *adjusted_mode) | |
141 | { | |
ea5b213a | 142 | struct intel_dvo *intel_dvo = enc_to_intel_dvo(encoder); |
79e53945 JB |
143 | |
144 | /* If we have timings from the BIOS for the panel, put them in | |
145 | * to the adjusted mode. The CRTC will be set up for this mode, | |
146 | * with the panel scaling set up to source from the H/VDisplay | |
147 | * of the original mode. | |
148 | */ | |
ea5b213a CW |
149 | if (intel_dvo->panel_fixed_mode != NULL) { |
150 | #define C(x) adjusted_mode->x = intel_dvo->panel_fixed_mode->x | |
79e53945 JB |
151 | C(hdisplay); |
152 | C(hsync_start); | |
153 | C(hsync_end); | |
154 | C(htotal); | |
155 | C(vdisplay); | |
156 | C(vsync_start); | |
157 | C(vsync_end); | |
158 | C(vtotal); | |
159 | C(clock); | |
160 | drm_mode_set_crtcinfo(adjusted_mode, CRTC_INTERLACE_HALVE_V); | |
161 | #undef C | |
162 | } | |
163 | ||
ea5b213a CW |
164 | if (intel_dvo->dev.dev_ops->mode_fixup) |
165 | return intel_dvo->dev.dev_ops->mode_fixup(&intel_dvo->dev, mode, adjusted_mode); | |
79e53945 JB |
166 | |
167 | return true; | |
168 | } | |
169 | ||
170 | static void intel_dvo_mode_set(struct drm_encoder *encoder, | |
171 | struct drm_display_mode *mode, | |
172 | struct drm_display_mode *adjusted_mode) | |
173 | { | |
174 | struct drm_device *dev = encoder->dev; | |
175 | struct drm_i915_private *dev_priv = dev->dev_private; | |
176 | struct intel_crtc *intel_crtc = to_intel_crtc(encoder->crtc); | |
ea5b213a | 177 | struct intel_dvo *intel_dvo = enc_to_intel_dvo(encoder); |
79e53945 JB |
178 | int pipe = intel_crtc->pipe; |
179 | u32 dvo_val; | |
ea5b213a | 180 | u32 dvo_reg = intel_dvo->dev.dvo_reg, dvo_srcdim_reg; |
79e53945 JB |
181 | int dpll_reg = (pipe == 0) ? DPLL_A : DPLL_B; |
182 | ||
183 | switch (dvo_reg) { | |
184 | case DVOA: | |
185 | default: | |
186 | dvo_srcdim_reg = DVOA_SRCDIM; | |
187 | break; | |
188 | case DVOB: | |
189 | dvo_srcdim_reg = DVOB_SRCDIM; | |
190 | break; | |
191 | case DVOC: | |
192 | dvo_srcdim_reg = DVOC_SRCDIM; | |
193 | break; | |
194 | } | |
195 | ||
ea5b213a | 196 | intel_dvo->dev.dev_ops->mode_set(&intel_dvo->dev, mode, adjusted_mode); |
79e53945 JB |
197 | |
198 | /* Save the data order, since I don't know what it should be set to. */ | |
199 | dvo_val = I915_READ(dvo_reg) & | |
200 | (DVO_PRESERVE_MASK | DVO_DATA_ORDER_GBRG); | |
201 | dvo_val |= DVO_DATA_ORDER_FP | DVO_BORDER_ENABLE | | |
202 | DVO_BLANK_ACTIVE_HIGH; | |
203 | ||
204 | if (pipe == 1) | |
205 | dvo_val |= DVO_PIPE_B_SELECT; | |
206 | dvo_val |= DVO_PIPE_STALL; | |
207 | if (adjusted_mode->flags & DRM_MODE_FLAG_PHSYNC) | |
208 | dvo_val |= DVO_HSYNC_ACTIVE_HIGH; | |
209 | if (adjusted_mode->flags & DRM_MODE_FLAG_PVSYNC) | |
210 | dvo_val |= DVO_VSYNC_ACTIVE_HIGH; | |
211 | ||
212 | I915_WRITE(dpll_reg, I915_READ(dpll_reg) | DPLL_DVO_HIGH_SPEED); | |
213 | ||
214 | /*I915_WRITE(DVOB_SRCDIM, | |
215 | (adjusted_mode->hdisplay << DVO_SRCDIM_HORIZONTAL_SHIFT) | | |
216 | (adjusted_mode->VDisplay << DVO_SRCDIM_VERTICAL_SHIFT));*/ | |
217 | I915_WRITE(dvo_srcdim_reg, | |
218 | (adjusted_mode->hdisplay << DVO_SRCDIM_HORIZONTAL_SHIFT) | | |
219 | (adjusted_mode->vdisplay << DVO_SRCDIM_VERTICAL_SHIFT)); | |
220 | /*I915_WRITE(DVOB, dvo_val);*/ | |
221 | I915_WRITE(dvo_reg, dvo_val); | |
222 | } | |
223 | ||
224 | /** | |
225 | * Detect the output connection on our DVO device. | |
226 | * | |
227 | * Unimplemented. | |
228 | */ | |
229 | static enum drm_connector_status intel_dvo_detect(struct drm_connector *connector) | |
230 | { | |
df0e9248 | 231 | struct intel_dvo *intel_dvo = intel_attached_dvo(connector); |
ea5b213a | 232 | return intel_dvo->dev.dev_ops->detect(&intel_dvo->dev); |
79e53945 JB |
233 | } |
234 | ||
235 | static int intel_dvo_get_modes(struct drm_connector *connector) | |
236 | { | |
df0e9248 | 237 | struct intel_dvo *intel_dvo = intel_attached_dvo(connector); |
79e53945 JB |
238 | |
239 | /* We should probably have an i2c driver get_modes function for those | |
240 | * devices which will have a fixed set of modes determined by the chip | |
241 | * (TV-out, for example), but for now with just TMDS and LVDS, | |
242 | * that's not the case. | |
243 | */ | |
ea5b213a | 244 | intel_ddc_get_modes(connector, intel_dvo->base.ddc_bus); |
79e53945 JB |
245 | if (!list_empty(&connector->probed_modes)) |
246 | return 1; | |
247 | ||
ea5b213a | 248 | if (intel_dvo->panel_fixed_mode != NULL) { |
79e53945 | 249 | struct drm_display_mode *mode; |
ea5b213a | 250 | mode = drm_mode_duplicate(connector->dev, intel_dvo->panel_fixed_mode); |
79e53945 JB |
251 | if (mode) { |
252 | drm_mode_probed_add(connector, mode); | |
253 | return 1; | |
254 | } | |
255 | } | |
ea5b213a | 256 | |
79e53945 JB |
257 | return 0; |
258 | } | |
259 | ||
ea5b213a | 260 | static void intel_dvo_destroy(struct drm_connector *connector) |
79e53945 | 261 | { |
79e53945 JB |
262 | drm_sysfs_connector_remove(connector); |
263 | drm_connector_cleanup(connector); | |
599be16c | 264 | kfree(connector); |
79e53945 | 265 | } |
79e53945 JB |
266 | |
267 | static const struct drm_encoder_helper_funcs intel_dvo_helper_funcs = { | |
268 | .dpms = intel_dvo_dpms, | |
269 | .mode_fixup = intel_dvo_mode_fixup, | |
270 | .prepare = intel_encoder_prepare, | |
271 | .mode_set = intel_dvo_mode_set, | |
272 | .commit = intel_encoder_commit, | |
273 | }; | |
274 | ||
275 | static const struct drm_connector_funcs intel_dvo_connector_funcs = { | |
c9fb15f6 | 276 | .dpms = drm_helper_connector_dpms, |
79e53945 JB |
277 | .detect = intel_dvo_detect, |
278 | .destroy = intel_dvo_destroy, | |
279 | .fill_modes = drm_helper_probe_single_connector_modes, | |
280 | }; | |
281 | ||
282 | static const struct drm_connector_helper_funcs intel_dvo_connector_helper_funcs = { | |
283 | .mode_valid = intel_dvo_mode_valid, | |
284 | .get_modes = intel_dvo_get_modes, | |
df0e9248 | 285 | .best_encoder = intel_best_encoder, |
79e53945 JB |
286 | }; |
287 | ||
b358d0a6 | 288 | static void intel_dvo_enc_destroy(struct drm_encoder *encoder) |
79e53945 | 289 | { |
ea5b213a CW |
290 | struct intel_dvo *intel_dvo = enc_to_intel_dvo(encoder); |
291 | ||
292 | if (intel_dvo->dev.dev_ops->destroy) | |
293 | intel_dvo->dev.dev_ops->destroy(&intel_dvo->dev); | |
294 | ||
295 | kfree(intel_dvo->panel_fixed_mode); | |
296 | ||
297 | intel_encoder_destroy(encoder); | |
79e53945 JB |
298 | } |
299 | ||
300 | static const struct drm_encoder_funcs intel_dvo_enc_funcs = { | |
301 | .destroy = intel_dvo_enc_destroy, | |
302 | }; | |
303 | ||
79e53945 JB |
304 | /** |
305 | * Attempts to get a fixed panel timing for LVDS (currently only the i830). | |
306 | * | |
307 | * Other chips with DVO LVDS will need to extend this to deal with the LVDS | |
308 | * chip being on DVOB/C and having multiple pipes. | |
309 | */ | |
310 | static struct drm_display_mode * | |
ea5b213a | 311 | intel_dvo_get_current_mode(struct drm_connector *connector) |
79e53945 JB |
312 | { |
313 | struct drm_device *dev = connector->dev; | |
314 | struct drm_i915_private *dev_priv = dev->dev_private; | |
df0e9248 | 315 | struct intel_dvo *intel_dvo = intel_attached_dvo(connector); |
ea5b213a | 316 | uint32_t dvo_val = I915_READ(intel_dvo->dev.dvo_reg); |
79e53945 JB |
317 | struct drm_display_mode *mode = NULL; |
318 | ||
319 | /* If the DVO port is active, that'll be the LVDS, so we can pull out | |
320 | * its timings to get how the BIOS set up the panel. | |
321 | */ | |
322 | if (dvo_val & DVO_ENABLE) { | |
323 | struct drm_crtc *crtc; | |
324 | int pipe = (dvo_val & DVO_PIPE_B_SELECT) ? 1 : 0; | |
325 | ||
f875c15a | 326 | crtc = intel_get_crtc_for_pipe(dev, pipe); |
79e53945 JB |
327 | if (crtc) { |
328 | mode = intel_crtc_mode_get(dev, crtc); | |
79e53945 JB |
329 | if (mode) { |
330 | mode->type |= DRM_MODE_TYPE_PREFERRED; | |
331 | if (dvo_val & DVO_HSYNC_ACTIVE_HIGH) | |
332 | mode->flags |= DRM_MODE_FLAG_PHSYNC; | |
333 | if (dvo_val & DVO_VSYNC_ACTIVE_HIGH) | |
334 | mode->flags |= DRM_MODE_FLAG_PVSYNC; | |
335 | } | |
336 | } | |
337 | } | |
ea5b213a | 338 | |
79e53945 JB |
339 | return mode; |
340 | } | |
341 | ||
342 | void intel_dvo_init(struct drm_device *dev) | |
343 | { | |
21d40d37 | 344 | struct intel_encoder *intel_encoder; |
ea5b213a | 345 | struct intel_dvo *intel_dvo; |
599be16c | 346 | struct intel_connector *intel_connector; |
f9c10a9b | 347 | struct i2c_adapter *i2cbus = NULL; |
79e53945 JB |
348 | int ret = 0; |
349 | int i; | |
79e53945 | 350 | int encoder_type = DRM_MODE_ENCODER_NONE; |
ea5b213a CW |
351 | |
352 | intel_dvo = kzalloc(sizeof(struct intel_dvo), GFP_KERNEL); | |
353 | if (!intel_dvo) | |
79e53945 JB |
354 | return; |
355 | ||
599be16c ZW |
356 | intel_connector = kzalloc(sizeof(struct intel_connector), GFP_KERNEL); |
357 | if (!intel_connector) { | |
ea5b213a | 358 | kfree(intel_dvo); |
599be16c ZW |
359 | return; |
360 | } | |
361 | ||
ea5b213a | 362 | intel_encoder = &intel_dvo->base; |
373a3cf7 CW |
363 | drm_encoder_init(dev, &intel_encoder->base, |
364 | &intel_dvo_enc_funcs, encoder_type); | |
ea5b213a | 365 | |
79e53945 | 366 | /* Set up the DDC bus */ |
890f3359 CW |
367 | intel_encoder->ddc_bus = intel_i2c_create(intel_encoder, |
368 | GPIOD, "DVODDC_D"); | |
21d40d37 | 369 | if (!intel_encoder->ddc_bus) |
79e53945 JB |
370 | goto free_intel; |
371 | ||
372 | /* Now, try to find a controller */ | |
373 | for (i = 0; i < ARRAY_SIZE(intel_dvo_devices); i++) { | |
599be16c | 374 | struct drm_connector *connector = &intel_connector->base; |
ea5b213a | 375 | const struct intel_dvo_device *dvo = &intel_dvo_devices[i]; |
79e53945 JB |
376 | int gpio; |
377 | ||
79e53945 JB |
378 | /* Allow the I2C driver info to specify the GPIO to be used in |
379 | * special cases, but otherwise default to what's defined | |
380 | * in the spec. | |
381 | */ | |
382 | if (dvo->gpio != 0) | |
383 | gpio = dvo->gpio; | |
384 | else if (dvo->type == INTEL_DVO_CHIP_LVDS) | |
385 | gpio = GPIOB; | |
386 | else | |
387 | gpio = GPIOE; | |
388 | ||
389 | /* Set up the I2C bus necessary for the chip we're probing. | |
390 | * It appears that everything is on GPIOE except for panels | |
391 | * on i830 laptops, which are on GPIOB (DVOA). | |
392 | */ | |
f9c10a9b KP |
393 | if (i2cbus != NULL) |
394 | intel_i2c_destroy(i2cbus); | |
890f3359 CW |
395 | i2cbus = intel_i2c_create(intel_encoder, gpio, |
396 | gpio == GPIOB ? "DVOI2C_B" : "DVOI2C_E"); | |
397 | if (i2cbus == NULL) | |
f9c10a9b | 398 | continue; |
79e53945 | 399 | |
ea5b213a CW |
400 | intel_dvo->dev = *dvo; |
401 | ret = dvo->dev_ops->init(&intel_dvo->dev, i2cbus); | |
79e53945 JB |
402 | if (!ret) |
403 | continue; | |
404 | ||
21d40d37 EA |
405 | intel_encoder->type = INTEL_OUTPUT_DVO; |
406 | intel_encoder->crtc_mask = (1 << 0) | (1 << 1); | |
79e53945 JB |
407 | switch (dvo->type) { |
408 | case INTEL_DVO_CHIP_TMDS: | |
21d40d37 | 409 | intel_encoder->clone_mask = |
f8aed700 ML |
410 | (1 << INTEL_DVO_TMDS_CLONE_BIT) | |
411 | (1 << INTEL_ANALOG_CLONE_BIT); | |
79e53945 JB |
412 | drm_connector_init(dev, connector, |
413 | &intel_dvo_connector_funcs, | |
414 | DRM_MODE_CONNECTOR_DVII); | |
415 | encoder_type = DRM_MODE_ENCODER_TMDS; | |
416 | break; | |
417 | case INTEL_DVO_CHIP_LVDS: | |
21d40d37 | 418 | intel_encoder->clone_mask = |
f8aed700 | 419 | (1 << INTEL_DVO_LVDS_CLONE_BIT); |
79e53945 JB |
420 | drm_connector_init(dev, connector, |
421 | &intel_dvo_connector_funcs, | |
422 | DRM_MODE_CONNECTOR_LVDS); | |
423 | encoder_type = DRM_MODE_ENCODER_LVDS; | |
424 | break; | |
425 | } | |
426 | ||
427 | drm_connector_helper_add(connector, | |
428 | &intel_dvo_connector_helper_funcs); | |
429 | connector->display_info.subpixel_order = SubPixelHorizontalRGB; | |
430 | connector->interlace_allowed = false; | |
431 | connector->doublescan_allowed = false; | |
432 | ||
4ef69c7a | 433 | drm_encoder_helper_add(&intel_encoder->base, |
79e53945 JB |
434 | &intel_dvo_helper_funcs); |
435 | ||
df0e9248 | 436 | intel_connector_attach_encoder(intel_connector, intel_encoder); |
79e53945 JB |
437 | if (dvo->type == INTEL_DVO_CHIP_LVDS) { |
438 | /* For our LVDS chipsets, we should hopefully be able | |
439 | * to dig the fixed panel mode out of the BIOS data. | |
440 | * However, it's in a different format from the BIOS | |
441 | * data on chipsets with integrated LVDS (stored in AIM | |
442 | * headers, likely), so for now, just get the current | |
443 | * mode being output through DVO. | |
444 | */ | |
ea5b213a | 445 | intel_dvo->panel_fixed_mode = |
79e53945 | 446 | intel_dvo_get_current_mode(connector); |
ea5b213a | 447 | intel_dvo->panel_wants_dither = true; |
79e53945 JB |
448 | } |
449 | ||
450 | drm_sysfs_connector_add(connector); | |
451 | return; | |
452 | } | |
453 | ||
21d40d37 | 454 | intel_i2c_destroy(intel_encoder->ddc_bus); |
79e53945 JB |
455 | /* Didn't find a chip, so tear down. */ |
456 | if (i2cbus != NULL) | |
457 | intel_i2c_destroy(i2cbus); | |
458 | free_intel: | |
373a3cf7 | 459 | drm_encoder_cleanup(&intel_encoder->base); |
ea5b213a | 460 | kfree(intel_dvo); |
599be16c | 461 | kfree(intel_connector); |
79e53945 | 462 | } |