> static struct ufshpb_driver ufshpb_drv; > > +unsigned int ufshpb_host_map_kbytes = 1 * 1024; > I think you've already declared this as a module parameter in 3/5. > > No need to fix this now, unless there will be some more comments, > And you'll issue a v4. OK, thanks!