From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753777AbbLJBXF (ORCPT ); Wed, 9 Dec 2015 20:23:05 -0500 Received: from mail-ob0-f181.google.com ([209.85.214.181]:34677 "EHLO mail-ob0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750970AbbLJBXD (ORCPT ); Wed, 9 Dec 2015 20:23:03 -0500 MIME-Version: 1.0 In-Reply-To: <20151119183106.GC24773@dtor-ws> References: <1447345535-2912-1-git-send-email-simon@mungewell.org> <1447345535-2912-2-git-send-email-simon@mungewell.org> <20151119183106.GC24773@dtor-ws> Date: Wed, 9 Dec 2015 17:23:02 -0800 Message-ID: Subject: Re: [Patch-V2 1/6] INPUT: xpad: Add minimal support for Logitech G920 Wheel From: Dmitry Torokhov To: Jiri Kosina Cc: Simon Wood , "linux-input@vger.kernel.org" , lkml , Edwin , =?UTF-8?B?TWljaGFsIE1hbMO9?= , elias vanderstuyft , Benjamin Tissoires Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Nov 19, 2015 at 10:31 AM, Dmitry Torokhov wrote: > On Thu, Nov 19, 2015 at 02:50:51PM +0100, Jiri Kosina wrote: >> On Thu, 12 Nov 2015, Simon Wood wrote: >> >> > When plugged in the Logitech G920 wheel starts with USBID 046d:c261 >> > and behaviors as a vendor specific class. If a 'magic' byte sequence >> > is sent the wheel will detach and reconnect as a HID device with the >> > USBID 046d:c262. >> > >> > Signed-off-by: Simon Wood >> >> Adding Dmitry to CC. >> >> Dmitry, I am planning to take this through my tree together with the rest >> of the actual HID support for that device if you Ack this. > > Hmm, I have an incoming series for xbox that night clash with this... If > you'll put it in a clean branch off 4.3 I'd pull it and then get more > changes on top. > > Can we also change the subject as it is not about adding a minimal > support. Something like "Input: xpad - switch Logitech G920 Wheel into > HID mode" > > Otherwise: > > Acked-by: Dmitry Torokhov Hmm, looking sat this some more why are we waiting to switch device mode until after userspace opens input device instead of when we are executing driver probe()? Thanks. -- Dmitry