Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2844ea3f authored by Dave Airlie's avatar Dave Airlie
Browse files

Merge branch 'primary-plane' of git://people.freedesktop.org/~robclark/linux into drm-next 

Here's the latest iteration of the universal planes work, which I believe is
finally ready for merging.  Aside from the minor driver patches to use the
new drm_for_each_legacy_plane() macro for plane loops, these should all have
an r-b from Rob Clark now.

Actual userspace-visibility is currently hidden behind a
drm.universal_planes module parameter so that we can do some experimental
testing of this before flipping it on universally.

* 'primary-plane' of git://people.freedesktop.org/~robclark/linux:
  drm/doc: Update plane documentation and add plane helper library
  drm: Allow userspace to ask for universal plane list (v2)
  drm: Remove unused drm_crtc->fb
  drm: Replace crtc fb with primary plane fb (v3)
  drm/msm: Switch to universal plane API's
  drm: Add drm_crtc_init_with_planes() (v2)
  drm: Add plane type property (v2)
  drm: Add drm_universal_plane_init()
  drm: Add primary plane helpers (v3)
  drm: Make drm_crtc_check_viewport non-static
  drm/shmobile: Restrict plane loops to only operate on legacy planes
  drm/i915: Restrict plane loops to only operate on overlay planes (v2)
  drm/exynos: Restrict plane loops to only operate on overlay planes (v2)
  drm: Add support for multiple plane types (v2)
parents e3d6ddb3 6efa1f2f

Documentation/DocBook/drm.tmpl

+43 −7
Original line number Diff line number Diff line
@@ -1194,7 +1194,7 @@ int max_width, max_height;</synopsis> 
          pointer to CRTC functions.

        </para>

      </sect3>

      <sect3>

      <sect3 id="drm-kms-crtcops">

        <title>CRTC Operations</title>

        <sect4>

          <title>Set Configuration</title>
@@ -1335,15 +1335,47 @@ int max_width, max_height;</synopsis> 
	optionally scale it to a destination size. The result is then blended

	with or overlayed on top of a CRTC.

      </para>

      <para>

      The DRM core recognizes three types of planes:

      <itemizedlist>

        <listitem>

        DRM_PLANE_TYPE_PRIMARY represents a "main" plane for a CRTC.  Primary

        planes are the planes operated upon by by CRTC modesetting and flipping

        operations described in <xref linkend="drm-kms-crtcops"/>.

        </listitem>

        <listitem>

        DRM_PLANE_TYPE_CURSOR represents a "cursor" plane for a CRTC.  Cursor

        planes are the planes operated upon by the DRM_IOCTL_MODE_CURSOR and

        DRM_IOCTL_MODE_CURSOR2 ioctls.

        </listitem>

        <listitem>

        DRM_PLANE_TYPE_OVERLAY represents all non-primary, non-cursor planes.

        Some drivers refer to these types of planes as "sprites" internally.

        </listitem>

      </itemizedlist>

      For compatibility with legacy userspace, only overlay planes are made

      available to userspace by default.  Userspace clients may set the

      DRM_CLIENT_CAP_UNIVERSAL_PLANES client capability bit to indicate that

      they wish to receive a universal plane list containing all plane types.

      </para>

      <sect3>

        <title>Plane Initialization</title>

        <para>

          Planes are optional. To create a plane, a KMS drivers allocates and

          To create a plane, a KMS drivers allocates and

          zeroes an instances of struct <structname>drm_plane</structname>

          (possibly as part of a larger structure) and registers it with a call

          to <function>drm_plane_init</function>. The function takes a bitmask

          to <function>drm_universal_plane_init</function>. The function takes a bitmask

          of the CRTCs that can be associated with the plane, a pointer to the

          plane functions and a list of format supported formats.

          plane functions, a list of format supported formats, and the type of

          plane (primary, cursor, or overlay) being initialized.

        </para>

        <para>

          Cursor and overlay planes are optional.  All drivers should provide

          one primary plane per CRTC (although this requirement may change in

          the future); drivers that do not wish to provide special handling for

          primary planes may make use of the helper functions described in

          <xref linkend="drm-kms-planehelpers"/> to create and register a

          primary plane with standard capabilities.

        </para>

      </sect3>

      <sect3>
@@ -1774,7 +1806,7 @@ void intel_crt_init(struct drm_device *dev) 
  <sect1>

    <title>Mode Setting Helper Functions</title>

    <para>

      The CRTC, encoder and connector functions provided by the drivers

      The plane, CRTC, encoder and connector functions provided by the drivers

      implement the DRM API. They're called by the DRM core and ioctl handlers

      to handle device state changes and configuration request. As implementing

      those functions often requires logic not specific to drivers, mid-layer
