Re: [PATCH] drm/amd/display: Fix unintialized max_bpc state values

2018-11-29 Thread Wentland, Harry
On 2018-11-29 10:44 a.m., Nicholas Kazlauskas wrote: > [Why] > If the "max bpc" isn't explicitly set in the atomic state then it > have a value of 0. This has the correct behavior of limiting a panel > to 8bpc in the case where the panel supports 8bpc. In the case of eDP > panels this isn't a true

Re: [PATCH] drm/amd/display: Fix unintialized max_bpc state values

2018-11-29 Thread Deucher, Alexander
Reviewed-by: Alex Deucher From: amd-gfx on behalf of Nicholas Kazlauskas Sent: Thursday, November 29, 2018 10:44:46 AM To: amd-gfx@lists.freedesktop.org Cc: Kazlauskas, Nicholas Subject: [PATCH] drm/amd/display: Fix unintialized max_bpc state values [Why] If

[PATCH] drm/amd/display: Fix unintialized max_bpc state values

2018-11-29 Thread Nicholas Kazlauskas
[Why] If the "max bpc" isn't explicitly set in the atomic state then it have a value of 0. This has the correct behavior of limiting a panel to 8bpc in the case where the panel supports 8bpc. In the case of eDP panels this isn't a true assumption - there are panels that can only do 6bpc. Banding o