On Tue, Jul 7, 2015 at 6:56 AM, David Binderman <dcb314@xxxxxxxxxxx> wrote:
> Hello there,
>
> [linux-4.2-rc1/drivers/gpu/drm/radeon/cik.c:1912] -> [linux-4.2-rc1/drivers/gpu/drm/radeon/cik.c:1913]: (warning) Opposite conditions in nested 'if' blocks lead to a dead code block.
>
> Source code is
>
> if (running == 0) {
> if (running) {
> blackout = RREG32(MC_SHARED_BLACKOUT_CNTL);
> WREG32(MC_SHARED_BLACKOUT_CNTL, blackout | 1);
> }
It's there mainly as a reminder that we need to blackout the mc before
updating the ucode and then unblack it out at the end. Originally
this code was divided into several functions and the checks were
necessary.
Alex
>
> Regards
>
> David Binderman
>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
