[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-12-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

Samantha McVey  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #13 from Samantha McVey  ---
Going to close this one as fixed.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

--- Comment #12 from Samantha McVey  ---
Nicholas,

Your last patch but with `new_state->max_bpc = state->max_bpc;` instead of
`new_state->max_bpc = cur->max_bpc;` resolves the issue.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

--- Comment #11 from Samantha McVey  ---
new_state->max_bpc = cur->max_bpc;

cur doesn't exist here (fails to compile with v2 patch). Did you mean
state->max_bpc?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

Nicholas Kazlauskas  changed:

   What|Removed |Added

 Attachment #142652|0   |1
is obsolete||

--- Comment #10 from Nicholas Kazlauskas  ---
Created attachment 142653
  --> https://bugs.freedesktop.org/attachment.cgi?id=142653=edit
v2-0001-drm-amd-display-Fix-unintialized-max_bpc-state-va.patch

I meant to copy the property for the dm connector state, sorry.

Try this patch instead.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

--- Comment #9 from Samantha McVey  ---
It doesn't compile with that patch:

drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c: In function
‘dm_crtc_duplicate_state’:
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:2991:7: error:
‘struct dm_crtc_state’ has no member named ‘max_bpc’
  state->max_bpc = cur->max_bpc;
   ^~
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:2991:22: error:
‘struct dm_crtc_state’ has no member named ‘max_bpc’
  state->max_bpc = cur->max_bpc;
  ^~

-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

--- Comment #8 from Nicholas Kazlauskas  ---
Created attachment 142652
  --> https://bugs.freedesktop.org/attachment.cgi?id=142652=edit
0001-drm-amd-display-Fix-unintialized-max_bpc-state-value.patch

Does this help fix the issue?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

--- Comment #7 from Samantha McVey  ---
Nicholas,

Reverting that commit (307638884f721b02b6a54ee8b351c5b4434bd4a6) on the top of
amd-staging-drm-next resolves the banding issues. So it is that commit that is
causing the problem.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

--- Comment #6 from Nicholas Kazlauskas  ---
(In reply to Samantha McVey from comment #5)
> I have bisected it to commit: drm/amd/display: Support amdgpu "max bpc"
> connector property
> 307638884f721b02b6a54ee8b351c5b4434bd4a6 Author: Nicholas Kazlauskas
> 

This isn't a regression from 4.19 but rather a fix for 4.18 where bpc was
uncapped from 8 unconditionally. It locked out high refresh modes for high
resolution panels with max bpc greater than 8.

The reported 6bpc in your dmesg log comes from DRM itself and not amdgpu. It's
what's being reported as being supported by your panel from its EDID. I'm not
sure if this is the panel itself reporting itself wrong or if there's a quirk
affecting it from the edid qurik list in drm_edid.c.

You can try reverting my patch on amd-staging-drm-next to see if you still get
the banding.

Or you could try setting the "max bpc" property to something higher (like 16).
You should be able to add this as an "Option" to a conf file in
/etc/X11/xorg.conf.d for your Monitor.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-11-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

--- Comment #5 from Samantha McVey  ---
I have bisected it to commit: drm/amd/display: Support amdgpu "max bpc"
connector property
307638884f721b02b6a54ee8b351c5b4434bd4a6 Author: Nicholas Kazlauskas


-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 108825] Regression Raven Ridge: Banding on eDP display

2018-11-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108825

Samantha McVey  changed:

   What|Removed |Added

Version|XOrg git|unspecified
Summary|Raven Ridge: Banding on eDP |Regression Raven Ridge:
   |display |Banding on eDP display

--- Comment #4 from Samantha McVey  ---
Looks like this is a regression. Dithering works in 4.19.2 but does not work in
amd-staging-drm-next.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel