[DragonFlyBSD - Bug #3102] sys/dev/drm/radeon/radeon_uvd.c:897: possible cut'n'paste issue ?

[bugtracker-admin at leaf.dragonflybsd.org]

Issue #3102 has been updated by dcb.


>Could you report it there too?

I had a look around for a bug button and couldn't see one, so no.

I am happy for someone who knows their way around github better than me
to have a go.


----------------------------------------
Bug #3102: sys/dev/drm/radeon/radeon_uvd.c:897: possible cut'n'paste issue ?
http://bugs.dragonflybsd.org/issues/3102#change-13320

* Author: dcb
* Status: New
* Priority: Normal
* Assignee: 
* Category: Kernel
* Target version: 
----------------------------------------
dragonfly/sys/dev/drm/radeon/radeon_uvd.c:897] -> [dragonfly/sys/dev/drm/radeon/radeon_uvd.c:903]: (warning) Identical condition 'vclk_div>pd_max', second condition is always false

Source code is

        if (vclk_div > pd_max)
            break; /* vco is too big, it has to stop */

        /* calc dclk divider with current vco freq */
        dclk_div = radeon_uvd_calc_upll_post_div(vco_freq, dclk,
                             pd_min, pd_even);
        if (vclk_div > pd_max)
            break; /* vco is too big, it has to stop */

Maybe the 2nd if would be better if it sanity checked dclk_div, like

        if (dclk_div > pd_max)
            break; /* vco is too big, it has to stop */
 



-- 
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://bugs.dragonflybsd.org/my/account