Skip to content

Commit 4856ebd

Browse files
committed
Merge tag 'drm-fixes-2025-05-24' of https://gitlab.freedesktop.org/drm/kernel
Pull drm fixes from Dave Airlie: "Weekly drm fixes pull, on target to be quiet, just one amdgpu, one edid and a few minor xe fixes. edid: - fix HDR metadata reset amdgpu: - Hibernate fix xe: - Make sure to check all forcewakes when dumping mocs - Fix wrong use of read64 on 32b register - Synchronize Panther Lake PCI IDs" * tag 'drm-fixes-2025-05-24' of https://gitlab.freedesktop.org/drm/kernel: drm/xe/ptl: Update the PTL pci id table drm/xe: Use xe_mmio_read32() to read mtcfg register drm/xe/mocs: Check if all domains awake Revert "drm/amd: Keep display off while going into S4" drm/edid: fixed the bug that hdr metadata was not reset
2 parents 7586ac7 + fe1e5a1 commit 4856ebd

File tree

5 files changed

+16
-15
lines changed

5 files changed

+16
-15
lines changed

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3469,11 +3469,6 @@ static int dm_resume(struct amdgpu_ip_block *ip_block)
34693469

34703470
return 0;
34713471
}
3472-
3473-
/* leave display off for S4 sequence */
3474-
if (adev->in_s4)
3475-
return 0;
3476-
34773472
/* Recreate dc_state - DC invalidates it when setting power state to S3. */
34783473
dc_state_release(dm_state->context);
34793474
dm_state->context = dc_state_create(dm->dc, NULL);

drivers/gpu/drm/drm_edid.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6596,6 +6596,7 @@ static void drm_reset_display_info(struct drm_connector *connector)
65966596
info->has_hdmi_infoframe = false;
65976597
info->rgb_quant_range_selectable = false;
65986598
memset(&info->hdmi, 0, sizeof(info->hdmi));
6599+
memset(&connector->hdr_sink_metadata, 0, sizeof(connector->hdr_sink_metadata));
65996600

66006601
info->edid_hdmi_rgb444_dc_modes = 0;
66016602
info->edid_hdmi_ycbcr444_dc_modes = 0;

drivers/gpu/drm/xe/xe_mmio.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -75,12 +75,12 @@ static void mmio_multi_tile_setup(struct xe_device *xe, size_t tile_mmio_size)
7575
* is fine as it's going to the root tile's mmio, that's
7676
* guaranteed to be initialized earlier in xe_mmio_probe_early()
7777
*/
78-
mtcfg = xe_mmio_read64_2x32(mmio, XEHP_MTCFG_ADDR);
78+
mtcfg = xe_mmio_read32(mmio, XEHP_MTCFG_ADDR);
7979
tile_count = REG_FIELD_GET(TILE_COUNT, mtcfg) + 1;
8080

8181
if (tile_count < xe->info.tile_count) {
8282
drm_info(&xe->drm, "tile_count: %d, reduced_tile_count %d\n",
83-
xe->info.tile_count, tile_count);
83+
xe->info.tile_count, tile_count);
8484
xe->info.tile_count = tile_count;
8585

8686
/*
@@ -128,7 +128,7 @@ int xe_mmio_probe_early(struct xe_device *xe)
128128
*/
129129
xe->mmio.size = pci_resource_len(pdev, GTTMMADR_BAR);
130130
xe->mmio.regs = pci_iomap(pdev, GTTMMADR_BAR, 0);
131-
if (xe->mmio.regs == NULL) {
131+
if (!xe->mmio.regs) {
132132
drm_err(&xe->drm, "failed to map registers\n");
133133
return -EIO;
134134
}
@@ -309,8 +309,8 @@ u64 xe_mmio_read64_2x32(struct xe_mmio *mmio, struct xe_reg reg)
309309
return (u64)udw << 32 | ldw;
310310
}
311311

312-
static int __xe_mmio_wait32(struct xe_mmio *mmio, struct xe_reg reg, u32 mask, u32 val, u32 timeout_us,
313-
u32 *out_val, bool atomic, bool expect_match)
312+
static int __xe_mmio_wait32(struct xe_mmio *mmio, struct xe_reg reg, u32 mask, u32 val,
313+
u32 timeout_us, u32 *out_val, bool atomic, bool expect_match)
314314
{
315315
ktime_t cur = ktime_get_raw();
316316
const ktime_t end = ktime_add_us(cur, timeout_us);

drivers/gpu/drm/xe/xe_mocs.c

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -775,22 +775,23 @@ void xe_mocs_init(struct xe_gt *gt)
775775
void xe_mocs_dump(struct xe_gt *gt, struct drm_printer *p)
776776
{
777777
struct xe_device *xe = gt_to_xe(gt);
778+
enum xe_force_wake_domains domain;
778779
struct xe_mocs_info table;
779780
unsigned int fw_ref, flags;
780781

781782
flags = get_mocs_settings(xe, &table);
782783

784+
domain = flags & HAS_LNCF_MOCS ? XE_FORCEWAKE_ALL : XE_FW_GT;
783785
xe_pm_runtime_get_noresume(xe);
784-
fw_ref = xe_force_wake_get(gt_to_fw(gt),
785-
flags & HAS_LNCF_MOCS ?
786-
XE_FORCEWAKE_ALL : XE_FW_GT);
787-
if (!fw_ref)
786+
fw_ref = xe_force_wake_get(gt_to_fw(gt), domain);
787+
788+
if (!xe_force_wake_ref_has_domain(fw_ref, domain))
788789
goto err_fw;
789790

790791
table.ops->dump(&table, flags, gt, p);
791792

792-
xe_force_wake_put(gt_to_fw(gt), fw_ref);
793793
err_fw:
794+
xe_force_wake_put(gt_to_fw(gt), fw_ref);
794795
xe_pm_runtime_put(xe);
795796
}
796797

include/drm/intel/pciids.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -861,6 +861,10 @@
861861
MACRO__(0xB081, ## __VA_ARGS__), \
862862
MACRO__(0xB082, ## __VA_ARGS__), \
863863
MACRO__(0xB083, ## __VA_ARGS__), \
864+
MACRO__(0xB084, ## __VA_ARGS__), \
865+
MACRO__(0xB085, ## __VA_ARGS__), \
866+
MACRO__(0xB086, ## __VA_ARGS__), \
867+
MACRO__(0xB087, ## __VA_ARGS__), \
864868
MACRO__(0xB08F, ## __VA_ARGS__), \
865869
MACRO__(0xB090, ## __VA_ARGS__), \
866870
MACRO__(0xB0A0, ## __VA_ARGS__), \

0 commit comments

Comments
 (0)