USB: coding style fixes (function naming)
authorarighi <arighi@38d2e660-2303-0410-9eaa-f027e97ec537>
Thu, 23 Sep 2010 17:12:36 +0000 (17:12 +0000)
committerarighi <arighi@38d2e660-2303-0410-9eaa-f027e97ec537>
Thu, 23 Sep 2010 17:12:36 +0000 (17:12 +0000)
git-svn-id: https://src.develer.com/svnoss/bertos/trunk@4276 38d2e660-2303-0410-9eaa-f027e97ec537

bertos/cpu/cortex-m3/drv/usb_stm32.c
bertos/drv/usb.h
bertos/drv/usb_keyboard.c
bertos/drv/usb_mouse.c
bertos/drv/usb_mouse.h
bertos/drv/usb_serial.c
bertos/drv/usb_serial.h

index 55e27788f5876f313921a03fd4538d801420a20f..a47237e9b51c2e1f2ec10428634ee411a13da91a 100644 (file)
@@ -137,20 +137,20 @@ static stm32_udc_t udc;
 static UsbDevice *usb_dev;
 
 /* USB packet memory management: list of allocated chunks */
-static pack_mem_slot_t *pPacketMemUse;
+static pack_mem_slot_t *mem_use;
 
 /* USB packet memory management: memory buffer metadata */
 #define EP_MAX_SLOTS   16
-static pack_mem_slot_t PacketMemBuff[EP_MAX_SLOTS];
+static pack_mem_slot_t memory_buffer[EP_MAX_SLOTS];
 
 /* Allocate a free block of the packet memory */
 static pack_mem_slot_t *usb_malloc(void)
 {
        unsigned int i;
 
-       for (i = 0; i < countof(PacketMemBuff); i++)
-               if (PacketMemBuff[i].Size == 0)
-                       return &PacketMemBuff[i];
+       for (i = 0; i < countof(memory_buffer); i++)
+               if (memory_buffer[i].Size == 0)
+                       return &memory_buffer[i];
        return NULL;
 }
 
@@ -161,92 +161,92 @@ static void usb_free(pack_mem_slot_t *pPntr)
 }
 
 /* Allocate a free chunk of the packet memory (inside a block) */
