Bulk End Point IN not working

Tsuneo,
thanks for the reply..
I found out the problem was this..

void USB_Configure_Event (void) {

  if (USB_Configuration) {                   /* Check if USB is configured */
    GetInReport();
    USB_WriteEP(HID_EP_IN, &InReport[0], sizeof(InReport));
  }
}

I did not the change the endpoint from HID_EP_IN to 0x82. After i changed it everything is working great. I guess this is called once during the enumeration? But i dont understand what exactly is the purpose if this configuration event. I have read couple of other posts where you explained it, but i still dont understand the purpose of it, all it is doing is reading the IN endpoint buffer.

Tsuneo,
thanks for the reply..
I found out the problem was this..

void USB_Configure_Event (void) {

  if (USB_Configuration) {                   /* Check if USB is configured */
    GetInReport();
    USB_WriteEP(HID_EP_IN, &InReport[0], sizeof(InReport));
  }
}

I did not the change the endpoint from HID_EP_IN to 0x82. After i changed it everything is working great. I guess this is called once during the enumeration? But i dont understand what exactly is the purpose if this configuration event. I have read couple of other posts where you explained it, but i still dont understand the purpose of it, all it is doing is reading the IN endpoint buffer.

USB_Configure_Event() is called from Set_Configuration request handler. The HID example writes (USB_WriteEP()) the first packet to the IN endpoint, here.

> I guess this is called once during the enumeration? But i dont understand what exactly is the purpose if this configuration event.

Set_Configuration request is a part of enumeration sequence.
On enumeration, host reads out descriptors from the target device to know about the device. And then, host checks if it has enough resources to run the device or not.
- PC device driver for the device
- Bus bandwidth required to run the device endpoints
etc.
Set_Configuration notifies to the device that host is prepared to run the device.

In this request, device is expected to do these jobs
- Enable and initialize the endpoints other than the default one
- Initialize the context (variables and flags) for the communication using these endpoints



> I did not the change the endpoint from HID_EP_IN to 0x82. After i changed it everything is working great.

Really everything ?
Surely, you've gotten IN packets from the device. But how about the contents of the packet?
Jan's host example expects that the device returns the same packet written by the host application, immediately (loop back). With your change, the host app gets the previous packet, not the current one.

1) Delete the contents of USB_Configure_Event()

usb_user.c

#if USB_CONFIGURE_EVENT
void USB_Configure_Event (void) {

// comment the contents of this call
/*
  if (USB_Configuration) {                   /* Check if USB is configured */
    GetInReport();
    USB_WriteEP(HID_EP_IN, &InReport, sizeof(InReport));
  }
*/
}
#endif

2) Modify endpoint ISR (callback) for OUT as follows.
In this snippet, the process for TX packet is light one (just copy, for loopback).
But when the process takes more steps, you may move USB_WriteEP() elsewhere, where the TX packet is actually made up.

usb_user.c

#include <string.h>

#define BULK_EP_OUT       0x02
#define BULK_EP_IN        0x82
#define USB_BUFFER_SIZE   64
U8 usb_RX_buffer[ USB_BUFFER_SIZE ];
U8 usb_TX_buffer[ USB_BUFFER_SIZE ];

void USB_EndPoint2( U32 event ) {
  U32 packet_size;

  switch (event) {
    case USB_EVT_OUT:
      packet_size = USB_ReadEP( BULK_EP_OUT, usb_RX_buffer );        // Read a packet from the OUT endpoint

      memcpy( usb_TX_buffer, usb_RX_buffer, (size_t)packet_size );   // In this snippet, just copy the data
                                                                     // from RX buffer to TX buffer for loopback
                                                                     // Usually, your firmware parses RX packet
                                                                     // And it makes up a TX packet on the TX buffer

      USB_WriteEP( BULK_EP_IN, usb_TX_buffer, packet_size );         // Return TX packet
      break;

    case USB_EVT_IN:
      //
      // nothing to do, here
      //
      break;
  }
}

Tsuneo

Thanks,
I did not check if i am getting "proper" data but i was getting data. But i will def look into..