Hi, ufshcd_pci_shutdown (in ufshcd_pci.c) is using ufshcd_hba_stop so it cannot be moved to ufshcd.c. Sorry for missing it last time I tested, I returned ufshcd_hba_stop to ufshcd.h by mistake when I resolved conflicts of the next patch so I didn't hit the compilation errors. Thanks, Maya > Move the ufshcd_hba_stop from header file. > > Signed-off-by: Seungwon Jeon <tgih.jun@xxxxxxxxxxx> > Reviewed-by: Subhash Jadavani <subhashj@xxxxxxxxxxxxxx> > Tested-by: Maya Erez <merez@xxxxxxxxxxxxxx> > --- > drivers/scsi/ufs/ufshcd.c | 9 +++++++++ > drivers/scsi/ufs/ufshcd.h | 9 --------- > 2 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c > index f244812..d932239 100644 > --- a/drivers/scsi/ufs/ufshcd.c > +++ b/drivers/scsi/ufs/ufshcd.c > @@ -280,6 +280,15 @@ static inline void ufshcd_hba_start(struct ufs_hba > *hba) > } > > /** > + * ufshcd_hba_stop - Send controller to reset state > + * @hba: per adapter instance > + */ > +static inline void ufshcd_hba_stop(struct ufs_hba *hba) > +{ > + writel(CONTROLLER_DISABLE, (hba->mmio_base + REG_CONTROLLER_ENABLE)); > +} > + > +/** > * ufshcd_is_hba_active - Get controller state > * @hba: per adapter instance > * > diff --git a/drivers/scsi/ufs/ufshcd.h b/drivers/scsi/ufs/ufshcd.h > index 336980b..3a1052d 100644 > --- a/drivers/scsi/ufs/ufshcd.h > +++ b/drivers/scsi/ufs/ufshcd.h > @@ -208,13 +208,4 @@ int ufshcd_init(struct device *, struct ufs_hba ** , > void __iomem * , > unsigned int); > void ufshcd_remove(struct ufs_hba *); > > -/** > - * ufshcd_hba_stop - Send controller to reset state > - * @hba: per adapter instance > - */ > -static inline void ufshcd_hba_stop(struct ufs_hba *hba) > -{ > - writel(CONTROLLER_DISABLE, (hba->mmio_base + REG_CONTROLLER_ENABLE)); > -} > - > #endif /* End of Header */ > -- > 1.7.0.4 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-scsi" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Maya Erez QUALCOMM ISRAEL, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html