NES gamepad Help

General discussions about V-USB, our firmware-only implementation of a low speed USB device on Atmel's AVR microcontrollers
Post Reply
EnigmaticManiac
Posts: 5
Joined: Sun Apr 17, 2011 4:01 am

NES gamepad Help

Post by EnigmaticManiac » Sun Apr 17, 2011 4:44 am

Hi, I am working on a very similar program for my senior design project as the NES/SNES gamepad example from OD. However, whenever I plug my chip into my computer it comes up as an Unknown Device and has a Code 43 error. This occurs on my 64bit Win7 computer and the 32bit XP computers at my school. I am using the circuit example that uses the 3.6v Zener diodes. I would appreciate an extra set of eyes for my code. I imagine it is something small and insignificant, but still quite the headache. :/

Main.c file

Code: Select all

#include <avr/io.h>
#include <avr/interrupt.h>
#include <stdbool.h>
#define F_CPU 12000000L
#include <util/delay.h>
#include <avr/eeprom.h>
#include <avr/pgmspace.h>
#include "usbdrv.h"

static unsigned char ReadController();

static unsigned char conStatus;

/* ------------------------------------------------------------------------- */
// The following is USB based code, that works in conjunction with the V-USB
// Device Stack.
PROGMEM char usbHidReportDescriptor[44] = {
    0x05, 0x01,                    // USAGE_PAGE (Generic Desktop)
    0x09, 0x05,                    // USAGE (Game Pad)
    0xa1, 0x01,                    // COLLECTION (Application)
    0x05, 0x09,                    //   USAGE_PAGE (Button)
    0x19, 0x01,                    //   USAGE_MINIMUM (Button 1)
    0x29, 0x04,                    //   USAGE_MAXIMUM (Button 4)
    0x15, 0x00,                    //   LOGICAL_MINIMUM (0)
    0x25, 0x01,                    //   LOGICAL_MAXIMUM (1)
    0x75, 0x01,                    //   REPORT_SIZE (1)
    0x95, 0x04,                    //   REPORT_COUNT (4)
    0x81, 0x02,                    //   INPUT (Data,Var,Abs)
    0x05, 0x01                     //   USAGE_PAGE (Generic Desktop)
    0x09, 0x01,                    //   USAGE (Pointer)
    0xa1, 0x00,                    //   COLLECTION (Physical)
    0x09, 0x31,                    //     USAGE (Y)
    0x09, 0x30,                    //     USAGE (X)
    0x15, 0x00,                    //     LOGICAL_MINIMUM (0)
    0x25, 0x02,                    //     LOGICAL_MAXIMUM (2)
    0x75, 0x02,                    //     REPORT_SIZE (2)
    0x95, 0x02,                    //     REPORT_COUNT (2)
    0x81, 0x02,                    //     INPUT (Data,Var,Abs)
    0xc0,                          //   END_COLLECTION
    0xc0                           // END_COLLECTION
};
//Based on this Report Descriptor, the report's LSBs will be buttons 4,3,2,and 1,where
//0x01 is button 1 and 0x08 is button 4. The report's MSBs deal with the D-pad. Because
//each axie is judged seperately, the top two bits are X and the bottom two bits are Y.
//Also, because of how the report is structured, 0b0101 represents no button being
//pushed, since 0b10 represents a logical maximum and 0b00 rep. a logical minimum.
//Example: 0b01010000 represents the controller with no buttons pressed.
//Example: 0b00001001 represents the user pressing down and left as well as start and A.
//Example: 0b10100110 represents the user pressing up and right as well as select and B.

static unsigned char idleRate;

usbMsgLen_t usbFunctionSetup(uchar data[8])
{
   usbRequest_t *rq = (void *)data;

   usbMsgPtr = conStatus;
    if((rq->bmRequestType & USBRQ_TYPE_MASK) == USBRQ_TYPE_CLASS){    /* class request type */
        if(rq->bRequest == USBRQ_HID_GET_REPORT){  /* wValue: ReportType (highbyte), ReportID (lowbyte) */
            /* we only have one report type, so don't look at wValue */
            conStatus = ReadController();
            return 1;
        }else if(rq->bRequest == USBRQ_HID_GET_IDLE){
            usbMsgPtr = &idleRate;
            return 1;
        }else if(rq->bRequest == USBRQ_HID_SET_IDLE){
            idleRate = rq->wValue.bytes[1];
        }
    }else{
        /* no vendor specific requests implemented */
    }
   return 0;
}

int main(){
   usbInit();
     sei();
   
   DDRA |= 0x01;//Latch
   DDRB |= 0x03;//Clk signal on pin0 and output on pin 1   

   while(1){
      usbPoll();
      if(usbInterruptIsReady()){
         conStatus = ReadController();
         usbSetInterrupt(&conStatus,1);
         //Loads data into interrupt buffer to send to host.
      }//end if
   }
return 42;
}


//Precondition: None
//Purpose: Manually pulses latch and generates artificial
//   clock signal. This allows for easy reading of the
//   status of the buttons on the NES game controller.
//Postconditions: Returns a byte containing the status of
//   all buttons.
static unsigned char ReadController(){
   unsigned char data = 0;
   PORTA = 0x01;//Latch pulse high
   _delay_us(1);
   PORTA = 0;//Latch close
   for(int i=0; i<8;i++){
      if(!(PINB & 0x02))//Buttons are normally open.
         data |= 1<<i;//Shifts in a 1
      else
         data |= 0<<i;//Shifts in a 0
      _delay_us(1);
      PORTB |= 0x01;//Clk high
      _delay_us(1);
      PORTB &= (0<<1);//Clk low
   }//for
   //This section rearranges the byte to conform with the HID Report that will be sent
   //to the computer
   bool up, down;
   up = data & 0x10; //stores status of up button
   down = data & 0x20; //stores status of down button
   data = (up << 6) | (down << 5); //switches up and down bits
   data ^= 0x50; //switches status of down and left bits
   return data;
}


usbconfig.h file

Code: Select all

/* Name: usbconfig.h
 * Project: V-USB, virtual USB port for Atmel's(r) AVR(r) microcontrollers
 * Author: Christian Starkjohann
 * Creation Date: 2005-04-01
 * Tabsize: 4
 * Copyright: (c) 2005 by OBJECTIVE DEVELOPMENT Software GmbH
 * License: GNU GPL v2 (see License.txt), GNU GPL v3 or proprietary (CommercialLicense.txt)
 * This Revision: $Id: usbconfig-prototype.h 785 2010-05-30 17:57:07Z cs $
 */

#ifndef __usbconfig_h_included__
#define __usbconfig_h_included__

/*
General Description:
This file is an example configuration (with inline documentation) for the USB
driver. It configures V-USB for USB D+ connected to Port D bit 2 (which is
also hardware interrupt 0 on many devices) and USB D- to Port D bit 4. You may
wire the lines to any other port, as long as D+ is also wired to INT0 (or any
other hardware interrupt, as long as it is the highest level interrupt, see
section at the end of this file).
+ To create your own usbconfig.h file, copy this file to your project's
+ firmware source directory) and rename it to "usbconfig.h".
+ Then edit it accordingly.
*/

/* ---------------------------- Hardware Config ---------------------------- */

#define USB_CFG_IOPORTNAME      D
/* This is the port where the USB bus is connected. When you configure it to
 * "B", the registers PORTB, PINB and DDRB will be used.
 */
#define USB_CFG_DMINUS_BIT      4
/* This is the bit number in USB_CFG_IOPORT where the USB D- line is connected.
 * This may be any bit in the port.
 */
#define USB_CFG_DPLUS_BIT       2
/* This is the bit number in USB_CFG_IOPORT where the USB D+ line is connected.
 * This may be any bit in the port. Please note that D+ must also be connected
 * to interrupt pin INT0! [You can also use other interrupts, see section
 * "Optional MCU Description" below, or you can connect D- to the interrupt, as
 * it is required if you use the USB_COUNT_SOF feature. If you use D- for the
 * interrupt, the USB interrupt will also be triggered at Start-Of-Frame
 * markers every millisecond.]
 */
#define USB_CFG_CLOCK_KHZ       (F_CPU/1000)
/* Clock rate of the AVR in kHz. Legal values are 12000, 12800, 15000, 16000,
 * 16500, 18000 and 20000. The 12.8 MHz and 16.5 MHz versions of the code
 * require no crystal, they tolerate +/- 1% deviation from the nominal
 * frequency. All other rates require a precision of 2000 ppm and thus a
 * crystal!
 * Since F_CPU should be defined to your actual clock rate anyway, you should
 * not need to modify this setting.
 */
#define USB_CFG_CHECK_CRC       0
/* Define this to 1 if you want that the driver checks integrity of incoming
 * data packets (CRC checks). CRC checks cost quite a bit of code size and are
 * currently only available for 18 MHz crystal clock. You must choose
 * USB_CFG_CLOCK_KHZ = 18000 if you enable this option.
 */

/* ----------------------- Optional Hardware Config ------------------------ */

/* #define USB_CFG_PULLUP_IOPORTNAME   D */
/* If you connect the 1.5k pullup resistor from D- to a port pin instead of
 * V+, you can connect and disconnect the device from firmware by calling
 * the macros usbDeviceConnect() and usbDeviceDisconnect() (see usbdrv.h).
 * This constant defines the port on which the pullup resistor is connected.
 */
/* #define USB_CFG_PULLUP_BIT          4 */
/* This constant defines the bit number in USB_CFG_PULLUP_IOPORT (defined
 * above) where the 1.5k pullup resistor is connected. See description
 * above for details.
 */

/* --------------------------- Functional Range ---------------------------- */

#define USB_CFG_HAVE_INTRIN_ENDPOINT    1
/* Define this to 1 if you want to compile a version with two endpoints: The
 * default control endpoint 0 and an interrupt-in endpoint (any other endpoint
 * number).
 */
#define USB_CFG_HAVE_INTRIN_ENDPOINT3   0
/* Define this to 1 if you want to compile a version with three endpoints: The
 * default control endpoint 0, an interrupt-in endpoint 3 (or the number
 * configured below) and a catch-all default interrupt-in endpoint as above.
 * You must also define USB_CFG_HAVE_INTRIN_ENDPOINT to 1 for this feature.
 */
#define USB_CFG_EP3_NUMBER              3
/* If the so-called endpoint 3 is used, it can now be configured to any other
 * endpoint number (except 0) with this macro. Default if undefined is 3.
 */
/* #define USB_INITIAL_DATATOKEN           USBPID_DATA1 */
/* The above macro defines the startup condition for data toggling on the
 * interrupt/bulk endpoints 1 and 3. Defaults to USBPID_DATA1.
 * Since the token is toggled BEFORE sending any data, the first packet is
 * sent with the oposite value of this configuration!
 */
#define USB_CFG_IMPLEMENT_HALT          0
/* Define this to 1 if you also want to implement the ENDPOINT_HALT feature
 * for endpoint 1 (interrupt endpoint). Although you may not need this feature,
 * it is required by the standard. We have made it a config option because it
 * bloats the code considerably.
 */
#define USB_CFG_SUPPRESS_INTR_CODE      0
/* Define this to 1 if you want to declare interrupt-in endpoints, but don't
 * want to send any data over them. If this macro is defined to 1, functions
 * usbSetInterrupt() and usbSetInterrupt3() are omitted. This is useful if
 * you need the interrupt-in endpoints in order to comply to an interface
 * (e.g. HID), but never want to send any data. This option saves a couple
 * of bytes in flash memory and the transmit buffers in RAM.
 */
#define USB_CFG_INTR_POLL_INTERVAL      10
/* If you compile a version with endpoint 1 (interrupt-in), this is the poll
 * interval. The value is in milliseconds and must not be less than 10 ms for
 * low speed devices.
 */
#define USB_CFG_IS_SELF_POWERED         0
/* Define this to 1 if the device has its own power supply. Set it to 0 if the
 * device is powered from the USB bus.
 */
#define USB_CFG_MAX_BUS_POWER           100
/* Set this variable to the maximum USB bus power consumption of your device.
 * The value is in milliamperes. [It will be divided by two since USB
 * communicates power requirements in units of 2 mA.]
 */
#define USB_CFG_IMPLEMENT_FN_WRITE      0
/* Set this to 1 if you want usbFunctionWrite() to be called for control-out
 * transfers. Set it to 0 if you don't need it and want to save a couple of
 * bytes.
 */
#define USB_CFG_IMPLEMENT_FN_READ       0
/* Set this to 1 if you need to send control replies which are generated
 * "on the fly" when usbFunctionRead() is called. If you only want to send
 * data from a static buffer, set it to 0 and return the data from
 * usbFunctionSetup(). This saves a couple of bytes.
 */
#define USB_CFG_IMPLEMENT_FN_WRITEOUT   0
/* Define this to 1 if you want to use interrupt-out (or bulk out) endpoints.
 * You must implement the function usbFunctionWriteOut() which receives all
 * interrupt/bulk data sent to any endpoint other than 0. The endpoint number
 * can be found in 'usbRxToken'.
 */
#define USB_CFG_HAVE_FLOWCONTROL        0
/* Define this to 1 if you want flowcontrol over USB data. See the definition
 * of the macros usbDisableAllRequests() and usbEnableAllRequests() in
 * usbdrv.h.
 */
#define USB_CFG_DRIVER_FLASH_PAGE       0
/* If the device has more than 64 kBytes of flash, define this to the 64 k page
 * where the driver's constants (descriptors) are located. Or in other words:
 * Define this to 1 for boot loaders on the ATMega128.
 */
#define USB_CFG_LONG_TRANSFERS          0
/* Define this to 1 if you want to send/receive blocks of more than 254 bytes
 * in a single control-in or control-out transfer. Note that the capability
 * for long transfers increases the driver size.
 */
/* #define USB_RX_USER_HOOK(data, len)     if(usbRxToken == (uchar)USBPID_SETUP) blinkLED(); */
/* This macro is a hook if you want to do unconventional things. If it is
 * defined, it's inserted at the beginning of received message processing.
 * If you eat the received message and don't want default processing to
 * proceed, do a return after doing your things. One possible application
 * (besides debugging) is to flash a status LED on each packet.
 */
/* #define USB_RESET_HOOK(resetStarts)     if(!resetStarts){hadUsbReset();} */
/* This macro is a hook if you need to know when an USB RESET occurs. It has
 * one parameter which distinguishes between the start of RESET state and its
 * end.
 */
/* #define USB_SET_ADDRESS_HOOK()              hadAddressAssigned(); */
/* This macro (if defined) is executed when a USB SET_ADDRESS request was
 * received.
 */
#define USB_COUNT_SOF                   0
/* define this macro to 1 if you need the global variable "usbSofCount" which
 * counts SOF packets. This feature requires that the hardware interrupt is
 * connected to D- instead of D+.
 */
/* #ifdef __ASSEMBLER__
 * macro myAssemblerMacro
 *     in      YL, TCNT0
 *     sts     timer0Snapshot, YL
 *     endm
 * #endif
 * #define USB_SOF_HOOK                    myAssemblerMacro
 * This macro (if defined) is executed in the assembler module when a
 * Start Of Frame condition is detected. It is recommended to define it to
 * the name of an assembler macro which is defined here as well so that more
 * than one assembler instruction can be used. The macro may use the register
 * YL and modify SREG. If it lasts longer than a couple of cycles, USB messages
 * immediately after an SOF pulse may be lost and must be retried by the host.
 * What can you do with this hook? Since the SOF signal occurs exactly every
 * 1 ms (unless the host is in sleep mode), you can use it to tune OSCCAL in
 * designs running on the internal RC oscillator.
 * Please note that Start Of Frame detection works only if D- is wired to the
 * interrupt, not D+. THIS IS DIFFERENT THAN MOST EXAMPLES!
 */
#define USB_CFG_CHECK_DATA_TOGGLING     0
/* define this macro to 1 if you want to filter out duplicate data packets
 * sent by the host. Duplicates occur only as a consequence of communication
 * errors, when the host does not receive an ACK. Please note that you need to
 * implement the filtering yourself in usbFunctionWriteOut() and
 * usbFunctionWrite(). Use the global usbCurrentDataToken and a static variable
 * for each control- and out-endpoint to check for duplicate packets.
 */
#define USB_CFG_HAVE_MEASURE_FRAME_LENGTH   0
/* define this macro to 1 if you want the function usbMeasureFrameLength()
 * compiled in. This function can be used to calibrate the AVR's RC oscillator.
 */
#define USB_USE_FAST_CRC                0
/* The assembler module has two implementations for the CRC algorithm. One is
 * faster, the other is smaller. This CRC routine is only used for transmitted
 * messages where timing is not critical. The faster routine needs 31 cycles
 * per byte while the smaller one needs 61 to 69 cycles. The faster routine
 * may be worth the 32 bytes bigger code size if you transmit lots of data and
 * run the AVR close to its limit.
 */

/* -------------------------- Device Description --------------------------- */

#define  USB_CFG_VENDOR_ID       0xc0, 0x16 /* = 0x16c0 = 5824 = voti.nl */
/* USB vendor ID for the device, low byte first. If you have registered your
 * own Vendor ID, define it here. Otherwise you may use one of obdev's free
 * shared VID/PID pairs. Be sure to read USB-IDs-for-free.txt for rules!
 * *** IMPORTANT NOTE ***
 * This template uses obdev's shared VID/PID pair for Vendor Class devices
 * with libusb: 0x16c0/0x5dc.  Use this VID/PID pair ONLY if you understand
 * the implications!
 */
#define  USB_CFG_DEVICE_ID       0xdf, 0x05 /* = 0x05dc = 1500 */
/* This is the ID of the product, low byte first. It is interpreted in the
 * scope of the vendor ID. If you have registered your own VID with usb.org
 * or if you have licensed a PID from somebody else, define it here. Otherwise
 * you may use one of obdev's free shared VID/PID pairs. See the file
 * USB-IDs-for-free.txt for details!
 * *** IMPORTANT NOTE ***
 * This template uses obdev's shared VID/PID pair for Vendor Class devices
 * with libusb: 0x16c0/0x5dc.  Use this VID/PID pair ONLY if you understand
 * the implications!
 */
#define USB_CFG_DEVICE_VERSION  0x00, 0x01
/* Version number of the device: Minor number first, then major number.
 */
#define USB_CFG_VENDOR_NAME     'd', 'w', 'i', 'e', 'h', 'e', '8', '8', '@', 'h', 'o',  't',  'm',  'a',  'i',  'l',  '.',  'c',  'o',  'm',   
#define USB_CFG_VENDOR_NAME_LEN 20
/* These two values define the vendor name returned by the USB device. The name
 * must be given as a list of characters under single quotes. The characters
 * are interpreted as Unicode (UTF-16) entities.
 * If you don't want a vendor name string, undefine these macros.
 * ALWAYS define a vendor name containing your Internet domain name if you use
 * obdev's free shared VID/PID pair. See the file USB-IDs-for-free.txt for
 * details.
 */
#define USB_CFG_DEVICE_NAME     'N', 'E', 'S', 't', 'o', 'U', 'S', 'B'
#define USB_CFG_DEVICE_NAME_LEN 8
/* Same as above for the device name. If you don't want a device name, undefine
 * the macros. See the file USB-IDs-for-free.txt before you assign a name if
 * you use a shared VID/PID.
 */
/*#define USB_CFG_SERIAL_NUMBER   'N', 'o', 'n', 'e' */
/*#define USB_CFG_SERIAL_NUMBER_LEN   0 */
/* Same as above for the serial number. If you don't want a serial number,
 * undefine the macros.
 * It may be useful to provide the serial number through other means than at
 * compile time. See the section about descriptor properties below for how
 * to fine tune control over USB descriptors such as the string descriptor
 * for the serial number.
 */
#define USB_CFG_DEVICE_CLASS        0    /* set to 0 if deferred to interface */
#define USB_CFG_DEVICE_SUBCLASS     0
/* See USB specification if you want to conform to an existing device class.
 * Class 0xff is "vendor specific".
 */
#define USB_CFG_INTERFACE_CLASS     3   /* define class here if not at device level */
#define USB_CFG_INTERFACE_SUBCLASS  0
#define USB_CFG_INTERFACE_PROTOCOL  0
/* See USB specification if you want to conform to an existing device class or
 * protocol. The following classes must be set at interface level:
 * HID class is 3, no subclass and protocol required (but may be useful!)
 * CDC class is 2, use subclass 2 and protocol 1 for ACM
 */
#define USB_CFG_HID_REPORT_DESCRIPTOR_LENGTH    44
/* Define this to the length of the HID report descriptor, if you implement
 * an HID device. Otherwise don't define it or define it to 0.
 * If you use this define, you must add a PROGMEM character array named
 * "usbHidReportDescriptor" to your code which contains the report descriptor.
 * Don't forget to keep the array and this define in sync!
 */

