]> Git Repo - J-linux.git/commitdiff
Merge tag 'amd-drm-next-5.20-2022-07-14' of https://gitlab.freedesktop.org/agd5f...
authorDave Airlie <[email protected]>
Fri, 15 Jul 2022 05:07:24 +0000 (15:07 +1000)
committerDave Airlie <[email protected]>
Fri, 15 Jul 2022 05:07:26 +0000 (15:07 +1000)
amd-drm-next-5.20-2022-07-14:

amdgpu:
- DCN3.2 updates
- DC SubVP support
- DP MST fixes
- Audio fixes
- DC code cleanup
- SMU13 updates
- Adjust GART size on newer APUs for S/G display
- Soft reset for GFX 11
- Soft reset for SDMA 6
- Add gfxoff status query for vangogh
- Improve BO domain pinning
- Fix timestamps for cursor only commits
- MES fixes
- DCN 3.1.4 support
- Misc fixes
- Misc code cleanup

amdkfd:
- Simplify GPUVM validation
- Unified memory for CWSR save/restore area
- fix possible list corruption on queue failure

radeon:
- Fix bogus power of two warning

UAPI:
- Unified memory for CWSR save/restore area for KFD
  Proposed userspace: https://lists.freedesktop.org/archives/amd-gfx/2022-June/080952.html

Signed-off-by: Dave Airlie <[email protected]>
From: Alex Deucher <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
1  2 
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c
drivers/gpu/drm/amd/display/dc/inc/dc_link_dp.h
drivers/gpu/drm/amd/pm/swsmu/smu11/smu_v11_0.c
drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
drivers/gpu/drm/radeon/radeon_device.c

index 41e4774abdb0cc504f71bd895ee600442858711c,c03f300851fad8e8c36b960f919f6ca418bfe61c..3e83fed540e889bd2576552e50cea2838fa7568f
  #include <linux/pci.h>
  #include <linux/firmware.h>
  #include <linux/component.h>
+ #include <linux/dmi.h>
  
  #include <drm/display/drm_dp_mst_helper.h>
  #include <drm/display/drm_hdmi_helper.h>
  #include <drm/drm_atomic.h>
  #include <drm/drm_atomic_uapi.h>
  #include <drm/drm_atomic_helper.h>
 +#include <drm/drm_blend.h>
  #include <drm/drm_fb_helper.h>
  #include <drm/drm_fourcc.h>
  #include <drm/drm_edid.h>
  #include <drm/drm_vblank.h>
  #include <drm/drm_audio_component.h>
 +#include <drm/drm_gem_atomic_helper.h>
  
  #include "ivsrcid/dcn/irqsrcs_dcn_1_0.h"
  
@@@ -119,6 -118,8 +120,8 @@@ MODULE_FIRMWARE(FIRMWARE_DIMGREY_CAVEFI
  MODULE_FIRMWARE(FIRMWARE_BEIGE_GOBY_DMUB);
  #define FIRMWARE_YELLOW_CARP_DMUB "amdgpu/yellow_carp_dmcub.bin"
  MODULE_FIRMWARE(FIRMWARE_YELLOW_CARP_DMUB);
+ #define FIRMWARE_DCN_314_DMUB "amdgpu/dcn_3_1_4_dmcub.bin"
+ MODULE_FIRMWARE(FIRMWARE_DCN_314_DMUB);
  #define FIRMWARE_DCN_315_DMUB "amdgpu/dcn_3_1_5_dmcub.bin"
  MODULE_FIRMWARE(FIRMWARE_DCN_315_DMUB);
  #define FIRMWARE_DCN316_DMUB "amdgpu/dcn_3_1_6_dmcub.bin"
@@@ -473,6 -474,26 +476,26 @@@ static void dm_pflip_high_irq(void *int
                     vrr_active, (int) !e);
  }
  
