On 2020-05-08 07:44, Bean Huo wrote: > 1: driver module parameter, while system booting, specify maximum HPB > cache size in kernel boot Parameters: > > static int max_hpb_mem = 128; > module_param(max_hpb_mem,int,0444); > > but this paramter is added in the ufshcd.c since the ufshpb.c is not a > independent module. looks not very natural. > [ ... ] > If you prefer to convert current HPB driver to be a kernel module > driver, I prefer first apporach. Hi Bean, A kernel module parameter sounds good to me. Thanks, Bart.