On 02/06/2022 07:32, Chanho Park wrote: > Put const qualifier of samsung_ufs_phy_cfg pointer because they will > not be changed from drvdata. > > Signed-off-by: Chanho Park <chanho61.park@xxxxxxxxxxx> > --- > drivers/phy/samsung/phy-samsung-ufs.c | 4 ++-- > drivers/phy/samsung/phy-samsung-ufs.h | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/phy/samsung/phy-samsung-ufs.c b/drivers/phy/samsung/phy-samsung-ufs.c > index 602ddef259eb..b7ddb91a02be 100644 > --- a/drivers/phy/samsung/phy-samsung-ufs.c > +++ b/drivers/phy/samsung/phy-samsung-ufs.c > @@ -75,7 +75,7 @@ static int samsung_ufs_phy_wait_for_lock_acq(struct phy *phy) > static int samsung_ufs_phy_calibrate(struct phy *phy) > { > struct samsung_ufs_phy *ufs_phy = get_samsung_ufs_phy(phy); > - struct samsung_ufs_phy_cfg **cfgs = ufs_phy->cfg; > + const struct samsung_ufs_phy_cfg **cfgs = ufs_phy->cfg; This should be rather array of const pointers to const data. Best regards, Krzysztof