From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751007AbdE3CpM (ORCPT ); Mon, 29 May 2017 22:45:12 -0400 Received: from mail-pf0-f172.google.com ([209.85.192.172]:36420 "EHLO mail-pf0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750797AbdE3CpJ (ORCPT ); Mon, 29 May 2017 22:45:09 -0400 Date: Mon, 29 May 2017 19:45:05 -0700 From: Dmitry Torokhov To: Andy Lutomirski Cc: Pali =?iso-8859-1?Q?Roh=E1r?= , Jiri Kosina , Darren Hart , platform-driver-x86@vger.kernel.org, Andy Shevchenko , Mario Limonciello , Rafael Wysocki , "linux-kernel@vger.kernel.org" , Linux ACPI Subject: Re: [PATCH 16/16] platform/x86: dell-wmi: Convert to the WMI bus infrastructure Message-ID: <20170530024505.GC32841@dtor-ws> References: <4a4da770c8d0af7e3f31e79261b43573d70a6ea6.1495862272.git.dvhart@infradead.org> <201705271250.20338@pali> <3669DF37-7D17-40AD-852A-F8EDBE15A785@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, May 27, 2017 at 11:40:52AM -0700, Andy Lutomirski wrote: > On Sat, May 27, 2017 at 9:17 AM, Dmitry Torokhov > wrote: > > 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? > > > > If BUS_HOST means that the device is part of the host as opposed to > being plugged in, then it seems entirely reasonable. Yes, it basically means platform-specific interface. -- Dmitry