Skip to content

Conversation

@Rapti
Copy link

@Rapti Rapti commented Oct 6, 2025

Starting with kernel version 6.16.8, the first kernel patch could no longer be applied because all EDID_QUIRK_NON_DESKTOP were wrapped in BIT( ) in this commit.

@Rapti Rapti changed the title Update 0001-drm-edid-Add-Vive-Cosmos-Vive-Pro-2-to-non-desktop-l.patch Update kernel patch for kernel version 6.16.8 Oct 12, 2025
@CertainLach
Copy link
Owner

You shouldn't need this patch on 6.16 kernel, since VivePro2 announces its own usage using EDID, and kernel is able to detect that automatically

@CertainLach
Copy link
Owner

You can check that by using edid-decode:

> edid-decode /sys/class/drm/card1-DP-3/edid

Block 1, DisplayID Extension Block:
  Version: 2.0
  Extension Count: 0
  Display Product Primary Use Case: Head-mounted Virtual Reality (VR) display

This patch was required when the kernel was not respecting this field.

@RononDex
Copy link

I just came across this issue when trying to upgrade my kernel. So are the other patches (0002-drm-edid-parse-DRM-VESA-dsc-bpp-target_fix.patch and 0003-drm-amd-use-fixed-dsc-bits-per-pixel-from-edid.patch) also not needed anymore?

@Rapti
Copy link
Author

Rapti commented Oct 28, 2025

You shouldn't need this patch on 6.16 kernel, since VivePro2 announces its own usage using EDID, and kernel is able to detect that automatically

You're right, it does work without it.

@CertainLach
Copy link
Owner

CertainLach commented Oct 28, 2025

I just came across this issue when trying to upgrade my kernel. So are the other patches (0002-drm-edid-parse-DRM-VESA-dsc-bpp-target_fix.patch and 0003-drm-amd-use-fixed-dsc-bits-per-pixel-from-edid.patch) also not needed anymore?

Those are needed for modes 4 and 5, but you'd better use the updated version: https://lore.kernel.org/all/20251017162736.45368-1-iam@lach.pw/T/#u
(Will add it to this repo later)

The patch I linked is for 6.18-rc3, rebased for 6.17 version is in this discord thread: https://discord.com/channels/1065291958328758352/1428185501008924672

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants