Re: [PATCH 2/2] Staging: mimio: Fix Coding style errors in mimio.c

[Ameya Palande]

Hi Chris,

On Sun, Feb 21, 2010 at 5:27 PM, Chris Nicholson
<chris.nicholson@xxxxxxxxxxxx> wrote:
> This is a patch to the mimio.c that fixes a couple of
> space errors picked up by checkpatch.pl
>
> Signed-off-by: Chris Nicholson <chris.nicholson@xxxxxxxxxxxx>
> ---
> Âdrivers/staging/mimio/mimio.c | Â Â4 ++--
> Â1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/mimio/mimio.c b/drivers/staging/mimio/mimio.c
> index 1ba8103..4d7a602 100644
> --- a/drivers/staging/mimio/mimio.c
> +++ b/drivers/staging/mimio/mimio.c
> @@ -145,7 +145,7 @@ static void mimio_rx_handler(struct mimio *, unsigned char *, unsigned int);
> Âstatic int mimio_tx(struct mimio *, const char *, int);
>
> Âstatic char mimio_name[] = "VirtualInk mimio-Xi";
> -static struct usb_device_id mimio_table [] = {
> +static struct usb_device_id mimio_table[] = {
> Â Â Â Â{ USB_DEVICE(MIMIO_VENDOR_ID, MIMIO_PRODUCT_ID) },
> Â Â Â Â{ USB_DEVICE(0x0525, 0xa4a0) }, /* gadget zero firmware */
> Â Â Â Â{ }
> @@ -588,7 +588,7 @@ static int handle_mimio_rx_penupdown(struct mimio *mimio,
> Â Â Â Â Â Â Â Â Â Â Â Â*(mimio->pktbuf.p + 2);
> Â Â Â Â Â Â Â Âif (x != *(mimio->pktbuf.p + 3)) {
> Â Â Â Â Â Â Â Â Â Â Â Âdev_dbg(&mimio->idev->dev, "EV_PEN%s: bad xsum.\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â down ? "DOWN":"UP");
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â down ? "DOWN" : "UP");
> Â Â Â Â Â Â Â Â Â Â Â Â/* skip this event data */
> Â Â Â Â Â Â Â Â Â Â Â Âmimio->pktbuf.p += 4;
> Â Â Â Â Â Â Â Â Â Â Â Â/* decode any remaining events */
> --
> 1.6.6

I sent the same patch in Dec 2009 :(
http://driverdev.linuxdriverproject.org/pipermail/devel/2009-December/003128.html

It seems that mimio driver is already removed from linux-next:

commit c5982142e80229a90cd1a73eedef886de9c8e271
Author: Greg Kroah-Hartman <gregkh@xxxxxxx>
Date:   Thu Feb 11 17:15:14 2010 -0800

    Staging: mimio: remove the mimio driver

Cheers,
Ameya.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/