unclear code in vega10 pp

[Tom St Denis <tom.stdenis-5C7GfCeVMHo@public.gmane.org>]

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@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx