AVR build warnings; minor USB fix

git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@3729 42af7a65-404d-4744-a932-0658087f49c3

arch/avr/src/at90usb/at90usb_usbdev.c

@@ -1102,6 +1102,7 @@ static void avr_dispatchrequest(FAR const struct usb_ctrlreq_s *ctrl)
 
 static int avr_ep0configure(void)
 {
+  FAR struct avr_ep_s *privep = &g_usbdev.eplist[AVR_EP0];
   uint8_t regval;
 
   /* Configure endpoint 0 */
@@ -1121,6 +1122,16 @@ static int avr_ep0configure(void)
       return -EINVAL;
     }
 
+  /* Initialize the endpoint data structure.  Mark EP0 as an IN endpoint so
+   * that the submit() logic will know that any enqueue packets are to be
+   * sent.
+   */
+
+  memset(privep, 0, sizeof(struct avr_ep_s));
+  privep->ep.ops       = &g_epops;
+  privep->ep.maxpacket = AVR_CTRLEP_SIZE;
+  privep->epin         = 1;
+
   /* Enable OUT interrupts */
 
   UEIENX |= (1 << RXOUTE);
@@ -2090,6 +2101,7 @@ static int avr_geninterrupt(int irq, FAR void *context)
 static int avr_epconfigure(FAR struct usbdev_ep_s *ep,
                            FAR const struct usb_epdesc_s *desc, bool last)
 {
+  FAR struct avr_ep_s *privep = (FAR struct avr_ep_s *)ep;
   uint16_t maxpacket = GETUINT16(desc->mxpacketsize);
   uint8_t uecfg0x;
   uint8_t uecfg1x;
@@ -2097,7 +2109,7 @@ static int avr_epconfigure(FAR struct usbdev_ep_s *ep,
   uint8_t regval;
 
   usbtrace(TRACE_EPCONFIGURE, ep->eplog);
-  DEBUGASSERT(desc->addr == ep->eplog);
+  DEBUGASSERT(ep->eplog != 0 && desc->addr == ep->eplog);
 
   /* Configure the endpoint */
 
@@ -2133,16 +2145,18 @@ static int avr_epconfigure(FAR struct usbdev_ep_s *ep,
 
   if (USB_ISEPIN(desc->addr))
     {
+      DEBUGASSERT(privep->epin != 0);
       uecfg0x |= AVR_DIR_IN;
       ueienx   = (1 << RXOUTE);
     }
+  else
+    {
+      DEBUGASSERT(privep->epin == 0);
+    }
 
   /* Handle banking (this needs to be revisited... Always double bank?) */
 
-  if (ep->eplog != 0)
+  uecfg1x |= AVR_DOUBLE_BANK;
-    {
-      uecfg1x |= AVR_DOUBLE_BANK;
-    }
 
   /* Handle the maximum packet size */
 
@@ -2191,9 +2205,10 @@ static int avr_epconfigure(FAR struct usbdev_ep_s *ep,
       return -EINVAL;
     }
 
-  /* Reset endpoint status */
+  /* Save the new max packet size and reset endpoint status */
 
-  g_usbdev.stalled = false;
+  privep->ep.maxpacket = maxpacket;
+  privep->stalled      = 0;
 
   /* Enable interrupts as appropriate for this endpoint */
 
@@ -2518,6 +2533,7 @@ static FAR struct usbdev_ep_s *avr_allocep(FAR struct usbdev_s *dev,
                                            uint8_t epno, bool in,
                                            uint8_t eptype)
 {
+  FAR struct avr_ep_s *privep;
   irqstate_t flags;
   uint8_t epset = g_usbdev.epavail;
   uint8_t epmask;
@@ -2567,22 +2583,33 @@ static FAR struct usbdev_ep_s *avr_allocep(FAR struct usbdev_s *dev,
           epmask = 1 << epndx;
           if ((epset & epmask) != 0)
             {
+               /* Initialize the endpoint structure */
+
+               privep           = &g_usbdev.eplist[epndx];
+               memset(privep, 0, sizeof(struct avr_ep_s));
+
+               privep->ep.ops       = &g_epops;
+               privep->ep.eplog     = epndx;
+               privep->ep.maxpacket = (epndx == 1) ? 256 : 64;
+
                /* Mark the IN/OUT endpoint no longer available */
 
                g_usbdev.epavail &= ~epmask;
                if (in)
                  {
                    g_usbdev.epinset |= epmask;
+                   privep->epin      = 1;
                  }
                else
                  {
                    g_usbdev.epoutset |= epmask;
+                   privep->epin       = 0;
                  }
 
                /* And return the pointer to the standard endpoint structure */
 
                irqrestore(flags);
-               return &g_usbdev.eplist[epndx].ep;
+               return &privep->ep;
             }
         }
 
@@ -2716,8 +2743,6 @@ static int avr_pullup(struct usbdev_s *dev, bool enable)
 
 void up_usbinitialize(void)
 {
-  int i;
-
   usbtrace(TRACE_DEVINIT, 0);
 
   /* Shutdown the USB interface to put it in a known initial state */
@@ -2735,25 +2760,6 @@ void up_usbinitialize(void)
   g_usbdev.usbdev.ep0 = &g_usbdev.eplist[AVR_EP0].ep;
   g_usbdev.epavail    = AVR_ALL_EPS & ~(1 << AVR_EP0);
 
-  /* Initialize the endpoint list */
-
-  for (i = 0; i < AVR_NENDPOINTS; i++)
-    {
-      /* Set endpoint operations, reference to driver structure (not really
-       * necessary because there is only one controller), and the physical
-       * endpoint number (which is just the index to the endpoint).
-       */
-
-      g_usbdev.eplist[i].ep.ops   = &g_epops;
-      g_usbdev.eplist[i].ep.eplog = i;
- 
-      /* The maximum packet size may depend on the endpoint.  Endpoint 1 has
-       * a larger maximum packet size than the other endpoints.
-       */
-
-       g_usbdev.eplist[i].ep.maxpacket = (i == 1) ? 256 : 64;
-    }
-
   /* Reset the interface to force re-enumeration */
 
   avr_usbreset();