-static bool USB_AllocateBuffer(uint16_t *pOffset, uint32_t *pPacketSize,
+static bool usb_alloc_buffer(uint16_t *pOffset, uint32_t *size,
                             int EndPoint)
 {
-       pack_mem_slot_t *pPacketMem = pPacketMemUse,
-                       *pPacketMemNext, *pPacketMemUseNew;
-       uint32_t MaxPacketSize = *pPacketSize;
+       pack_mem_slot_t *mem = mem_use,
+                       *memNext, *mem_useNew;
+       uint32_t max_size = *size;
 
        /*
         * Packet size alignment:
         *  - fine-granularity allocation: size alignment by 2;
         *  - coarse-granularity allocation: size alignment by 32.
         */
-       if (MaxPacketSize < 62)
-               MaxPacketSize = ALIGN_UP(MaxPacketSize, 2);
+       if (max_size < 62)
+               max_size = ALIGN_UP(max_size, 2);
        else
-               MaxPacketSize = ALIGN_UP(MaxPacketSize, 32);
+               max_size = ALIGN_UP(max_size, 32);
        /*
         * Finding free memory chunks from the allocated blocks of the USB
         * packet memory.
         */
        *pOffset = 0;
-       while (pPacketMem != NULL)
+       while (mem != NULL)
        {
                /* Offset alignment by 4 */
-               *pOffset = ALIGN_UP(pPacketMem->Start + pPacketMem->Size, 4);
-               pPacketMemNext = pPacketMem->next;
-               if ((pPacketMem->next == NULL) ||
-                               (pPacketMemNext->Start >=
-                                       *pOffset + MaxPacketSize))
+               *pOffset = ALIGN_UP(mem->Start + mem->Size, 4);
+               memNext = mem->next;
+               if ((mem->next == NULL) ||
+                               (memNext->Start >=
+                                       *pOffset + max_size))
                        break;
-               pPacketMem = pPacketMem->next;
+               mem = mem->next;
        }
        /* Check for out-of-memory condition */
-       if ((*pOffset + MaxPacketSize) >= USB_BDT_OFFSET)
+       if ((*pOffset + max_size) >= USB_BDT_OFFSET)
                return false;
        /*
         * Allocate a new memory block, next to the last allocated block.
         */
-       pPacketMemUseNew = usb_malloc();
-       if (pPacketMemUseNew == NULL)
+       mem_useNew = usb_malloc();
+       if (mem_useNew == NULL)
                return false;
        /* Insert the block to the list of allocated blocks */
-       if (pPacketMemUse == NULL)
+       if (mem_use == NULL)
        {
-               pPacketMemUse = pPacketMemUseNew;
-               pPacketMemUse->next = NULL;
+               mem_use = mem_useNew;
+               mem_use->next = NULL;
        }
        else
        {
-               pPacketMemUseNew->next = pPacketMem->next;
-               pPacketMem->next = pPacketMemUseNew;
+               mem_useNew->next = mem->next;
+               mem->next = mem_useNew;
        }
        /* Update block's metadata */
-       pPacketMemUseNew->ep_addr = EndPoint;
-       pPacketMemUseNew->Start = *pOffset;
-       pPacketMemUseNew->Size = MaxPacketSize;
+       mem_useNew->ep_addr = EndPoint;
+       mem_useNew->Start = *pOffset;
+       mem_useNew->Size = max_size;
 
-       *pPacketSize = MaxPacketSize;
+       *size = max_size;
 
        return true;
 }
 
 /* Release a chunk of the packet memory (inside a block) */
-static void USB_ReleaseBuffer(int EndPoint)
+static void usb_free_buffer(int EndPoint)
 {
-       pack_mem_slot_t *pPacketMem, *pPacketMemPrev = NULL;
-       pPacketMem = pPacketMemUse;
+       pack_mem_slot_t *mem, *memPrev = NULL;
+       mem = mem_use;
 
-       while (pPacketMem != NULL)
+       while (mem != NULL)
        {
-               if (pPacketMem->ep_addr == EndPoint)
+               if (mem->ep_addr == EndPoint)
                {
-                       if (UNLIKELY(pPacketMemPrev == NULL))
+                       if (UNLIKELY(memPrev == NULL))
                        {
                                /* Free the first element of the list */
-                               pPacketMemUse = pPacketMemUse->next;
-                               usb_free(pPacketMem);
-                               pPacketMem = pPacketMemUse;
+                               mem_use = mem_use->next;
+                               usb_free(mem);
+                               mem = mem_use;
                                continue;
                        }
-                       pPacketMemPrev->next = pPacketMem->next;
-                       usb_free(pPacketMem);
+                       memPrev->next = mem->next;
+                       usb_free(mem);
                }
                else
-                       pPacketMemPrev = pPacketMem;
-               pPacketMem = pPacketMemPrev->next;
+                       memPrev = mem;
+               mem = memPrev->next;
        }
 }
 
@@ -286,7 +286,7 @@ static void usb_resume(void)
 }
 
 /* Convert logical EP address to physical EP address */