//#define USB_PUBLIC static
/* Use the define above if you #include usbdrv.c instead of linking against it.
 * This technique saves a couple of bytes in flash memory.
 */

/* ------------------- Fine Control over USB Descriptors ------------------- */
/* If you don't want to use the driver's default USB descriptors, you can
 * provide our own. These can be provided as (1) fixed length static data in
 * flash memory, (2) fixed length static data in RAM or (3) dynamically at
 * runtime in the function usbFunctionDescriptor(). See usbdrv.h for more
 * information about this function.
 * Descriptor handling is configured through the descriptor's properties. If
 * no properties are defined or if they are 0, the default descriptor is used.
 * Possible properties are:
 *   + USB_PROP_IS_DYNAMIC: The data for the descriptor should be fetched
 *     at runtime via usbFunctionDescriptor(). If the usbMsgPtr mechanism is
 *     used, the data is in FLASH by default. Add property USB_PROP_IS_RAM if
 *     you want RAM pointers.
 *   + USB_PROP_IS_RAM: The data returned by usbFunctionDescriptor() or found
 *     in static memory is in RAM, not in flash memory.
 *   + USB_PROP_LENGTH(len): If the data is in static memory (RAM or flash),
 *     the driver must know the descriptor's length. The descriptor itself is
 *     found at the address of a well known identifier (see below).
 * List of static descriptor names (must be declared PROGMEM if in flash):
 *   char usbDescriptorDevice[];
 *   char usbDescriptorConfiguration[];
 *   char usbDescriptorHidReport[];
 *   char usbDescriptorString0[];
 *   int usbDescriptorStringVendor[];
 *   int usbDescriptorStringDevice[];
 *   int usbDescriptorStringSerialNumber[];
 * Other descriptors can't be provided statically, they must be provided
 * dynamically at runtime.
 *
 * Descriptor properties are or-ed or added together, e.g.:
 * #define USB_CFG_DESCR_PROPS_DEVICE   (USB_PROP_IS_RAM | USB_PROP_LENGTH(18))
 *
 * The following descriptors are defined:
 *   USB_CFG_DESCR_PROPS_DEVICE
 *   USB_CFG_DESCR_PROPS_CONFIGURATION
 *   USB_CFG_DESCR_PROPS_STRINGS
 *   USB_CFG_DESCR_PROPS_STRING_0
 *   USB_CFG_DESCR_PROPS_STRING_VENDOR
 *   USB_CFG_DESCR_PROPS_STRING_PRODUCT
 *   USB_CFG_DESCR_PROPS_STRING_SERIAL_NUMBER
 *   USB_CFG_DESCR_PROPS_HID
 *   USB_CFG_DESCR_PROPS_HID_REPORT
 *   USB_CFG_DESCR_PROPS_UNKNOWN (for all descriptors not handled by the driver)
 *
 * Note about string descriptors: String descriptors are not just strings, they
 * are Unicode strings prefixed with a 2 byte header. Example:
 * int  serialNumberDescriptor[] = {
 *     USB_STRING_DESCRIPTOR_HEADER(6),
 *     'S', 'e', 'r', 'i', 'a', 'l'
 * };
 */

#define USB_CFG_DESCR_PROPS_DEVICE                  0
#define USB_CFG_DESCR_PROPS_CONFIGURATION           0
#define USB_CFG_DESCR_PROPS_STRINGS                 0
#define USB_CFG_DESCR_PROPS_STRING_0                0
#define USB_CFG_DESCR_PROPS_STRING_VENDOR           0
#define USB_CFG_DESCR_PROPS_STRING_PRODUCT          0
#define USB_CFG_DESCR_PROPS_STRING_SERIAL_NUMBER    0
#define USB_CFG_DESCR_PROPS_HID                     0
#define USB_CFG_DESCR_PROPS_HID_REPORT              0
#define USB_CFG_DESCR_PROPS_UNKNOWN                 0

/* ----------------------- Optional MCU Description ------------------------ */

/* The following configurations have working defaults in usbdrv.h. You
 * usually don't need to set them explicitly. Only if you want to run
 * the driver on a device which is not yet supported or with a compiler
 * which is not fully supported (such as IAR C) or if you use a differnt
 * interrupt than INT0, you may have to define some of these.
 */
