From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sayali Lokhande Subject: Re: [PATCH V5 2/2] scsi: ufs: Add configfs support for ufs provisioning Date: Wed, 11 Jul 2018 15:20:06 +0530 Message-ID: <4964583a-d13f-7b92-9838-e4398ec8c682@codeaurora.org> References: <1530858040-13971-1-git-send-email-sayalil@codeaurora.org> <1530858040-13971-3-git-send-email-sayalil@codeaurora.org> <20180708202136.GA1546@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20180708202136.GA1546@infradead.org> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Christoph Hellwig Cc: subhashj@codeaurora.org, cang@codeaurora.org, vivek.gautam@codeaurora.org, rnayak@codeaurora.org, vinholikatti@gmail.com, jejb@linux.vnet.ibm.com, martin.petersen@oracle.com, asutoshd@codeaurora.org, evgreen@chromium.org, riteshh@codeaurora.org, linux-scsi@vger.kernel.org, open list List-Id: linux-scsi@vger.kernel.org On 7/9/2018 1:51 AM, Christoph Hellwig wrote: > On Fri, Jul 06, 2018 at 11:50:40AM +0530, Sayali Lokhande wrote: >> This patch adds configfs support to provision ufs device at >> runtime. This feature can be primarily useful in factory or >> assembly line as some devices may be required to be configured >> multiple times during initial system development phase. >> Configuration Descriptors can be written multiple times until >> bConfigDescrLock attribute is zero. >> >> Configuration descriptor buffer consists of Device and Unit >> descriptor configurable parameters which are parsed from vendor >> specific provisioning file and then passed via configfs node at >> runtime to provision ufs device. >> CONFIG_CONFIGFS_FS needs to be enabled for using this feature. > As mentioned before: this absolutely needs to be a separate > config option so that the code can be compiled out entirely. Agree. Will add separate config for UFS Provision and set it dependent on CONFIGFS_FS