-static int USB_EpLogToPhysAdd(uint8_t ep_addr)
+static int usb_ep_logical_to_hw(uint8_t ep_addr)
 {
        int addr = (ep_addr & 0x0f) << 1;
        return (ep_addr & 0x80) ? addr + 1 : addr;
@@ -746,9 +746,9 @@ static int usb_ep_configure(const UsbEndpointDesc *epd, bool enable)
        stm32_usb_ep_t *ep_hw;
        reg32_t *hw;
        uint16_t Offset;
-       uint32_t MaxPacketSizeTmp;
+       uint32_t size;
 
-       EP = USB_EpLogToPhysAdd(epd->bEndpointAddress);
+       EP = usb_ep_logical_to_hw(epd->bEndpointAddress);
        ep_hw = &ep_cnfg[EP];
 
        if (enable)
@@ -757,8 +757,8 @@ static int usb_ep_configure(const UsbEndpointDesc *epd, bool enable)
                 * Allocate packet memory for EP buffer/s calculate actual size
                 * only for the OUT EPs.
                 */
-               MaxPacketSizeTmp = epd->wMaxPacketSize;
-               if (!USB_AllocateBuffer(&Offset, &MaxPacketSizeTmp, EP))
+               size = epd->wMaxPacketSize;
+               if (!usb_alloc_buffer(&Offset, &size, EP))
                        return -USB_MEMORY_FULL;
 
                /* Set EP status */
@@ -785,7 +785,7 @@ static int usb_ep_configure(const UsbEndpointDesc *epd, bool enable)
                                __func__, EP >> 1, EP & 1 ? "IN" : "OUT");
 
                /* Low-level endpoint configuration */
-               usb_ep_low_level_config(EP, Offset, MaxPacketSizeTmp);
+               usb_ep_low_level_config(EP, Offset, size);
 
                /* Set EP Kind & enable */
                switch (ep_hw->type)
@@ -847,7 +847,7 @@ static int usb_ep_configure(const UsbEndpointDesc *epd, bool enable)
                        EpCtrlClr_CTR_RX(hw);
                }
                /* Release buffer */
-               USB_ReleaseBuffer(EP);
+               usb_free_buffer(EP);
                ep_cnfg[EP].hw = NULL;
        }
        return 0;
@@ -1085,7 +1085,7 @@ static void USB_StatusHandler(UNUSED_ARG(int, EP))
 static bool rx_done;
 static size_t rx_size;
 
