Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Mime-Version: 1.0
Subject: RE:(2) (2) [PATCH v6 5/6] scsi: ufs: wb: Modify messages
Reply-To: j-young.choi@samsung.com
Sender: Jinyoung CHOI <j-young.choi@samsung.com>
From:   Jinyoung CHOI <j-young.choi@samsung.com>
To:     Stanley Chu <chu.stanley@gmail.com>
CC:     ALIM AKHTAR <alim.akhtar@samsung.com>,
        "avri.altman@wdc.com" <avri.altman@wdc.com>,
        "bvanassche@acm.org" <bvanassche@acm.org>,
        "jejb@linux.ibm.com" <jejb@linux.ibm.com>,
        "martin.petersen@oracle.com" <martin.petersen@oracle.com>,
        "beanhuo@micron.com" <beanhuo@micron.com>,
        "adrian.hunter@intel.com" <adrian.hunter@intel.com>,
        "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
In-Reply-To: <CAGaU9a8Xxvtc0=Ci=ztwya=GqFZ0WB3p4wSo-QvLADdF3Fj+4Q@mail.gmail.com>
Message-ID: <20220803060739epcms2p4881142ec0b1c015612b8ccefa95cf819@epcms2p4>
Date:   Wed, 03 Aug 2022 15:07:39 +0900
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 102P
DLP-Filter: Pass
References: <CAGaU9a8Xxvtc0=Ci=ztwya=GqFZ0WB3p4wSo-QvLADdF3Fj+4Q@mail.gmail.com>
        <20220802080146epcms2p24b86bfce3d3c09c79b91d861cb3b2cce@epcms2p2>
        <20220802080927epcms2p1d0d89c32a9bd07c07f233801bb954807@epcms2p1>
        <CAGaU9a_YZDxaeOCErnQwFF9mE7bARDf4sw3F3ai1DiWwNVMFcw@mail.gmail.com>
        <20220803041126epcms2p82bafb63e4f135d09017519ed3066f914@epcms2p8>
        <CGME20220802080146epcms2p24b86bfce3d3c09c79b91d861cb3b2cce@epcms2p4>
Precedence: bulk

