Greg,
Many thanks for your reply. I will try to install git and download the latest release candidate. I see that 2.6.37.rc7 is already available. I think I will also hope to find get_icount there to address Pete's concern,
Thank you again,
Vadim.
--- On Sun, 12/26/10, Greg KH <gregkh@xxxxxxx> wrote:
> From: Greg KH <gregkh@xxxxxxx>
> Subject: Re: [PATCH 1/1] mct_u232: IOCTL implementation
> To: "Tsozik" <tsozik@xxxxxxxxx>
> Cc: linux-usb@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
> Date: Sunday, December 26, 2010, 1:41 PM
> On Sat, Dec 25, 2010 at 09:39:39PM
> -0800, Tsozik wrote:
> > From: Vadim Tsozik <tsozik@xxxxxxxxx>
> >
> > Added mct_u232_ioctl function and implemented
> TIOCMIWAIT and TIOCGICOUNT commands. MCT u232 p9 is one of a
> few usb to serail adapters which converts USB +/-5v voltage
> levels to COM +/-15 voltages. So it can also power COM
> interfaced devices. This makes it very usable for legacy COM
> interfaced data-acquisition hardware. I tested new
> implementation with AWARE Electronics RM-60 radiation meter,
> which sends pulse via RNG COM line whenever new particle is
> registered.
> >
> > Patch below is based on
> linux-2.6.35.10-72.fc14.x86_64.
>
> Care to make sure this applies on a kernel that we can
> actually apply
> patches to (i.e 2.6.37-rc6?) The .35 kernel was a
> long time ago.
>
> >
> > Signed-off-by: Vadim Tsozik <tsozik@xxxxxxxxx>
> >
> > ---
> > --- original/mct_u232.c 2010-12-25 21:31:13.744174626
> -0500
> > +++ mct_u232.c 2010-12-25 21:44:57.714640343
> -0500
>
> Care to read over Documentation/SubmittingPatches which
> shows you the
> proper patch level to make a patch against?
>
> > @@ -24,6 +24,12 @@
> > * Basic tests have
> been performed with minicom/zmodem transfers and
> > * modem dialing
> under Linux 2.4.0-test10 (for me it works fine).
> > *
> > + * 24-Apr-2010 Vadim Tsozik <tsozik@xxxxxxxxx>
> > + * - Added implementation of
> 'TIOCMIWAIT' and 'TIOCGICOUNT' ioctls.
> > + * This routines are
> necessary if you use mct u232 p9 as data
> > + * acquisition interface.
> These routines were tested with RM-60 AWARE
> > + * Electronics Radiation
> Monitor.
> > + *
>
> No need to ever add an entry here, your changelog
> information is now
> stored in git, not in the files themselves.
>
> > * 04-Nov-2003 Bill Marr <marr at
> flex dot com>
> > * - Mimic Windows
> driver by sending 2 USB 'device request' messages
> > * following
> normal 'baud rate change' message. This allows data to
> be
> > @@ -78,6 +84,8 @@
> > #include <asm/unaligned.h>
> > #include <linux/usb.h>
> > #include <linux/usb/serial.h>
> > +#include <linux/serial.h>
> > +#include <linux/ioctl.h>
> > #include "mct_u232.h"
> >
> > /*
> > @@ -104,6 +112,8 @@ static void
> mct_u232_break_ctl(struct tt
> > static int mct_u232_tiocmget(struct
> tty_struct *tty, struct file *file);
> > static int mct_u232_tiocmset(struct
> tty_struct *tty, struct file *file,
> >
> unsigned int set,
> unsigned int clear);
> > +static int mct_u232_ioctl(struct tty_struct
> *tty, struct file *file,
> > +
> unsigned int cmd,
> unsigned long arg);
> > static void mct_u232_throttle(struct tty_struct
> *tty);
> > static void mct_u232_unthrottle(struct
> tty_struct *tty);
> >
> > @@ -150,6 +160,7 @@ static struct usb_serial_driver
> mct_u232
> > .tiocmset
> = mct_u232_tiocmset,
> > .attach =
> mct_u232_startup,
> > .release =
> mct_u232_release,
> > + .ioctl =
> mct_u232_ioctl,
>
> Your email client converted the tabs to spaces, making this
> patch
> unable to apply. Please fix your email client and
> also run your patch
> through the scripts/checkpatch.pl script before sending it
> to fix up the
> formatting errors that are in it.
>
> Care to try again after doing this, and addressing Pete's
> concerns?
>
> thanks,
>
> greg k-h
>
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
