On Fri, Jan 29, 2021 at 03:11:07PM +0800, Peng Zhou wrote: > @@ -2770,6 +2781,15 @@ static int __maybe_unused msdc_runtime_resume(struct device *dev) > > msdc_ungate_clock(host); > msdc_restore_reg(host); > + > + /* > + * 1: MSDC_AES_CTL_INIT > + * 4: cap_id, no-meaning now > + * 1: cfg_id, we choose the second cfg group > + */ > + if (mmc->caps2 & MMC_CAP2_CRYPTO) > + arm_smccc_smc(MTK_SIP_MMC_CONTROL, > + 1, 4, 1, 0, 0, 0, 0, &smccc_res); > return 0; > } > This part doesn't compile because there is no smccc_res variable. It worked if I added it as a local variable in this function. - Eric