> +static int ufshpb_lu_hpb_init(struct ufs_hba *hba, struct ufshpb_lu *hpb, > > + struct ufshpb_dev_info *hpb_dev_info) > > +{ > > + int ret; > > + > > + spin_lock_init(&hpb->hpb_state_lock); > > + > > + ret = ufshpb_alloc_region_tbl(hba, hpb); > > + if (ret) > > + goto release_m_page_cache; > This label is added only on 4/5 > > > +static int __init ufshpb_init(void) > > +{ > > + unsigned int pool_size; > Unused variable > > > + int ret; > > + > > + ret = driver_register(&ufshpb_drv.drv); > > + if (ret) > > + pr_err("ufshpb: driver register failed\n"); > > + > > + return ret; > > +} OK, I will fix it.