+ static void dm_crtc_handle_vblank(struct amdgpu_crtc *acrtc)
+ {
+       struct drm_crtc *crtc = &acrtc->base;
+       struct drm_device *dev = crtc->dev;
+       unsigned long flags;
+       drm_crtc_handle_vblank(crtc);
+       spin_lock_irqsave(&dev->event_lock, flags);
+       /* Send completion event for cursor-only commits */
+       if (acrtc->event && acrtc->pflip_status != AMDGPU_FLIP_SUBMITTED) {
+               drm_crtc_send_vblank_event(crtc, acrtc->event);
+               drm_crtc_vblank_put(crtc);
+               acrtc->event = NULL;
+       }
+       spin_unlock_irqrestore(&dev->event_lock, flags);
+ }
  static void dm_vupdate_high_irq(void *interrupt_params)
  {
        struct common_irq_params *irq_params = interrupt_params;
                 * if a pageflip happened inside front-porch.
                 */
                if (vrr_active) {
-                       drm_crtc_handle_vblank(&acrtc->base);
+                       dm_crtc_handle_vblank(acrtc);
  
                        /* BTR processing for pre-DCE12 ASICs */
                        if (acrtc->dm_irq_params.stream &&
@@@ -563,7 -584,7 +586,7 @@@ static void dm_crtc_high_irq(void *inte
         * to dm_vupdate_high_irq after end of front-porch.
         */
        if (!vrr_active)
-               drm_crtc_handle_vblank(&acrtc->base);
+               dm_crtc_handle_vblank(acrtc);
  
        /**
         * Following stuff must happen at start of vblank, for crc
@@@ -1403,6 -1424,41 +1426,41 @@@ static bool dm_should_disable_stutter(s
        return false;
  }
  
+ static const struct dmi_system_id hpd_disconnect_quirk_table[] = {
+       {
+               .matches = {
+                       DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+                       DMI_MATCH(DMI_PRODUCT_NAME, "Precision 3660"),
+               },
+       },
+       {
+               .matches = {
+                       DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+                       DMI_MATCH(DMI_PRODUCT_NAME, "Precision 3260"),
+               },
+       },
+       {
+               .matches = {
+                       DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+                       DMI_MATCH(DMI_PRODUCT_NAME, "Precision 3460"),
+               },
+       },
+       {}
+ };
+ static void retrieve_dmi_info(struct amdgpu_display_manager *dm)
+ {
+       const struct dmi_system_id *dmi_id;
+       dm->aux_hpd_discon_quirk = false;
+       dmi_id = dmi_first_match(hpd_disconnect_quirk_table);
+       if (dmi_id) {
+               dm->aux_hpd_discon_quirk = true;
+               DRM_INFO("aux_hpd_discon_quirk attached\n");
+       }
+ }
  static int amdgpu_dm_init(struct amdgpu_device *adev)
  {
        struct dc_init_data init_data;
  
        init_data.flags.enable_mipi_converter_optimization = true;
  
+       init_data.dcn_reg_offsets = adev->reg_offset[DCE_HWIP][0];
+       init_data.nbio_reg_offsets = adev->reg_offset[NBIO_HWIP][0];
        INIT_LIST_HEAD(&adev->dm.da_list);
+       retrieve_dmi_info(&adev->dm);
        /* Display Core create. */
        adev->dm.dc = dc_create(&init_data);
  
        if (amdgpu_dc_debug_mask & DC_DISABLE_CLOCK_GATING)
                adev->dm.dc->debug.disable_clock_gate = true;
  
+       if (amdgpu_dc_debug_mask & DC_FORCE_SUBVP_MCLK_SWITCH)
+               adev->dm.dc->debug.force_subvp_mclk_switch = true;
        r = dm_dmub_hw_init(adev);
        if (r) {
                DRM_ERROR("DMUB interface failed to initialize: status=%d\n", r);
  #if defined(CONFIG_DRM_AMD_SECURE_DISPLAY)
        adev->dm.crc_rd_wrk = amdgpu_dm_crtc_secure_display_create_work();
  #endif
-       if (dc_enable_dmub_notifications(adev->dm.dc)) {
+       if (dc_is_dmub_outbox_supported(adev->dm.dc)) {
                init_completion(&adev->dm.dmub_aux_transfer_done);
                adev->dm.dmub_notify = kzalloc(sizeof(struct dmub_notification), GFP_KERNEL);
                if (!adev->dm.dmub_notify) {
                goto error;
        }
  
+       /* Enable outbox notification only after IRQ handlers are registered and DMUB is alive.
+        * It is expected that DMUB will resend any pending notifications at this point, for
+        * example HPD from DPIA.
+        */
+       if (dc_is_dmub_outbox_supported(adev->dm.dc))
+               dc_enable_dmub_outbox(adev->dm.dc);
        /* create fake encoders for MST */
        dm_dp_create_fake_mst_encoders(adev);
  
@@@ -1941,6 -2013,10 +2015,10 @@@ static int dm_dmub_sw_init(struct amdgp
                dmub_asic = (adev->external_rev_id == YELLOW_CARP_B0) ? DMUB_ASIC_DCN31B : DMUB_ASIC_DCN31;
                fw_name_dmub = FIRMWARE_YELLOW_CARP_DMUB;
                break;
+       case IP_VERSION(3, 1, 4):
+               dmub_asic = DMUB_ASIC_DCN314;
+               fw_name_dmub = FIRMWARE_DCN_314_DMUB;
+               break;
        case IP_VERSION(3, 1, 5):
                dmub_asic = DMUB_ASIC_DCN315;
                fw_name_dmub = FIRMWARE_DCN_315_DMUB;
@@@ -2625,9 -2701,6 +2703,6 @@@ static int dm_resume(void *handle
                 */
                link_enc_cfg_copy(adev->dm.dc->current_state, dc_state);
  
-               if (dc_enable_dmub_notifications(adev->dm.dc))
-                       amdgpu_dm_outbox_init(adev);
                r = dm_dmub_hw_init(adev);
                if (r)
                        DRM_ERROR("DMUB interface failed to initialize: status=%d\n", r);
                        }
                }
  
+               if (dc_is_dmub_outbox_supported(adev->dm.dc)) {
+                       amdgpu_dm_outbox_init(adev);
+                       dc_enable_dmub_outbox(adev->dm.dc);
+               }
                WARN_ON(!dc_commit_state(dm->dc, dc_state));
  
                dm_gpureset_commit_state(dm->cached_dc_state, dm);
        /* TODO: Remove dc_state->dccg, use dc->dccg directly. */
        dc_resource_state_construct(dm->dc, dm_state->context);
  
-       /* Re-enable outbox interrupts for DPIA. */
-       if (dc_enable_dmub_notifications(adev->dm.dc))
-               amdgpu_dm_outbox_init(adev);
        /* Before powering on DC we need to re-initialize DMUB. */
        dm_dmub_hw_resume(adev);
  
+       /* Re-enable outbox interrupts for DPIA. */
+       if (dc_is_dmub_outbox_supported(adev->dm.dc)) {
+               amdgpu_dm_outbox_init(adev);
+               dc_enable_dmub_outbox(adev->dm.dc);
+       }
        /* power on hardware */
        dc_set_power_state(dm->dc, DC_ACPI_CM_POWER_STATE_D0);
  
                if (!dc_link_detect_sink(aconnector->dc_link, &new_connection_type))
                        DRM_ERROR("KMS: Failed to detect connector\n");
  
-               if (aconnector->base.force && new_connection_type == dc_connection_none)
+               if (aconnector->base.force && new_connection_type == dc_connection_none) {
                        emulated_link_detect(aconnector->dc_link);
-               else
+               } else {
+                       mutex_lock(&dm->dc_lock);
                        dc_link_detect(aconnector->dc_link, DETECT_REASON_HPD);
+                       mutex_unlock(&dm->dc_lock);
+               }
  
                if (aconnector->fake_enable && aconnector->dc_link->local_sink)
                        aconnector->fake_enable = false;
@@@ -3039,6 -3122,7 +3124,7 @@@ static void handle_hpd_irq_helper(struc
  #ifdef CONFIG_DRM_AMD_DC_HDCP
        struct dm_connector_state *dm_con_state = to_dm_connector_state(connector->state);
  #endif
+       bool ret = false;
  
        if (adev->dm.disable_hpd_irq)
                return;
  
                if (aconnector->base.force == DRM_FORCE_UNSPECIFIED)
                        drm_kms_helper_connector_hotplug_event(connector);
+       } else {
+               mutex_lock(&adev->dm.dc_lock);
+               ret = dc_link_detect(aconnector->dc_link, DETECT_REASON_HPD);
+               mutex_unlock(&adev->dm.dc_lock);
+               if (ret) {
+                       amdgpu_dm_update_connector_after_detect(aconnector);
  
-       } else if (dc_link_detect(aconnector->dc_link, DETECT_REASON_HPD)) {
-               amdgpu_dm_update_connector_after_detect(aconnector);
-               drm_modeset_lock_all(dev);
-               dm_restore_drm_connector_state(dev, connector);
-               drm_modeset_unlock_all(dev);
+                       drm_modeset_lock_all(dev);
+                       dm_restore_drm_connector_state(dev, connector);
+                       drm_modeset_unlock_all(dev);
  
-               if (aconnector->base.force == DRM_FORCE_UNSPECIFIED)
-                       drm_kms_helper_connector_hotplug_event(connector);
+                       if (aconnector->base.force == DRM_FORCE_UNSPECIFIED)
+                               drm_kms_helper_connector_hotplug_event(connector);
+               }
        }
        mutex_unlock(&aconnector->hpd_lock);
  
                        drm_modeset_unlock_all(dev);
  
                        drm_kms_helper_connector_hotplug_event(connector);
-               } else if (dc_link_detect(dc_link, DETECT_REASON_HPDRX)) {
+               } else {
+                       bool ret = false;
  
-                       if (aconnector->fake_enable)
-                               aconnector->fake_enable = false;
+                       mutex_lock(&adev->dm.dc_lock);
+                       ret = dc_link_detect(dc_link, DETECT_REASON_HPDRX);
+                       mutex_unlock(&adev->dm.dc_lock);
  
-                       amdgpu_dm_update_connector_after_detect(aconnector);
+                       if (ret) {
+                               if (aconnector->fake_enable)
+                                       aconnector->fake_enable = false;
  
+                               amdgpu_dm_update_connector_after_detect(aconnector);
  
-                       drm_modeset_lock_all(dev);
-                       dm_restore_drm_connector_state(dev, connector);
-                       drm_modeset_unlock_all(dev);
+                               drm_modeset_lock_all(dev);
+                               dm_restore_drm_connector_state(dev, connector);
+                               drm_modeset_unlock_all(dev);
  
-                       drm_kms_helper_connector_hotplug_event(connector);
+                               drm_kms_helper_connector_hotplug_event(connector);
+                       }
                }
        }
  #ifdef CONFIG_DRM_AMD_DC_HDCP
@@@ -3820,7 -3914,8 +3916,8 @@@ static int amdgpu_dm_mode_config_init(s
        adev_to_drm(adev)->mode_config.max_height = 16384;
  
        adev_to_drm(adev)->mode_config.preferred_depth = 24;
-       adev_to_drm(adev)->mode_config.prefer_shadow = 1;
+       /* disable prefer shadow for now due to hibernation issues */
+       adev_to_drm(adev)->mode_config.prefer_shadow = 0;
        /* indicates support for immediate flip */
        adev_to_drm(adev)->mode_config.async_page_flip = true;
  
@@@ -4221,6 -4316,7 +4318,7 @@@ static int amdgpu_dm_initialize_drm_dev
        case IP_VERSION(3, 0, 0):
        case IP_VERSION(3, 1, 2):
        case IP_VERSION(3, 1, 3):
+       case IP_VERSION(3, 1, 4):
        case IP_VERSION(3, 1, 5):
        case IP_VERSION(3, 1, 6):
        case IP_VERSION(3, 2, 0):
                switch (adev->ip_versions[DCE_HWIP][0]) {
                case IP_VERSION(3, 1, 2):
                case IP_VERSION(3, 1, 3):
+               case IP_VERSION(3, 1, 4):
                case IP_VERSION(3, 1, 5):
                case IP_VERSION(3, 1, 6):
                case IP_VERSION(3, 2, 0):
                if (aconnector->base.force && new_connection_type == dc_connection_none) {
                        emulated_link_detect(link);
                        amdgpu_dm_update_connector_after_detect(aconnector);
+               } else {
+                       bool ret = false;
  
-               } else if (dc_link_detect(link, DETECT_REASON_BOOT)) {
-                       amdgpu_dm_update_connector_after_detect(aconnector);
-                       register_backlight_device(dm, link);
-                       if (dm->num_of_edps)
-                               update_connector_ext_caps(aconnector);
-                       if (psr_feature_enabled)
-                               amdgpu_dm_set_psr_caps(link);
-                       /* TODO: Fix vblank control helpers to delay PSR entry to allow this when
-                        * PSR is also supported.
-                        */
-                       if (link->psr_settings.psr_feature_enabled)
-                               adev_to_drm(adev)->vblank_disable_immediate = false;
-               }
+                       mutex_lock(&dm->dc_lock);
+                       ret = dc_link_detect(link, DETECT_REASON_BOOT);
+                       mutex_unlock(&dm->dc_lock);
+                       if (ret) {
+                               amdgpu_dm_update_connector_after_detect(aconnector);
+                               register_backlight_device(dm, link);
+                               if (dm->num_of_edps)
+                                       update_connector_ext_caps(aconnector);
  
+                               if (psr_feature_enabled)
+                                       amdgpu_dm_set_psr_caps(link);
  
+                               /* TODO: Fix vblank control helpers to delay PSR entry to allow this when
+                                * PSR is also supported.
+                                */
+                               if (link->psr_settings.psr_feature_enabled)
+                                       adev_to_drm(adev)->vblank_disable_immediate = false;
+                       }
+               }
        }
  
        /* Software is initialized. Now we can register interrupt handlers. */
                case IP_VERSION(3, 0, 1):
                case IP_VERSION(3, 1, 2):
                case IP_VERSION(3, 1, 3):
+               case IP_VERSION(3, 1, 4):
                case IP_VERSION(3, 1, 5):
                case IP_VERSION(3, 1, 6):
                case IP_VERSION(3, 2, 0):
@@@ -4545,6 -4650,7 +4652,7 @@@ static int dm_early_init(void *handle
                case IP_VERSION(2, 1, 0):
                case IP_VERSION(3, 1, 2):
                case IP_VERSION(3, 1, 3):
+               case IP_VERSION(3, 1, 4):
                case IP_VERSION(3, 1, 5):
                case IP_VERSION(3, 1, 6):
                case IP_VERSION(3, 2, 0):
@@@ -5295,6 -5401,7 +5403,7 @@@ get_plane_modifiers(struct amdgpu_devic
                        add_gfx10_1_modifiers(adev, mods, &size, &capacity);
                break;
        case AMDGPU_FAMILY_GC_11_0_0:
+       case AMDGPU_FAMILY_GC_11_0_2:
                add_gfx11_modifiers(adev, mods, &size, &capacity);
                break;
        }
@@@ -5474,7 -5581,7 +5583,7 @@@ fill_blending_from_plane_state(const st
                        }
                }
  
-               if (per_pixel_alpha && plane_state->pixel_blend_mode == DRM_MODE_BLEND_COVERAGE)
+               if (*per_pixel_alpha && plane_state->pixel_blend_mode == DRM_MODE_BLEND_COVERAGE)
                        *pre_multiplied_alpha = false;
        }
  
@@@ -6767,7 -6874,7 +6876,7 @@@ dm_crtc_duplicate_state(struct drm_crt
        return &state->base;
  }
  
 -#ifdef CONFIG_DRM_AMD_SECURE_DISPLAY
 +#ifdef CONFIG_DEBUG_FS
  static int amdgpu_dm_crtc_late_register(struct drm_crtc *crtc)
  {
        crtc_debugfs_init(crtc);
@@@ -6868,7 -6975,7 +6977,7 @@@ static const struct drm_crtc_funcs amdg
        .enable_vblank = dm_enable_vblank,
        .disable_vblank = dm_disable_vblank,
        .get_vblank_timestamp = drm_crtc_vblank_helper_get_vblank_timestamp,
 -#if defined(CONFIG_DRM_AMD_SECURE_DISPLAY)
 +#if defined(CONFIG_DEBUG_FS)
        .late_register = amdgpu_dm_crtc_late_register,
  #endif
  };
@@@ -7213,12 -7320,10 +7322,10 @@@ create_validate_stream_for_sink(struct 
                        break;
                }
  
-               if (stream->signal == SIGNAL_TYPE_DISPLAY_PORT_MST)
+               dc_result = dc_validate_stream(adev->dm.dc, stream);
+               if (dc_result == DC_OK && stream->signal == SIGNAL_TYPE_DISPLAY_PORT_MST)
                        dc_result = dm_dp_mst_is_port_support_mode(aconnector, stream);
  
-               if (dc_result == DC_OK)
-                       dc_result = dc_validate_stream(adev->dm.dc, stream);
                if (dc_result != DC_OK) {
                        DRM_DEBUG_KMS("Mode %dx%d (clk %d) failed DC validation with error %d (%s)\n",
                                      drm_mode->hdisplay,
@@@ -7775,10 -7880,6 +7882,10 @@@ static int dm_plane_helper_prepare_fb(s
                goto error_unpin;
        }
  
 +      r = drm_gem_plane_helper_prepare_fb(plane, new_state);
 +      if (unlikely(r != 0))
 +              goto error_unpin;
 +
        amdgpu_bo_unreserve(rbo);
  
        afb->address = amdgpu_bo_gpu_offset(rbo);
@@@ -8651,7 -8752,7 +8758,7 @@@ static int amdgpu_dm_i2c_xfer(struct i2
  
        if (dc_submit_i2c(
                        ddc_service->ctx->dc,
-                       ddc_service->ddc_pin->hw_info.ddc_channel,
+                       ddc_service->link->link_index,
                        &cmd))
                result = num;
  
@@@ -8687,8 -8788,6 +8794,6 @@@ create_i2c(struct ddc_service *ddc_serv
        snprintf(i2c->base.name, sizeof(i2c->base.name), "AMDGPU DM i2c hw bus %d", link_index);
        i2c_set_adapdata(&i2c->base, i2c);
        i2c->ddc_service = ddc_service;
-       if (i2c->ddc_service->ddc_pin)
-               i2c->ddc_service->ddc_pin->hw_info.ddc_channel = link_index;
  
        return i2c;
  }
@@@ -9313,9 -9412,12 +9418,10 @@@ static void amdgpu_dm_commit_planes(str
        struct dm_crtc_state *dm_old_crtc_state =
                        to_dm_crtc_state(drm_atomic_get_old_crtc_state(state, pcrtc));
        int planes_count = 0, vpos, hpos;
 -      long r;
        unsigned long flags;
 -      struct amdgpu_bo *abo;
        uint32_t target_vblank, last_flip_vblank;
        bool vrr_active = amdgpu_dm_vrr_active(acrtc_state);
+       bool cursor_update = false;
        bool pflip_present = false;
        struct {
                struct dc_surface_update surface_updates[MAX_SURFACES];
                struct dm_plane_state *dm_new_plane_state = to_dm_plane_state(new_plane_state);
  
                /* Cursor plane is handled after stream updates */
-               if (plane->type == DRM_PLANE_TYPE_CURSOR)
+               if (plane->type == DRM_PLANE_TYPE_CURSOR) {
+                       if ((fb && crtc == pcrtc) ||
+                           (old_plane_state->fb && old_plane_state->crtc == pcrtc))
+                               cursor_update = true;
                        continue;
+               }
  
                if (!fb || !crtc || pcrtc != crtc)
                        continue;
                        continue;
                }
  
 -              abo = gem_to_amdgpu_bo(fb->obj[0]);
 -
 -              /*
 -               * Wait for all fences on this FB. Do limited wait to avoid
 -               * deadlock during GPU reset when this fence will not signal
 -               * but we hold reservation lock for the BO.
 -               */
 -              r = dma_resv_wait_timeout(abo->tbo.base.resv,
 -                                        DMA_RESV_USAGE_WRITE, false,
 -                                        msecs_to_jiffies(5000));
 -              if (unlikely(r <= 0))
 -                      DRM_ERROR("Waiting for fences timed out!");
 -
                fill_dc_plane_info_and_addr(
                        dm->adev, new_plane_state,
                        afb->tiling_flags,
                                bundle->stream_update.vrr_infopacket =
                                        &acrtc_state->stream->vrr_infopacket;
                }
+       } else if (cursor_update && acrtc_state->active_planes > 0 &&
+                  acrtc_attach->base.state->event) {
+               drm_crtc_vblank_get(pcrtc);
+               spin_lock_irqsave(&pcrtc->dev->event_lock, flags);
+               acrtc_attach->event = acrtc_attach->base.state->event;
+               acrtc_attach->base.state->event = NULL;
+               spin_unlock_irqrestore(&pcrtc->dev->event_lock, flags);
        }
  
        /* Update the planes if changed or disable if we don't have any. */
@@@ -9742,14 -9872,9 +9863,14 @@@ static void amdgpu_dm_atomic_commit_tai
        struct dm_crtc_state *dm_old_crtc_state, *dm_new_crtc_state;
        int crtc_disable_count = 0;
        bool mode_set_reset_required = false;
 +      int r;
  
        trace_amdgpu_dm_atomic_commit_tail_begin(state);
  
 +      r = drm_atomic_helper_wait_for_fences(dev, state, false);
 +      if (unlikely(r))
 +              DRM_ERROR("Waiting for fences timed out!");
 +
        drm_atomic_helper_update_legacy_modeset_state(dev, state);
  
        dm_state = dm_atomic_get_new_state(state);
index b64507f294ca86143c257af140233aca28aca28e,49bdcbf0a5920b1f46506cf19c3e716ba86a6bf4..a1f40d0cd41cf97a912d580f7a8ca810c28b28fa
@@@ -871,18 -871,28 +871,18 @@@ static int psr_capability_show(struct s
  }
  
  /*
 - * Returns the current and maximum output bpc for the connector.
 - * Example usage: cat /sys/kernel/debug/dri/0/DP-1/output_bpc
 + * Returns the current bpc for the crtc.
 + * Example usage: cat /sys/kernel/debug/dri/0/crtc-0/amdgpu_current_bpc
   */
 -static int output_bpc_show(struct seq_file *m, void *data)
 +static int amdgpu_current_bpc_show(struct seq_file *m, void *data)
  {
 -      struct drm_connector *connector = m->private;
 -      struct drm_device *dev = connector->dev;
 -      struct drm_crtc *crtc = NULL;
 +      struct drm_crtc *crtc = m->private;
 +      struct drm_device *dev = crtc->dev;
        struct dm_crtc_state *dm_crtc_state = NULL;
        int res = -ENODEV;
        unsigned int bpc;
  
        mutex_lock(&dev->mode_config.mutex);
 -      drm_modeset_lock(&dev->mode_config.connection_mutex, NULL);
 -
 -      if (connector->state == NULL)
 -              goto unlock;
 -
 -      crtc = connector->state->crtc;
 -      if (crtc == NULL)
 -              goto unlock;
 -
        drm_modeset_lock(&crtc->mutex, NULL);
        if (crtc->state == NULL)
                goto unlock;
        }
  
        seq_printf(m, "Current: %u\n", bpc);
 -      seq_printf(m, "Maximum: %u\n", connector->display_info.bpc);
        res = 0;
  
  unlock:
 -      if (crtc)
 -              drm_modeset_unlock(&crtc->mutex);
 -
 -      drm_modeset_unlock(&dev->mode_config.connection_mutex);
 +      drm_modeset_unlock(&crtc->mutex);
        mutex_unlock(&dev->mode_config.mutex);
  
        return res;
  }
 +DEFINE_SHOW_ATTRIBUTE(amdgpu_current_bpc);
  
  /*
   * Example usage:
@@@ -1226,12 -1239,14 +1226,14 @@@ static ssize_t trigger_hotplug(struct f
        struct drm_connector *connector = &aconnector->base;
        struct dc_link *link = NULL;
        struct drm_device *dev = connector->dev;
+       struct amdgpu_device *adev = drm_to_adev(dev);
        enum dc_connection_type new_connection_type = dc_connection_none;
        char *wr_buf = NULL;
        uint32_t wr_buf_size = 42;
        int max_param_num = 1;
        long param[1] = {0};
        uint8_t param_nums = 0;
+       bool ret = false;
  
        if (!aconnector || !aconnector->dc_link)
                return -EINVAL;
                        new_connection_type != dc_connection_none)
                        goto unlock;
  
-               if (!dc_link_detect(aconnector->dc_link, DETECT_REASON_HPD))
+               mutex_lock(&adev->dm.dc_lock);
+               ret = dc_link_detect(aconnector->dc_link, DETECT_REASON_HPD);
+               mutex_unlock(&adev->dm.dc_lock);
+               if (!ret)
                        goto unlock;
  
                amdgpu_dm_update_connector_after_detect(aconnector);
@@@ -2526,6 -2545,7 +2532,6 @@@ static int target_backlight_show(struc
  DEFINE_SHOW_ATTRIBUTE(dp_dsc_fec_support);
  DEFINE_SHOW_ATTRIBUTE(dmub_fw_state);
  DEFINE_SHOW_ATTRIBUTE(dmub_tracebuffer);
 -DEFINE_SHOW_ATTRIBUTE(output_bpc);
  DEFINE_SHOW_ATTRIBUTE(dp_lttpr_status);
  #ifdef CONFIG_DRM_AMD_DC_HDCP
  DEFINE_SHOW_ATTRIBUTE(hdcp_sink_capability);
@@@ -2772,6 -2792,7 +2778,6 @@@ static const struct 
        const struct file_operations *fops;
  } connector_debugfs_entries[] = {
                {"force_yuv420_output", &force_yuv420_output_fops},
 -              {"output_bpc", &output_bpc_fops},
                {"trigger_hotplug", &trigger_hotplug_debugfs_fops},
                {"internal_display", &internal_display_fops}
  };
@@@ -3155,10 -3176,9 +3161,10 @@@ static int crc_win_update_get(void *dat
  
  DEFINE_DEBUGFS_ATTRIBUTE(crc_win_update_fops, crc_win_update_get,
                         crc_win_update_set, "%llu\n");
 -
 +#endif
  void crtc_debugfs_init(struct drm_crtc *crtc)
  {
 +#ifdef CONFIG_DRM_AMD_SECURE_DISPLAY
        struct dentry *dir = debugfs_lookup("crc", crtc->debugfs_entry);
  
        if (!dir)
                                   &crc_win_y_end_fops);
        debugfs_create_file_unsafe("crc_win_update", 0644, dir, crtc,
                                   &crc_win_update_fops);
 -
 -}
  #endif
 +      debugfs_create_file("amdgpu_current_bpc", 0644, crtc->debugfs_entry,
 +                          crtc, &amdgpu_current_bpc_fops);
 +}
 +
  /*
   * Writes DTN log state to the user supplied buffer.
   * Example usage: cat /sys/kernel/debug/dri/0/amdgpu_dm_dtn_log
@@@ -3290,7 -3308,10 +3296,10 @@@ static int trigger_hpd_mst_set(void *da
                        aconnector = to_amdgpu_dm_connector(connector);
                        if (aconnector->dc_link->type == dc_connection_mst_branch &&
                            aconnector->mst_mgr.aux) {
+                               mutex_lock(&adev->dm.dc_lock);
                                dc_link_detect(aconnector->dc_link, DETECT_REASON_HPD);
+                               mutex_unlock(&adev->dm.dc_lock);
                                drm_dp_mst_topology_mgr_set_mst(&aconnector->mst_mgr, true);
                        }
                }
index 710797b2f0df592f296a40e8501d8562f4b0ac70,d380b8bc6f3952ce15d5b432b2cbff61b6985d23..dfc74aea2852a36cf11e46572f78d5ba86b3a321
@@@ -695,7 -695,7 +695,7 @@@ bool dp_is_interlane_aligned(union lane
  void dp_hw_to_dpcd_lane_settings(
                const struct link_training_settings *lt_settings,
                const struct dc_lane_settings hw_lane_settings[LANE_COUNT_DP_MAX],
 -              union dpcd_training_lane dpcd_lane_settings[LANE_COUNT_DP_MAX])
 +              union dpcd_training_lane dpcd_lane_settings[])
  {
        uint8_t lane = 0;
  
@@@ -725,7 -725,7 +725,7 @@@ void dp_decide_lane_settings
                const struct link_training_settings *lt_settings,
                const union lane_adjust ln_adjust[LANE_COUNT_DP_MAX],
                struct dc_lane_settings hw_lane_settings[LANE_COUNT_DP_MAX],
 -              union dpcd_training_lane dpcd_lane_settings[LANE_COUNT_DP_MAX])
 +              union dpcd_training_lane dpcd_lane_settings[])
  {
        uint32_t lane;
  
@@@ -3880,15 -3880,13 +3880,13 @@@ static bool decide_mst_link_settings(co
        return true;
  }
  
void decide_link_settings(struct dc_stream_state *stream,
bool decide_link_settings(struct dc_stream_state *stream,
        struct dc_link_settings *link_setting)
  {
-       struct dc_link *link;
-       uint32_t req_bw;
-       req_bw = dc_bandwidth_in_kbps_from_timing(&stream->timing);
+       struct dc_link *link = stream->link;
+       uint32_t req_bw = dc_bandwidth_in_kbps_from_timing(&stream->timing);
  
-       link = stream->link;
+       memset(link_setting, 0, sizeof(*link_setting));
  
        /* if preferred is specified through AMDDP, use it, if it's enough
         * to drive the mode
                        LANE_COUNT_UNKNOWN &&
                        link->preferred_link_setting.link_rate !=
                                        LINK_RATE_UNKNOWN) {
-               *link_setting =  link->preferred_link_setting;
-               return;
+               *link_setting = link->preferred_link_setting;
+               return true;
        }
  
        /* MST doesn't perform link training for now
         * TODO: add MST specific link training routine
         */
        if (stream->signal == SIGNAL_TYPE_DISPLAY_PORT_MST) {
-               if (decide_mst_link_settings(link, link_setting))
-                       return;
+               decide_mst_link_settings(link, link_setting);
        } else if (link->connector_signal == SIGNAL_TYPE_EDP) {
                /* enable edp link optimization for DSC eDP case */
                if (stream->timing.flags.DSC) {
                                decide_edp_link_settings(link, &tmp_link_setting, orig_req_bw);
                                max_link_rate = tmp_link_setting.link_rate;
                        }
-                       if (decide_edp_link_settings_with_dsc(link, link_setting, req_bw, max_link_rate))
-                               return;
-               } else if (decide_edp_link_settings(link, link_setting, req_bw))
-                       return;
-       } else if (decide_dp_link_settings(link, link_setting, req_bw))
-               return;
-       BREAK_TO_DEBUGGER();
-       ASSERT(link->verified_link_cap.lane_count != LANE_COUNT_UNKNOWN);
+                       decide_edp_link_settings_with_dsc(link, link_setting, req_bw, max_link_rate);
+               } else {
+                       decide_edp_link_settings(link, link_setting, req_bw);
+               }
+       } else {
+               decide_dp_link_settings(link, link_setting, req_bw);
+       }
  
-       *link_setting = link->verified_link_cap;
+       return link_setting->lane_count != LANE_COUNT_UNKNOWN &&
+                       link_setting->link_rate != LINK_RATE_UNKNOWN;
  }
  
  /*************************Short Pulse IRQ***************************/
@@@ -4509,7 -4505,6 +4505,6 @@@ void dc_link_dp_handle_link_loss(struc
  {
        int i;
        struct pipe_ctx *pipe_ctx;
-       struct dc_link_settings prev_link_settings = link->preferred_link_setting;
  
        for (i = 0; i < MAX_PIPES; i++) {
                pipe_ctx = &link->dc->current_state->res_ctx.pipe_ctx[i];
        if (pipe_ctx == NULL || pipe_ctx->stream == NULL)
                return;
  
-       /* toggle stream state with the preference for current link settings */
-       dc_link_set_preferred_training_settings((struct dc *)link->dc,
-                                       &link->cur_link_settings, NULL, link, true);
        for (i = 0; i < MAX_PIPES; i++) {
                pipe_ctx = &link->dc->current_state->res_ctx.pipe_ctx[i];
                if (pipe_ctx && pipe_ctx->stream && !pipe_ctx->stream->dpms_off &&
                        core_link_enable_stream(link->dc->current_state, pipe_ctx);
                }
        }
-       /* restore previous link settings preference */
-       dc_link_set_preferred_training_settings((struct dc *)link->dc,
-                                       &prev_link_settings, NULL, link, true);
  }
  
  bool dc_link_handle_hpd_rx_irq(struct dc_link *link, union hpd_irq_data *out_hpd_irq_dpcd_data, bool *out_link_loss,
@@@ -4890,7 -4877,7 +4877,7 @@@ static void get_active_converter_info
                                                        hdmi_color_caps.bits.MAX_BITS_PER_COLOR_COMPONENT);
  
  #if defined(CONFIG_DRM_AMD_DC_DCN)
-                                       if (link->dc->caps.hdmi_frl_pcon_support) {
+                                       if (link->dc->caps.dp_hdmi21_pcon_support) {
                                                union hdmi_encoded_link_bw hdmi_encoded_link_bw;
  
                                                link->dpcd_caps.dongle_caps.dp_hdmi_frl_max_link_bw_in_kbps =
@@@ -5553,7 -5540,7 +5540,7 @@@ static bool retrieve_link_cap(struct dc
                 * only if required.
                 */
                if (link->ep_type == DISPLAY_ENDPOINT_USB4_DPIA &&
-                               !link->dc->debug.dpia_debug.bits.disable_force_tbt3_work_around &&
+                               link->dc->debug.dpia_debug.bits.enable_force_tbt3_work_around &&
                                link->dpcd_caps.is_branch_dev &&
                                link->dpcd_caps.branch_dev_id == DP_BRANCH_DEVICE_ID_90CC24 &&
                                link->dpcd_caps.branch_hw_revision == DP_BRANCH_HW_REV_10 &&
@@@ -6971,13 -6958,14 +6958,14 @@@ bool is_dp_128b_132b_signal(struct pipe
                        dc_is_dp_signal(pipe_ctx->stream->signal));
  }
  
- void edp_panel_backlight_power_on(struct dc_link *link)
+ void edp_panel_backlight_power_on(struct dc_link *link, bool wait_for_hpd)
  {
        if (link->connector_signal != SIGNAL_TYPE_EDP)
                return;
  
        link->dc->hwss.edp_power_control(link, true);
-       link->dc->hwss.edp_wait_for_hpd_ready(link, true);
+       if (wait_for_hpd)
+               link->dc->hwss.edp_wait_for_hpd_ready(link, true);
        if (link->dc->hwss.edp_backlight_control)
                link->dc->hwss.edp_backlight_control(link, true);
  }
index a3c1e9c56d8b02ea04c28a60c2cb729744e20df6,b44c7b43f7db2175dbb0cea9ceec43341ccbbb6e..6682d9e181c6e328c6343bb61c173cbc38ef3e74
@@@ -70,7 -70,7 +70,7 @@@ bool decide_edp_link_settings(struct dc
                struct dc_link_settings *link_setting,
                uint32_t req_bw);
  
void decide_link_settings(
bool decide_link_settings(
        struct dc_stream_state *stream,
        struct dc_link_settings *link_setting);
  
@@@ -148,12 -148,12 +148,12 @@@ bool dp_is_max_vs_reached
  void dp_hw_to_dpcd_lane_settings(
        const struct link_training_settings *lt_settings,
        const struct dc_lane_settings hw_lane_settings[LANE_COUNT_DP_MAX],
 -      union dpcd_training_lane dpcd_lane_settings[LANE_COUNT_DP_MAX]);
 +      union dpcd_training_lane dpcd_lane_settings[]);
  void dp_decide_lane_settings(
        const struct link_training_settings *lt_settings,
        const union lane_adjust ln_adjust[LANE_COUNT_DP_MAX],
        struct dc_lane_settings hw_lane_settings[LANE_COUNT_DP_MAX],
 -      union dpcd_training_lane dpcd_lane_settings[LANE_COUNT_DP_MAX]);
 +      union dpcd_training_lane dpcd_lane_settings[]);
  
  uint32_t dp_translate_training_aux_read_interval(uint32_t dpcd_aux_read_interval);
  
@@@ -193,6 -193,7 +193,7 @@@ enum dc_status dpcd_configure_lttpr_mod
                struct link_training_settings *lt_settings);
  
  enum dp_link_encoding dp_get_link_encoding_format(const struct dc_link_settings *link_settings);
+ bool dp_retrieve_lttpr_cap(struct dc_link *link);
  bool dpcd_write_128b_132b_sst_payload_allocation_table(
                const struct dc_stream_state *stream,
                struct dc_link *link,
@@@ -214,11 -215,10 +215,10 @@@ void enable_dp_hpo_output(struct dc_lin
  void disable_dp_hpo_output(struct dc_link *link,
                const struct link_resource *link_res,
                enum signal_type signal);
  void setup_dp_hpo_stream(struct pipe_ctx *pipe_ctx, bool enable);
  bool is_dp_128b_132b_signal(struct pipe_ctx *pipe_ctx);
- bool dp_retrieve_lttpr_cap(struct dc_link *link);
- void edp_panel_backlight_power_on(struct dc_link *link);
+ void edp_panel_backlight_power_on(struct dc_link *link, bool wait_for_hpd);
  void dp_receiver_power_ctrl(struct dc_link *link, bool on);
  void dp_source_sequence_trace(struct dc_link *link, uint8_t dp_test_mode);
  void dp_enable_link_phy(
index 8f828ec76c355136b0c7fd0f0940ded5f2ed0876,6008450370e88cef2a4917cb84181d43b20a0f89..dccbd9f707238922da482404622241aa09926f4a
@@@ -781,7 -781,7 +781,7 @@@ int smu_v11_0_set_allowed_mask(struct s
                goto failed;
        }
  
 -      bitmap_copy((unsigned long *)feature_mask, feature->allowed, 64);
 +      bitmap_to_arr32(feature_mask, feature->allowed, 64);
  
        ret = smu_cmn_send_smc_msg_with_param(smu, SMU_MSG_SetAllowedFeaturesMaskHigh,
                                          feature_mask[1], NULL);
@@@ -1228,6 -1228,8 +1228,8 @@@ int smu_v11_0_set_fan_speed_rpm(struct 
        uint32_t crystal_clock_freq = 2500;
        uint32_t tach_period;
  
+       if (speed == 0)
+               return -EINVAL;
        /*
         * To prevent from possible overheat, some ASICs may have requirement
         * for minimum fan speed:
index 8342703ce7d6f3c175d4e134ed609d6df63ea02a,0370482dd52bc01693131681e9ebc2cf958f597a..e8fe84f806d172f98b56b5c411f566c26e742b56
@@@ -856,7 -856,7 +856,7 @@@ int smu_v13_0_set_allowed_mask(struct s
            feature->feature_num < 64)
                return -EINVAL;
  
 -      bitmap_copy((unsigned long *)feature_mask, feature->allowed, 64);
 +      bitmap_to_arr32(feature_mask, feature->allowed, 64);
  
        ret = smu_cmn_send_smc_msg_with_param(smu, SMU_MSG_SetAllowedFeaturesMaskHigh,
                                              feature_mask[1], NULL);
@@@ -1084,9 -1084,33 +1084,33 @@@ int smu_v13_0_set_power_limit(struct sm
        return 0;
  }
  
+ static int smu_v13_0_allow_ih_interrupt(struct smu_context *smu)
+ {
+       return smu_cmn_send_smc_msg(smu,
+                                   SMU_MSG_AllowIHHostInterrupt,
+                                   NULL);
+ }
+ static int smu_v13_0_process_pending_interrupt(struct smu_context *smu)
+ {
+       int ret = 0;
+       if (smu->dc_controlled_by_gpio &&
+           smu_cmn_feature_is_enabled(smu, SMU_FEATURE_ACDC_BIT))
+               ret = smu_v13_0_allow_ih_interrupt(smu);
+       return ret;
+ }
  int smu_v13_0_enable_thermal_alert(struct smu_context *smu)
  {
-       return amdgpu_irq_get(smu->adev, &smu->irq_source, 0);
+       int ret = 0;
+       ret = amdgpu_irq_get(smu->adev, &smu->irq_source, 0);
+       if (ret)
+               return ret;
+       return smu_v13_0_process_pending_interrupt(smu);
  }
  
  int smu_v13_0_disable_thermal_alert(struct smu_context *smu)
index f508aefcf786da8720c626c956a5d3be4b0a91d7,429644d5ddc6947ca16da8500b991d05df8266c9..2b12389f841ae73eabdab7479df1c9987da18eb3
@@@ -38,7 -38,6 +38,7 @@@
  #include <drm/drm_crtc_helper.h>
  #include <drm/drm_device.h>
  #include <drm/drm_file.h>
 +#include <drm/drm_framebuffer.h>
  #include <drm/drm_probe_helper.h>
  #include <drm/radeon_drm.h>
  
@@@ -1114,7 -1113,7 +1114,7 @@@ static int radeon_gart_size_auto(enum r
  static void radeon_check_arguments(struct radeon_device *rdev)
  {
        /* vramlimit must be a power of two */
-       if (!is_power_of_2(radeon_vram_limit)) {
+       if (radeon_vram_limit != 0 && !is_power_of_2(radeon_vram_limit)) {
                dev_warn(rdev->dev, "vram limit (%d) must be a power of 2\n",
                                radeon_vram_limit);
                radeon_vram_limit = 0;
This page took 0.22562 seconds and 4 git commands to generate.