RE: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Sharma, Shashank
Subject: Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100 yeah, ideally. Just need to get support for analog encoders. Alex From: Koenig, Christian mailto:christian.koe...@amd.com>> Sent: Wednesday, November 4, 2020 9:31 AM To: Deucher, Ale

Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Christian König
M *To:* Sharma, Shashank <mailto:shashank.sha...@amd.com>; amd-gfx@lists.freedesktop.org <mailto:amd-gfx@lists.freedesktop.org> <mailto:amd-gfx@lists.freedesktop.org> *Cc:* Deucher, Alexander <mailto:alexander.deuc...@amd.com>; Qin, Eddy <mailto:eddy....@amd.com> *

Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Deucher, Alexander
sktop.org<mailto:amd-gfx@lists.freedesktop.org> <mailto:amd-gfx@lists.freedesktop.org> Cc: Deucher, Alexander <mailto:alexander.deuc...@amd.com>; Qin, Eddy <mailto:eddy@amd.com> Subject: Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100 Am 04.11.20 um 11:40 schrie

Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Christian König
Christian ; amd-gfx@lists.freedesktop.org *Cc:* Qin, Eddy *Subject:* Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100 Thanks Alex, Same question here, Should we go through some extensive test routine due to change in PLL values, or is it OK to go ahead based on our experience f

Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Deucher, Alexander
harma, Shashank Sent: Wednesday, November 4, 2020 9:11 AM To: Deucher, Alexander ; Koenig, Christian ; amd-gfx@lists.freedesktop.org Cc: Qin, Eddy Subject: Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100 Thanks Alex, Same question here, Should we go through some extensive test routin

Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Shashank Sharma
Christian > *Sent:* Wednesday, November 4, 2020 6:54 AM > *To:* Sharma, Shashank ; > amd-gfx@lists.freedesktop.org > *Cc:* Deucher, Alexander ; Qin, Eddy > > *Subject:* Re: [PATC

Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Shashank Sharma
deon driver change ? Regards Shashank >> Regards >> Shashank >> -Original Message- >> From: Koenig, Christian >> Sent: Wednesday, November 4, 2020 3:35 PM >> To: Sharma, Shashank ; amd-gfx@lists.freedesktop.org >> Cc: Deucher, Alexander ; Qin, Eddy

Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Deucher, Alexander
Acked-by: Alex Deucher From: Koenig, Christian Sent: Wednesday, November 4, 2020 6:54 AM To: Sharma, Shashank ; amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Qin, Eddy Subject: Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100 Am 04.11.20

Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Christian König
From: Koenig, Christian Sent: Wednesday, November 4, 2020 3:35 PM To: Sharma, Shashank ; amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Qin, Eddy Subject: Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100 Am 03.11.20 um 18:13 schrieb Shashank Sharma: This patch limits the ref_di

RE: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Sharma, Shashank
, Christian Sent: Wednesday, November 4, 2020 3:35 PM To: Sharma, Shashank ; amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Qin, Eddy Subject: Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100 Am 03.11.20 um 18:13 schrieb Shashank Sharma: > This patch limits the ref_div_

Re: [PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-04 Thread Christian König
Am 03.11.20 um 18:13 schrieb Shashank Sharma: This patch limits the ref_div_max value to 100, during the calculation of PLL feedback reference divider. With current value (128), the produced fb_ref_div value generates unstable output at particular frequencies. Radeon driver limits this value at 1

[PATCH] drm/amdgpu: clip the ref divider max value at 100

2020-11-03 Thread Shashank Sharma
This patch limits the ref_div_max value to 100, during the calculation of PLL feedback reference divider. With current value (128), the produced fb_ref_div value generates unstable output at particular frequencies. Radeon driver limits this value at 100. On Oland, when we try to setup mode 2048x12