Signed-off-by: Luca Ellero <luca.ell...@brickedbrain.com>
---
 drivers/staging/ced1401/ced_ioc.c |   21 ++++++++++++---------
 drivers/staging/ced1401/usb1401.c |    9 ++++++---
 drivers/staging/ced1401/usb1401.h |    6 +++---
 3 files changed, 21 insertions(+), 15 deletions(-)

diff --git a/drivers/staging/ced1401/ced_ioc.c 
b/drivers/staging/ced1401/ced_ioc.c
index 36cf974..f6f681f 100644
--- a/drivers/staging/ced1401/ced_ioc.c
+++ b/drivers/staging/ced1401/ced_ioc.c
@@ -593,7 +593,8 @@ int ced_clear_area(DEVICE_EXTENSION *pdx, int nArea)
                dev_err(&pdx->interface->dev, "%s: Attempt to clear area %d\n",
                        __func__, nArea);
        } else {
-               TRANSAREA *pTA = &pdx->rTransDef[nArea];        /*  to save 
typing */
+               /* to save typing */
+               struct transarea *pTA = &pdx->rTransDef[nArea];
                if (!pTA->bUsed)        /*  if not used... */
                        iReturn = U14ERR_NOTSET;        /*  ...nothing to be 
done */
                else {
@@ -622,10 +623,10 @@ int ced_clear_area(DEVICE_EXTENSION *pdx, int nArea)
                                    && (pdx->rDMAInfo.wIdent == nArea))
                                        pdx->bXFerWaiting = false;      /*  
Cannot have pending xfer if area cleared */
 
-                               /*  Clean out the TRANSAREA except for the wait 
queue, which is at the end */
+                               /*  Clean out the struct transarea except for 
the wait queue, which is at the end */
                                /*  This sets bUsed to false and dwEventSz to 0 
to say area not used and no events. */
                                memset(pTA, 0,
-                                      sizeof(TRANSAREA) -
+                                      sizeof(struct transarea) -
                                       sizeof(wait_queue_head_t));
                        }
                        spin_unlock_irq(&pdx->stagedLock);
