Hi Avri, > -----Original Message----- > From: Avri Altman <Avri.Altman@xxxxxxx> > Sent: 20 April 2020 15:56 > To: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>; robh@xxxxxxxxxx > Cc: devicetree@xxxxxxxxxxxxxxx; linux-scsi@xxxxxxxxxxxxxxx; krzk@xxxxxxxxxx; > martin.petersen@xxxxxxxxxx; kwmad.kim@xxxxxxxxxxx; > stanley.chu@xxxxxxxxxxxx; cang@xxxxxxxxxxxxxx; linux-samsung- > soc@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux- > kernel@xxxxxxxxxxxxxxx > Subject: RE: [PATCH v6 09/10] scsi: ufs-exynos: add UFS host support for Exynos > SoCs > > > + struct exynos_ufs_drv_data *drv_data; > > + > > + u32 opts; > > +#define EXYNOS_UFS_OPT_HAS_APB_CLK_CTRL BIT(0) > > +#define EXYNOS_UFS_OPT_SKIP_CONNECTION_ESTAB BIT(1) > > +#define EXYNOS_UFS_OPT_BROKEN_AUTO_CLK_CTRL BIT(2) > > +#define EXYNOS_UFS_OPT_BROKEN_RX_SEL_IDX BIT(3) > > +#define EXYNOS_UFS_OPT_USE_SW_HIBERN8_TIMER BIT(4) > Could not find where the last 2 are being used. > The assignment is done on line# 988 and 989 in exynos_ufs_init() and used in {pre,post}_hibern8, let me see if I can add these in the list of .opts in drv_data. Thanks for pointing this out. > Thanks, > Avri