>hi=C2=A0Jinyoung,=0D=0A>=0D=0A>On=C2=A0Wed,=C2=A0Aug=C2=A03,=C2=A02022=C2=
=A0at=C2=A012:11=C2=A0PM=C2=A0Jinyoung=C2=A0CHOI=C2=A0<j-young.choi=40samsu=
ng.com>=C2=A0wrote:=0D=0A>>=0D=0A>>=C2=A0Hi,=C2=A0Stanley,=0D=0A>>=0D=0A>>=
=C2=A0>Hi,=0D=0A>>=C2=A0>=0D=0A>>=C2=A0>On=C2=A0Tue,=C2=A0Aug=C2=A02,=C2=A0=
2022=C2=A0at=C2=A04:29=C2=A0PM=C2=A0Jinyoung=C2=A0CHOI=C2=A0<j-young.choi=
=40samsung.com>=C2=A0wrote:=0D=0A>>=C2=A0>>=0D=0A>>=C2=A0>>=C2=A0Messages=
=C2=A0are=C2=A0modified=C2=A0to=C2=A0fit=C2=A0the=C2=A0format=C2=A0of=C2=A0=
others.=0D=0A>>=C2=A0>>=0D=0A>>=C2=A0>>=C2=A0Reviewed-by:=C2=A0Avri=C2=A0Al=
tman=C2=A0<avri.altman=40wdc.com>=0D=0A>>=C2=A0>>=C2=A0Reviewed-by:=C2=A0Ba=
rt=C2=A0Van=C2=A0Assche=C2=A0<bvanassche=40acm.org>=0D=0A>>=C2=A0>>=C2=A0Si=
gned-off-by:=C2=A0Jinyoung=C2=A0Choi=C2=A0<j-young.choi=40samsung.com>=0D=
=0A>>=C2=A0>>=C2=A0---=0D=0A>>=C2=A0>>=C2=A0=C2=A0drivers/ufs/core/ufs-sysf=
s.c=C2=A0=7C=C2=A0=C2=A02=C2=A0+-=0D=0A>>=C2=A0>>=C2=A0=C2=A0drivers/ufs/co=
re/ufshcd.c=C2=A0=C2=A0=C2=A0=C2=A0=7C=C2=A023=C2=A0+++++++++++------------=
=0D=0A>>=C2=A0>>=C2=A0=C2=A02=C2=A0files=C2=A0changed,=C2=A012=C2=A0inserti=
ons(+),=C2=A013=C2=A0deletions(-)=0D=0A>>=C2=A0>>=0D=0A>>=C2=A0>>=C2=A0diff=
=C2=A0--git=C2=A0a/drivers/ufs/core/ufs-sysfs.c=C2=A0b/drivers/ufs/core/ufs=
-sysfs.c=0D=0A>>=C2=A0>>=C2=A0index=C2=A02c0b7f45de4b..117272cf7d61=C2=A010=
0644=0D=0A>>=C2=A0>>=C2=A0---=C2=A0a/drivers/ufs/core/ufs-sysfs.c=0D=0A>>=
=C2=A0>>=C2=A0+++=C2=A0b/drivers/ufs/core/ufs-sysfs.c=0D=0A>>=C2=A0>>=C2=A0=
=40=40=C2=A0-230,7=C2=A0+230,7=C2=A0=40=40=C2=A0static=C2=A0ssize_t=C2=A0wb=
_on_store(struct=C2=A0device=C2=A0*dev,=C2=A0struct=C2=A0device_attribute=
=C2=A0*attr,=0D=0A>>=C2=A0>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0*=C2=A0If=C2=
=A0the=C2=A0platform=C2=A0supports=C2=A0UFSHCD_CAP_CLK_SCALING,=C2=A0turn=
=C2=A0WB=0D=0A>>=C2=A0>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0*=C2=A0on/off=C2=
=A0will=C2=A0be=C2=A0done=C2=A0while=C2=A0clock=C2=A0scaling=C2=A0up/down.=
=0D=0A>>=C2=A0>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0*/=0D=0A>>=C2=A0>>=C2=A0=
-=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0dev_warn(dev,=C2=A0=22To=C2=A0control=C2=A0WB=C2=A0throug=
h=C2=A0wb_on=C2=A0is=C2=A0not=C2=A0allowed=21=5Cn=22);=0D=0A>>=C2=A0>>=C2=
=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0dev_warn(dev,=C2=A0=22It=C2=A0is=C2=A0not=C2=A0allowed=
=C2=A0to=C2=A0configure=C2=A0WB=21=5Cn=22);=0D=0A>>=C2=A0>>=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0return=C2=A0-EOPNOTSUPP;=0D=0A>>=C2=A0>>=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=7D=0D=0A>>=C2=A0>>=0D=0A>>=C2=A0>>=C2=A0d=
iff=C2=A0--git=C2=A0a/drivers/ufs/core/ufshcd.c=C2=A0b/drivers/ufs/core/ufs=
hcd.c=0D=0A>>=C2=A0>>=C2=A0index=C2=A05099d161f115..dcd7f03db2a2=C2=A010064=
4=0D=0A>>=C2=A0>>=C2=A0---=C2=A0a/drivers/ufs/core/ufshcd.c=0D=0A>>=C2=A0>>=
=C2=A0+++=C2=A0b/drivers/ufs/core/ufshcd.c=0D=0A>>=C2=A0>>=C2=A0=40=40=C2=
=A0-5737,13=C2=A0+5737,13=C2=A0=40=40=C2=A0int=C2=A0ufshcd_wb_toggle(struct=
=C2=A0ufs_hba=C2=A0*hba,=C2=A0bool=C2=A0enable)=0D=0A>>=C2=A0>>=0D=0A>>=C2=
=A0>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0ret=C2=A0=3D=C2=
=A0__ufshcd_wb_toggle(hba,=C2=A0enable,=C2=A0QUERY_FLAG_IDN_WB_EN);=0D=0A>>=
=C2=A0>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if=C2=A0(ret)=
=C2=A0=7B=0D=0A>>=C2=A0>>=C2=A0-=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0dev_err(hba->dev,=C2=A0=22%=
s=C2=A0Write=C2=A0Booster=C2=A0%s=C2=A0failed=C2=A0%d=5Cn=22,=0D=0A>>=C2=A0=
>>=C2=A0-=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0__f=
unc__,=C2=A0enable=C2=A0?=C2=A0=22enable=22=C2=A0:=C2=A0=22disable=22,=C2=
=A0ret);=0D=0A>>=C2=A0>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0dev_err(hba->dev,=C2=A0=22%=
s:=C2=A0Write=C2=A0Booster=C2=A0%s=C2=A0failed=C2=A0%d=5Cn=22,=0D=0A>>=C2=
=A0>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
__func__,=C2=A0enable=C2=A0?=C2=A0=22enabling=22=C2=A0:=C2=A0=22disabling=
=22,=C2=A0ret);=0D=0A>>=C2=A0>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0return=C2=A0ret=
;=0D=0A>>=C2=A0>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=7D=
=0D=0A>>=C2=A0>>=0D=0A>>=C2=A0>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0hba->dev_info.wb_enabled=C2=A0=3D=C2=A0enable;=0D=0A>>=C2=A0>>=
=C2=A0-=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0dev_info(hba->dev,=C2=A0=
=22%s=C2=A0Write=C2=A0Booster=C2=A0%s=5Cn=22,=0D=0A>>=C2=A0>>=C2=A0+=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0dev_info(hba->dev,=C2=A0=22%s:=C2=A0Wri=
te=C2=A0Booster=C2=A0%s=5Cn=22,=0D=0A>>=C2=A0>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0__func__,=C2=A0enable=C2=A0=
?=C2=A0=22enabled=22=C2=A0:=C2=A0=22disabled=22);=0D=0A>>=C2=A0>=0D=0A>>=C2=
=A0>You=C2=A0need=C2=A0to=C2=A0rebase=C2=A0this=C2=A0patch=C2=A0to=C2=A0fol=
low=C2=A0the=C2=A0latest=C2=A0change=C2=A0as=0D=0A>>=C2=A0>https://lore.ker=
nel.org/all/20220709000027.3929970-1-bjorn.andersson=40linaro.org/=0D=0A>>=
=0D=0A>>=C2=A0I=C2=A0am=C2=A0currently=C2=A0working=C2=A0on=C2=A0the=C2=A0l=
atest=C2=A05.20/scsi-staging.=0D=0A>>=C2=A0In=C2=A0this=C2=A0case,=C2=A0can=
=C2=A0I=C2=A0refer=C2=A0the=C2=A0commit=C2=A0of=C2=A05.19/scsi-fixes=0D=0A>=
>=C2=A0and=C2=A0add=C2=A0commit=C2=A0to=C2=A05.20/scsi-staging?=0D=0A>>=C2=
=A0Or=C2=A0can=C2=A0I=C2=A0reflect=C2=A0it=C2=A0in=C2=A0my=C2=A0change?=0D=
=0A>>=C2=A0I=C2=A0have=C2=A0no=C2=A0experience,=C2=A0so=C2=A0please=C2=A0gu=
ide.=C2=A0:)=0D=0A>>=0D=0A>>=C2=A0Thank=C2=A0you=C2=A0for=C2=A0checking.=0D=
=0A>>=C2=A0Jinyoung.=0D=0A>>=0D=0A>>=0D=0A>=0D=0A>I=C2=A0did=C2=A0not=C2=A0=
notice=C2=A0that=C2=A0the=C2=A0below=C2=A0patch=C2=A0is=C2=A0merged=C2=A0to=
=C2=A05.19/scsi-fixes,=C2=A0so=0D=0A>I=C2=A0guess=C2=A0perhaps=C2=A0you=C2=
=A0could=C2=A0keep=C2=A0working=C2=A0on=C2=A0the=C2=A0latest=0D=0A>5.20/scs=
i-staging,=C2=A0and=C2=A0then=C2=A0Martin=C2=A0would=C2=A0help=C2=A0resolve=
=C2=A0the=C2=A0conflict.=0D=0A>https://lore.kernel.org/all/20220709000027.3=
929970-1-bjorn.andersson=40linaro.org/=0D=0A>=0D=0A>Thanks,=0D=0A>=0D=0A>St=
anley=0D=0A=0D=0AIt=20has=20been=20applied=20to=205.19/scsi-fixes,=0D=0Aso=
=20I=20will=20modify=20the=20debug=20level=20at=20my=20change=20for=20sync.=
=0D=0A=0D=0ABest=20Regards,=0D=0AJinyoung.