Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2023-01-05 Thread Sebastian Wick
On Fri, Dec 23, 2022 at 8:10 PM Harry Wentland wrote: > > > > On 12/14/22 04:01, Pekka Paalanen wrote: > > On Tue, 13 Dec 2022 18:20:59 +0100 > > Michel Dänzer wrote: > > > >> On 12/12/22 19:21, Harry Wentland wrote: > >>> This will let us pass kms_hdr.bpc_switch. > >>> > >>> I don't see any good

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2023-01-04 Thread Michel Dänzer
On 12/23/22 20:10, Harry Wentland wrote: > On 12/14/22 04:01, Pekka Paalanen wrote: >> On Tue, 13 Dec 2022 18:20:59 +0100 >> Michel Dänzer wrote: >>> On 12/12/22 19:21, Harry Wentland wrote: This will let us pass kms_hdr.bpc_switch. I don't see any good reasons why we still need to

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-23 Thread Harry Wentland
On 12/14/22 04:01, Pekka Paalanen wrote: > On Tue, 13 Dec 2022 18:20:59 +0100 > Michel Dänzer wrote: > >> On 12/12/22 19:21, Harry Wentland wrote: >>> This will let us pass kms_hdr.bpc_switch. >>> >>> I don't see any good reasons why we still need to >>> limit bpc to 8 bpc and doing so is prob

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-16 Thread Michel Dänzer
On 12/15/22 10:07, Michel Dänzer wrote: > On 12/14/22 16:46, Alex Deucher wrote: >> On Wed, Dec 14, 2022 at 4:01 AM Pekka Paalanen wrote: >>> On Tue, 13 Dec 2022 18:20:59 +0100 >>> Michel Dänzer wrote: On 12/12/22 19:21, Harry Wentland wrote: > This will let us pass kms_hdr.bpc_switch. >

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-15 Thread Michel Dänzer
On 12/15/22 18:42, Michel Dänzer wrote: > On 12/15/22 18:33, Alex Deucher wrote: >> On Thu, Dec 15, 2022 at 4:17 AM Pekka Paalanen wrote: >>> >>> On Wed, 14 Dec 2022 10:46:55 -0500 >>> Alex Deucher wrote: >>> On Wed, Dec 14, 2022 at 4:01 AM Pekka Paalanen wrote: > > On Tue, 13 Dec 2

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-15 Thread Michel Dänzer
On 12/15/22 18:33, Alex Deucher wrote: > On Thu, Dec 15, 2022 at 4:17 AM Pekka Paalanen wrote: >> >> On Wed, 14 Dec 2022 10:46:55 -0500 >> Alex Deucher wrote: >> >>> On Wed, Dec 14, 2022 at 4:01 AM Pekka Paalanen wrote: On Tue, 13 Dec 2022 18:20:59 +0100 Michel Dänzer wrote:

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-15 Thread Alex Deucher
On Thu, Dec 15, 2022 at 4:17 AM Pekka Paalanen wrote: > > On Wed, 14 Dec 2022 10:46:55 -0500 > Alex Deucher wrote: > > > On Wed, Dec 14, 2022 at 4:01 AM Pekka Paalanen wrote: > > > > > > On Tue, 13 Dec 2022 18:20:59 +0100 > > > Michel Dänzer wrote: > > > > > > > On 12/12/22 19:21, Harry Wentlan

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-15 Thread Alex Deucher
On Thu, Dec 15, 2022 at 4:07 AM Michel Dänzer wrote: > > On 12/14/22 16:46, Alex Deucher wrote: > > On Wed, Dec 14, 2022 at 4:01 AM Pekka Paalanen wrote: > >> On Tue, 13 Dec 2022 18:20:59 +0100 > >> Michel Dänzer wrote: > >>> On 12/12/22 19:21, Harry Wentland wrote: > This will let us pass

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-15 Thread Pekka Paalanen
On Wed, 14 Dec 2022 10:46:55 -0500 Alex Deucher wrote: > On Wed, Dec 14, 2022 at 4:01 AM Pekka Paalanen wrote: > > > > On Tue, 13 Dec 2022 18:20:59 +0100 > > Michel Dänzer wrote: > > > > > On 12/12/22 19:21, Harry Wentland wrote: > > > > This will let us pass kms_hdr.bpc_switch. > > > > > >

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-15 Thread Michel Dänzer
On 12/14/22 16:46, Alex Deucher wrote: > On Wed, Dec 14, 2022 at 4:01 AM Pekka Paalanen wrote: >> On Tue, 13 Dec 2022 18:20:59 +0100 >> Michel Dänzer wrote: >>> On 12/12/22 19:21, Harry Wentland wrote: This will let us pass kms_hdr.bpc_switch. I don't see any good reasons why we st

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-14 Thread Alex Deucher
On Wed, Dec 14, 2022 at 4:01 AM Pekka Paalanen wrote: > > On Tue, 13 Dec 2022 18:20:59 +0100 > Michel Dänzer wrote: > > > On 12/12/22 19:21, Harry Wentland wrote: > > > This will let us pass kms_hdr.bpc_switch. > > > > > > I don't see any good reasons why we still need to > > > limit bpc to 8 bpc

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-14 Thread Pekka Paalanen
On Tue, 13 Dec 2022 18:20:59 +0100 Michel Dänzer wrote: > On 12/12/22 19:21, Harry Wentland wrote: > > This will let us pass kms_hdr.bpc_switch. > > > > I don't see any good reasons why we still need to > > limit bpc to 8 bpc and doing so is problematic when > > we enable HDR. > > > > If I reme

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-13 Thread Michel Dänzer
On 12/12/22 19:21, Harry Wentland wrote: > This will let us pass kms_hdr.bpc_switch. > > I don't see any good reasons why we still need to > limit bpc to 8 bpc and doing so is problematic when > we enable HDR. > > If I remember correctly there might have been some > displays out there where the a

Re: [PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-13 Thread Pekka Paalanen
On Mon, 12 Dec 2022 13:21:37 -0500 Harry Wentland wrote: > This will let us pass kms_hdr.bpc_switch. > > I don't see any good reasons why we still need to > limit bpc to 8 bpc and doing so is problematic when > we enable HDR. > > If I remember correctly there might have been some > displays out

[PATCH 16/16] drm/amd/display: Don't restrict bpc to 8 bpc

2022-12-12 Thread Harry Wentland
This will let us pass kms_hdr.bpc_switch. I don't see any good reasons why we still need to limit bpc to 8 bpc and doing so is problematic when we enable HDR. If I remember correctly there might have been some displays out there where the advertised link bandwidth was not large enough to drive th