From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751567AbdE0QRb (ORCPT ); Sat, 27 May 2017 12:17:31 -0400 Received: from mail-pg0-f65.google.com ([74.125.83.65]:34635 "EHLO mail-pg0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750724AbdE0QR1 (ORCPT ); Sat, 27 May 2017 12:17:27 -0400 Date: Sat, 27 May 2017 09:17:21 -0700 User-Agent: K-9 Mail for Android In-Reply-To: References: <4a4da770c8d0af7e3f31e79261b43573d70a6ea6.1495862272.git.dvhart@infradead.org> <201705271250.20338@pali> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Subject: Re: [PATCH 16/16] platform/x86: dell-wmi: Convert to the WMI bus infrastructure To: Andy Lutomirski , =?ISO-8859-1?Q?Pali_Roh=E1r?= , Jiri Kosina CC: Darren Hart , platform-driver-x86@vger.kernel.org, Andy Shevchenko , Mario Limonciello , Rafael Wysocki , "linux-kernel@vger.kernel.org" , Linux ACPI From: Dmitry Torokhov Message-ID: <3669DF37-7D17-40AD-852A-F8EDBE15A785@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v4RGHePG027467 On May 27, 2017 9:04:38 AM PDT, Andy Lutomirski wrote: >On Sat, May 27, 2017 at 3:50 AM, Pali Rohár >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? Thanks. -- Dmitry