Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: q5O0GQSgSMwVIlDKYEgBch4qcH2FbR99BROzDF0tMmSaMWQRoE0j6ew6areU98s8kQZFQUR3FN
 eZg+1zJTQlqlgio/YQojKoRYffo6KgdO202iWX3aqXy/RxXFEtiWWFqHwr+6p6EShu+doO6Kbl
 HMsemQLf2TmSB+CH8LQANg4iK92dobJ11OtTHmOVYCY+mS8dMFOHPTMpA/amnoeHd+6pi1vTC1
 GN3g0I981IQ6VZm2F4XY10WPzXDqhR/swM26WQAk+pwoHJo4kpMo/s5d+p+XmEsTe3RBEjSEac
 eec=
From:   Avri Altman <Avri.Altman@wdc.com>
To:     Stanley Chu <stanley.chu@mediatek.com>,
        "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "martin.petersen@oracle.com" <martin.petersen@oracle.com>,
        "alim.akhtar@samsung.com" <alim.akhtar@samsung.com>,
        "jejb@linux.ibm.com" <jejb@linux.ibm.com>,
        "asutoshd@codeaurora.org" <asutoshd@codeaurora.org>
CC:     "beanhuo@micron.com" <beanhuo@micron.com>,
        "cang@codeaurora.org" <cang@codeaurora.org>,
        "matthias.bgg@gmail.com" <matthias.bgg@gmail.com>,
        "bvanassche@acm.org" <bvanassche@acm.org>,
        "linux-mediatek@lists.infradead.org" 
        <linux-mediatek@lists.infradead.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kuohong.wang@mediatek.com" <kuohong.wang@mediatek.com>,
        "peter.wang@mediatek.com" <peter.wang@mediatek.com>,
        "chun-hung.wu@mediatek.com" <chun-hung.wu@mediatek.com>,
        "andy.teng@mediatek.com" <andy.teng@mediatek.com>,
        "cc.chou@mediatek.com" <cc.chou@mediatek.com>,
        "chaotian.jing@mediatek.com" <chaotian.jing@mediatek.com>
Subject: RE: [PATCH v3 0/2] scsi: ufs: Add trace event for UIC commands and
 cleanup UIC struct
Thread-Topic: [PATCH v3 0/2] scsi: ufs: Add trace event for UIC commands and
 cleanup UIC struct
Thread-Index: AQHWQuXCGsZjEECOEkK/CSP7H3WFf6jZSDGA
Date:   Mon, 15 Jun 2020 07:31:57 +0000
Message-ID: <SN6PR04MB4640ABE5681594C4C1A53F7BFC9C0@SN6PR04MB4640.namprd04.prod.outlook.com>
References: <20200615072235.23042-1-stanley.chu@mediatek.com>
In-Reply-To: <20200615072235.23042-1-stanley.chu@mediatek.com>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 3e02f386-0fe0-489c-1964-08d810fe2f4e
X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Jun 2020 07:31:57.3180
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: OGvK99Qa9rdojRwJ5mYFdwZ4r+0JI/TCSjzHfwfVBdfwupKN0b1DF1Y4BeHBviTyIw1IzwiqgtpSH2QkpWYhvw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR04MB4574
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Looks good to me.
Thanks,
Avri

>=20
>=20
> Hi,
> This series adds trace event for UIC commands and do a small cleanup in
> struct uic_command.
>=20
> v2 -> v3:
>   - Refactor "complete" event hooks in ufshcd_uic_cmd_compl() (Avri Altma=
n)
>=20
> v1 -> v2:
>   - Rename "uic_send" to "send" and "uic_complete" to "complete"
>   - Move "send" trace before UIC command is sent otherwise "send" trace m=
ay
> log incorrect arguments
>   - Move "complete" trace to UIC interrupt handler to make logging time
> precise
>=20
> Stanley Chu (2):
>   scsi: ufs: Remove unused field in struct uic_command
>   scsi: ufs: Add trace event for UIC commands
>=20
>  drivers/scsi/ufs/ufshcd.c  | 26 ++++++++++++++++++++++++++
>  drivers/scsi/ufs/ufshcd.h  |  4 ----
>  include/trace/events/ufs.h | 31 +++++++++++++++++++++++++++++++
>  3 files changed, 57 insertions(+), 4 deletions(-)
>=20
> --
> 2.18.0