[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <0f061fc0-d7ee-aa48-278b-4ca7dc3ce53f@acm.org>
Date: Sat, 23 Apr 2022 14:38:46 -0700
From: Bart Van Assche <bvanassche@....org>
To: jin young choi <ychoijy@...il.com>,
Avri Altman <Avri.Altman@....com>
Cc: "j-young.choi@...sung.com" <j-young.choi@...sung.com>,
ALIM AKHTAR <alim.akhtar@...sung.com>,
"jejb@...ux.ibm.com" <jejb@...ux.ibm.com>,
"martin.petersen@...cle.com" <martin.petersen@...cle.com>,
"beanhuo@...ron.com" <beanhuo@...ron.com>,
Daejun Park <daejun7.park@...sung.com>,
"adrian.hunter@...el.com" <adrian.hunter@...el.com>,
"cang@...eaurora.org" <cang@...eaurora.org>,
"asutoshd@...eaurora.org" <asutoshd@...eaurora.org>,
"linux-scsi@...r.kernel.org" <linux-scsi@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] scsi: ufs: wb: Add Manual Flush sysfs and cleanup toggle
functions
On 4/23/22 07:24, jin young choi wrote:
>>> +static DEVICE_ATTR_RW(wb_flush_on);
>> Maybe wb_flush_enable ?
>>
> 'wb_on' sysfs already existed. So I named it in the same format (_on)
> I'll change both. (_on -> _enable)
sysfs attributes constitute an ABI. Breaking the user space ABI is not
allowed. Hence, renaming existing sysfs attributes is not an option.
Thanks,
Bart.
Powered by blists - more mailing lists