Re: [PATCH 16/16] platform/x86: dell-wmi: Convert to the WMI bus infrastructure
From: Andy Lutomirski
Date: Sat May 27 2017 - 14:44:01 EST
On Sat, May 27, 2017 at 9:17 AM, Dmitry Torokhov
<dmitry.torokhov@xxxxxxxxx> wrote:
> On May 27, 2017 9:04:38 AM PDT, Andy Lutomirski <luto@xxxxxxxxxx> wrote:
>>On Sat, May 27, 2017 at 3:50 AM, Pali RohÃr <pali.rohar@xxxxxxxxx>
>>wrote:
>>> On Saturday 27 May 2017 07:31:30 Darren Hart wrote:
>>>> - dell_wmi_input_dev->name = "Dell WMI hotkeys";
>>>> - dell_wmi_input_dev->phys = "wmi/input0";
>>>> - dell_wmi_input_dev->id.bustype = BUS_HOST;
>>>> + priv->input_dev->name = "Dell WMI hotkeys";
>>>> + priv->input_dev->id.bustype = BUS_HOST;
>>>
>>> Is not there BUS_WMI, or something like that? (Just asking)
>>>
>>
>>Jiri and/or Dmitry, what is bustype for, anyway?
>
> The bus type could be used to help further identifying device if it used same vendor/product for spi and i2c, for example, but there are not many if them. I'm not sure if anyone actually makes decisions based on it, but it is part of abi now.
>
>>I suppose we could add BUS_PLATFORM.
>
> What would be the difference from BUS_HOST?
>
If BUS_HOST means that the device is part of the host as opposed to
being plugged in, then it seems entirely reasonable.