/* #define USB_INTR_CFG            MCUCR */
/* #define USB_INTR_CFG_SET        ((1 << ISC00) | (1 << ISC01)) */
/* #define USB_INTR_CFG_CLR        0 */
/* #define USB_INTR_ENABLE         GIMSK */
/* #define USB_INTR_ENABLE_BIT     INT0 */
/* #define USB_INTR_PENDING        GIFR */
/* #define USB_INTR_PENDING_BIT    INTF0 */
/* #define USB_INTR_VECTOR         INT0_vect */

#endif /* __usbconfig_h_included__ */


Thanks in advance.

EnigmaticManiac
Posts: 5
Joined: Sun Apr 17, 2011 4:01 am

Re: NES gamepad Help

Post by EnigmaticManiac » Mon Apr 18, 2011 6:48 am

If it helps, I attached a logic analyzer to the circuit to see what was happening. When I connect it, the host tries to talk to the circuit 3 times but doesn't seem to get what it wants, which after some research suggest that something is wrong with my descriptors. Which might also have something to do with my code no longer compiling because "../usbdrvasm.S:391:9: error: #error "USB_CFG_CLOCK_KHZ is not one of the supported non-crc-rates!"." Which is extremely puzzling because F_CPU is defined in my main.c file.
I am using AVR Studio ver 4.18 with GCC compiler.
In source files folder:
main.c
oddebug.c
usbdrv.c
usbdrvasm.s

in header files folder:
asmcommon.inc
oddebug.h
usbconfig.h
usbdrv.h
usbdrvasm12.inc
usbportability.h

and they are all in the same location on my hard drive.

_frank26080115

Re: NES gamepad Help

Post by _frank26080115 » Mon Apr 18, 2011 3:39 pm

EnigmaticManiac wrote:Which might also have something to do with my code no longer compiling because "../usbdrvasm.S:391:9: error: #error "USB_CFG_CLOCK_KHZ is not one of the supported non-crc-rates!"." Which is extremely puzzling because F_CPU is defined in my main.c file.


You can't define F_CPU in one file and expect other files to share the same definition. You need to place F_CPU as a compiler option.

EnigmaticManiac
Posts: 5
Joined: Sun Apr 17, 2011 4:01 am

Re: NES gamepad Help

Post by EnigmaticManiac » Mon Apr 18, 2011 8:09 pm

Thanks for that tip. No change though. For some reason, Windows doesn't seem to like my descriptors. Haven't modified any of the other files. I'm using an ATmega644pa, btw.

ulao
Rank 4
Rank 4
Posts: 481
Joined: Mon Aug 25, 2008 8:45 pm

Re: NES gamepad Help

Post by ulao » Tue Apr 19, 2011 2:52 am

See what usbLyser says...

EnigmaticManiac
Posts: 5
Joined: Sun Apr 17, 2011 4:01 am

Re: NES gamepad Help

Post by EnigmaticManiac » Tue Apr 19, 2011 5:12 am

USBlyzer tells me that Enumeration of device failed.

Description: Unknown Device
Instance Status: 01806400h DN_HAS_PROBLEM
DN_DISABLEABLE
DN_REMOVABLE
DN_NT_ENUMERATOR
DN_NT_DRIVER
Instance ID: USB\VID_0000&PID_0000\5&32417976&0&1
Hardware IDs: USB\UNKNOWN
Compatible IDs: USB\UNKNOWN
Service Name:
Setup Class: USB
Setup Class Description: Universal Serial Bus controllers
Setup Class GUID: {36fc9e60-c465-11cf-8056-444553540000}
Software Key: {36fc9e60-c465-11cf-8056-444553540000}\0035
Manufacturer: (Standard USB Host Controller)
Friendly Name:
Hardware Location: Port_#0001.Hub_#0002
PDO Name: \Device\USBPDO-5

Also, for shits, I changed the chip to an ATmega16a, no change. Assuming the code isn't wrong, what else could cause this kind of error, where the device doesn't even get past the first stage of enumeration?

Daid
Rank 2
Rank 2
Posts: 55
Joined: Mon Apr 18, 2011 12:19 pm

Re: NES gamepad Help

Post by Daid » Tue Apr 19, 2011 10:56 am

