1 // SPDX-License-Identifier: GPL-2.0-or-later 2 /* 3 * Copyright (C) 2016 Marek Vasut <marex@denx.de> 4 * 5 * This code is based on drivers/video/fbdev/mxsfb.c : 6 * Copyright (C) 2010 Juergen Beisert, Pengutronix 7 * Copyright (C) 2008-2009 Freescale Semiconductor, Inc. All Rights Reserved. 8 * Copyright (C) 2008 Embedded Alley Solutions, Inc All Rights Reserved. 9 */ 10 11 #include <linux/clk.h> 12 #include <linux/io.h> 13 #include <linux/iopoll.h> 14 #include <linux/pm_runtime.h> 15 #include <linux/spinlock.h> 16 17 #include <drm/drm_atomic.h> 18 #include <drm/drm_atomic_helper.h> 19 #include <drm/drm_bridge.h> 20 #include <drm/drm_crtc.h> 21 #include <drm/drm_encoder.h> 22 #include <drm/drm_fb_cma_helper.h> 23 #include <drm/drm_fourcc.h> 24 #include <drm/drm_gem_cma_helper.h> 25 #include <drm/drm_plane.h> 26 #include <drm/drm_plane_helper.h> 27 #include <drm/drm_vblank.h> 28 29 #include "mxsfb_drv.h" 30 #include "mxsfb_regs.h" 31 32 /* 1 second delay should be plenty of time for block reset */ 33 #define RESET_TIMEOUT 1000000 34 35 /* ----------------------------------------------------------------------------- 36 * CRTC 37 */ 38 39 static u32 set_hsync_pulse_width(struct mxsfb_drm_private *mxsfb, u32 val) 40 { 41 return (val & mxsfb->devdata->hs_wdth_mask) << 42 mxsfb->devdata->hs_wdth_shift; 43 } 44 45 /* 46 * Setup the MXSFB registers for decoding the pixels out of the framebuffer and 47 * outputting them on the bus. 48 */ 49 static void mxsfb_set_formats(struct mxsfb_drm_private *mxsfb) 50 { 51 struct drm_device *drm = mxsfb->drm; 52 const u32 format = mxsfb->crtc.primary->state->fb->format->format; 53 u32 bus_format = MEDIA_BUS_FMT_RGB888_1X24; 54 u32 ctrl, ctrl1; 55 56 if (mxsfb->connector->display_info.num_bus_formats) 57 bus_format = mxsfb->connector->display_info.bus_formats[0]; 58 59 DRM_DEV_DEBUG_DRIVER(drm->dev, "Using bus_format: 0x%08X\n", 60 bus_format); 61 62 ctrl = CTRL_BYPASS_COUNT | CTRL_MASTER; 63 64 /* CTRL1 contains IRQ config and status bits, preserve those. */ 65 ctrl1 = readl(mxsfb->base + LCDC_CTRL1); 66 ctrl1 &= CTRL1_CUR_FRAME_DONE_IRQ_EN | CTRL1_CUR_FRAME_DONE_IRQ; 67 68 switch (format) { 69 case DRM_FORMAT_RGB565: 70 dev_dbg(drm->dev, "Setting up RGB565 mode\n"); 71 ctrl |= CTRL_WORD_LENGTH_16; 72 ctrl1 |= CTRL1_SET_BYTE_PACKAGING(0xf); 73 break; 74 case DRM_FORMAT_XRGB8888: 75 dev_dbg(drm->dev, "Setting up XRGB8888 mode\n"); 76 ctrl |= CTRL_WORD_LENGTH_24; 77 /* Do not use packed pixels = one pixel per word instead. */ 78 ctrl1 |= CTRL1_SET_BYTE_PACKAGING(0x7); 79 break; 80 } 81 82 switch (bus_format) { 83 case MEDIA_BUS_FMT_RGB565_1X16: 84 ctrl |= CTRL_BUS_WIDTH_16; 85 break; 86 case MEDIA_BUS_FMT_RGB666_1X18: 87 ctrl |= CTRL_BUS_WIDTH_18; 88 break; 89 case MEDIA_BUS_FMT_RGB888_1X24: 90 ctrl |= CTRL_BUS_WIDTH_24; 91 break; 92 default: 93 dev_err(drm->dev, "Unknown media bus format %d\n", bus_format); 94 break; 95 } 96 97 writel(ctrl1, mxsfb->base + LCDC_CTRL1); 98 writel(ctrl, mxsfb->base + LCDC_CTRL); 99 } 100 101 static void mxsfb_enable_controller(struct mxsfb_drm_private *mxsfb) 102 { 103 u32 reg; 104 105 if (mxsfb->clk_disp_axi) 106 clk_prepare_enable(mxsfb->clk_disp_axi); 107 clk_prepare_enable(mxsfb->clk); 108 109 /* If it was disabled, re-enable the mode again */ 110 writel(CTRL_DOTCLK_MODE, mxsfb->base + LCDC_CTRL + REG_SET); 111 112 /* Enable the SYNC signals first, then the DMA engine */ 113 reg = readl(mxsfb->base + LCDC_VDCTRL4); 114 reg |= VDCTRL4_SYNC_SIGNALS_ON; 115 writel(reg, mxsfb->base + LCDC_VDCTRL4); 116 117 writel(CTRL_RUN, mxsfb->base + LCDC_CTRL + REG_SET); 118 } 119 120 static void mxsfb_disable_controller(struct mxsfb_drm_private *mxsfb) 121 { 122 u32 reg; 123 124 /* 125 * Even if we disable the controller here, it will still continue 126 * until its FIFOs are running out of data 127 */ 128 writel(CTRL_DOTCLK_MODE, mxsfb->base + LCDC_CTRL + REG_CLR); 129 130 readl_poll_timeout(mxsfb->base + LCDC_CTRL, reg, !(reg & CTRL_RUN), 131 0, 1000); 132 133 reg = readl(mxsfb->base + LCDC_VDCTRL4); 134 reg &= ~VDCTRL4_SYNC_SIGNALS_ON; 135 writel(reg, mxsfb->base + LCDC_VDCTRL4); 136 137 clk_disable_unprepare(mxsfb->clk); 138 if (mxsfb->clk_disp_axi) 139 clk_disable_unprepare(mxsfb->clk_disp_axi); 140 } 141 142 /* 143 * Clear the bit and poll it cleared. This is usually called with 144 * a reset address and mask being either SFTRST(bit 31) or CLKGATE 145 * (bit 30). 146 */ 147 static int clear_poll_bit(void __iomem *addr, u32 mask) 148 { 149 u32 reg; 150 151 writel(mask, addr + REG_CLR); 152 return readl_poll_timeout(addr, reg, !(reg & mask), 0, RESET_TIMEOUT); 153 } 154 155 static int mxsfb_reset_block(struct mxsfb_drm_private *mxsfb) 156 { 157 int ret; 158 159 ret = clear_poll_bit(mxsfb->base + LCDC_CTRL, CTRL_SFTRST); 160 if (ret) 161 return ret; 162 163 writel(CTRL_CLKGATE, mxsfb->base + LCDC_CTRL + REG_CLR); 164 165 ret = clear_poll_bit(mxsfb->base + LCDC_CTRL, CTRL_SFTRST); 166 if (ret) 167 return ret; 168 169 return clear_poll_bit(mxsfb->base + LCDC_CTRL, CTRL_CLKGATE); 170 } 171 172 static dma_addr_t mxsfb_get_fb_paddr(struct drm_plane *plane) 173 { 174 struct drm_framebuffer *fb = plane->state->fb; 175 struct drm_gem_cma_object *gem; 176 177 if (!fb) 178 return 0; 179 180 gem = drm_fb_cma_get_gem_obj(fb, 0); 181 if (!gem) 182 return 0; 183 184 return gem->paddr; 185 } 186 187 static void mxsfb_crtc_mode_set_nofb(struct mxsfb_drm_private *mxsfb) 188 { 189 struct drm_device *drm = mxsfb->crtc.dev; 190 struct drm_display_mode *m = &mxsfb->crtc.state->adjusted_mode; 191 u32 bus_flags = mxsfb->connector->display_info.bus_flags; 192 u32 vdctrl0, vsync_pulse_len, hsync_pulse_len; 193 int err; 194 195 /* 196 * It seems, you can't re-program the controller if it is still 197 * running. This may lead to shifted pictures (FIFO issue?), so 198 * first stop the controller and drain its FIFOs. 199 */ 200 201 /* Mandatory eLCDIF reset as per the Reference Manual */ 202 err = mxsfb_reset_block(mxsfb); 203 if (err) 204 return; 205 206 /* Clear the FIFOs */ 207 writel(CTRL1_FIFO_CLEAR, mxsfb->base + LCDC_CTRL1 + REG_SET); 208 209 if (mxsfb->devdata->has_overlay) 210 writel(0, mxsfb->base + LCDC_AS_CTRL); 211 212 mxsfb_set_formats(mxsfb); 213 214 clk_set_rate(mxsfb->clk, m->crtc_clock * 1000); 215 216 if (mxsfb->bridge && mxsfb->bridge->timings) 217 bus_flags = mxsfb->bridge->timings->input_bus_flags; 218 219 DRM_DEV_DEBUG_DRIVER(drm->dev, "Pixel clock: %dkHz (actual: %dkHz)\n", 220 m->crtc_clock, 221 (int)(clk_get_rate(mxsfb->clk) / 1000)); 222 DRM_DEV_DEBUG_DRIVER(drm->dev, "Connector bus_flags: 0x%08X\n", 223 bus_flags); 224 DRM_DEV_DEBUG_DRIVER(drm->dev, "Mode flags: 0x%08X\n", m->flags); 225 226 writel(TRANSFER_COUNT_SET_VCOUNT(m->crtc_vdisplay) | 227 TRANSFER_COUNT_SET_HCOUNT(m->crtc_hdisplay), 228 mxsfb->base + mxsfb->devdata->transfer_count); 229 230 vsync_pulse_len = m->crtc_vsync_end - m->crtc_vsync_start; 231 232 vdctrl0 = VDCTRL0_ENABLE_PRESENT | /* Always in DOTCLOCK mode */ 233 VDCTRL0_VSYNC_PERIOD_UNIT | 234 VDCTRL0_VSYNC_PULSE_WIDTH_UNIT | 235 VDCTRL0_SET_VSYNC_PULSE_WIDTH(vsync_pulse_len); 236 if (m->flags & DRM_MODE_FLAG_PHSYNC) 237 vdctrl0 |= VDCTRL0_HSYNC_ACT_HIGH; 238 if (m->flags & DRM_MODE_FLAG_PVSYNC) 239 vdctrl0 |= VDCTRL0_VSYNC_ACT_HIGH; 240 /* Make sure Data Enable is high active by default */ 241 if (!(bus_flags & DRM_BUS_FLAG_DE_LOW)) 242 vdctrl0 |= VDCTRL0_ENABLE_ACT_HIGH; 243 /* 244 * DRM_BUS_FLAG_PIXDATA_DRIVE_ defines are controller centric, 245 * controllers VDCTRL0_DOTCLK is display centric. 246 * Drive on positive edge -> display samples on falling edge 247 * DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE -> VDCTRL0_DOTCLK_ACT_FALLING 248 */ 249 if (bus_flags & DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE) 250 vdctrl0 |= VDCTRL0_DOTCLK_ACT_FALLING; 251 252 writel(vdctrl0, mxsfb->base + LCDC_VDCTRL0); 253 254 /* Frame length in lines. */ 255 writel(m->crtc_vtotal, mxsfb->base + LCDC_VDCTRL1); 256 257 /* Line length in units of clocks or pixels. */ 258 hsync_pulse_len = m->crtc_hsync_end - m->crtc_hsync_start; 259 writel(set_hsync_pulse_width(mxsfb, hsync_pulse_len) | 260 VDCTRL2_SET_HSYNC_PERIOD(m->crtc_htotal), 261 mxsfb->base + LCDC_VDCTRL2); 262 263 writel(SET_HOR_WAIT_CNT(m->crtc_htotal - m->crtc_hsync_start) | 264 SET_VERT_WAIT_CNT(m->crtc_vtotal - m->crtc_vsync_start), 265 mxsfb->base + LCDC_VDCTRL3); 266 267 writel(SET_DOTCLK_H_VALID_DATA_CNT(m->hdisplay), 268 mxsfb->base + LCDC_VDCTRL4); 269 } 270 271 static int mxsfb_crtc_atomic_check(struct drm_crtc *crtc, 272 struct drm_atomic_state *state) 273 { 274 struct drm_crtc_state *crtc_state = drm_atomic_get_new_crtc_state(state, 275 crtc); 276 bool has_primary = crtc_state->plane_mask & 277 drm_plane_mask(crtc->primary); 278 279 /* The primary plane has to be enabled when the CRTC is active. */ 280 if (crtc_state->active && !has_primary) 281 return -EINVAL; 282 283 /* TODO: Is this needed ? */ 284 return drm_atomic_add_affected_planes(state, crtc); 285 } 286 287 static void mxsfb_crtc_atomic_flush(struct drm_crtc *crtc, 288 struct drm_atomic_state *state) 289 { 290 struct drm_pending_vblank_event *event; 291 292 event = crtc->state->event; 293 crtc->state->event = NULL; 294 295 if (!event) 296 return; 297 298 spin_lock_irq(&crtc->dev->event_lock); 299 if (drm_crtc_vblank_get(crtc) == 0) 300 drm_crtc_arm_vblank_event(crtc, event); 301 else 302 drm_crtc_send_vblank_event(crtc, event); 303 spin_unlock_irq(&crtc->dev->event_lock); 304 } 305 306 static void mxsfb_crtc_atomic_enable(struct drm_crtc *crtc, 307 struct drm_atomic_state *state) 308 { 309 struct mxsfb_drm_private *mxsfb = to_mxsfb_drm_private(crtc->dev); 310 struct drm_device *drm = mxsfb->drm; 311 dma_addr_t paddr; 312 313 pm_runtime_get_sync(drm->dev); 314 mxsfb_enable_axi_clk(mxsfb); 315 316 drm_crtc_vblank_on(crtc); 317 318 mxsfb_crtc_mode_set_nofb(mxsfb); 319 320 /* Write cur_buf as well to avoid an initial corrupt frame */ 321 paddr = mxsfb_get_fb_paddr(crtc->primary); 322 if (paddr) { 323 writel(paddr, mxsfb->base + mxsfb->devdata->cur_buf); 324 writel(paddr, mxsfb->base + mxsfb->devdata->next_buf); 325 } 326 327 mxsfb_enable_controller(mxsfb); 328 } 329 330 static void mxsfb_crtc_atomic_disable(struct drm_crtc *crtc, 331 struct drm_atomic_state *state) 332 { 333 struct mxsfb_drm_private *mxsfb = to_mxsfb_drm_private(crtc->dev); 334 struct drm_device *drm = mxsfb->drm; 335 struct drm_pending_vblank_event *event; 336 337 mxsfb_disable_controller(mxsfb); 338 339 spin_lock_irq(&drm->event_lock); 340 event = crtc->state->event; 341 if (event) { 342 crtc->state->event = NULL; 343 drm_crtc_send_vblank_event(crtc, event); 344 } 345 spin_unlock_irq(&drm->event_lock); 346 347 drm_crtc_vblank_off(crtc); 348 349 mxsfb_disable_axi_clk(mxsfb); 350 pm_runtime_put_sync(drm->dev); 351 } 352 353 static int mxsfb_crtc_enable_vblank(struct drm_crtc *crtc) 354 { 355 struct mxsfb_drm_private *mxsfb = to_mxsfb_drm_private(crtc->dev); 356 357 /* Clear and enable VBLANK IRQ */ 358 writel(CTRL1_CUR_FRAME_DONE_IRQ, mxsfb->base + LCDC_CTRL1 + REG_CLR); 359 writel(CTRL1_CUR_FRAME_DONE_IRQ_EN, mxsfb->base + LCDC_CTRL1 + REG_SET); 360 361 return 0; 362 } 363 364 static void mxsfb_crtc_disable_vblank(struct drm_crtc *crtc) 365 { 366 struct mxsfb_drm_private *mxsfb = to_mxsfb_drm_private(crtc->dev); 367 368 /* Disable and clear VBLANK IRQ */ 369 writel(CTRL1_CUR_FRAME_DONE_IRQ_EN, mxsfb->base + LCDC_CTRL1 + REG_CLR); 370 writel(CTRL1_CUR_FRAME_DONE_IRQ, mxsfb->base + LCDC_CTRL1 + REG_CLR); 371 } 372 373 static const struct drm_crtc_helper_funcs mxsfb_crtc_helper_funcs = { 374 .atomic_check = mxsfb_crtc_atomic_check, 375 .atomic_flush = mxsfb_crtc_atomic_flush, 376 .atomic_enable = mxsfb_crtc_atomic_enable, 377 .atomic_disable = mxsfb_crtc_atomic_disable, 378 }; 379 380 static const struct drm_crtc_funcs mxsfb_crtc_funcs = { 381 .reset = drm_atomic_helper_crtc_reset, 382 .destroy = drm_crtc_cleanup, 383 .set_config = drm_atomic_helper_set_config, 384 .page_flip = drm_atomic_helper_page_flip, 385 .atomic_duplicate_state = drm_atomic_helper_crtc_duplicate_state, 386 .atomic_destroy_state = drm_atomic_helper_crtc_destroy_state, 387 .enable_vblank = mxsfb_crtc_enable_vblank, 388 .disable_vblank = mxsfb_crtc_disable_vblank, 389 }; 390 391 /* ----------------------------------------------------------------------------- 392 * Encoder 393 */ 394 395 static const struct drm_encoder_funcs mxsfb_encoder_funcs = { 396 .destroy = drm_encoder_cleanup, 397 }; 398 399 /* ----------------------------------------------------------------------------- 400 * Planes 401 */ 402 403 static int mxsfb_plane_atomic_check(struct drm_plane *plane, 404 struct drm_plane_state *plane_state) 405 { 406 struct mxsfb_drm_private *mxsfb = to_mxsfb_drm_private(plane->dev); 407 struct drm_crtc_state *crtc_state; 408 409 crtc_state = drm_atomic_get_new_crtc_state(plane_state->state, 410 &mxsfb->crtc); 411 412 return drm_atomic_helper_check_plane_state(plane_state, crtc_state, 413 DRM_PLANE_HELPER_NO_SCALING, 414 DRM_PLANE_HELPER_NO_SCALING, 415 false, true); 416 } 417 418 static void mxsfb_plane_primary_atomic_update(struct drm_plane *plane, 419 struct drm_plane_state *old_pstate) 420 { 421 struct mxsfb_drm_private *mxsfb = to_mxsfb_drm_private(plane->dev); 422 dma_addr_t paddr; 423 424 paddr = mxsfb_get_fb_paddr(plane); 425 if (paddr) 426 writel(paddr, mxsfb->base + mxsfb->devdata->next_buf); 427 } 428 429 static void mxsfb_plane_overlay_atomic_update(struct drm_plane *plane, 430 struct drm_plane_state *old_pstate) 431 { 432 struct mxsfb_drm_private *mxsfb = to_mxsfb_drm_private(plane->dev); 433 struct drm_plane_state *state = plane->state; 434 dma_addr_t paddr; 435 u32 ctrl; 436 437 paddr = mxsfb_get_fb_paddr(plane); 438 if (!paddr) { 439 writel(0, mxsfb->base + LCDC_AS_CTRL); 440 return; 441 } 442 443 /* 444 * HACK: The hardware seems to output 64 bytes of data of unknown 445 * origin, and then to proceed with the framebuffer. Until the reason 446 * is understood, live with the 16 initial invalid pixels on the first 447 * line and start 64 bytes within the framebuffer. 448 */ 449 paddr += 64; 450 451 writel(paddr, mxsfb->base + LCDC_AS_NEXT_BUF); 452 453 /* 454 * If the plane was previously disabled, write LCDC_AS_BUF as well to 455 * provide the first buffer. 456 */ 457 if (!old_pstate->fb) 458 writel(paddr, mxsfb->base + LCDC_AS_BUF); 459 460 ctrl = AS_CTRL_AS_ENABLE | AS_CTRL_ALPHA(255); 461 462 switch (state->fb->format->format) { 463 case DRM_FORMAT_XRGB4444: 464 ctrl |= AS_CTRL_FORMAT_RGB444 | AS_CTRL_ALPHA_CTRL_OVERRIDE; 465 break; 466 case DRM_FORMAT_ARGB4444: 467 ctrl |= AS_CTRL_FORMAT_ARGB4444 | AS_CTRL_ALPHA_CTRL_EMBEDDED; 468 break; 469 case DRM_FORMAT_XRGB1555: 470 ctrl |= AS_CTRL_FORMAT_RGB555 | AS_CTRL_ALPHA_CTRL_OVERRIDE; 471 break; 472 case DRM_FORMAT_ARGB1555: 473 ctrl |= AS_CTRL_FORMAT_ARGB1555 | AS_CTRL_ALPHA_CTRL_EMBEDDED; 474 break; 475 case DRM_FORMAT_RGB565: 476 ctrl |= AS_CTRL_FORMAT_RGB565 | AS_CTRL_ALPHA_CTRL_OVERRIDE; 477 break; 478 case DRM_FORMAT_XRGB8888: 479 ctrl |= AS_CTRL_FORMAT_RGB888 | AS_CTRL_ALPHA_CTRL_OVERRIDE; 480 break; 481 case DRM_FORMAT_ARGB8888: 482 ctrl |= AS_CTRL_FORMAT_ARGB8888 | AS_CTRL_ALPHA_CTRL_EMBEDDED; 483 break; 484 } 485 486 writel(ctrl, mxsfb->base + LCDC_AS_CTRL); 487 } 488 489 static bool mxsfb_format_mod_supported(struct drm_plane *plane, 490 uint32_t format, 491 uint64_t modifier) 492 { 493 return modifier == DRM_FORMAT_MOD_LINEAR; 494 } 495 496 static const struct drm_plane_helper_funcs mxsfb_plane_primary_helper_funcs = { 497 .atomic_check = mxsfb_plane_atomic_check, 498 .atomic_update = mxsfb_plane_primary_atomic_update, 499 }; 500 501 static const struct drm_plane_helper_funcs mxsfb_plane_overlay_helper_funcs = { 502 .atomic_check = mxsfb_plane_atomic_check, 503 .atomic_update = mxsfb_plane_overlay_atomic_update, 504 }; 505 506 static const struct drm_plane_funcs mxsfb_plane_funcs = { 507 .format_mod_supported = mxsfb_format_mod_supported, 508 .update_plane = drm_atomic_helper_update_plane, 509 .disable_plane = drm_atomic_helper_disable_plane, 510 .destroy = drm_plane_cleanup, 511 .reset = drm_atomic_helper_plane_reset, 512 .atomic_duplicate_state = drm_atomic_helper_plane_duplicate_state, 513 .atomic_destroy_state = drm_atomic_helper_plane_destroy_state, 514 }; 515 516 static const uint32_t mxsfb_primary_plane_formats[] = { 517 DRM_FORMAT_RGB565, 518 DRM_FORMAT_XRGB8888, 519 }; 520 521 static const uint32_t mxsfb_overlay_plane_formats[] = { 522 DRM_FORMAT_XRGB4444, 523 DRM_FORMAT_ARGB4444, 524 DRM_FORMAT_XRGB1555, 525 DRM_FORMAT_ARGB1555, 526 DRM_FORMAT_RGB565, 527 DRM_FORMAT_XRGB8888, 528 DRM_FORMAT_ARGB8888, 529 }; 530 531 static const uint64_t mxsfb_modifiers[] = { 532 DRM_FORMAT_MOD_LINEAR, 533 DRM_FORMAT_MOD_INVALID 534 }; 535 536 /* ----------------------------------------------------------------------------- 537 * Initialization 538 */ 539 540 int mxsfb_kms_init(struct mxsfb_drm_private *mxsfb) 541 { 542 struct drm_encoder *encoder = &mxsfb->encoder; 543 struct drm_crtc *crtc = &mxsfb->crtc; 544 int ret; 545 546 drm_plane_helper_add(&mxsfb->planes.primary, 547 &mxsfb_plane_primary_helper_funcs); 548 ret = drm_universal_plane_init(mxsfb->drm, &mxsfb->planes.primary, 1, 549 &mxsfb_plane_funcs, 550 mxsfb_primary_plane_formats, 551 ARRAY_SIZE(mxsfb_primary_plane_formats), 552 mxsfb_modifiers, DRM_PLANE_TYPE_PRIMARY, 553 NULL); 554 if (ret) 555 return ret; 556 557 if (mxsfb->devdata->has_overlay) { 558 drm_plane_helper_add(&mxsfb->planes.overlay, 559 &mxsfb_plane_overlay_helper_funcs); 560 ret = drm_universal_plane_init(mxsfb->drm, 561 &mxsfb->planes.overlay, 1, 562 &mxsfb_plane_funcs, 563 mxsfb_overlay_plane_formats, 564 ARRAY_SIZE(mxsfb_overlay_plane_formats), 565 mxsfb_modifiers, DRM_PLANE_TYPE_OVERLAY, 566 NULL); 567 if (ret) 568 return ret; 569 } 570 571 drm_crtc_helper_add(crtc, &mxsfb_crtc_helper_funcs); 572 ret = drm_crtc_init_with_planes(mxsfb->drm, crtc, 573 &mxsfb->planes.primary, NULL, 574 &mxsfb_crtc_funcs, NULL); 575 if (ret) 576 return ret; 577 578 encoder->possible_crtcs = drm_crtc_mask(crtc); 579 return drm_encoder_init(mxsfb->drm, encoder, &mxsfb_encoder_funcs, 580 DRM_MODE_ENCODER_NONE, NULL); 581 } 582