From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stanley Chu Subject: RE: [PATCH v1 0/3] scsi: ufs: Re-factor ufshcd_get_pwr_dev_param Date: Mon, 25 Feb 2019 17:13:58 +0800 Message-ID: <1551086038.30977.23.camel@mtkswgap22> References: <1551067825-22654-1-git-send-email-stanley.chu@mediatek.com> <1551067825-22654-2-git-send-email-stanley.chu@mediatek.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+glpam-linux-mediatek=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Avri Altman Cc: "linux-scsi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "martin.petersen-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org" , "chun-hung.wu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org" , "kuohong.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org" , "subhashj-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org" , "linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "liwei213-hv44wF8Li93QT0dZR+AlfA@public.gmane.org" , "alim.akhtar-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org" , "matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org" , "pedrom.sousa-HKixBCOQz3hWk0Htik3J/w@public.gmane.org" , "vivek.gautam-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org" , "peter.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org" List-Id: linux-scsi@vger.kernel.org Hi Avri, Thanks so much for consideration. Then I will combine this series to "scsi: ufs-mediatek: Add UFS support for Mediatek SoC chips" and upload a completed version later. On Mon, 2019-02-25 at 08:54 +0000, Avri Altman wrote: > Hi, > > > Hello, > > > > This series re-factors ufshcd_get_pwr_dev_param in vendors' drivers. This > > function is for vendors to decide finally agreed attributes for power mode > > change if vendors define > > their host capability in struct ufs_dev_params. Because it is using by both > > ufs-qcom and ufs-hisi currently and may be used in other vendors in the > > future, simply re-factor it and then evevry vendors can reuse it. > > > > Stanley Chu (3): > > scsi: ufs: Introduce ufshcd_get_pwr_dev_param > > scsi: ufs-qcom: Re-factor ufshcd_get_pwr_dev_param > > scsi: ufs-hisi: Re-factor ufshcd_get_pwr_dev_param > I think its fine to do all this in a single patch as part of your "Add Mediatek..." series, > So it won't get unnecessarily delayed. > > Looks fine to me. > > Thanks, > Avri Thanks, Stanley > > _______________________________________________ > Linux-mediatek mailing list > Linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > http://lists.infradead.org/mailman/listinfo/linux-mediatek