> > - struct samsung_ufs_phy_cfg **cfgs; > > + const struct samsung_ufs_phy_cfg **cfgs; > > I still think this should be array of const pointers to const data, so: > const struct samsung_ufs_phy_cfg * const *cfgs; I proposed this in > last email. Make sense. I'll change this as you suggested. Best Regards, Chanho Park
