mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Maxin B John <maxin.john@gmail.com>
Cc: devel@linuxdriverproject.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Support for OWI/Maplin USB Robotic Arm
Date: Mon, 22 Oct 2012 15:40:12 -0700	[thread overview]
Message-ID: <20121022224012.GB24489@kroah.com> (raw)
In-Reply-To: <CAMhs6YxeUJiqzMAj=zBCpZkivDsjHk7Td_Ov50uwFw8vv+qrrg@mail.gmail.com>

On Mon, Oct 08, 2012 at 11:52:56PM -0400, Maxin B John wrote:
> Hi Greg,
> 
> On Mon, Oct 8, 2012 at 7:55 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> > On Mon, Oct 08, 2012 at 05:29:43PM -0400, Maxin B. John wrote:
> >> Hi Greg,
> >>
> >> This driver helps us to control the OWI/Maplin USB Robotic Arm device.
> >>
> >> I have tested this driver in my x86 Laptop and it works as expected
> >> (Some clean-ups and modifications are in the pipeline, though).
> >>
> >> After review process and the life in staging,I hope this driver will
> >> find it's place in "drivers/usb/misc/" :)
> >>
> >> The userspace utility program is available from the below URL:
> >> https://github.com/maxinbjohn/robotic_arm_driver
> >>
> >> OWI/Maplin USB Robotic arm is available from:
> >> http://www.maplin.co.uk/robotic-arm-kit-with-usb-pc-interface-266257
> >> http://www.owirobots.com/cart/html/owi-535pc-robotic-arm-kit-with-usb-pc-interface-assembled.html
> >>
> >> Please let me know your comments.
> >
> > Very nice, but are there any real requirement to have this be a kernel
> > driver, and not just be driven from userspace through libusb/usbfs
> > instead?
> 
> Thanks. A number of people are using the libusb based userspace
> applications to control these devices. This was an attempt to remove
> the libusb dependency and control the device easily like:
> # echo 1 > /sys/bus/usb/drivers/robotic_arm/5-1:1.0/led

I really think that libusb is the best way to control this device.  We
removed a bunch of drivers from the kernel a number of years ago that
could be better controlled by userspace.  Because of that, I can't take
this driver either.

Sorry,

greg k-h

      reply	other threads:[~2012-10-22 22:40 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-08 21:29 Maxin B. John
2012-10-08 23:55 ` Greg KH
2012-10-09  3:52   ` Maxin B John
2012-10-22 22:40     ` Greg KH [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20121022224012.GB24489@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=devel@linuxdriverproject.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=maxin.john@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome