unclear code in vega10 pp

[Tom St Denis]

After seeing this style a few other places in the same file are they 
just padding out the rest of the array with the highest setting?

Tom

On 05/09/17 12:18 PM, Tom St Denis wrote:
> In vega10_populate_smc_link_levels(): there's this bit
> 
>      j = i - 1;
>      while (i < NUM_LINK_LEVELS) {
>          pp_table->PcieGenSpeed[i] = pcie_table->pcie_gen[j];
>          pp_table->PcieLaneCount[i] = pcie_table->pcie_lane[j];
> 
>          result = vega10_populate_single_lclk_level(hwmgr,
>                  pcie_table->lclk[j], &(pp_table->LclkDid[i]));
>          if (result) {
>              pr_info("Populate LClock Level %d Failed!\n", i);
>              return result;
>          }
>          i++;
>      }
> 
> It seems 'j' isn't changing so it's spamming multiple entries in the 
> lclkdid[] array based on the same entries from the pcie_table->pcie*[j].
> 
> Is that intentional or is a ++ or -- to j missing?
> 
> Cheers,
> Tom
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx