PC Button Box Not Working, need help

I'm building a button box for sim racing using a ProMicro and some push-buttons. I've got the Arduino IDE set up with the necessary libraries for a game controller. However, when I press the buttons on the box, they're not registering in Windows Game Controllers. I've double-checked the wiring, and the buttons work when tested with multimeter. Any ideas on what might be causing this issue?

Below is a photo of the wiring schematic I followed and the code.

#include <Keypad.h>
#include <Joystick.h>

#define ENABLE_PULLUPS
#define NUMROTARIES 2
#define NUMBUTTONS 13
#define NUMROWS 4
#define NUMCOLS 4


byte buttons[NUMROWS][NUMCOLS] = {
  {0,1,2,3},
  {4,5},
  {6,7,8,9},
  {10,11,12,13},
};

struct rotariesdef {
  byte pin1;
  byte pin2;
  int ccwchar;
  int cwchar;
  volatile unsigned char state;
};

rotariesdef rotaries[NUMROTARIES] {
  {0,1,24,25,0}, //For rotary encoder 1, {Pin #A, Pin #B, PC Btn #, PC Btn #, 0}
  {2,3,26,27,0}, //For rotary encoder 2, {Pin #A, Pin #B, PC Btn #, PC Btn #, 0}
};

#define DIR_CCW 0x10 //0x10 = 16
#define DIR_CW 0x20 //0x20 = 32
#define R_START 0x0 //0x0 = 0

#ifdef HALF_STEP
#define R_CCW_BEGIN 0x1 //0x1 = 1
#define R_CW_BEGIN 0x2 //0x2 = 2
#define R_START_M 0x3 //0x3 = 3
#define R_CW_BEGIN_M 0x4 //0x4 = 4
#define R_CCW_BEGIN_M 0x5 //0x5 = 5
const unsigned char ttable[6][4] = {
  // R_START (00)
  {R_START_M,            R_CW_BEGIN,     R_CCW_BEGIN,  R_START},
  // R_CCW_BEGIN
  {R_START_M | DIR_CCW, R_START,        R_CCW_BEGIN,  R_START},
  // R_CW_BEGIN
  {R_START_M | DIR_CW,  R_CW_BEGIN,     R_START,      R_START},
  // R_START_M (11)
  {R_START_M,            R_CCW_BEGIN_M,  R_CW_BEGIN_M, R_START},
  // R_CW_BEGIN_M
  {R_START_M,            R_START_M,      R_CW_BEGIN_M, R_START | DIR_CW},
  // R_CCW_BEGIN_M
  {R_START_M,            R_CCW_BEGIN_M,  R_START_M,    R_START | DIR_CCW},
};
#else
#define R_CW_FINAL 0x1
#define R_CW_BEGIN 0x2
#define R_CW_NEXT 0x3
#define R_CCW_BEGIN 0x4
#define R_CCW_FINAL 0x5
#define R_CCW_NEXT 0x6

const unsigned char ttable[7][4] = {
  // R_START
  {R_START,    R_CW_BEGIN,  R_CCW_BEGIN, R_START},
  // R_CW_FINAL
  {R_CW_NEXT,  R_START,     R_CW_FINAL,  R_START | DIR_CW},
  // R_CW_BEGIN
  {R_CW_NEXT,  R_CW_BEGIN,  R_START,     R_START},
  // R_CW_NEXT
  {R_CW_NEXT,  R_CW_BEGIN,  R_CW_FINAL,  R_START},
  // R_CCW_BEGIN
  {R_CCW_NEXT, R_START,     R_CCW_BEGIN, R_START},
  // R_CCW_FINAL
  {R_CCW_NEXT, R_CCW_FINAL, R_START,     R_START | DIR_CCW},
  // R_CCW_NEXT
  {R_CCW_NEXT, R_CCW_FINAL, R_CCW_BEGIN, R_START},
};
#endif

//Button to ProMicro inputs from the matrix
byte rowPins[NUMROWS] = {21,20,19,10}; 
byte colPins[NUMCOLS] = {16,14,15,18}; 

Keypad buttbx = Keypad( makeKeymap(buttons), rowPins, colPins, NUMROWS, NUMCOLS); 


//The list below enables or disables all the axis on the PC game controller.
//These axes  are X,Y,Z, Rx, Ry, Rz, Rudder, Throttle, Accl, Braking, Steering
Joystick_ Joystick(JOYSTICK_DEFAULT_REPORT_ID, 
  JOYSTICK_TYPE_JOYSTICK, 32, 0,
  false, false, false, false, false, false,
  false, false, false, false, false);

void setup() {
  Joystick.begin();
  rotary_init();}

void loop() { 
  CheckAllEncoders();
  CheckAllButtons();
}

void CheckAllButtons(void) {
      if (buttbx.getKeys())
    {
       for (int i=0; i<LIST_MAX; i++)   
        {
           if ( buttbx.key[i].stateChanged )   
            {
            switch (buttbx.key[i].kstate) {  
                    case PRESSED:
                    case HOLD:
                              Joystick.setButton(buttbx.key[i].kchar, 1);
                              break;
                    case RELEASED:
                    case IDLE:
                              Joystick.setButton(buttbx.key[i].kchar, 0);
                              break;
            }
           }   
         }
     }
}


void rotary_init() {
  for (int i=0;i<NUMROTARIES;i++) {
    pinMode(rotaries[i].pin1, INPUT);
    pinMode(rotaries[i].pin2, INPUT);
    #ifdef ENABLE_PULLUPS
      digitalWrite(rotaries[i].pin1, HIGH);
      digitalWrite(rotaries[i].pin2, HIGH);
    #endif
  }
}


unsigned char rotary_process(int _i) {
   unsigned char pinstate = (digitalRead(rotaries[_i].pin2) << 1) | digitalRead(rotaries[_i].pin1);
  rotaries[_i].state = ttable[rotaries[_i].state & 0xf][pinstate];
  return (rotaries[_i].state & 0x30);
}

void CheckAllEncoders(void) {
  for (int i=0;i<NUMROTARIES;i++) {
    unsigned char result = rotary_process(i);
    if (result == DIR_CCW) {
      Joystick.setButton(rotaries[i].ccwchar, 1); delay(100); Joystick.setButton(rotaries[i].ccwchar, 0);
    };
    if (result == DIR_CW) {
      Joystick.setButton(rotaries[i].cwchar, 1); delay(100); Joystick.setButton(rotaries[i].cwchar, 0);
    };
  }
}


#include <Keypad.h>
#include <Joystick.h>


#define ENABLE_PULLUPS
#define NUMROTARIES 2
#define NUMBUTTONS 13
#define NUMROWS 4
#define NUMCOLS 4



byte buttons[NUMROWS][NUMCOLS] = {
  {0,1,2,3},
  {4,5},
  {6,7,8,9},
  {10,11,12,13},
};


struct rotariesdef {
  byte pin1;
  byte pin2;
  int ccwchar;
  int cwchar;
  volatile unsigned char state;
};


rotariesdef rotaries[NUMROTARIES] {
  {0,1,24,25,0}, //For rotary encoder 1, {Pin #A, Pin #B, PC Btn #, PC Btn #, 0}
  {2,3,26,27,0}, //For rotary encoder 2, {Pin #A, Pin #B, PC Btn #, PC Btn #, 0}
};


#define DIR_CCW 0x10 //0x10 = 16
#define DIR_CW 0x20 //0x20 = 32
#define R_START 0x0 //0x0 = 0


#ifdef HALF_STEP
#define R_CCW_BEGIN 0x1 //0x1 = 1
#define R_CW_BEGIN 0x2 //0x2 = 2
#define R_START_M 0x3 //0x3 = 3
#define R_CW_BEGIN_M 0x4 //0x4 = 4
#define R_CCW_BEGIN_M 0x5 //0x5 = 5
const unsigned char ttable[6][4] = {
  // R_START (00)
  {R_START_M,            R_CW_BEGIN,     R_CCW_BEGIN,  R_START},
  // R_CCW_BEGIN
  {R_START_M | DIR_CCW, R_START,        R_CCW_BEGIN,  R_START},
  // R_CW_BEGIN
  {R_START_M | DIR_CW,  R_CW_BEGIN,     R_START,      R_START},
  // R_START_M (11)
  {R_START_M,            R_CCW_BEGIN_M,  R_CW_BEGIN_M, R_START},
  // R_CW_BEGIN_M
  {R_START_M,            R_START_M,      R_CW_BEGIN_M, R_START | DIR_CW},
  // R_CCW_BEGIN_M
  {R_START_M,            R_CCW_BEGIN_M,  R_START_M,    R_START | DIR_CCW},
};
#else
#define R_CW_FINAL 0x1
#define R_CW_BEGIN 0x2
#define R_CW_NEXT 0x3
#define R_CCW_BEGIN 0x4
#define R_CCW_FINAL 0x5
#define R_CCW_NEXT 0x6


const unsigned char ttable[7][4] = {
  // R_START
  {R_START,    R_CW_BEGIN,  R_CCW_BEGIN, R_START},
  // R_CW_FINAL
  {R_CW_NEXT,  R_START,     R_CW_FINAL,  R_START | DIR_CW},
  // R_CW_BEGIN
  {R_CW_NEXT,  R_CW_BEGIN,  R_START,     R_START},
  // R_CW_NEXT
  {R_CW_NEXT,  R_CW_BEGIN,  R_CW_FINAL,  R_START},
  // R_CCW_BEGIN
  {R_CCW_NEXT, R_START,     R_CCW_BEGIN, R_START},
  // R_CCW_FINAL
  {R_CCW_NEXT, R_CCW_FINAL, R_START,     R_START | DIR_CCW},
  // R_CCW_NEXT
  {R_CCW_NEXT, R_CCW_FINAL, R_CCW_BEGIN, R_START},
};
#endif


//Button to ProMicro inputs from the matrix
byte rowPins[NUMROWS] = {21,20,19,10}; 
byte colPins[NUMCOLS] = {16,14,15,18}; 


Keypad buttbx = Keypad( makeKeymap(buttons), rowPins, colPins, NUMROWS, NUMCOLS); 



//The list below enables or disables all the axis on the PC game controller.
//These axes  are X,Y,Z, Rx, Ry, Rz, Rudder, Throttle, Accl, Braking, Steering
Joystick_ Joystick(JOYSTICK_DEFAULT_REPORT_ID, 
  JOYSTICK_TYPE_JOYSTICK, 32, 0,
  false, false, false, false, false, false,
  false, false, false, false, false);


void setup() {
  Joystick.begin();
  rotary_init();}


void loop() { 
  CheckAllEncoders();
  CheckAllButtons();
}


void CheckAllButtons(void) {
      if (buttbx.getKeys())
    {
       for (int i=0; i<LIST_MAX; i++)   
        {
           if ( buttbx.key[i].stateChanged )   
            {
            switch (buttbx.key[i].kstate) {  
                    case PRESSED:
                    case HOLD:
                              Joystick.setButton(buttbx.key[i].kchar, 1);
                              break;
                    case RELEASED:
                    case IDLE:
                              Joystick.setButton(buttbx.key[i].kchar, 0);
                              break;
            }
           }   
         }
     }
}



void rotary_init() {
  for (int i=0;i<NUMROTARIES;i++) {
    pinMode(rotaries[i].pin1, INPUT);
    pinMode(rotaries[i].pin2, INPUT);
    #ifdef ENABLE_PULLUPS
      digitalWrite(rotaries[i].pin1, HIGH);
      digitalWrite(rotaries[i].pin2, HIGH);
    #endif
  }
}



unsigned char rotary_process(int _i) {
   unsigned char pinstate = (digitalRead(rotaries[_i].pin2) << 1) | digitalRead(rotaries[_i].pin1);
  rotaries[_i].state = ttable[rotaries[_i].state & 0xf][pinstate];
  return (rotaries[_i].state & 0x30);
}


