[v1,0/4] scsi: ufs-mediatek: provide power management

Message

Stanley Chu Dec. 13, 2019, 8:11 a.m. UTC

The patch set provides power management on MediaTek Chipsets by

1. Introduce reference clock control
2. Configure customized auto-hibernate timer
3. Enable clk-gating with customized delayed timer value

Stanley Chu (4):
  scsi: ufs-mediatek: introduce reference clock control
  scsi: ufs: export ufshcd_auto_hibern8_update for vendor usage
  scsi: ufs-mediatek: configure customized auto-hibern8 timer
  scsi: ufs-mediatek: configure and enable clk-gating

 drivers/scsi/ufs/ufs-mediatek.c | 93 +++++++++++++++++++++++++++++++--
 drivers/scsi/ufs/ufs-mediatek.h | 20 ++++++-
 drivers/scsi/ufs/ufs-sysfs.c    | 20 -------
 drivers/scsi/ufs/ufshcd.c       | 18 +++++++
 drivers/scsi/ufs/ufshcd.h       |  1 +
 5 files changed, 126 insertions(+), 26 deletions(-)

Comments

Martin K. Petersen Dec. 19, 2019, 11:17 p.m. UTC | #1

Stanley,

> The patch set provides power management on MediaTek Chipsets by

Had to apply this by hand. Please make sure you prepare patch
submissions against my "queue" branch.

Stanley Chu Dec. 20, 2019, 1:25 a.m. UTC | #2

Hi Martin,

Thank you so much and sorry for your inconvenience.

I was based on the latest linux-next commit in my submission time. I
will be more careful and use your "queue" branch instead for submission.

BTW, sorry again because this series actually require a header file
present by below patch in another series which was submitted earlier
than this series,

"soc: mediatek: add header for SiP service interface"

Otherwise missing header "include/linux/soc/mediatek/mtk_sip_svc.h" will
cause build error if MediaTek UFS driver is enabled.

Hope "soc: mediatek: add header for SiP service interface" could be
merged soon, or please rollback this series first if build error happens
and wait until above patch is merged.

To prevent this error, I shall merge both series and provide a new
combined series. If you want me to do so, please kindly let me know.
Sorry for this again.

On Thu, 2019-12-19 at 18:17 -0500, Martin K. Petersen wrote:
> Stanley,
> 
> > The patch set provides power management on MediaTek Chipsets by
> 
> Had to apply this by hand. Please make sure you prepare patch
> submissions against my "queue" branch.
> 

Thanks,
Stanley

Martin K. Petersen Dec. 20, 2019, 2:25 a.m. UTC | #3

Stanley,

> Otherwise missing header "include/linux/soc/mediatek/mtk_sip_svc.h"
> will cause build error if MediaTek UFS driver is enabled.

Thanks for the heads-up. I obviously don't have an easy way to verify. I
did check after applying to see if there was a way I could trigger a
build of the driver on a non MediaTek platform. But that didn't appear
to be trivial.