@@ -1782,8 +1814,8 @@ void intel_crt_init(struct drm_device *dev) 
    </para>

    <para>

      The DRM core contains one mid-layer implementation. The mid-layer provides

      implementations of several CRTC, encoder and connector functions (called

      from the top of the mid-layer) that pre-process requests and call

      implementations of several plane, CRTC, encoder and connector functions

      (called from the top of the mid-layer) that pre-process requests and call

      lower-level functions provided by the driver (at the bottom of the

      mid-layer). For instance, the

      <function>drm_crtc_helper_set_config</function> function can be used to
@@ -2293,6 +2325,10 @@ void intel_crt_init(struct drm_device *dev) 
!Iinclude/linux/hdmi.h

!Edrivers/video/hdmi.c

    </sect2>

    <sect2>

      <title id="drm-kms-planehelpers">Plane Helper Reference</title>

!Edrivers/gpu/drm/drm_plane_helper.c Plane Helpers

    </sect2>

  </sect1>



  <!-- Internals: kms properties -->

drivers/gpu/drm/Makefile

+2 −1
Original line number Diff line number Diff line
@@ -13,7 +13,8 @@ drm-y := drm_auth.o drm_buffer.o drm_bufs.o drm_cache.o \ 
		drm_crtc.o drm_modes.o drm_edid.o \

		drm_info.o drm_debugfs.o drm_encoder_slave.o \

		drm_trace_points.o drm_global.o drm_prime.o \

		drm_rect.o drm_vma_manager.o drm_flip_work.o

		drm_rect.o drm_vma_manager.o drm_flip_work.o \

		drm_plane_helper.o



drm-$(CONFIG_COMPAT) += drm_ioc32.o

drm-$(CONFIG_DRM_GEM_CMA_HELPER) += drm_gem_cma_helper.o

drivers/gpu/drm/armada/armada_crtc.c

