• State Not Answered
  • Locked Locked
  • Replies 14 replies
  • Subscribers 15 subscribers
  • Views 1977 views
  • Users 0 members are here
Options
Related
How was your experience today?
This discussion has been locked.
You can no longer post new replies to this discussion. If you have a question you can start a new discussion

Problem when EP1 is used for OUT EP in RTX HID program

Lingaraju CH

Good day to all,

I am using MCB1760 board.

I want to use EP1 for OUT EP in my application.

As a start I have modified the RTX_HID code as below.

The device is detecting.

When I run the HID_Client on the PC I can select the device.

But after selecting the device it is getting struct.

Changes are made as given below:
USB_ConfigDescriptor in usbdesc.c: Added EP1 output descriptor.
USB_EndPoint1 task of usbuser.c: Added USB_EVT_OUT functionality.

Please suggest

const U8 USB_ConfigDescriptor[] = {
/* Configuration 1 */ USB_CONFIGUARTION_DESC_SIZE, /* bLength */ USB_CONFIGURATION_DESCRIPTOR_TYPE, /* bDescriptorType */ WBVAL( /* wTotalLength */ 1*USB_CONFIGUARTION_DESC_SIZE + 1*USB_INTERFACE_DESC_SIZE + 1*HID_DESC_SIZE + 2*USB_ENDPOINT_DESC_SIZE //Updated on 09-11-11 ), 0x01, /* bNumInterfaces */ 0x01, /* bConfigurationValue: 0x01 is used to select this configuration */ 0x00, /* iConfiguration: no string to describe this configuration */ USB_CONFIG_BUS_POWERED /*|*/ /* bmAttributes */
/*USB_CONFIG_REMOTE_WAKEUP*/, USB_CONFIG_POWER_MA(100), /* bMaxPower, device power consumption is 100 mA */

/* Interface 0, Alternate Setting 0, HID Class */ USB_INTERFACE_DESC_SIZE, /* bLength */ USB_INTERFACE_DESCRIPTOR_TYPE, /* bDescriptorType */ 0x00, /* bInterfaceNumber */ 0x00, /* bAlternateSetting */ 0x02, /* bNumEndpoints */ //Updated to 2 on 09-11-11 USB_DEVICE_CLASS_HUMAN_INTERFACE, /* bInterfaceClass */ HID_SUBCLASS_NONE, /* bInterfaceSubClass */ HID_PROTOCOL_NONE, /* bInterfaceProtocol */ 0x04, /* iInterface */
/* HID Class Descriptor */
/* HID_DESC_OFFSET = 0x0012 */ HID_DESC_SIZE, /* bLength */ HID_HID_DESCRIPTOR_TYPE, /* bDescriptorType */ WBVAL(0x0100), /* 1.00 */ /* bcdHID */ 0x00, /* bCountryCode */ 0x01, /* bNumDescriptors */ HID_REPORT_DESCRIPTOR_TYPE, /* bDescriptorType */ WBVAL(HID_REPORT_DESC_SIZE), /* wDescriptorLength */
/* Endpoint, HID Interrupt In */ USB_ENDPOINT_DESC_SIZE, /* bLength */ USB_ENDPOINT_DESCRIPTOR_TYPE, /* bDescriptorType */ USB_ENDPOINT_IN(1), /* bEndpointAddress */ USB_ENDPOINT_TYPE_INTERRUPT, /* bmAttributes */ WBVAL(0x0040), /* wMaxPacketSize */ 0x20, /* 32ms */ /* bInterval */
/* Endpoint, HID Interrupt Out */ USB_ENDPOINT_DESC_SIZE, /* bLength */ USB_ENDPOINT_DESCRIPTOR_TYPE, /* bDescriptorType */ USB_ENDPOINT_OUT(1), /* bEndpointAddress */ USB_ENDPOINT_TYPE_INTERRUPT, /* bmAttributes */ WBVAL(0x0040), /* wMaxPacketSize */ 0x20, /* 32ms */ /* bInterval */
/* Terminator */ 0 /* bLength */
};

#if (USB_EP_EVENT & (1 << 1))
__task void USB_EndPoint1 (void) { U16 evt;

for (;;) { os_evt_wait_or(0xFFFF, 0xFFFF); /* Wait for an Event */ evt = os_evt_get(); /* Get Event Flags */

if (evt & USB_EVT_IN) { GetInReport(); USB_WriteEP(HID_EP_IN, &InReport, sizeof(InReport)); } if (evt & USB_EVT_OUT) { USB_ReadEP(HID_EP_IN, &OutReport); SetOutReport(); } }
} #endif

Parents
  • 0 in reply to Lingaraju CH

    Seems (almost) fine. You've taken the code form this topic, and modified a little.
    http://www.keil.com/forum/19729/

    Did you modify USB_Core() task for USB_Configuration event, too?

    The original Keil RTX_HID continuously sends input report.
    To start this sequential reports, it puts the first report at USB_Configuration handler in USB_Core() task.
    But your modification sends the input report just when an output report completes, for loop back.
    The first report in USB_Configuration event is deleted.

    usbuser.c

U32 out_report_size = 0;
U32 in_report_size = 0;
U32 in_index = 0;
OS_SEM USB_IN_EP1_Semaphore;

__task void USB_Core (void) {
   ...
   ...

#if USB_CONFIGURE_EVENT
    if (evt & USB_EVT_SET_CFG) {
      if (USB_Configuration) {              /* Check if USB is configured */
                                     // <--- comment these two lines
//        GetInReport();
//        USB_WriteEP(HID_EP_IN, &InReport, sizeof(InReport));
                                     // <--- add these three lines
        out_report_size = 0;
        in_report_size = 0;
        in_index = 0;
        os_sem_init( USB_IN_EP1_Semaphore, 1 );
     }
    }
#endif

    To speed up the response, set bInterval value on the endpoint descriptors to 1 ms.

    Tsuneo

Reply
  • 0 in reply to Lingaraju CH

    Seems (almost) fine. You've taken the code form this topic, and modified a little.
    http://www.keil.com/forum/19729/

    Did you modify USB_Core() task for USB_Configuration event, too?

    The original Keil RTX_HID continuously sends input report.
    To start this sequential reports, it puts the first report at USB_Configuration handler in USB_Core() task.
    But your modification sends the input report just when an output report completes, for loop back.
    The first report in USB_Configuration event is deleted.

    usbuser.c

U32 out_report_size = 0;
U32 in_report_size = 0;
U32 in_index = 0;
OS_SEM USB_IN_EP1_Semaphore;

__task void USB_Core (void) {
   ...
   ...

#if USB_CONFIGURE_EVENT
    if (evt & USB_EVT_SET_CFG) {
      if (USB_Configuration) {              /* Check if USB is configured */
                                     // <--- comment these two lines
//        GetInReport();
//        USB_WriteEP(HID_EP_IN, &InReport, sizeof(InReport));
                                     // <--- add these three lines
        out_report_size = 0;
        in_report_size = 0;
        in_index = 0;
        os_sem_init( USB_IN_EP1_Semaphore, 1 );
     }
    }
#endif

    To speed up the response, set bInterval value on the endpoint descriptors to 1 ms.

    Tsuneo

Children