@@ -670,7 +671,7 @@ static int ced_set_area(DEVICE_EXTENSION *pdx, int nArea, 
char __user *puBuf,
        unsigned int ulOffset = ((unsigned long)puBuf) & (PAGE_SIZE - 1);
        int len = (dwLength + ulOffset + PAGE_SIZE - 1) >> PAGE_SHIFT;
 
-       TRANSAREA *pTA = &pdx->rTransDef[nArea];        /*  to save typing */
+       struct transarea *pTA = &pdx->rTransDef[nArea]; /*  to save typing */
        struct page **pPages = NULL;    /*  space for page tables */
        int nPages = 0;         /*  and number of pages */
 
@@ -791,7 +792,7 @@ int ced_set_event(DEVICE_EXTENSION *pdx, struct 
transfer_event __user *pTE)
        if (te.wAreaNum >= MAX_TRANSAREAS)      /*  the area must exist */
                return U14ERR_BADAREA;
        else {
-               TRANSAREA *pTA = &pdx->rTransDef[te.wAreaNum];
+               struct transarea *pTA = &pdx->rTransDef[te.wAreaNum];
                mutex_lock(&pdx->io_mutex);     /*  make sure we have no 
competitor */
                spin_lock_irq(&pdx->stagedLock);
                if (pTA->bUsed) {       /*  area must be in use */
@@ -821,7 +822,7 @@ int ced_wait_event(DEVICE_EXTENSION *pdx, int nArea, int 
msTimeOut)
                return U14ERR_BADAREA;
        else {
                int iWait;
-               TRANSAREA *pTA = &pdx->rTransDef[nArea];
+               struct transarea *pTA = &pdx->rTransDef[nArea];
                msTimeOut = (msTimeOut * HZ + 999) / 1000;      /*  convert 
timeout to jiffies */
 
                /*  We cannot wait holding the mutex, but we check the flags 
while holding */
@@ -867,7 +868,7 @@ int ced_test_event(DEVICE_EXTENSION *pdx, int nArea)
        if ((unsigned)nArea >= MAX_TRANSAREAS)
                iReturn = U14ERR_BADAREA;
        else {
-               TRANSAREA *pTA = &pdx->rTransDef[nArea];
+               struct transarea *pTA = &pdx->rTransDef[nArea];
                mutex_lock(&pdx->io_mutex);     /*  make sure we have no 
competitor */
                spin_lock_irq(&pdx->stagedLock);
                iReturn = pTA->iWakeUp; /*  get wakeup count since last call */
@@ -1354,7 +1355,8 @@ int ced_get_circ_block(DEVICE_EXTENSION *pdx, TCIRCBLOCK 
__user *pCB)
        cb.dwSize = 0;
 
        if (nArea < MAX_TRANSAREAS) {   /*  The area number must be OK */
-               TRANSAREA *pArea = &pdx->rTransDef[nArea];      /*  Pointer to 
relevant info */
+               /* Pointer to relevant info */
+               struct transarea *pArea = &pdx->rTransDef[nArea];
                spin_lock_irq(&pdx->stagedLock);        /*  Lock others out */
 
                if ((pArea->bUsed) && (pArea->bCircular) &&     /*  Must be 
circular area */
@@ -1405,7 +1407,8 @@ int ced_free_circ_block(DEVICE_EXTENSION *pdx, TCIRCBLOCK 
__user *pCB)
        cb.dwSize = 0;
 
        if (nArea < MAX_TRANSAREAS) {   /*  The area number must be OK */
-               TRANSAREA *pArea = &pdx->rTransDef[nArea];      /*  Pointer to 
relevant info */
+               /* Pointer to relevant info */
+               struct transarea *pArea = &pdx->rTransDef[nArea];
                spin_lock_irq(&pdx->stagedLock);        /*  Lock others out */
 
                if ((pArea->bUsed) && (pArea->bCircular) &&     /*  Must be 
circular area */
diff --git a/drivers/staging/ced1401/usb1401.c 
b/drivers/staging/ced1401/usb1401.c
index b44140f..ee16353 100644
--- a/drivers/staging/ced1401/usb1401.c
+++ b/drivers/staging/ced1401/usb1401.c
@@ -441,7 +441,8 @@ static void ced_copy_user_space(DEVICE_EXTENSION *pdx, int 
n)
 {
        unsigned int nArea = pdx->StagedId;
        if (nArea < MAX_TRANSAREAS) {
-               TRANSAREA *pArea = &pdx->rTransDef[nArea];      /*  area to be 
used */
+               /*  area to be used */
+               struct transarea *pArea = &pdx->rTransDef[nArea];
                unsigned int dwOffset =
                    pdx->StagedDone + pdx->StagedOffset + pArea->dwBaseOffset;
                char *pCoherBuf = pdx->pCoherStagedIO;  /*  coherent buffer */
@@ -541,7 +542,8 @@ static void staged_callback(struct urb *pUrb)
 
        if ((pdx->StagedDone == pdx->StagedLength) ||   /*  If no more to do */
            (bCancel)) {                /*  or this IRP was cancelled */
-               TRANSAREA *pArea = &pdx->rTransDef[pdx->StagedId];      /*  
Transfer area info */
+               /*  Transfer area info */
+               struct transarea *pArea = &pdx->rTransDef[pdx->StagedId];
                dev_dbg(&pdx->interface->dev,
                        "%s: transfer done, bytes %d, cancel %d\n",
                        __func__, pdx->StagedDone, bCancel);
@@ -761,7 +763,8 @@ static int ced_stage_chunk(DEVICE_EXTENSION *pdx)
 int ced_read_write_mem(DEVICE_EXTENSION *pdx, bool Read, unsigned short wIdent,
                 unsigned int dwOffs, unsigned int dwLen)
 {
-       TRANSAREA *pArea = &pdx->rTransDef[wIdent];     /*  Transfer area info 
*/
+       /* Transfer area info */
+       struct transarea *pArea = &pdx->rTransDef[wIdent];
 
        if (!can_accept_io_requests(pdx)) {     /*  Are we in a state to accept 
new requests? */
                dev_err(&pdx->interface->dev, "%s: can't accept requests\n",
diff --git a/drivers/staging/ced1401/usb1401.h 
b/drivers/staging/ced1401/usb1401.h
index 59f90b1..1aab3e3 100644
--- a/drivers/staging/ced1401/usb1401.h
+++ b/drivers/staging/ced1401/usb1401.h
@@ -101,7 +101,7 @@ struct circ_blk {
 
 /*  A structure holding all of the information about a transfer area - an area 
of */
 /*   memory set up for use either as a source or destination in DMA transfers. 
*/
-typedef struct transarea {
+struct transarea {
        void    *lpvBuff;                /*  User address of xfer area saved 
for completeness */
        UINT        dwBaseOffset;           /*  offset to start of xfer area in 
first page */
        UINT        dwLength;               /*  Length of xfer area, in bytes */
@@ -116,7 +116,7 @@ typedef struct transarea {
        UINT        dwEventSz;              /*  ...notification by the event SZ 
is 0 if unset */
        struct circ_blk aBlocks[2];         /*  Info on a pair of circular 
blocks */
        wait_queue_head_t wqEvent;          /*  The wait queue for events in 
this area MUST BE LAST */
-} TRANSAREA;
+};
 
 /*  The DMADESC structure is used to hold information on the transfer in 
progress. It */
 /*  is set up by ReadDMAInfo, using information sent by the 1401 in an escape 
sequence. */
@@ -156,7 +156,7 @@ typedef struct _DEVICE_EXTENSION {
        __u8 bInterval;                     /* Interrupt end point interval */
 
        volatile unsigned int dwDMAFlag;    /* state of DMA */
-       TRANSAREA rTransDef[MAX_TRANSAREAS];/* transfer area info */
+       struct transarea rTransDef[MAX_TRANSAREAS];  /* transfer area info */
        volatile DMADESC rDMAInfo;          /*  info on current DMA transfer */
        volatile bool bXFerWaiting;         /*  Flag set if DMA transfer 
stalled */
        volatile bool bInDrawDown;          /*  Flag that we want to halt 
transfers */
-- 
1.7.10.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to