Re: [PATCH v2 9/9] scsi: ufshpb: Make host mode parameters configurable

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Avri Altman <avri.altman@xxxxxxx>
Subject: Re: [PATCH v2 9/9] scsi: ufshpb: Make host mode parameters configurable
From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
Date: Tue, 2 Feb 2021 12:18:53 +0100
Cc: "James E . J . Bottomley" <jejb@xxxxxxxxxxxxxxxxxx>, "Martin K . Petersen" <martin.petersen@xxxxxxxxxx>, linux-scsi@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Bart Van Assche <bvanassche@xxxxxxx>, yongmyung lee <ymhungry.lee@xxxxxxxxxxx>, Daejun Park <daejun7.park@xxxxxxxxxxx>, alim.akhtar@xxxxxxxxxxx, asutoshd@xxxxxxxxxxxxxx, Zang Leigang <zangleigang@xxxxxxxxxxxxx>, Avi Shchislowski <avi.shchislowski@xxxxxxx>, Bean Huo <beanhuo@xxxxxxxxxx>, cang@xxxxxxxxxxxxxx, stanley.chu@xxxxxxxxxxxx
In-reply-to: <20210202083007.104050-10-avri.altman@wdc.com>

On Tue, Feb 02, 2021 at 10:30:07AM +0200, Avri Altman wrote:
> +struct attribute_group ufs_sysfs_hpb_param_group = {
> +	.name = "hpb_param_sysfs",

Shouldn't this be "hpb_param"?  Why the trailing "_sysfs", doesn't that
look odd in the directory path?

thanks,

greg k-h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Follow-Ups:
- RE: [PATCH v2 9/9] scsi: ufshpb: Make host mode parameters configurable
  - From: Avri Altman

References:
- [PATCH v2 0/9] Add Host control mode to HPB
  - From: Avri Altman
- [PATCH v2 9/9] scsi: ufshpb: Make host mode parameters configurable
  - From: Avri Altman

Prev by Date: Re: [PATCH v2 9/9] scsi: ufshpb: Make host mode parameters configurable
Next by Date: RE: [PATCH v2 9/9] scsi: ufshpb: Make host mode parameters configurable
Previous by thread: RE: RE: [PATCH v2 9/9] scsi: ufshpb: Make host mode parameters configurable
Next by thread: RE: [PATCH v2 9/9] scsi: ufshpb: Make host mode parameters configurable
Index(es):
- Date
- Thread

[Index of Archives] [SCSI Target Devel] [Linux SCSI Target Infrastructure] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Samba] [Device Mapper]