Commit 77dd1143 authored by Linus Torvalds's avatar Linus Torvalds
Browse files

Merge tag 'drm-fixes-2021-08-27' of git://anongit.freedesktop.org/drm/drm 

Pull drm fixes from Dave Airlie:
 "Last set of fixes for 5.14, nothing major a couple of i915, couple of
  imx and a few amdgpu. All pretty small.

  i915:
   - Fix syncmap memory leak
   - Drop redundant display port debug print

  amdgpu:
   - Fix for pinning display buffers multiple times
   - Fix delayed work handling for GFXOFF
   - Fix build when CONFIG_SUSPEND is not set

  imx:
   - fix planar offset calculations
   - fix accidental partial revert"

* tag 'drm-fixes-2021-08-27' of git://anongit.freedesktop.org/drm/drm:
  drm/i915/dp: Drop redundant debug print
  drm/i915: Fix syncmap memory leak
  drm/amdgpu: Fix build with missing pm_suspend_target_state module export
  drm/amdgpu: Cancel delayed work when GFXOFF is disabled
  drm/amdgpu: use the preferred pin domain after the check
  drm/imx: ipuv3-plane: fix accidental partial revert of 8 pixel alignment fix
  gpu: ipu-v3: Fix i.MX IPU-v3 offset calculations for (semi)planar U/V formats
parents 73367f05 9fe4f5a2

drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c

+1 −1
Original line number Diff line number Diff line
@@ -1040,7 +1040,7 @@ void amdgpu_acpi_detect(void) 
 */

bool amdgpu_acpi_is_s0ix_supported(struct amdgpu_device *adev)

