Re: [PATCH 2/3] scsi: ufs: ufs-pci: Fix default runtime and system PM levels

[Bart Van Assche <bvanassche@acm.org>]

On 7/2/25 11:43 PM, Adrian Hunter wrote:
>   static int ufs_intel_mtl_init(struct ufs_hba *hba)
>   {
> +	struct ufs_host *ufs_host;
> +	int err;
> +
>   	hba->caps |= UFSHCD_CAP_CRYPTO | UFSHCD_CAP_WB_EN;
> -	return ufs_intel_common_init(hba);
> +	err = ufs_intel_common_init(hba);
> +	ufs_host = ufshcd_get_variant(hba);
> +	ufs_host->late_init = ufs_intel_mtl_late_init;
> +	return err;
>   }

Shouldn't the ufs_host declaration and assignment be combined as
follows?

struct ufs_host *ufs_host = ufshcd_get_variant(hba);

Otherwise this patch looks good to me.

Thanks,

Bart.