+12 −11
Original line number Diff line number Diff line
@@ -478,11 +478,12 @@ static int armada_drm_crtc_mode_set(struct drm_crtc *crtc, 
	unsigned i;

	bool interlaced;



	drm_framebuffer_reference(crtc->fb);

	drm_framebuffer_reference(crtc->primary->fb);



	interlaced = !!(adj->flags & DRM_MODE_FLAG_INTERLACE);



	i = armada_drm_crtc_calc_fb(dcrtc->crtc.fb, x, y, regs, interlaced);

	i = armada_drm_crtc_calc_fb(dcrtc->crtc.primary->fb,

				    x, y, regs, interlaced);



	rm = adj->crtc_hsync_start - adj->crtc_hdisplay;

	lm = adj->crtc_htotal - adj->crtc_hsync_end;
@@ -567,10 +568,10 @@ static int armada_drm_crtc_mode_set(struct drm_crtc *crtc, 
	}



	val = CFG_GRA_ENA | CFG_GRA_HSMOOTH;

	val |= CFG_GRA_FMT(drm_fb_to_armada_fb(dcrtc->crtc.fb)->fmt);

	val |= CFG_GRA_MOD(drm_fb_to_armada_fb(dcrtc->crtc.fb)->mod);

	val |= CFG_GRA_FMT(drm_fb_to_armada_fb(dcrtc->crtc.primary->fb)->fmt);

	val |= CFG_GRA_MOD(drm_fb_to_armada_fb(dcrtc->crtc.primary->fb)->mod);



	if (drm_fb_to_armada_fb(dcrtc->crtc.fb)->fmt > CFG_420)

	if (drm_fb_to_armada_fb(dcrtc->crtc.primary->fb)->fmt > CFG_420)

		val |= CFG_PALETTE_ENA;



	if (interlaced)
@@ -608,7 +609,7 @@ static int armada_drm_crtc_mode_set_base(struct drm_crtc *crtc, int x, int y, 
	struct armada_regs regs[4];

	unsigned i;



	i = armada_drm_crtc_calc_fb(crtc->fb, crtc->x, crtc->y, regs,

	i = armada_drm_crtc_calc_fb(crtc->primary->fb, crtc->x, crtc->y, regs,

				    dcrtc->interlaced);

	armada_reg_queue_end(regs, i);
@@ -616,7 +617,7 @@ static int armada_drm_crtc_mode_set_base(struct drm_crtc *crtc, int x, int y, 
	wait_event(dcrtc->frame_wait, !dcrtc->frame_work);



	/* Take a reference to the new fb as we're using it */

	drm_framebuffer_reference(crtc->fb);

	drm_framebuffer_reference(crtc->primary->fb);



	/* Update the base in the CRTC */

	armada_drm_crtc_update_regs(dcrtc, regs);
@@ -637,7 +638,7 @@ static void armada_drm_crtc_disable(struct drm_crtc *crtc) 
	struct armada_crtc *dcrtc = drm_to_armada_crtc(crtc);



	armada_drm_crtc_dpms(crtc, DRM_MODE_DPMS_OFF);

	armada_drm_crtc_finish_fb(dcrtc, crtc->fb, true);

	armada_drm_crtc_finish_fb(dcrtc, crtc->primary->fb, true);



	/* Power down most RAMs and FIFOs */

	writel_relaxed(CFG_PDWN256x32 | CFG_PDWN256x24 | CFG_PDWN256x8 |
@@ -904,7 +905,7 @@ static int armada_drm_crtc_page_flip(struct drm_crtc *crtc, 
	int ret;



	/* We don't support changing the pixel format */

	if (fb->pixel_format != crtc->fb->pixel_format)

	if (fb->pixel_format != crtc->primary->fb->pixel_format)

		return -EINVAL;



	work = kmalloc(sizeof(*work), GFP_KERNEL);
@@ -912,7 +913,7 @@ static int armada_drm_crtc_page_flip(struct drm_crtc *crtc, 
		return -ENOMEM;



	work->event = event;

	work->old_fb = dcrtc->crtc.fb;

	work->old_fb = dcrtc->crtc.primary->fb;



	i = armada_drm_crtc_calc_fb(fb, crtc->x, crtc->y, work->regs,

				    dcrtc->interlaced);
@@ -941,7 +942,7 @@ static int armada_drm_crtc_page_flip(struct drm_crtc *crtc, 
	 * will _not_ drop that reference on successful return from this

	 * function.  Simply mark this new framebuffer as the current one.

	 */

	dcrtc->crtc.fb = fb;

	dcrtc->crtc.primary->fb = fb;



	/*

	 * Finally, if the display is blanked, we won't receive an

drivers/gpu/drm/ast/ast_mode.c

+6 −6
Original line number Diff line number Diff line
@@ -81,7 +81,7 @@ static bool ast_get_vbios_mode_info(struct drm_crtc *crtc, struct drm_display_mo 
	u32 refresh_rate_index = 0, mode_id, color_index, refresh_rate;

	u32 hborder, vborder;



	switch (crtc->fb->bits_per_pixel) {

	switch (crtc->primary->fb->bits_per_pixel) {

	case 8:

		vbios_mode->std_table = &vbios_stdtable[VGAModeIndex];

		color_index = VGAModeIndex - 1;
@@ -176,7 +176,7 @@ static bool ast_get_vbios_mode_info(struct drm_crtc *crtc, struct drm_display_mo 
		ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0x8e, mode_id & 0xff);



		ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0x91, 0xa8);

		ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0x92, crtc->fb->bits_per_pixel);

		ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0x92, crtc->primary->fb->bits_per_pixel);

		ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0x93, adjusted_mode->clock / 1000);

		ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0x94, adjusted_mode->crtc_hdisplay);

		ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0x95, adjusted_mode->crtc_hdisplay >> 8);
@@ -340,7 +340,7 @@ static void ast_set_offset_reg(struct drm_crtc *crtc) 


	u16 offset;



	offset = crtc->fb->pitches[0] >> 3;

	offset = crtc->primary->fb->pitches[0] >> 3;

	ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0x13, (offset & 0xff));

	ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0xb0, (offset >> 8) & 0x3f);

}
@@ -365,7 +365,7 @@ static void ast_set_ext_reg(struct drm_crtc *crtc, struct drm_display_mode *mode 
	struct ast_private *ast = crtc->dev->dev_private;

	u8 jregA0 = 0, jregA3 = 0, jregA8 = 0;



	switch (crtc->fb->bits_per_pixel) {

	switch (crtc->primary->fb->bits_per_pixel) {

	case 8:

		jregA0 = 0x70;

		jregA3 = 0x01;
@@ -418,7 +418,7 @@ static void ast_set_sync_reg(struct drm_device *dev, struct drm_display_mode *mo 
static bool ast_set_dac_reg(struct drm_crtc *crtc, struct drm_display_mode *mode,

		     struct ast_vbios_mode_info *vbios_mode)

{

	switch (crtc->fb->bits_per_pixel) {

	switch (crtc->primary->fb->bits_per_pixel) {

	case 8:

		break;

	default:
@@ -490,7 +490,7 @@ static int ast_crtc_do_set_base(struct drm_crtc *crtc, 
		ast_bo_unreserve(bo);

	}



	ast_fb = to_ast_framebuffer(crtc->fb);

	ast_fb = to_ast_framebuffer(crtc->primary->fb);

	obj = ast_fb->obj;

	bo = gem_to_ast_bo(obj);

drivers/gpu/drm/bochs/bochs_kms.c

+2 −2
Original line number Diff line number Diff line
@@ -62,10 +62,10 @@ static int bochs_crtc_mode_set_base(struct drm_crtc *crtc, int x, int y, 
		}

	}



	if (WARN_ON(crtc->fb == NULL))

	if (WARN_ON(crtc->primary->fb == NULL))

		return -EINVAL;



	bochs_fb = to_bochs_framebuffer(crtc->fb);

	bochs_fb = to_bochs_framebuffer(crtc->primary->fb);

	bo = gem_to_bochs_bo(bochs_fb->obj);

	ret = ttm_bo_reserve(&bo->bo, true, false, false, 0);

	if (ret)