| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Mon, 18 Apr 2011 01:09:02 -0300
From: Thiago Farina <tfransosi@...il.com>
To: wanlong.gao@...il.com
Cc: linux-kernel@...r.kernel.org, Greg Kroah-Hartman <gregkh@...e.de>,
Arnd Bergmann <arnd@...db.de>, linux-usb@...r.kernel.org
Subject: Re: [PATCH 1/1] drivers/usb/image/mdc800.c: Remove "typdef enum".
On Sun, Apr 17, 2011 at 9:54 PM, Wanlong Gao <wanlong.gao@...il.com> wrote:
> On 4/18/11, Thiago Farina <tfransosi@...il.com> wrote:
>> Instead use the keyword "enum" where we use it.
>>
>> Signed-off-by: Thiago Farina <tfransosi@...il.com>
>> ---
>> 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?
Powered by blists - more mailing lists