<div dir="ltr"><div>FYI, this fixes tiling on VanGogh.</div><div><br></div><div>Reviewed-by: Marek Olšák <<a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a>><br></div><div><br></div><div>Marek<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 20, 2020 at 6:31 PM Bas Nieuwenhuizen <<a href="mailto:bas@basnieuwenhuizen.nl">bas@basnieuwenhuizen.nl</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">As far a I can tell uses a variant of DCN3xx which uses num_pkrs.<br>
<br>
If we do not initialize the variable we will set the register field<br>
to ilog2(0) = -1, though the mask will reduce that to 7. Pretty sure<br>
7 is not the value we want here.<br>
<br>
Signed-off-by: Bas Nieuwenhuizen <<a href="mailto:bas@basnieuwenhuizen.nl" target="_blank">bas@basnieuwenhuizen.nl</a>><br>
---<br>
<br>
Found while rebasing my modifiers series. Not tested on HW.<br>
<br>
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 ++-<br>
 1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c<br>
index 6855aad7f312..2713caac4f2a 100644<br>
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c<br>
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c<br>
@@ -4074,7 +4074,8 @@ fill_plane_buffer_attributes(struct amdgpu_device *adev,<br>
 #ifdef CONFIG_DRM_AMD_DC_DCN3_0<br>
                if (adev->asic_type == CHIP_SIENNA_CICHLID ||<br>
                    adev->asic_type == CHIP_NAVY_FLOUNDER ||<br>
-                   adev->asic_type == CHIP_DIMGREY_CAVEFISH)<br>
+                   adev->asic_type == CHIP_DIMGREY_CAVEFISH ||<br>
+                   adev->asic_type == CHIP_VANGOGH)<br>
                        tiling_info->gfx9.num_pkrs = adev->gfx.config.gb_addr_config_fields.num_pkrs;<br>
 #endif<br>
                ret = fill_plane_dcc_attributes(adev, afb, format, rotation,<br>
-- <br>
2.28.0<br>
<br>
_______________________________________________<br>
amd-gfx mailing list<br>
<a href="mailto:amd-gfx@lists.freedesktop.org" target="_blank">amd-gfx@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a><br>
</blockquote></div>