Wrong shift in dce_v8_0_afmt_update_ACR() under dce_v8_0.c?

Alex Deucher alexdeucher at gmail.com
Tue Aug 9 04:30:16 UTC 2016


On Mon, Aug 8, 2016 at 5:43 PM, Alexandre Demers
<alexandre.f.demers at gmail.com> wrote:
> Hi,
>
> While looking into dce_v8_0.c, I stumbled upon something that might be wrong
> dce_v8_0_afmt_update_ACR(). We can read:
>
> WREG32(mmHDMI_ACR_32_0 + offset, (acr.cts_32khz <<
> HDMI_ACR_44_0__HDMI_ACR_CTS_44__SHIFT));
>
> However, I'm mostly sure it should be (luckily, the shift is of the same
> value):
>
> WREG32(mmHDMI_ACR_32_0 + offset, (acr.cts_32khz <<
> HDMI_ACR_32_0__HDMI_ACR_CTS_32__SHIFT));

Yeah, copy/paste error.

Alex

>
> Cheers
>
> --
> Alexandre Demers
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list