On Fri, Apr 09, 2021 at 03:55:22PM +0800, Ye Bin wrote: > There is a error message within devm_ioremap_resource > > already, so remove the dev_err call to avoid redundant > > error message. > > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: Ye Bin <yebin10@xxxxxxxxxx> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> Thanks, Mani > --- > drivers/scsi/ufs/ufs-qcom.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/drivers/scsi/ufs/ufs-qcom.c b/drivers/scsi/ufs/ufs-qcom.c > index 9b711d6aac54..2a3dd21da6a6 100644 > --- a/drivers/scsi/ufs/ufs-qcom.c > +++ b/drivers/scsi/ufs/ufs-qcom.c > @@ -1071,13 +1071,8 @@ static int ufs_qcom_init(struct ufs_hba *hba) > if (res) { > host->dev_ref_clk_ctrl_mmio = > devm_ioremap_resource(dev, res); > - if (IS_ERR(host->dev_ref_clk_ctrl_mmio)) { > - dev_warn(dev, > - "%s: could not map dev_ref_clk_ctrl_mmio, err %ld\n", > - __func__, > - PTR_ERR(host->dev_ref_clk_ctrl_mmio)); > + if (IS_ERR(host->dev_ref_clk_ctrl_mmio)) > host->dev_ref_clk_ctrl_mmio = NULL; > - } > host->dev_ref_clk_en_mask = BIT(5); > } > } >