USB enumeration happens as soon as you connect the resistor from D- to VCC. So if you remove the chip you'll still get a enumeration failed error.

So if the VID/PID pair isn't found then the chip isn't communicating with the host yet. Which could be one of the following:
  • Wrong fuse settings (internal RC configured when you need to use an crystal, DIV8 setting in ATTiny2313)
  • Mismatch in crystal frequency and F_CPU.
  • Bad wiring, wiring USB cables directly to my board has caused me some grief. Try a very short USB cable. I also managed to switch around power and ground once...
  • Wrong pin/interrupt settings in usbconfig.h
  • Not calling usbPoll() fast enough (or any other general bug in your code) try loading the hid-mouse example. Which doesn't require any additional hardware and should tell you if your USB hardware is working.
  • No code is running in the chip. Crystal not working, power not connected, try a blinking led example. (Even without a LED you should be able to use your logic analyzer to see this pin working or not)

EnigmaticManiac
Posts: 5
Joined: Sun Apr 17, 2011 4:01 am

Re: NES gamepad Help

Post by EnigmaticManiac » Wed Apr 20, 2011 1:52 am

Apparently the issue was my zener diodes. Switched to a 3.3v regulator, works properly. Thanks for the help guys! Learned a lot of useful stuff.

l0l0l0l
Posts: 2
Joined: Sat May 07, 2011 6:23 am

Re: NES gamepad Help

Post by l0l0l0l » Sat May 07, 2011 6:31 am

EnigmaticManiac wrote:Apparently the issue was my zener diodes. Switched to a 3.3v regulator, works properly. Thanks for the help guys! Learned a lot of useful stuff.


Hey Enigmatic, did you use the schematic from http://www.raphnet.net/electronique/sne ... dex_en.php? If not, would you mind sending me your schematic and source code? I am interested in putting something like this together too.

ulao
Rank 4
Rank 4
Posts: 481
Joined: Mon Aug 25, 2008 8:45 pm

Re: NES gamepad Help

Post by ulao » Sat May 07, 2011 3:14 pm

I am interested in putting something like this together too.
Why woudl you not want to use Ralhp's ? His adapters are some of the best around.

l0l0l0l
Posts: 2
Joined: Sat May 07, 2011 6:23 am

Re: NES gamepad Help

Post by l0l0l0l » Sun May 08, 2011 12:41 am

ulao wrote:
I am interested in putting something like this together too.
Why woudl you not want to use Ralhp's ? His adapters are some of the best around.


I am attempting to, I am having trouble with it so I was wondering if the OP had any suggestions or made any changes to get it to work.

ulao
Rank 4
Rank 4
Posts: 481
Joined: Mon Aug 25, 2008 8:45 pm

Re: NES gamepad Help

Post by ulao » Sun May 08, 2011 2:08 am

Well, I'm not going to steal the touch from EnigmaticManiac but I will offer help if you need it. I have used all of Ralf's project and added a lot of features to them.

Here is my project if you are curios
http://bliss-box.net/Bliss-Box/new.html

Ralf has been MIA for some time, I hope all is well he was a great mentor.

enkae
Posts: 1
Joined: Mon Oct 24, 2011 3:40 pm

Re: NES gamepad Help

Post by enkae » Mon Oct 24, 2011 3:43 pm

hanks for that tip. No change though. Windows doesn't seem to like my descriptors. Haven't modified any of the other files. I'm using an ATmega644pa, btw. buy cialis now
Last edited by enkae on Wed Jan 04, 2012 9:24 am, edited 1 time in total.

Putois
Posts: 1
Joined: Wed Oct 26, 2011 4:41 pm

Re: NES gamepad Help

Post by Putois » Wed Oct 26, 2011 4:42 pm

Thanks for that tip. But I have the same reason, Windows doesn't seem to like my descriptors. I'm using an ATmega644pa too. cialis
Last edited by Putois on Tue Jan 10, 2012 1:50 pm, edited 1 time in total.

ulao
Rank 4
Rank 4
Posts: 481
Joined: Mon Aug 25, 2008 8:45 pm

Re: NES gamepad Help

Post by ulao » Thu Oct 27, 2011 4:10 am

I seem to remember something about the pico chips causing issues. My advice would be to try a atmega8/168/328 and see if it works. If its the chip then you may have to email Christian.

Post Reply