{

#if IS_ENABLED(CONFIG_AMD_PMC) && IS_ENABLED(CONFIG_PM_SLEEP)

#if IS_ENABLED(CONFIG_AMD_PMC) && IS_ENABLED(CONFIG_SUSPEND)

	if (acpi_gbl_FADT.flags & ACPI_FADT_LOW_POWER_S0) {

		if (adev->flags & AMD_IS_APU)

			return pm_suspend_target_state == PM_SUSPEND_TO_IDLE;

drivers/gpu/drm/amd/amdgpu/amdgpu_device.c

+5 −6
Original line number Diff line number Diff line
@@ -2777,13 +2777,12 @@ static void amdgpu_device_delay_enable_gfx_off(struct work_struct *work) 
	struct amdgpu_device *adev =

		container_of(work, struct amdgpu_device, gfx.gfx_off_delay_work.work);



	mutex_lock(&adev->gfx.gfx_off_mutex);

	if (!adev->gfx.gfx_off_state && !adev->gfx.gfx_off_req_count) {

	WARN_ON_ONCE(adev->gfx.gfx_off_state);

	WARN_ON_ONCE(adev->gfx.gfx_off_req_count);



	if (!amdgpu_dpm_set_powergating_by_smu(adev, AMD_IP_BLOCK_TYPE_GFX, true))

		adev->gfx.gfx_off_state = true;

}

	mutex_unlock(&adev->gfx.gfx_off_mutex);

}



/**

 * amdgpu_device_ip_suspend_phase1 - run suspend for hardware IPs (phase 1)

drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c

+25 −11
Original line number Diff line number Diff line
@@ -563,24 +563,38 @@ void amdgpu_gfx_off_ctrl(struct amdgpu_device *adev, bool enable) 


	mutex_lock(&adev->gfx.gfx_off_mutex);



	if (!enable)

		adev->gfx.gfx_off_req_count++;

	else if (adev->gfx.gfx_off_req_count > 0)

	if (enable) {

		/* If the count is already 0, it means there's an imbalance bug somewhere.

		 * Note that the bug may be in a different caller than the one which triggers the

		 * WARN_ON_ONCE.

		 */

		if (WARN_ON_ONCE(adev->gfx.gfx_off_req_count == 0))

			goto unlock;



		adev->gfx.gfx_off_req_count--;



	if (enable && !adev->gfx.gfx_off_state && !adev->gfx.gfx_off_req_count) {

		if (adev->gfx.gfx_off_req_count == 0 && !adev->gfx.gfx_off_state)

			schedule_delayed_work(&adev->gfx.gfx_off_delay_work, GFX_OFF_DELAY_ENABLE);

	} else if (!enable && adev->gfx.gfx_off_state) {

		if (!amdgpu_dpm_set_powergating_by_smu(adev, AMD_IP_BLOCK_TYPE_GFX, false)) {

	} else {

		if (adev->gfx.gfx_off_req_count == 0) {

			cancel_delayed_work_sync(&adev->gfx.gfx_off_delay_work);



			if (adev->gfx.gfx_off_state &&

			    !amdgpu_dpm_set_powergating_by_smu(adev, AMD_IP_BLOCK_TYPE_GFX, false)) {

				adev->gfx.gfx_off_state = false;



				if (adev->gfx.funcs->init_spm_golden) {

				dev_dbg(adev->dev, "GFXOFF is disabled, re-init SPM golden settings\n");

					dev_dbg(adev->dev,

						"GFXOFF is disabled, re-init SPM golden settings\n");

					amdgpu_gfx_init_spm_golden(adev);

				}

			}

		}



		adev->gfx.gfx_off_req_count++;

	}



unlock:

	mutex_unlock(&adev->gfx.gfx_off_mutex);

}

drivers/gpu/drm/amd/amdgpu/amdgpu_object.c

+5 −5
Original line number Diff line number Diff line
@@ -920,11 +920,6 @@ int amdgpu_bo_pin_restricted(struct amdgpu_bo *bo, u32 domain, 
			return -EINVAL;

	}



	/* This assumes only APU display buffers are pinned with (VRAM|GTT).

	 * See function amdgpu_display_supported_domains()

	 */

	domain = amdgpu_bo_get_preferred_pin_domain(adev, domain);



	if (bo->tbo.pin_count) {

		uint32_t mem_type = bo->tbo.resource->mem_type;

		uint32_t mem_flags = bo->tbo.resource->placement;
@@ -949,6 +944,11 @@ int amdgpu_bo_pin_restricted(struct amdgpu_bo *bo, u32 domain, 
		return 0;

	}



	/* This assumes only APU display buffers are pinned with (VRAM|GTT).

	 * See function amdgpu_display_supported_domains()

	 */

	domain = amdgpu_bo_get_preferred_pin_domain(adev, domain);



	if (bo->tbo.base.import_attach)

		dma_buf_pin(bo->tbo.base.import_attach);

drivers/gpu/drm/i915/display/intel_dp.c

+2 −7
Original line number Diff line number Diff line
@@ -3850,23 +3850,18 @@ static void intel_dp_check_device_service_irq(struct intel_dp *intel_dp) 


static void intel_dp_check_link_service_irq(struct intel_dp *intel_dp)

{

	struct drm_i915_private *i915 = dp_to_i915(intel_dp);

	u8 val;



	if (intel_dp->dpcd[DP_DPCD_REV] < 0x11)

		return;



	if (drm_dp_dpcd_readb(&intel_dp->aux,

			      DP_LINK_SERVICE_IRQ_VECTOR_ESI0, &val) != 1 || !val) {

		drm_dbg_kms(&i915->drm, "Error in reading link service irq vector\n");

			      DP_LINK_SERVICE_IRQ_VECTOR_ESI0, &val) != 1 || !val)

		return;

	}



	if (drm_dp_dpcd_writeb(&intel_dp->aux,

			       DP_LINK_SERVICE_IRQ_VECTOR_ESI0, val) != 1) {

		drm_dbg_kms(&i915->drm, "Error in writing link service irq vector\n");

			       DP_LINK_SERVICE_IRQ_VECTOR_ESI0, val) != 1)

		return;

	}



	if (val & HDMI_LINK_STATUS_CHANGED)

		intel_dp_handle_hdmi_link_status_change(intel_dp);