2011-04-17 22:34:38

by Thiago Farina

[permalink] [raw]
Subject: [PATCH 1/1] drivers/usb/image/mdc800.c: Remove "typdef enum".

Instead use the keyword "enum" where we use it.

Signed-off-by: Thiago Farina <[email protected]>
---
drivers/usb/image/mdc800.c | 111 +++++++++++++++++++++----------------------
1 files changed, 54 insertions(+), 57 deletions(-)

diff --git a/drivers/usb/image/mdc800.c b/drivers/usb/image/mdc800.c
index 575b56c..4b0a032 100644
--- a/drivers/usb/image/mdc800.c
+++ b/drivers/usb/image/mdc800.c
@@ -122,59 +122,57 @@
/* Minor Number of the device (create with mknod /dev/mustek c 180 32) */
#define MDC800_DEVICE_MINOR_BASE 32

+/*
+ * Data and structs.
+ */

-/**************************************************************************
- Data and structs
-***************************************************************************/
-
-
-typedef enum {
+enum mdc800_state {
NOT_CONNECTED, READY, WORKING, DOWNLOAD
-} mdc800_state;
+};


-/* Data for the driver */
+/* Data for the driver. */
struct mdc800_data
{
- struct usb_device * dev; // Device Data
- mdc800_state state;
+ struct usb_device* dev; /* Device Data. */
+ enum mdc800_state state;

- unsigned int endpoint [4];
+ unsigned int endpoint[4];

- struct urb * irq_urb;
+ struct urb* irq_urb;
wait_queue_head_t irq_wait;
int irq_woken;
char* irq_urb_buffer;

- int camera_busy; // is camera busy ?
- int camera_request_ready; // Status to synchronize with irq
- char camera_response [8]; // last Bytes send after busy
+ int camera_busy; /* Is camera busy? */
+ int camera_request_ready; /* Status to synchronize with irq. */
+ char camera_response[8]; /* last Bytes send after busy. */

- struct urb * write_urb;
+ struct urb* write_urb;
char* write_urb_buffer;
wait_queue_head_t write_wait;
int written;


- struct urb * download_urb;
+ struct urb* download_urb;
char* download_urb_buffer;
wait_queue_head_t download_wait;
int downloaded;
- int download_left; // Bytes left to download ?
+ int download_left; /* Bytes left to download? */


/* Device Data */
- char out [64]; // Answer Buffer
- int out_ptr; // Index to the first not readen byte
- int out_count; // Bytes in the buffer
+ char out[64]; /* Answer buffer. */
+ int out_ptr; /* Index to the first not readen byte. */
+ int out_count; /* Bytes in the buffer. */

- int open; // Camera device open ?
- struct mutex io_lock; // IO -lock
+ int open; /* Camera device open. */
+ struct mutex io_lock; /* IO -lock. */

- char in [8]; // Command Input Buffer
- int in_count;
+ char in [8]; /* Command Input Buffer. */
+ int in_count;

- int pic_index; // Cache for the Imagesize (-1 for nothing cached )
+ int pic_index; /* Cache for the Imagesize (-1 for nothing cached) */
int pic_len;
int minor;
};
@@ -183,52 +181,51 @@ struct mdc800_data
/* Specification of the Endpoints */
static struct usb_endpoint_descriptor mdc800_ed [4] =
{
- {
- .bLength = 0,
+ {
+ .bLength = 0,
.bDescriptorType = 0,
.bEndpointAddress = 0x01,
- .bmAttributes = 0x02,
+ .bmAttributes = 0x02,
.wMaxPacketSize = cpu_to_le16(8),
- .bInterval = 0,
- .bRefresh = 0,
- .bSynchAddress = 0,
+ .bInterval = 0,
+ .bRefresh = 0,
+ .bSynchAddress = 0,
},
{
- .bLength = 0,
- .bDescriptorType = 0,
- .bEndpointAddress = 0x82,
- .bmAttributes = 0x03,
- .wMaxPacketSize = cpu_to_le16(8),
- .bInterval = 0,
- .bRefresh = 0,
- .bSynchAddress = 0,
+ .bLength = 0,
+ .bDescriptorType = 0,
+ .bEndpointAddress = 0x82,
+ .bmAttributes = 0x03,
+ .wMaxPacketSize = cpu_to_le16(8),
+ .bInterval = 0,
+ .bRefresh = 0,
+ .bSynchAddress = 0,
},
{
- .bLength = 0,
- .bDescriptorType = 0,
- .bEndpointAddress = 0x03,
- .bmAttributes = 0x02,
- .wMaxPacketSize = cpu_to_le16(64),
- .bInterval = 0,
- .bRefresh = 0,
- .bSynchAddress = 0,
+ .bLength = 0,
+ .bDescriptorType = 0,
+ .bEndpointAddress = 0x03,
+ .bmAttributes = 0x02,
+ .wMaxPacketSize = cpu_to_le16(64),
+ .bInterval = 0,
+ .bRefresh = 0,
+ .bSynchAddress = 0,
},
{
- .bLength = 0,
- .bDescriptorType = 0,
- .bEndpointAddress = 0x84,
- .bmAttributes = 0x02,
- .wMaxPacketSize = cpu_to_le16(64),
- .bInterval = 0,
- .bRefresh = 0,
- .bSynchAddress = 0,
+ .bLength = 0,
+ .bDescriptorType = 0,
+ .bEndpointAddress = 0x84,
+ .bmAttributes = 0x02,
+ .wMaxPacketSize = cpu_to_le16(64),
+ .bInterval = 0,
+ .bRefresh = 0,
+ .bSynchAddress = 0,
},
};

/* The Variable used by the driver */
static struct mdc800_data* mdc800;

-
/***************************************************************************
The USB Part of the driver
****************************************************************************/
--
1.7.3.2.343.g7d43d


2011-04-18 00:54:23

by Wanlong Gao

[permalink] [raw]
Subject: Re: [PATCH 1/1] drivers/usb/image/mdc800.c: Remove "typdef enum".

On 4/18/11, Thiago Farina <[email protected]> wrote:
> Instead use the keyword "enum" where we use it.
>
> Signed-off-by: Thiago Farina <[email protected]>
> ---
> drivers/usb/image/mdc800.c | 111
> +++++++++++++++++++++----------------------
> 1 files changed, 54 insertions(+), 57 deletions(-)
>
> diff --git a/drivers/usb/image/mdc800.c b/drivers/usb/image/mdc800.c
> index 575b56c..4b0a032 100644
> --- a/drivers/usb/image/mdc800.c
> +++ b/drivers/usb/image/mdc800.c
> @@ -122,59 +122,57 @@
> /* Minor Number of the device (create with mknod /dev/mustek c 180 32) */
> #define MDC800_DEVICE_MINOR_BASE 32
>
> +/*
> + * Data and structs.
> + */
>
> -/**************************************************************************
> - Data and structs
> -***************************************************************************/
> -
> -
> -typedef enum {
> +enum mdc800_state {
> NOT_CONNECTED, READY, WORKING, DOWNLOAD
> -} mdc800_state;
> +};
>
>
> -/* Data for the driver */
> +/* Data for the driver. */
> struct mdc800_data
> {
> - struct usb_device * dev; // Device Data
> - mdc800_state state;
> + struct usb_device* dev; /* Device Data. */
> + enum mdc800_state state;
>
> - unsigned int endpoint [4];
> + unsigned int endpoint[4];
>
> - struct urb * irq_urb;
> + struct urb* irq_urb;
> wait_queue_head_t irq_wait;
> int irq_woken;
> char* irq_urb_buffer;
>
> - int camera_busy; // is camera busy ?
> - int camera_request_ready; // Status to synchronize with irq
> - char camera_response [8]; // last Bytes send after busy
> + int camera_busy; /* Is camera busy? */
> + int camera_request_ready; /* Status to synchronize with irq. */
> + char camera_response[8]; /* last Bytes send after busy. */
>
> - struct urb * write_urb;
> + struct urb* write_urb;
> char* write_urb_buffer;
> wait_queue_head_t write_wait;
> int written;
>
>
> - struct urb * download_urb;
> + struct urb* download_urb;
> char* download_urb_buffer;
> wait_queue_head_t download_wait;
> int downloaded;
> - int download_left; // Bytes left to download ?
> + int download_left; /* Bytes left to download? */
>
>
> /* Device Data */
> - char out [64]; // Answer Buffer
> - int out_ptr; // Index to the first not readen byte
> - int out_count; // Bytes in the buffer
> + char out[64]; /* Answer buffer. */
> + int out_ptr; /* Index to the first not readen byte. */
> + int out_count; /* Bytes in the buffer. */
>
> - int open; // Camera device open ?
> - struct mutex io_lock; // IO -lock
> + int open; /* Camera device open. */
> + struct mutex io_lock; /* IO -lock. */
>
> - char in [8]; // Command Input Buffer
> - int in_count;
> + char in [8]; /* Command Input Buffer. */
> + int in_count;
>
> - int pic_index; // Cache for the Imagesize (-1 for nothing cached )
> + int pic_index; /* Cache for the Imagesize (-1 for nothing cached) */
> int pic_len;
> int minor;
> };
> @@ -183,52 +181,51 @@ struct mdc800_data
> /* Specification of the Endpoints */
> static struct usb_endpoint_descriptor mdc800_ed [4] =
> {
> - {
> - .bLength = 0,
> + {
> + .bLength = 0,
> .bDescriptorType = 0,
> .bEndpointAddress = 0x01,
> - .bmAttributes = 0x02,
> + .bmAttributes = 0x02,
> .wMaxPacketSize = cpu_to_le16(8),
> - .bInterval = 0,
> - .bRefresh = 0,
> - .bSynchAddress = 0,
> + .bInterval = 0,
> + .bRefresh = 0,
> + .bSynchAddress = 0,
> },
> {
> - .bLength = 0,
> - .bDescriptorType = 0,
> - .bEndpointAddress = 0x82,
> - .bmAttributes = 0x03,
> - .wMaxPacketSize = cpu_to_le16(8),
> - .bInterval = 0,
> - .bRefresh = 0,
> - .bSynchAddress = 0,
> + .bLength = 0,
> + .bDescriptorType = 0,
> + .bEndpointAddress = 0x82,
> + .bmAttributes = 0x03,
> + .wMaxPacketSize = cpu_to_le16(8),
> + .bInterval = 0,
> + .bRefresh = 0,
> + .bSynchAddress = 0,
> },
> {
> - .bLength = 0,
> - .bDescriptorType = 0,
> - .bEndpointAddress = 0x03,
> - .bmAttributes = 0x02,
> - .wMaxPacketSize = cpu_to_le16(64),
> - .bInterval = 0,
> - .bRefresh = 0,
> - .bSynchAddress = 0,
> + .bLength = 0,
> + .bDescriptorType = 0,
> + .bEndpointAddress = 0x03,
> + .bmAttributes = 0x02,
> + .wMaxPacketSize = cpu_to_le16(64),
> + .bInterval = 0,
> + .bRefresh = 0,
> + .bSynchAddress = 0,
> },
> {
> - .bLength = 0,
> - .bDescriptorType = 0,
> - .bEndpointAddress = 0x84,
> - .bmAttributes = 0x02,
> - .wMaxPacketSize = cpu_to_le16(64),
> - .bInterval = 0,
> - .bRefresh = 0,
> - .bSynchAddress = 0,
> + .bLength = 0,
> + .bDescriptorType = 0,
> + .bEndpointAddress = 0x84,
> + .bmAttributes = 0x02,
> + .wMaxPacketSize = cpu_to_le16(64),
> + .bInterval = 0,
> + .bRefresh = 0,
> + .bSynchAddress = 0,
> },
> };
>
> /* The Variable used by the driver */
> static struct mdc800_data* mdc800;
>
> -
> /***************************************************************************
> The USB Part of the driver
> ****************************************************************************/
> --
> 1.7.3.2.343.g7d43d
>
Why did you change it ? and why change so many lines ?
It had some thing wrong ?
Thanks
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>

2011-04-18 04:09:10

by Thiago Farina

[permalink] [raw]
Subject: Re: [PATCH 1/1] drivers/usb/image/mdc800.c: Remove "typdef enum".

On Sun, Apr 17, 2011 at 9:54 PM, Wanlong Gao <[email protected]> wrote:
> On 4/18/11, Thiago Farina <[email protected]> wrote:
>> Instead use the keyword "enum" where we use it.
>>
>> Signed-off-by: Thiago Farina <[email protected]>
>> ---
>>  drivers/usb/image/mdc800.c |  111
>> +++++++++++++++++++++----------------------
>>  1 files changed, 54 insertions(+), 57 deletions(-)
>>
>> diff --git a/drivers/usb/image/mdc800.c b/drivers/usb/image/mdc800.c
>> index 575b56c..4b0a032 100644
>> --- a/drivers/usb/image/mdc800.c
>> +++ b/drivers/usb/image/mdc800.c
>> @@ -122,59 +122,57 @@
>>  /* Minor Number of the device (create with mknod /dev/mustek c 180 32) */
>>  #define MDC800_DEVICE_MINOR_BASE 32
>>
>> +/*
>> + * Data and structs.
>> + */
>>
>> -/**************************************************************************
>> -     Data and structs
>> -***************************************************************************/
>> -
>> -
>> -typedef enum {
>> +enum mdc800_state {
>>       NOT_CONNECTED, READY, WORKING, DOWNLOAD
>> -} mdc800_state;
>> +};
>>
>>
>> -/* Data for the driver */
>> +/* Data for the driver. */
>>  struct mdc800_data
>>  {
>> -     struct usb_device *     dev;                    // Device Data
>> -     mdc800_state            state;
>> +     struct usb_device*      dev;                  /* Device Data. */
>> +     enum mdc800_state       state;
>>
>> -     unsigned int            endpoint [4];
>> +     unsigned int            endpoint[4];
>>
>> -     struct urb *            irq_urb;
>> +     struct urb*             irq_urb;
>>       wait_queue_head_t       irq_wait;
>>       int                     irq_woken;
>>       char*                   irq_urb_buffer;
>>
>> -     int                     camera_busy;          // is camera busy ?
>> -     int                     camera_request_ready; // Status to synchronize with irq
>> -     char                    camera_response [8];  // last Bytes send after busy
>> +     int                     camera_busy;          /* Is camera busy? */
>> +     int                     camera_request_ready; /* Status to synchronize with irq. */
>> +     char                    camera_response[8];   /* last Bytes send after busy. */
>>
>> -     struct urb *            write_urb;
>> +     struct urb*             write_urb;
>>       char*                   write_urb_buffer;
>>       wait_queue_head_t       write_wait;
>>       int                     written;
>>
>>
>> -     struct urb *            download_urb;
>> +     struct urb*             download_urb;
>>       char*                   download_urb_buffer;
>>       wait_queue_head_t       download_wait;
>>       int                     downloaded;
>> -     int                     download_left;          // Bytes left to download ?
>> +     int                     download_left;       /* Bytes left to download? */
>>
>>
>>       /* Device Data */
>> -     char                    out [64];       // Answer Buffer
>> -     int                     out_ptr;        // Index to the first not readen byte
>> -     int                     out_count;      // Bytes in the buffer
>> +     char                    out[64];        /* Answer buffer. */
>> +     int                     out_ptr;        /* Index to the first not readen byte. */
>> +     int                     out_count;      /* Bytes in the buffer. */
>>
>> -     int                     open;           // Camera device open ?
>> -     struct mutex            io_lock;        // IO -lock
>> +     int                     open;           /* Camera device open. */
>> +     struct mutex            io_lock;        /* IO -lock. */
>>
>> -     char                    in [8];         // Command Input Buffer
>> -     int                     in_count;
>> +     char                    in [8];         /* Command Input Buffer. */
>> +     int                     in_count;
>>
>> -     int                     pic_index;      // Cache for the Imagesize (-1 for nothing cached )
>> +     int                     pic_index;      /* Cache for the Imagesize (-1 for nothing cached) */
>>       int                     pic_len;
>>       int                     minor;
>>  };
>> @@ -183,52 +181,51 @@ struct mdc800_data
>>  /* Specification of the Endpoints */
>>  static struct usb_endpoint_descriptor mdc800_ed [4] =
>>  {
>> -     {
>> -             .bLength =              0,
>> +     {
>> +             .bLength =              0,
>>               .bDescriptorType =      0,
>>               .bEndpointAddress =     0x01,
>> -             .bmAttributes =         0x02,
>> +             .bmAttributes =         0x02,
>>               .wMaxPacketSize =       cpu_to_le16(8),
>> -             .bInterval =            0,
>> -             .bRefresh =             0,
>> -             .bSynchAddress =        0,
>> +             .bInterval =            0,
>> +             .bRefresh =             0,
>> +             .bSynchAddress =        0,
>>       },
>>       {
>> -             .bLength =              0,
>> -             .bDescriptorType =      0,
>> -             .bEndpointAddress =     0x82,
>> -             .bmAttributes =         0x03,
>> -             .wMaxPacketSize =       cpu_to_le16(8),
>> -             .bInterval =            0,
>> -             .bRefresh =             0,
>> -             .bSynchAddress =        0,
>> +             .bLength =              0,
>> +             .bDescriptorType =      0,
>> +             .bEndpointAddress =     0x82,
>> +             .bmAttributes =         0x03,
>> +             .wMaxPacketSize =       cpu_to_le16(8),
>> +             .bInterval =            0,
>> +             .bRefresh =             0,
>> +             .bSynchAddress =        0,
>>       },
>>       {
>> -             .bLength =              0,
>> -             .bDescriptorType =      0,
>> -             .bEndpointAddress =     0x03,
>> -             .bmAttributes =         0x02,
>> -             .wMaxPacketSize =       cpu_to_le16(64),
>> -             .bInterval =            0,
>> -             .bRefresh =             0,
>> -             .bSynchAddress =        0,
>> +             .bLength =              0,
>> +             .bDescriptorType =      0,
>> +             .bEndpointAddress =     0x03,
>> +             .bmAttributes =         0x02,
>> +             .wMaxPacketSize =       cpu_to_le16(64),
>> +             .bInterval =            0,
>> +             .bRefresh =             0,
>> +             .bSynchAddress =        0,
>>       },
>>       {
>> -             .bLength =              0,
>> -             .bDescriptorType =      0,
>> -             .bEndpointAddress =     0x84,
>> -             .bmAttributes =         0x02,
>> -             .wMaxPacketSize =       cpu_to_le16(64),
>> -             .bInterval =            0,
>> -             .bRefresh =             0,
>> -             .bSynchAddress =        0,
>> +             .bLength =              0,
>> +             .bDescriptorType =      0,
>> +             .bEndpointAddress =     0x84,
>> +             .bmAttributes =         0x02,
>> +             .wMaxPacketSize =       cpu_to_le16(64),
>> +             .bInterval =            0,
>> +             .bRefresh =             0,
>> +             .bSynchAddress =        0,
>>       },
>>  };
>>
>>  /* The Variable used by the driver */
>>  static struct mdc800_data* mdc800;
>>
>> -
>> /***************************************************************************
>>       The USB Part of the driver
>> ****************************************************************************/
>> --
>> 1.7.3.2.343.g7d43d
>>
> Why did you change it ?

To get rid of the typedef?

> and why change so many lines ?

Removed whitespaces, and converted // to /* */

> It had some thing wrong ?

Hum?
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?

2011-04-18 04:23:04

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH 1/1] drivers/usb/image/mdc800.c: Remove "typdef enum".

On Monday 18 April 2011, Thiago Farina wrote:

> > and why change so many lines ?
>
> Removed whitespaces, and converted // to /* */

You forgot to list these in the changelog.

Arnd

2011-04-18 04:28:51

by jiaweiwei

[permalink] [raw]
Subject: Re: [PATCH 1/1] drivers/usb/image/mdc800.c: Remove "typdef enum".

On Sun, Apr 17, 2011 at 07:34:14PM -0300, Thiago Farina wrote:
> Instead use the keyword "enum" where we use it.
>
> Signed-off-by: Thiago Farina <[email protected]>
> ---
> drivers/usb/image/mdc800.c | 111 +++++++++++++++++++++----------------------
This is well for me.
> NOT_CONNECTED, READY, WORKING, DOWNLOAD
> -} mdc800_state;
> +};
>
>
> -/* Data for the driver */
> +/* Data for the driver. */
> struct mdc800_data
> {
> - struct usb_device * dev; // Device Data
> - mdc800_state state;
> + struct usb_device* dev; /* Device Data. */
This should be 'struct usb_device *dev'. The same as
following. See Documentation/CodingStyle for details.
> + enum mdc800_state state;
>
> - unsigned int endpoint [4];
> + unsigned int endpoint[4];
>
> - struct urb * irq_urb;
> + struct urb* irq_urb;
> wait_queue_head_t irq_wait;

2011-04-18 07:25:08

by Oliver Neukum

[permalink] [raw]
Subject: Re: [PATCH 1/1] drivers/usb/image/mdc800.c: Remove "typdef enum".

Am Montag, 18. April 2011, 02:54:16 schrieb Wanlong Gao:
> > -
> > /***************************************************************************
> > The USB Part of the driver
> > ****************************************************************************/
> > --
> > 1.7.3.2.343.g7d43d
> >
> Why did you change it ? and why change so many lines ?
> It had some thing wrong ?

Exactly. This is a driver for legacy hardware. Could you leave it alone
unless you fix a real, as in makes a difference to code generation, bug?

Regards
Oliver

2011-04-18 17:27:52

by Wanlong Gao

[permalink] [raw]
Subject: Re: [PATCH 1/1] drivers/usb/image/mdc800.c: Remove "typdef enum".

2011-4-18 15:26, Oliver Neukum wroted:
> Am Montag, 18. April 2011, 02:54:16 schrieb Wanlong Gao:
>>> -
>>> /***************************************************************************
>>> The USB Part of the driver
>>> ****************************************************************************/
>>> --
>>> 1.7.3.2.343.g7d43d
>>>
>> Why did you change it ? and why change so many lines ?
>> It had some thing wrong ?
>
> Exactly. This is a driver for legacy hardware. Could you leave it alone
> unless you fix a real, as in makes a difference to code generation, bug?
>
> Regards
> Oliver
Hmm..Maybe he just want to make to code nice . and didn't think as more
about this as you .

Thanks
Regards

Wanlong Gao

2011-04-19 07:07:53

by Oliver Neukum

[permalink] [raw]
Subject: Re: [PATCH 1/1] drivers/usb/image/mdc800.c: Remove "typdef enum".

Am Montag, 18. April 2011, 19:27:35 schrieb Wanlong Gao:
> 2011-4-18 15:26, Oliver Neukum wroted:
> > Am Montag, 18. April 2011, 02:54:16 schrieb Wanlong Gao:
> >>> -
> >>> /***************************************************************************
> >>> The USB Part of the driver
> >>> ****************************************************************************/
> >>> --
> >>> 1.7.3.2.343.g7d43d
> >>>
> >> Why did you change it ? and why change so many lines ?
> >> It had some thing wrong ?
> >
> > Exactly. This is a driver for legacy hardware. Could you leave it alone
> > unless you fix a real, as in makes a difference to code generation, bug?
> >
> > Regards
> > Oliver
> Hmm..Maybe he just want to make to code nice . and didn't think as more
> about this as you .

Yes, I appreciate that, but the number of testers for these devices is low,
but there are still users.

Regards
Oliver