Hi Mika, > + > + if (intel_c20_use_mplla(hw_state->clock)) { > + for (i = 0; i < ARRAY_SIZE(hw_state->mplla); i++) > + drm_dbg_kms(&i915->drm, "mplla[%d] = 0x%.4x\n", i, hw_state->mplla[i]); > + } else { > + for (i = 0; i < ARRAY_SIZE(hw_state->mpllb); i++) > + drm_dbg_kms(&i915->drm, "mpllb[%d] = 0x%.4x\n", i, hw_state->mpllb[i]); > + } if you're going to resend it, brackets are nnot needed here. Andi