intel_dvo.c 16 KB

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