Change in coreboot[master]: soc/rockchip/rk3399/sdram: Use rank_mask in WDQL training

[Moritz Fischer (Code Review)]

Moritz Fischer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50888 )

Change subject: soc/rockchip/rk3399/sdram: Use rank_mask in WDQL training
......................................................................

Patch Set 9:

(1 comment)

Patchset:

PS9:
> How many ranks does that board have?

The config file(s) says 0x2 for rank on the 4G models. This should result in get_rank_mask() return 0x3 and should make the loop here do the same?

Could it be one of the other changes that went in (like the phy_ctrl_reset, or the clearing of the IRQ bits) or did you manage to bisect it to this commit? This change was supposed to be mostly a no-op for LPDDR3 case.

Very sorry about the breakage.