[radeonsi] si_populate_sq_ramping_values

Alex Deucher alexdeucher at gmail.com
Tue Feb 18 07:17:23 PST 2014


On Mon, Feb 17, 2014 at 5:08 PM, Sylvain BERTRAND <sylware at legeek.net> wrote:
> Hi,
>
> In si_dpm.c, si_populate_sq_ramping_values function,
>
> It should be SISLANDS_DPM2_SQ_RAMP_LTI_RATIO instead of
> NISLANDS_DPM2_SQ_RAMP_LTI_RATIO.
>

This is already changed.

> ----
>
> Moreover should it be:
> if (SISLANDS_DPM2_SQ_RAMP_LTI_RATIO > (LTI_RATIO_MASK >>
> LTI_RATIO_SHIFT))
>
> instead of:
> if (NISLANDS_DPM2_SQ_RAMP_LTI_RATIO <= (LTI_RATIO_MASK >>
> LTI_RATIO_SHIFT))
>
> ----
>
> The previous does disable "sq ramping" for good on all SI,
> expected?
>

You are correct.  fixed.

Thanks!

Alex

> regards,
>
> --
> Sylvain
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel


More information about the dri-devel mailing list