> > 
> > +void ufshpb_init(struct ufs_hba *hba)
> > +{
> > +        struct ufshpb_dev_info *hpb_dev_info = &hba->ufshpb_dev;
> > +        int try;
> > +        int ret;
> > +
> > +        if (!ufshpb_is_allowed(hba))
> > +                return;
> > +
>  
> Here it is better to check "dev_info->hpb_enable", if HPB is not
> enabled from UFS device level,  doesn't need to create mempool and take
> other memory resource.

I will add checking dev_info->hpb_enable value.
if (!ufshpb_is_allowed(hba) || !dev_info->hpb_enable)

Thanks,
Daejun

> Bean
>  
>  
>  
>  
>  
>   

Reply via email to