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 
28 #include <linux/i2c.h>
29 #include <linux/slab.h>
30 
31 #include <drm/drm_atomic_helper.h>
32 #include <drm/drm_crtc.h>
33 
34 #include "i915_drv.h"
35 #include "intel_connector.h"
36 #include "intel_display_types.h"
37 #include "intel_dvo.h"
38 #include "intel_dvo_dev.h"
39 #include "intel_gmbus.h"
40 #include "intel_panel.h"
41 
42 #define INTEL_DVO_CHIP_NONE	0
43 #define INTEL_DVO_CHIP_LVDS	1
44 #define INTEL_DVO_CHIP_TMDS	2
45 #define INTEL_DVO_CHIP_TVOUT	4
46 #define INTEL_DVO_CHIP_LVDS_NO_FIXED	5
47 
48 #define SIL164_ADDR	0x38
49 #define CH7xxx_ADDR	0x76
50 #define TFP410_ADDR	0x38
51 #define NS2501_ADDR     0x38
52 
53 static const struct intel_dvo_device intel_dvo_devices[] = {
54 	{
55 		.type = INTEL_DVO_CHIP_TMDS,
56 		.name = "sil164",
57 		.dvo_reg = DVOC,
58 		.dvo_srcdim_reg = DVOC_SRCDIM,
59 		.slave_addr = SIL164_ADDR,
60 		.dev_ops = &sil164_ops,
61 	},
62 	{
63 		.type = INTEL_DVO_CHIP_TMDS,
64 		.name = "ch7xxx",
65 		.dvo_reg = DVOC,
66 		.dvo_srcdim_reg = DVOC_SRCDIM,
67 		.slave_addr = CH7xxx_ADDR,
68 		.dev_ops = &ch7xxx_ops,
69 	},
70 	{
71 		.type = INTEL_DVO_CHIP_TMDS,
72 		.name = "ch7xxx",
73 		.dvo_reg = DVOC,
74 		.dvo_srcdim_reg = DVOC_SRCDIM,
75 		.slave_addr = 0x75, /* For some ch7010 */
76 		.dev_ops = &ch7xxx_ops,
77 	},
78 	{
79 		.type = INTEL_DVO_CHIP_LVDS,
80 		.name = "ivch",
81 		.dvo_reg = DVOA,
82 		.dvo_srcdim_reg = DVOA_SRCDIM,
83 		.slave_addr = 0x02, /* Might also be 0x44, 0x84, 0xc4 */
84 		.dev_ops = &ivch_ops,
85 	},
86 	{
87 		.type = INTEL_DVO_CHIP_TMDS,
88 		.name = "tfp410",
89 		.dvo_reg = DVOC,
90 		.dvo_srcdim_reg = DVOC_SRCDIM,
91 		.slave_addr = TFP410_ADDR,
92 		.dev_ops = &tfp410_ops,
93 	},
94 	{
95 		.type = INTEL_DVO_CHIP_LVDS,
96 		.name = "ch7017",
97 		.dvo_reg = DVOC,
98 		.dvo_srcdim_reg = DVOC_SRCDIM,
99 		.slave_addr = 0x75,
100 		.gpio = GMBUS_PIN_DPB,
101 		.dev_ops = &ch7017_ops,
102 	},
103 	{
104 		.type = INTEL_DVO_CHIP_LVDS_NO_FIXED,
105 		.name = "ns2501",
106 		.dvo_reg = DVOB,
107 		.dvo_srcdim_reg = DVOB_SRCDIM,
108 		.slave_addr = NS2501_ADDR,
109 		.dev_ops = &ns2501_ops,
110 	},
111 };
112 
113 struct intel_dvo {
114 	struct intel_encoder base;
115 
116 	struct intel_dvo_device dev;
117 
118 	struct intel_connector *attached_connector;
119 
120 	bool panel_wants_dither;
121 };
122 
123 static struct intel_dvo *enc_to_dvo(struct intel_encoder *encoder)
124 {
125 	return container_of(encoder, struct intel_dvo, base);
126 }
127 
128 static struct intel_dvo *intel_attached_dvo(struct intel_connector *connector)
129 {
130 	return enc_to_dvo(intel_attached_encoder(connector));
131 }
132 
133 static bool intel_dvo_connector_get_hw_state(struct intel_connector *connector)
134 {
135 	struct drm_device *dev = connector->base.dev;
136 	struct drm_i915_private *dev_priv = to_i915(dev);
137 	struct intel_dvo *intel_dvo = intel_attached_dvo(connector);
138 	u32 tmp;
139 
140 	tmp = intel_de_read(dev_priv, intel_dvo->dev.dvo_reg);
141 
142 	if (!(tmp & DVO_ENABLE))
143 		return false;
144 
145 	return intel_dvo->dev.dev_ops->get_hw_state(&intel_dvo->dev);
146 }
147 
148 static bool intel_dvo_get_hw_state(struct intel_encoder *encoder,
149 				   enum pipe *pipe)
150 {
151 	struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
152 	struct intel_dvo *intel_dvo = enc_to_dvo(encoder);
153 	u32 tmp;
154 
155 	tmp = intel_de_read(dev_priv, intel_dvo->dev.dvo_reg);
156 
157 	*pipe = (tmp & DVO_PIPE_SEL_MASK) >> DVO_PIPE_SEL_SHIFT;
158 
159 	return tmp & DVO_ENABLE;
160 }
161 
162 static void intel_dvo_get_config(struct intel_encoder *encoder,
163 				 struct intel_crtc_state *pipe_config)
164 {
165 	struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
166 	struct intel_dvo *intel_dvo = enc_to_dvo(encoder);
167 	u32 tmp, flags = 0;
168 
169 	pipe_config->output_types |= BIT(INTEL_OUTPUT_DVO);
170 
171 	tmp = intel_de_read(dev_priv, intel_dvo->dev.dvo_reg);
172 	if (tmp & DVO_HSYNC_ACTIVE_HIGH)
173 		flags |= DRM_MODE_FLAG_PHSYNC;
174 	else
175 		flags |= DRM_MODE_FLAG_NHSYNC;
176 	if (tmp & DVO_VSYNC_ACTIVE_HIGH)
177 		flags |= DRM_MODE_FLAG_PVSYNC;
178 	else
179 		flags |= DRM_MODE_FLAG_NVSYNC;
180 
181 	pipe_config->hw.adjusted_mode.flags |= flags;
182 
183 	pipe_config->hw.adjusted_mode.crtc_clock = pipe_config->port_clock;
184 }
185 
186 static void intel_disable_dvo(struct intel_atomic_state *state,
187 			      struct intel_encoder *encoder,
188 			      const struct intel_crtc_state *old_crtc_state,
189 			      const struct drm_connector_state *old_conn_state)
190 {
191 	struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
192 	struct intel_dvo *intel_dvo = enc_to_dvo(encoder);
193 	i915_reg_t dvo_reg = intel_dvo->dev.dvo_reg;
194 	u32 temp = intel_de_read(dev_priv, dvo_reg);
195 
196 	intel_dvo->dev.dev_ops->dpms(&intel_dvo->dev, false);
197 	intel_de_write(dev_priv, dvo_reg, temp & ~DVO_ENABLE);
198 	intel_de_read(dev_priv, dvo_reg);
199 }
200 
201 static void intel_enable_dvo(struct intel_atomic_state *state,
202 			     struct intel_encoder *encoder,
203 			     const struct intel_crtc_state *pipe_config,
204 			     const struct drm_connector_state *conn_state)
205 {
206 	struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
207 	struct intel_dvo *intel_dvo = enc_to_dvo(encoder);
208 	i915_reg_t dvo_reg = intel_dvo->dev.dvo_reg;
209 	u32 temp = intel_de_read(dev_priv, dvo_reg);
210 
211 	intel_dvo->dev.dev_ops->mode_set(&intel_dvo->dev,
212 					 &pipe_config->hw.mode,
213 					 &pipe_config->hw.adjusted_mode);
214 
215 	intel_de_write(dev_priv, dvo_reg, temp | DVO_ENABLE);
216 	intel_de_read(dev_priv, dvo_reg);
217 
218 	intel_dvo->dev.dev_ops->dpms(&intel_dvo->dev, true);
219 }
220 
221 static enum drm_mode_status
222 intel_dvo_mode_valid(struct drm_connector *connector,
223 		     struct drm_display_mode *mode)
224 {
225 	struct intel_dvo *intel_dvo = intel_attached_dvo(to_intel_connector(connector));
226 	const struct drm_display_mode *fixed_mode =
227 		to_intel_connector(connector)->panel.fixed_mode;
228 	int max_dotclk = to_i915(connector->dev)->max_dotclk_freq;
229 	int target_clock = mode->clock;
230 
231 	if (mode->flags & DRM_MODE_FLAG_DBLSCAN)
232 		return MODE_NO_DBLESCAN;
233 
234 	/* XXX: Validate clock range */
235 
236 	if (fixed_mode) {
237 		if (mode->hdisplay > fixed_mode->hdisplay)
238 			return MODE_PANEL;
239 		if (mode->vdisplay > fixed_mode->vdisplay)
240 			return MODE_PANEL;
241 
242 		target_clock = fixed_mode->clock;
243 	}
244 
245 	if (target_clock > max_dotclk)
246 		return MODE_CLOCK_HIGH;
247 
248 	return intel_dvo->dev.dev_ops->mode_valid(&intel_dvo->dev, mode);
249 }
250 
251 static int intel_dvo_compute_config(struct intel_encoder *encoder,
252 				    struct intel_crtc_state *pipe_config,
253 				    struct drm_connector_state *conn_state)
254 {
255 	struct intel_dvo *intel_dvo = enc_to_dvo(encoder);
256 	const struct drm_display_mode *fixed_mode =
257 		intel_dvo->attached_connector->panel.fixed_mode;
258 	struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode;
259 
260 	/*
261 	 * If we have timings from the BIOS for the panel, put them in
262 	 * to the adjusted mode.  The CRTC will be set up for this mode,
263 	 * with the panel scaling set up to source from the H/VDisplay
264 	 * of the original mode.
265 	 */
266 	if (fixed_mode)
267 		intel_fixed_panel_mode(fixed_mode, adjusted_mode);
268 
269 	if (adjusted_mode->flags & DRM_MODE_FLAG_DBLSCAN)
270 		return -EINVAL;
271 
272 	pipe_config->output_format = INTEL_OUTPUT_FORMAT_RGB;
273 
274 	return 0;
275 }
276 
277 static void intel_dvo_pre_enable(struct intel_atomic_state *state,
278 				 struct intel_encoder *encoder,
279 				 const struct intel_crtc_state *pipe_config,
280 				 const struct drm_connector_state *conn_state)
281 {
282 	struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
283 	struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc);
284 	const struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode;
285 	struct intel_dvo *intel_dvo = enc_to_dvo(encoder);
286 	enum pipe pipe = crtc->pipe;
287 	u32 dvo_val;
288 	i915_reg_t dvo_reg = intel_dvo->dev.dvo_reg;
289 	i915_reg_t dvo_srcdim_reg = intel_dvo->dev.dvo_srcdim_reg;
290 
291 	/* Save the data order, since I don't know what it should be set to. */
292 	dvo_val = intel_de_read(dev_priv, dvo_reg) &
293 		  (DVO_PRESERVE_MASK | DVO_DATA_ORDER_GBRG);
294 	dvo_val |= DVO_DATA_ORDER_FP | DVO_BORDER_ENABLE |
295 		   DVO_BLANK_ACTIVE_HIGH;
296 
297 	dvo_val |= DVO_PIPE_SEL(pipe);
298 	dvo_val |= DVO_PIPE_STALL;
299 	if (adjusted_mode->flags & DRM_MODE_FLAG_PHSYNC)
300 		dvo_val |= DVO_HSYNC_ACTIVE_HIGH;
301 	if (adjusted_mode->flags & DRM_MODE_FLAG_PVSYNC)
302 		dvo_val |= DVO_VSYNC_ACTIVE_HIGH;
303 
304 	/*I915_WRITE(DVOB_SRCDIM,
305 	  (adjusted_mode->crtc_hdisplay << DVO_SRCDIM_HORIZONTAL_SHIFT) |
306 	  (adjusted_mode->crtc_vdisplay << DVO_SRCDIM_VERTICAL_SHIFT));*/
307 	intel_de_write(dev_priv, dvo_srcdim_reg,
308 		       (adjusted_mode->crtc_hdisplay << DVO_SRCDIM_HORIZONTAL_SHIFT) | (adjusted_mode->crtc_vdisplay << DVO_SRCDIM_VERTICAL_SHIFT));
309 	/*I915_WRITE(DVOB, dvo_val);*/
310 	intel_de_write(dev_priv, dvo_reg, dvo_val);
311 }
312 
313 static enum drm_connector_status
314 intel_dvo_detect(struct drm_connector *connector, bool force)
315 {
316 	struct intel_dvo *intel_dvo = intel_attached_dvo(to_intel_connector(connector));
317 	DRM_DEBUG_KMS("[CONNECTOR:%d:%s]\n",
318 		      connector->base.id, connector->name);
319 	return intel_dvo->dev.dev_ops->detect(&intel_dvo->dev);
320 }
321 
322 static int intel_dvo_get_modes(struct drm_connector *connector)
323 {
324 	struct drm_i915_private *dev_priv = to_i915(connector->dev);
325 	const struct drm_display_mode *fixed_mode =
326 		to_intel_connector(connector)->panel.fixed_mode;
327 	int num_modes;
328 
329 	/*
330 	 * We should probably have an i2c driver get_modes function for those
331 	 * devices which will have a fixed set of modes determined by the chip
332 	 * (TV-out, for example), but for now with just TMDS and LVDS,
333 	 * that's not the case.
334 	 */
335 	num_modes = intel_ddc_get_modes(connector,
336 					intel_gmbus_get_adapter(dev_priv, GMBUS_PIN_DPC));
337 	if (num_modes)
338 		return num_modes;
339 
340 	if (fixed_mode) {
341 		struct drm_display_mode *mode;
342 
343 		mode = drm_mode_duplicate(connector->dev, fixed_mode);
344 		if (mode) {
345 			drm_mode_probed_add(connector, mode);
346 			num_modes++;
347 		}
348 	}
349 
350 	return num_modes;
351 }
352 
353 static const struct drm_connector_funcs intel_dvo_connector_funcs = {
354 	.detect = intel_dvo_detect,
355 	.late_register = intel_connector_register,
356 	.early_unregister = intel_connector_unregister,
357 	.destroy = intel_connector_destroy,
358 	.fill_modes = drm_helper_probe_single_connector_modes,
359 	.atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
360 	.atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state,
361 };
362 
363 static const struct drm_connector_helper_funcs intel_dvo_connector_helper_funcs = {
364 	.mode_valid = intel_dvo_mode_valid,
365 	.get_modes = intel_dvo_get_modes,
366 };
367 
368 static void intel_dvo_enc_destroy(struct drm_encoder *encoder)
369 {
370 	struct intel_dvo *intel_dvo = enc_to_dvo(to_intel_encoder(encoder));
371 
372 	if (intel_dvo->dev.dev_ops->destroy)
373 		intel_dvo->dev.dev_ops->destroy(&intel_dvo->dev);
374 
375 	intel_encoder_destroy(encoder);
376 }
377 
378 static const struct drm_encoder_funcs intel_dvo_enc_funcs = {
379 	.destroy = intel_dvo_enc_destroy,
380 };
381 
382 /*
383  * Attempts to get a fixed panel timing for LVDS (currently only the i830).
384  *
385  * Other chips with DVO LVDS will need to extend this to deal with the LVDS
386  * chip being on DVOB/C and having multiple pipes.
387  */
388 static struct drm_display_mode *
389 intel_dvo_get_current_mode(struct intel_encoder *encoder)
390 {
391 	struct drm_display_mode *mode;
392 
393 	mode = intel_encoder_current_mode(encoder);
394 	if (mode) {
395 		DRM_DEBUG_KMS("using current (BIOS) mode: ");
396 		drm_mode_debug_printmodeline(mode);
397 		mode->type |= DRM_MODE_TYPE_PREFERRED;
398 	}
399 
400 	return mode;
401 }
402 
403 static enum port intel_dvo_port(i915_reg_t dvo_reg)
404 {
405 	if (i915_mmio_reg_equal(dvo_reg, DVOA))
406 		return PORT_A;
407 	else if (i915_mmio_reg_equal(dvo_reg, DVOB))
408 		return PORT_B;
409 	else
410 		return PORT_C;
411 }
412 
413 void intel_dvo_init(struct drm_i915_private *dev_priv)
414 {
415 	struct intel_encoder *intel_encoder;
416 	struct intel_dvo *intel_dvo;
417 	struct intel_connector *intel_connector;
418 	int i;
419 	int encoder_type = DRM_MODE_ENCODER_NONE;
420 
421 	intel_dvo = kzalloc(sizeof(*intel_dvo), GFP_KERNEL);
422 	if (!intel_dvo)
423 		return;
424 
425 	intel_connector = intel_connector_alloc();
426 	if (!intel_connector) {
427 		kfree(intel_dvo);
428 		return;
429 	}
430 
431 	intel_dvo->attached_connector = intel_connector;
432 
433 	intel_encoder = &intel_dvo->base;
434 
435 	intel_encoder->disable = intel_disable_dvo;
436 	intel_encoder->enable = intel_enable_dvo;
437 	intel_encoder->get_hw_state = intel_dvo_get_hw_state;
438 	intel_encoder->get_config = intel_dvo_get_config;
439 	intel_encoder->compute_config = intel_dvo_compute_config;
440 	intel_encoder->pre_enable = intel_dvo_pre_enable;
441 	intel_connector->get_hw_state = intel_dvo_connector_get_hw_state;
442 
443 	/* Now, try to find a controller */
444 	for (i = 0; i < ARRAY_SIZE(intel_dvo_devices); i++) {
445 		struct drm_connector *connector = &intel_connector->base;
446 		const struct intel_dvo_device *dvo = &intel_dvo_devices[i];
447 		struct i2c_adapter *i2c;
448 		int gpio;
449 		bool dvoinit;
450 		enum pipe pipe;
451 		u32 dpll[I915_MAX_PIPES];
452 		enum port port;
453 
454 		/*
455 		 * Allow the I2C driver info to specify the GPIO to be used in
456 		 * special cases, but otherwise default to what's defined
457 		 * in the spec.
458 		 */
459 		if (intel_gmbus_is_valid_pin(dev_priv, dvo->gpio))
460 			gpio = dvo->gpio;
461 		else if (dvo->type == INTEL_DVO_CHIP_LVDS)
462 			gpio = GMBUS_PIN_SSC;
463 		else
464 			gpio = GMBUS_PIN_DPB;
465 
466 		/*
467 		 * Set up the I2C bus necessary for the chip we're probing.
468 		 * It appears that everything is on GPIOE except for panels
469 		 * on i830 laptops, which are on GPIOB (DVOA).
470 		 */
471 		i2c = intel_gmbus_get_adapter(dev_priv, gpio);
472 
473 		intel_dvo->dev = *dvo;
474 
475 		/*
476 		 * GMBUS NAK handling seems to be unstable, hence let the
477 		 * transmitter detection run in bit banging mode for now.
478 		 */
479 		intel_gmbus_force_bit(i2c, true);
480 
481 		/*
482 		 * ns2501 requires the DVO 2x clock before it will
483 		 * respond to i2c accesses, so make sure we have
484 		 * have the clock enabled before we attempt to
485 		 * initialize the device.
486 		 */
487 		for_each_pipe(dev_priv, pipe) {
488 			dpll[pipe] = intel_de_read(dev_priv, DPLL(pipe));
489 			intel_de_write(dev_priv, DPLL(pipe),
490 				       dpll[pipe] | DPLL_DVO_2X_MODE);
491 		}
492 
493 		dvoinit = dvo->dev_ops->init(&intel_dvo->dev, i2c);
494 
495 		/* restore the DVO 2x clock state to original */
496 		for_each_pipe(dev_priv, pipe) {
497 			intel_de_write(dev_priv, DPLL(pipe), dpll[pipe]);
498 		}
499 
500 		intel_gmbus_force_bit(i2c, false);
501 
502 		if (!dvoinit)
503 			continue;
504 
505 		port = intel_dvo_port(dvo->dvo_reg);
506 		drm_encoder_init(&dev_priv->drm, &intel_encoder->base,
507 				 &intel_dvo_enc_funcs, encoder_type,
508 				 "DVO %c", port_name(port));
509 
510 		intel_encoder->type = INTEL_OUTPUT_DVO;
511 		intel_encoder->power_domain = POWER_DOMAIN_PORT_OTHER;
512 		intel_encoder->port = port;
513 		intel_encoder->pipe_mask = ~0;
514 
515 		if (dvo->type != INTEL_DVO_CHIP_LVDS)
516 			intel_encoder->cloneable = (1 << INTEL_OUTPUT_ANALOG) |
517 				(1 << INTEL_OUTPUT_DVO);
518 
519 		switch (dvo->type) {
520 		case INTEL_DVO_CHIP_TMDS:
521 			intel_connector->polled = DRM_CONNECTOR_POLL_CONNECT |
522 				DRM_CONNECTOR_POLL_DISCONNECT;
523 			drm_connector_init(&dev_priv->drm, connector,
524 					   &intel_dvo_connector_funcs,
525 					   DRM_MODE_CONNECTOR_DVII);
526 			encoder_type = DRM_MODE_ENCODER_TMDS;
527 			break;
528 		case INTEL_DVO_CHIP_LVDS_NO_FIXED:
529 		case INTEL_DVO_CHIP_LVDS:
530 			drm_connector_init(&dev_priv->drm, connector,
531 					   &intel_dvo_connector_funcs,
532 					   DRM_MODE_CONNECTOR_LVDS);
533 			encoder_type = DRM_MODE_ENCODER_LVDS;
534 			break;
535 		}
536 
537 		drm_connector_helper_add(connector,
538 					 &intel_dvo_connector_helper_funcs);
539 		connector->display_info.subpixel_order = SubPixelHorizontalRGB;
540 		connector->interlace_allowed = false;
541 		connector->doublescan_allowed = false;
542 
543 		intel_connector_attach_encoder(intel_connector, intel_encoder);
544 		if (dvo->type == INTEL_DVO_CHIP_LVDS) {
545 			/*
546 			 * For our LVDS chipsets, we should hopefully be able
547 			 * to dig the fixed panel mode out of the BIOS data.
548 			 * However, it's in a different format from the BIOS
549 			 * data on chipsets with integrated LVDS (stored in AIM
550 			 * headers, likely), so for now, just get the current
551 			 * mode being output through DVO.
552 			 */
553 			intel_panel_init(&intel_connector->panel,
554 					 intel_dvo_get_current_mode(intel_encoder),
555 					 NULL);
556 			intel_dvo->panel_wants_dither = true;
557 		}
558 
559 		return;
560 	}
561 
562 	kfree(intel_dvo);
563 	kfree(intel_connector);
564 }
565