Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1315671

Re: [PATCH 0/2] Add a skeleton module to perform a basic initialization on certain USB devices

From Michal Malý <madcatxster@devoid-pointer.net>
Newsgroups linux.kernel
Subject Re: [PATCH 0/2] Add a skeleton module to perform a basic initialization on certain USB devices
Date 2016-01-23 17:00 +0100
Message-ID <qU8Lw-4P0-3@gated-at.bofh.it> (permalink)
References <qU3LQ-1mq-3@gated-at.bofh.it> <qU5Xk-2Mq-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On sobota 23. ledna 2016 13:46:32 CET Bjørn Mork wrote:
> Michal Malý <madcatxster@devoid-pointer.net> writes:
> > This mini series adds a simple skeleton module whose only purpose is to
> > bring devices that at first appear as a generic USB device into another
> > mode that can be handled by a more specific subsystem.
> > 
> > This patch was originally requested by Dmitry, reasoning that loading the
> > entire xpad module just to switch a Logitech G920 wheel into HID mode is
> > excessive and does not make much sense.
> > 
> > The module can be extended to handle any other USB device that might
> > require such a switch.
> 
> Can this switching be done in userspace? That's what we normally do, ref
> usb_modeswitch.

I briefly considered leaving the switch up to the userspace and handling the 
device in the kernel only once it's been switched. I am however uncertain how 
to advertise this to the users. Writing a tiny app that would currently handle 
just one device seems like an overkill, abusing usb_modeswitch seems counter-
intuitive as it's purpose is to handle various USB modems. Having a tiny 
module in the kernel looks like the most straightforward thing to do as far as 
user experience is concerned.

I would not object to deferring the switch to userspace as long as there is an 
easy way how to communicate the need for a small switching tool to the users.

Michal

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/2] Add a skeleton module to perform a basic initialization on certain USB devices Michal Malý <madcatxster@devoid-pointer.net> - 2016-01-23 11:40 +0100
  [PATCH 1/2] Add usb_skelswitch skeleton module to do basic initialization of devices that at first appear as a generic USB device. Michal Malý <madcatxster@devoid-pointer.net> - 2016-01-23 11:40 +0100
  Re: [PATCH 0/2] Add a skeleton module to perform a basic initialization on certain USB devices Bjørn Mork <bjorn@mork.no> - 2016-01-23 14:00 +0100
    Re: [PATCH 0/2] Add a skeleton module to perform a basic initialization on certain USB devices Michal Malý <madcatxster@devoid-pointer.net> - 2016-01-23 17:00 +0100
      Re: [PATCH 0/2] Add a skeleton module to perform a basic  initialization on certain USB devices Greg KH <gregkh@linuxfoundation.org> - 2016-01-23 17:50 +0100
        Re: [PATCH 0/2] Add a skeleton module to perform a basic  initialization on certain USB devices Oliver Neukum <oneukum@suse.com> - 2016-01-24 23:10 +0100
          Re: [PATCH 0/2] Add a skeleton module to perform a basic  initialization on certain USB devices Greg KH <gregkh@linuxfoundation.org> - 2016-01-24 23:50 +0100
            Re: [PATCH 0/2] Add a skeleton module to perform a basic  initialization on certain USB devices Oliver Neukum <oneukum@suse.com> - 2016-01-25 12:30 +0100
      Re: [PATCH 0/2] Add a skeleton module to perform a basic initialization  on certain USB devices Jiri Kosina <jikos@kernel.org> - 2016-01-25 15:20 +0100
        Re: [PATCH 0/2] Add a skeleton module to perform a basic  initialization on certain USB devices Michal Malý <madcatxster@devoid-pointer.net> - 2016-01-25 22:10 +0100

csiph-web