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;
Subject: Re: [PATCH v4] Introduce support for Systems Management Driver over
 WMI for Dell Systems
To:     "Limonciello, Mario" <Mario.Limonciello@dell.com>,
        Divya Bharathi <divya27392@gmail.com>,
        "dvhart@infradead.org" <dvhart@infradead.org>
Cc:     LKML <linux-kernel@vger.kernel.org>,
        "platform-driver-x86@vger.kernel.org" 
        <platform-driver-x86@vger.kernel.org>,
        "Bharathi, Divya" <Divya.Bharathi@Dell.com>,
        Hans de Goede <hdegoede@redhat.com>,
        Andy Shevchenko <andy.shevchenko@gmail.com>,
        mark gross <mgross@linux.intel.com>,
        "Ksr, Prasanth" <Prasanth.Ksr@dell.com>
References: <20200923113015.110980-1-divya.bharathi@dell.com>
 <9b015765-de6f-3b2a-8804-e23864eb8806@infradead.org>
 <DM6PR19MB2636C149F84D46718774E7D2FA380@DM6PR19MB2636.namprd19.prod.outlook.com>
From:   Randy Dunlap <rdunlap@infradead.org>
Message-ID: <23df654d-e7ce-dcc8-df81-1d4cdb62f001@infradead.org>
Date:   Wed, 23 Sep 2020 11:41:33 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.12.0
MIME-Version: 1.0
In-Reply-To: <DM6PR19MB2636C149F84D46718774E7D2FA380@DM6PR19MB2636.namprd19.prod.outlook.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 9/23/20 11:39 AM, Limonciello, Mario wrote:
>> -----Original Message-----
>> From: Randy Dunlap <rdunlap@infradead.org>
>> Sent: Wednesday, September 23, 2020 12:12
>> To: Divya Bharathi; dvhart@infradead.org
>> Cc: LKML; platform-driver-x86@vger.kernel.org; Bharathi, Divya; Hans de Goede;
>> Andy Shevchenko; mark gross; Limonciello, Mario; Ksr, Prasanth
>> Subject: Re: [PATCH v4] Introduce support for Systems Management Driver over
>> WMI for Dell Systems
>>
>>
>> [EXTERNAL EMAIL]
>>
>> Hi,
>>
>> On 9/23/20 4:30 AM, Divya Bharathi wrote:
>>> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
>>> index 40219bba6801..4fd7a3f0a904 100644
>>> --- a/drivers/platform/x86/Kconfig
>>> +++ b/drivers/platform/x86/Kconfig
>>> @@ -430,6 +430,18 @@ config DELL_WMI
>>>  	  To compile this driver as a module, choose M here: the module will
>>>  	  be called dell-wmi.
>>>
>>> +config DELL_WMI_SYSMAN
>>> +	tristate "Dell WMI Systems management WMI driver"
>>
>> Could you drop one of those "WMI" strings or are both of them needed?
>>
> 
> Thanks, that's a good suggestion.  How about instead:
> 
> "Dell WMI based Systems management driver"

        WMI-based

OK. Thanks.

-- 
~Randy