void CheckAllEncoders(void) {
  for (int i=0;i<NUMROTARIES;i++) {
    unsigned char result = rotary_process(i);
    if (result == DIR_CCW) {
      Joystick.setButton(rotaries[i].ccwchar, 1); delay(100); Joystick.setButton(rotaries[i].ccwchar, 0);
    };
    if (result == DIR_CW) {
      Joystick.setButton(rotaries[i].cwchar, 1); delay(100); Joystick.setButton(rotaries[i].cwchar, 0);
    };
  }
}
byte rowPins[NUMROWS] = {21,20,19,10}; 
byte colPins[NUMCOLS] = {16,14,15,18};

Verify your pin groups.

This may help.

#include <Joystick.h>

const byte ROWS = 4;
const byte COLS = 3;
char Keys[ROWS][COLS] = {
  {'1','2','3'},
  {'4','5','6'},
  {'7','8','9'},
  {'A','B','C'}
};
byte rowPins[ROWS] = {8,7,6,5};
byte colPins[COLS] = {4,3,2};
char customKey = 0;

Joystick_ Buttons(JOYSTICK_DEFAULT_REPORT_ID, 
  JOYSTICK_TYPE_JOYSTICK, 13, 0,false,false,false,false,false,false,false,false,false,false,false);

void setup(){
  Buttons.begin();
  pinMode(4,INPUT_PULLUP);
  pinMode(2,INPUT_PULLUP);
  pinMode(6,INPUT_PULLUP);
  pinMode(3,OUTPUT);
  pinMode(8,OUTPUT);
  pinMode(5,OUTPUT);
  pinMode(7,OUTPUT);
}



void loop(){
  digitalWrite(3,HIGH);
  digitalWrite(8,HIGH);
  digitalWrite(7,HIGH);
  digitalWrite(5,LOW);
  if(digitalRead(4) == LOW){
    Buttons.pressButton(9);
    delay(100);
    Buttons.releaseButton(9);   
  }
  if(digitalRead(2) == LOW){
    Buttons.pressButton(10);
    delay(100);
    Buttons.releaseButton(10);   
  }
  if(digitalRead(6) == LOW){
    Buttons.pressButton(11);
    delay(100);
    Buttons.releaseButton(11);   
  }    
  digitalWrite(3,HIGH);
  digitalWrite(8,HIGH);
  digitalWrite(7,LOW);
  digitalWrite(5,HIGH);
  if(digitalRead(4) == LOW){
    Buttons.pressButton(6);
    delay(100);
    Buttons.releaseButton(6);   
  }
  if(digitalRead(2) == LOW){
    Buttons.pressButton(7);
    delay(100);
    Buttons.releaseButton(7);   
  }
  if(digitalRead(6) == LOW){
    Buttons.pressButton(8);
    delay(100);
    Buttons.releaseButton(8);   
  } 
  digitalWrite(3,HIGH);
  digitalWrite(8,LOW);
  digitalWrite(7,HIGH);
  digitalWrite(5,HIGH);
  if(digitalRead(4) == LOW){
    Buttons.pressButton(3);
    delay(100);
    Buttons.releaseButton(3);   
  }
  if(digitalRead(2) == LOW){
    Buttons.pressButton(4);
    delay(100);
    Buttons.releaseButton(4);   
  }
  if(digitalRead(6) == LOW){
    Buttons.pressButton(5);
    delay(100);
    Buttons.releaseButton(5);   
  }
  digitalWrite(3,LOW);
  digitalWrite(8,HIGH);
  digitalWrite(7,HIGH);
  digitalWrite(5,HIGH);
  if(digitalRead(4) == LOW){
    Buttons.pressButton(0);
    delay(100);
    Buttons.releaseButton(0);   
  }
  if(digitalRead(2) == LOW){
    Buttons.pressButton(1);
    delay(100);
    Buttons.releaseButton(1);   
  }
  if(digitalRead(6) == LOW){
    Buttons.pressButton(2);
    delay(100);
    Buttons.releaseButton(2);   
  }
}

This topic was automatically closed 180 days after the last reply. New replies are no longer allowed.