-static void usb_ep_read_complete(int ep)
+static void usb_endpointRead_complete(int ep)
 {
        if (UNLIKELY(ep >= ENP_MAX_NUMB))
        {
@@ -1098,9 +1098,9 @@ static void usb_ep_read_complete(int ep)
        rx_size = ep_cnfg[ep].size;
 }
 
-ssize_t usb_ep_read(int ep, void *buffer, ssize_t size)
+ssize_t usb_endpointRead(int ep, void *buffer, ssize_t size)
 {
-       int ep_num = USB_EpLogToPhysAdd(ep);
+       int ep_num = usb_ep_logical_to_hw(ep);
 
        /* Non-blocking read for EP0 */
        if (ep_num == CTRL_ENP_OUT)
@@ -1120,7 +1120,7 @@ ssize_t usb_ep_read(int ep, void *buffer, ssize_t size)
        rx_size = 0;
 
        /* Blocking read */
-       __usb_ep_read(ep_num, buffer, size, usb_ep_read_complete);
+       __usb_ep_read(ep_num, buffer, size, usb_endpointRead_complete);
        while (!rx_done)
                cpu_relax();
 
@@ -1130,7 +1130,7 @@ ssize_t usb_ep_read(int ep, void *buffer, ssize_t size)
 static bool tx_done;
 static size_t tx_size;
 
-static void usb_ep_write_complete(int ep)
+static void usb_endpointWrite_complete(int ep)
 {
        if (UNLIKELY(ep >= ENP_MAX_NUMB))
        {
@@ -1143,9 +1143,9 @@ static void usb_ep_write_complete(int ep)
        tx_size = ep_cnfg[ep].size;
 }
 
-ssize_t usb_ep_write(int ep, const void *buffer, ssize_t size)
+ssize_t usb_endpointWrite(int ep, const void *buffer, ssize_t size)
 {
-       int ep_num = USB_EpLogToPhysAdd(ep);
+       int ep_num = usb_ep_logical_to_hw(ep);
 
        /* Non-blocking write for EP0 */
        if (ep_num == CTRL_ENP_IN)
@@ -1165,7 +1165,7 @@ ssize_t usb_ep_write(int ep, const void *buffer, ssize_t size)
        tx_size = 0;
 
        /* Blocking write */
-       __usb_ep_write(ep_num, buffer, size, usb_ep_write_complete);
+       __usb_ep_write(ep_num, buffer, size, usb_endpointWrite_complete);
        while (!tx_done)
                cpu_relax();
 
@@ -1205,7 +1205,7 @@ static int UsbEpStatus(uint16_t index)
                return -USB_NODEV_ERROR;
 
        InData = 0;
-       USB_GetStallEP(USB_EpLogToPhysAdd(index), (bool *)&InData);
+       USB_GetStallEP(usb_ep_logical_to_hw(index), (bool *)&InData);
        __usb_ep_write(CTRL_ENP_IN,
                        (uint8_t *)&InData, sizeof(uint16_t),
                        USB_StatusHandler);
@@ -1657,10 +1657,10 @@ static void usb_hw_reset(void)
                ep_cnfg[i].hw = NULL;
 
        /* Initialize USB memory */
-       for (i = 0; i < countof(PacketMemBuff); i++)
-               PacketMemBuff[i].Size = 0;
+       for (i = 0; i < countof(memory_buffer); i++)
+               memory_buffer[i].Size = 0;
        usb->BTABLE = USB_BDT_OFFSET;
-       pPacketMemUse = NULL;
+       mem_use = NULL;
 
        /* Endpoint initialization */
        ret = usb_ep_configure(&USB_CtrlEpDescr0, true);
@@ -1861,7 +1861,7 @@ static void usb_init(void)
 }
 
 /* Register an upper layer USB device into the driver */
-int usb_device_register(UsbDevice *dev)
+int usb_deviceRegister(UsbDevice *dev)
 {
 #if CONFIG_KERN
        MOD_CHECK(proc);
index 58e5c3178a608db773bf2e9eb0d8304b5c896aac..3804fdce960b2f4942c18214580c1f9dc0680c4c 100644 (file)
@@ -457,18 +457,18 @@ INLINE int usb_endpointIsIsocOut(const struct UsbEndpointDesc *epd)
 }
 
 /*
- * usb_ep_read - configure endponint and perform the read operation
+ * usb_endpointRead - configure endponint and perform the read operation
  */
-ssize_t usb_ep_read(int ep, void *buffer, ssize_t size);
+ssize_t usb_endpointRead(int ep, void *buffer, ssize_t size);
 
 /*
- * usb_ep_write - configure endponint and perform the write operation
+ * usb_endpointWrite - configure endponint and perform the write operation
  */
-ssize_t usb_ep_write(int ep, const void *buffer, ssize_t size);
+ssize_t usb_endpointWrite(int ep, const void *buffer, ssize_t size);
 
 /*
- * usb_device_register - register a generic USB device driver
+ * usb_deviceRegister - register a generic USB device driver
  */
-int usb_device_register(UsbDevice *dev);
+int usb_deviceRegister(UsbDevice *dev);
 
 #endif /* USB_H */
index fca0eb5e51bb43d1ae937b37c80c6107c1588607..89264c8e4c96fe5db3b1fa4ee1d8b1579ed2d8b0 100644 (file)
@@ -217,13 +217,13 @@ static void usb_hid_event_cb(UsbCtrlRequest *ctrl)
                {
                case HID_DT_HID:
                        LOG_INFO("%s: HID_DT_HID\n", __func__);
-                       usb_ep_write(USB_DIR_IN | 0,
+                       usb_endpointWrite(USB_DIR_IN | 0,
                                        &usb_hid_descriptor,
                                        sizeof(usb_hid_descriptor));
                        break;
                case HID_DT_REPORT:
                        LOG_INFO("%s: HID_DT_REPORT\n", __func__);
-                       usb_ep_write(USB_DIR_IN | 0,
+                       usb_endpointWrite(USB_DIR_IN | 0,
                                        &hid_report_descriptor,
                                        sizeof(hid_report_descriptor));
                        hid_keyboard_configured = true;
@@ -238,14 +238,14 @@ static void usb_hid_event_cb(UsbCtrlRequest *ctrl)
                break;
        case HID_REQ_SET_REPORT:
                LOG_INFO("%s: HID_REQ_SET_REPORT\n", __func__);
-               usb_ep_write(USB_DIR_IN | 0, NULL, 0);
+               usb_endpointWrite(USB_DIR_IN | 0, NULL, 0);
                break;
        case HID_REQ_GET_IDLE:
                LOG_INFO("%s: HID_REQ_GET_IDLE\n", __func__);
                break;
        case HID_REQ_SET_IDLE:
                LOG_INFO("%s: HID_REQ_SET_IDLE\n", __func__);
-               usb_ep_write(USB_DIR_IN | 0, NULL, 0);
+               usb_endpointWrite(USB_DIR_IN | 0, NULL, 0);
                break;
        case HID_REQ_GET_PROTOCOL:
                LOG_INFO("%s: HID_REQ_GET_PROTOCOL\n", __func__);
@@ -271,7 +271,7 @@ static UsbDevice usb_keyboard = {
 /* Low-level usb-hid device initialization */
 static int usb_keyboard_hw_init(void)
 {
-       if (usb_device_register(&usb_keyboard) < 0)
+       if (usb_deviceRegister(&usb_keyboard) < 0)
                return -1;
        LOG_INFO("usb-hid: registered new USB keyboard device\n");
        return 0;
@@ -282,7 +282,7 @@ void usb_keyboard_send_event(uint8_t mod, uint8_t code)
 {
        report[0] = mod;
        report[2] = code;
-       usb_ep_write(USB_HID_REPORT_EP, &report, sizeof(report));
+       usb_endpointWrite(USB_HID_REPORT_EP, &report, sizeof(report));
 }
 
 /*
index f8e62855cb16763c3bdcbc2b4971ed6da7b74d65..7ab6c7cdc08d2670d87b7ac64968f30326fe59dc 100644 (file)
@@ -212,13 +212,13 @@ static void usb_hid_event_cb(UsbCtrlRequest *ctrl)
                {
                case HID_DT_HID:
                        LOG_INFO("%s: HID_DT_HID\n", __func__);
-                       usb_ep_write(USB_DIR_IN | 0,
+                       usb_endpointWrite(USB_DIR_IN | 0,
                                        &usb_hid_descriptor,
                                        sizeof(usb_hid_descriptor));
                        break;
                case HID_DT_REPORT:
                        LOG_INFO("%s: HID_DT_REPORT\n", __func__);
-                       usb_ep_write(USB_DIR_IN | 0,
+                       usb_endpointWrite(USB_DIR_IN | 0,
                                        &hid_report_descriptor,
                                        sizeof(hid_report_descriptor));
                        hid_mouse_configured = true;
@@ -236,7 +236,7 @@ static void usb_hid_event_cb(UsbCtrlRequest *ctrl)
                break;
        case HID_REQ_SET_IDLE:
                LOG_INFO("%s: HID_REQ_SET_IDLE\n", __func__);
-               usb_ep_write(USB_DIR_IN | 0, NULL, 0);
+               usb_endpointWrite(USB_DIR_IN | 0, NULL, 0);
                break;
        case HID_REQ_GET_PROTOCOL:
                LOG_INFO("%s: HID_REQ_GET_PROTOCOL\n", __func__);
@@ -262,19 +262,19 @@ static UsbDevice usb_mouse = {
 /* Low-level usb-hid device initialization */
 static int usb_mouse_hw_init(void)
 {
-       if (usb_device_register(&usb_mouse) < 0)
+       if (usb_deviceRegister(&usb_mouse) < 0)
                return -1;
        LOG_INFO("usb-hid: registered new USB mouse device\n");
        return 0;
 }
 
 /* Send a mouse event */
-void usb_mouse_send_event(int8_t x, int8_t y, int8_t buttons)
+void usb_mouseSendEvent(int8_t x, int8_t y, int8_t buttons)
 {
        report.x = x;
        report.y = y;
        report.buttons = buttons;
-       usb_ep_write(USB_HID_REPORT_EP, &report, sizeof(report));
+       usb_endpointWrite(USB_HID_REPORT_EP, &report, sizeof(report));
 }
 
 /*
@@ -282,7 +282,7 @@ void usb_mouse_send_event(int8_t x, int8_t y, int8_t buttons)
  *
  * TODO: support more than one device at the same time.
  */
-int usb_mouse_init(UNUSED_ARG(int, unit))
+int usb_mouseInit(UNUSED_ARG(int, unit))
 {
 #if CONFIG_KERN
        MOD_CHECK(proc);
index 4ffbe81ff93479e66a504d89ad5db3ec2c1511b3..010c34b585403f7400673cfc9f29265aa05c2a07 100644 (file)
@@ -42,7 +42,7 @@
 #ifndef USB_MOUSE_H
 #define USB_MOUSE_H
 
-void usb_mouse_send_event(int8_t x, int8_t y, int8_t buttons);
-int usb_mouse_init(int unit);
+void usb_mouseSendEvent(int8_t x, int8_t y, int8_t buttons);
+int usb_mouseInit(int unit);
 
 #endif /* USB_MOUSE_H */
index fd1ae517b07b481404e7c819db11ccbfda667b36..0a57a7b6da04c5dd76c26a36838dc757cbcac2ed 100644 (file)
@@ -178,7 +178,7 @@ static int usb_serial_hw_init(void)
 #if CONFIG_KERN
        MOD_CHECK(proc);
 #endif
-       if (usb_device_register(&usb_serial) < 0)
+       if (usb_deviceRegister(&usb_serial) < 0)
                return -1;
        LOG_INFO("usb-serial: registered new USB interface driver\n");
        return 0;
@@ -197,7 +197,7 @@ static size_t usb_serial_write(struct KFile *fd,
        /* Silent compiler warnings if _DEBUG is not enabled */
        (void)fd;
        ASSERT(fds->is_open);
-       return usb_ep_write(usb_serial_ep_in_descriptor.bEndpointAddress,
+       return usb_endpointWrite(usb_serial_ep_in_descriptor.bEndpointAddress,
                                buf, size);
 }
 
@@ -213,7 +213,7 @@ static size_t usb_serial_read(struct KFile *fd, void *buf, size_t size)
        /* Silent compiler warnings if _DEBUG is not enabled */
        (void)fd;
        ASSERT(fds->is_open);
-       return usb_ep_read(usb_serial_ep_out_descriptor.bEndpointAddress,
+       return usb_endpointRead(usb_serial_ep_out_descriptor.bEndpointAddress,
                                buf, size);
 }
 
@@ -293,7 +293,7 @@ static struct KFile *usb_serial_reopen(struct KFile *fd)
  *
  * \return 0 if OK, a negative value in case of error.
  */
-int usb_serial_init(struct USBSerial *fds, int unit)
+int usb_serialInit(struct USBSerial *fds, int unit)
 {
        memset(fds, 0, sizeof(*fds));
 
index 0406cebfca0865bf70e631fece9c89b30a780857..86ed05b9288d40e82efe83a0fb1e9ff4ddc70826 100644 (file)
@@ -71,6 +71,6 @@ INLINE USBSerial *USB_SERIAL_CAST(KFile *fd)
         return (USBSerial *)fd;
 }
 
-int usb_serial_init(struct USBSerial *fds, int unit);
+int usb_serialInit(struct USBSerial *fds, int unit);
 
 #endif /* USB_SERIAL_H */