Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-20 Thread Jaegeuk Kim
On 12/20, gre...@linuxfoundation.org wrote: > On Tue, Dec 19, 2017 at 02:46:44PM -0800, Jaegeuk Kim wrote: > > >From 3368207da5988b8fed4e41e6c0f49a60ac014222 Mon Sep 17 00:00:00 2001 > > From: Jaegeuk Kim > > Date: Tue, 26 Sep 2017 20:53:48 -0700 > > Subject: [PATCH 2/2] scsi:

Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-20 Thread Jaegeuk Kim
On 12/20, gre...@linuxfoundation.org wrote: > On Tue, Dec 19, 2017 at 02:46:44PM -0800, Jaegeuk Kim wrote: > > >From 3368207da5988b8fed4e41e6c0f49a60ac014222 Mon Sep 17 00:00:00 2001 > > From: Jaegeuk Kim > > Date: Tue, 26 Sep 2017 20:53:48 -0700 > > Subject: [PATCH 2/2] scsi: ufs: introduce

Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-20 Thread gre...@linuxfoundation.org
On Tue, Dec 19, 2017 at 02:46:44PM -0800, Jaegeuk Kim wrote: > >From 3368207da5988b8fed4e41e6c0f49a60ac014222 Mon Sep 17 00:00:00 2001 > From: Jaegeuk Kim > Date: Tue, 26 Sep 2017 20:53:48 -0700 > Subject: [PATCH 2/2] scsi: ufs: introduce sysfs entries exposing UFS health >

Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-20 Thread gre...@linuxfoundation.org
On Tue, Dec 19, 2017 at 02:46:44PM -0800, Jaegeuk Kim wrote: > >From 3368207da5988b8fed4e41e6c0f49a60ac014222 Mon Sep 17 00:00:00 2001 > From: Jaegeuk Kim > Date: Tue, 26 Sep 2017 20:53:48 -0700 > Subject: [PATCH 2/2] scsi: ufs: introduce sysfs entries exposing UFS health > info > > This patch

Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-19 Thread Bart Van Assche
On Tue, 2017-12-19 at 14:46 -0800, Jaegeuk Kim wrote: > Subject: [PATCH 2/2] scsi: ufs: introduce sysfs entries exposing UFS health > info > > This patch adds a new sysfs group, namely health, via: > >/sys/devices/soc/X.ufshc/health/ Thanks for the quick respin. This looks a lot better to

Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-19 Thread Bart Van Assche
On Tue, 2017-12-19 at 14:46 -0800, Jaegeuk Kim wrote: > Subject: [PATCH 2/2] scsi: ufs: introduce sysfs entries exposing UFS health > info > > This patch adds a new sysfs group, namely health, via: > >/sys/devices/soc/X.ufshc/health/ Thanks for the quick respin. This looks a lot better to

Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-19 Thread Jaegeuk Kim
>From 3368207da5988b8fed4e41e6c0f49a60ac014222 Mon Sep 17 00:00:00 2001 From: Jaegeuk Kim Date: Tue, 26 Sep 2017 20:53:48 -0700 Subject: [PATCH 2/2] scsi: ufs: introduce sysfs entries exposing UFS health info This patch adds a new sysfs group, namely health, via:

Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-19 Thread Jaegeuk Kim
>From 3368207da5988b8fed4e41e6c0f49a60ac014222 Mon Sep 17 00:00:00 2001 From: Jaegeuk Kim Date: Tue, 26 Sep 2017 20:53:48 -0700 Subject: [PATCH 2/2] scsi: ufs: introduce sysfs entries exposing UFS health info This patch adds a new sysfs group, namely health, via:

Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-19 Thread Jaegeuk Kim
On 12/19, Bart Van Assche wrote: > On Tue, 2017-12-19 at 12:02 -0800, Jaegeuk Kim wrote: > > This patch introduces sysfs entries to show the information. > > What information does "the information" refer to? > > Regarding the patch title: I think this patch introduces new sysfs attributes >

Re: [PATCH 2/2 v2] scsi: ufs: use sysfs entry for health info

2017-12-19 Thread Jaegeuk Kim
On 12/19, Bart Van Assche wrote: > On Tue, 2017-12-19 at 12:02 -0800, Jaegeuk Kim wrote: > > This patch introduces sysfs entries to show the information. > > What information does "the information" refer to? > > Regarding the patch title: I think this patch introduces new sysfs attributes >