On Fri, Aug 21, 2020 at 09:27:18AM +0200, Pavel Machek wrote:
Hi!
From: Denis Efremov efremov@linux.com
commit f29aa08852e1953e461f2d47ab13c34e14bc08b3 upstream.
clk_s is checked twice in a row in ni_init_smc_spll_table(). fb_div should be checked instead.
Fixes: 69e0b57a91ad ("drm/radeon/kms: add dpm support for cayman (v5)") Cc: stable@vger.kernel.org Signed-off-by: Denis Efremov efremov@linux.com Signed-off-by: Alex Deucher alexander.deucher@amd.com Signed-off-by: Greg Kroah-Hartman gregkh@linuxfoundation.org
No, this is wrong.
We already have the fix in -stable, as:
commit a083deda0b4179fb6780bc53d900794c4952339f Author: Denis Efremov efremov@linux.com Date: Mon Jun 22 23:31:22 2020 +0300
drm/radeon: fix fb_div check in ni_init_smc_spll_table() commit 35f760b44b1b9cb16a306bdcc7220fbbf78c4789 upstream.Result is that we now convert _second_ copy clk_s check, and check fb_div twice. This introduces error, rather than fixing one.
I hate the drm patchflow, it causes this type of issue to happen every release. It's their fault, they give me no way of detecting this type of crap and seem to ignore my complaints :(
ugh.
greg k-h