Kmini refactor and Configurator support (#3388)

* default keymap refactor: QMK_KEYBOARD_H include; readability

* Configurator support

* info.json was missing a comma

* Added matrix functions to matrix.c per @drashna

keyboards/kmini/keymaps/default/keymap.c

@@ -13,10 +13,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
-#include "kmini.h"
-
-// Helpful defines
-#define _____ KC_TRNS
+#include QMK_KEYBOARD_H
 
 // Each layer gets a name for readability, which is then used in the keymap matrix below.
 // The underscores don't mean anything - you can have a layer called STUFF or any other name.
@@ -26,20 +23,20 @@
 #define _FN 1
 
 const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
-    [_MA] = LAYOUT(
-		KC_F1,	KC_F2,		KC_ESC,		KC_1,		KC_2,	KC_3,	KC_4,	KC_5,	KC_6,	KC_7,	KC_8,		KC_9,		KC_0,		KC_MINS,	KC_EQL,		KC_BSPC,	KC_INS,		\
-		KC_F3,	KC_F4,		KC_TAB,		KC_Q,		KC_W,	KC_E,	KC_R,	KC_T,	KC_Y,	KC_U,	KC_I,		KC_O,		KC_P,		KC_LBRC,	KC_RBRC,	KC_BSLS,	KC_DEL,		\
-		KC_F5,	KC_F6,		KC_CAPS,	KC_A,		KC_S,	KC_D,	KC_F,	KC_G,	KC_H,	KC_J,	KC_K,		KC_L,		KC_SCLN,	KC_QUOT,				KC_ENT,		KC_PGUP,	\
-		KC_F7,	KC_F8,		KC_LSFT,	KC_Z,		KC_X,	KC_C,	KC_V,	KC_B,	KC_N,	KC_M,	KC_COMM,	KC_DOT,		KC_SLSH,				KC_RSFT,	KC_UP,		KC_PGDN,	\
-		KC_F9,	KC_F10,		KC_LCTL,	KC_LGUI,	KC_LALT,				KC_SPC,											MO(_FN),				KC_LEFT,	KC_DOWN,	KC_RGHT		\
-    ),
-    [_FN] = LAYOUT(
-		_____,	_____,		_____,		KC_F1,		KC_F2,	KC_F3,	KC_F4,	KC_F5,	KC_F6,	KC_F7,	KC_F8,		KC_F9,		KC_F10,		KC_F11,		KC_F12,		RESET,		_____,		\
-		_____,	_____,		_____,		_____,		_____,	_____,	_____,	_____,	_____,	_____,	_____,		_____,		_____,		_____,		_____,		_____,		_____,		\
-		_____,	_____,		_____,		_____,		_____,	_____,	_____,	_____,	_____,	_____,	_____,		_____,		_____,		_____,					_____,		_____,		\
-		_____,	_____,		_____,		_____,		_____,	_____,	_____,	_____,	_____,	_____,	_____,		_____,		_____,					_____,		_____,		_____,		\
-		_____,	_____,		_____,		_____,		_____,					_____,											_____,					_____,		_____,		_____		\
-    ),
+  [_MA] = LAYOUT(
+    KC_F1,   KC_F2,   KC_ESC,  KC_1,    KC_2,    KC_3,    KC_4,    KC_5,    KC_6,    KC_7,    KC_8,    KC_9,    KC_0,    KC_MINS, KC_EQL,  KC_BSPC, KC_INS,  \
+    KC_F3,   KC_F4,   KC_TAB,  KC_Q,    KC_W,    KC_E,    KC_R,    KC_T,    KC_Y,    KC_U,    KC_I,    KC_O,    KC_P,    KC_LBRC, KC_RBRC, KC_BSLS, KC_DEL,  \
+    KC_F5,   KC_F6,   KC_CAPS, KC_A,    KC_S,    KC_D,    KC_F,    KC_G,    KC_H,    KC_J,    KC_K,    KC_L,    KC_SCLN, KC_QUOT,          KC_ENT,  KC_PGUP, \
+    KC_F7,   KC_F8,   KC_LSFT, KC_Z,    KC_X,    KC_C,    KC_V,    KC_B,    KC_N,    KC_M,    KC_COMM, KC_DOT,  KC_SLSH,          KC_RSFT, KC_UP,   KC_PGDN, \
+    KC_F9,   KC_F10,  KC_LCTL, KC_LGUI, KC_LALT,                            KC_SPC,                             MO(_FN),          KC_LEFT, KC_DOWN, KC_RGHT  \
+  ),
+  [_FN] = LAYOUT(
+    _______, _______, _______, KC_F1,   KC_F2,   KC_F3,   KC_F4,   KC_F5,   KC_F6,   KC_F7,   KC_F8,   KC_F9,   KC_F10,  KC_F11,  KC_F12,  RESET,   _______, \
+    _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, \
+    _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______,          _______, _______, \
+    _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______,          _______, _______, _______, \
+    _______, _______, _______, _______, _______,                            _______,                            _______,          _______, _______, _______  \
+  ),
 };
 
 void led_set_user(uint8_t usb_led) {

keyboards/kmini/matrix.c

@@ -48,6 +48,22 @@ static bool read_rows_on_col(matrix_row_t current_matrix[], uint8_t current_col)
 static void unselect_cols(void);
 static void select_col(uint8_t col);
 
+__attribute__ ((weak))
+void matrix_init_user(void) {}
+
+__attribute__ ((weak))
+void matrix_scan_user(void) {}
+
+__attribute__ ((weak))
+void matrix_init_kb(void) {
+  matrix_init_user();
+}
+
+__attribute__ ((weak))
+void matrix_scan_kb(void) {
+  matrix_scan_user();
+}
+
 inline
 uint8_t matrix_rows(void) {
     return MATRIX_ROWS;
@@ -219,16 +235,16 @@ static void init_rows(void) {
 static void unselect_cols(void) {
     DDRB  |= (1<<5) | (1<<6) | (1<<0) | (1<<7) | (1<<4); // OUT
     PORTB &= ~((1<<5) | (1<<6) | (1<<0) |  (1<<7) | (1<<4)); // LO
-	
+
     DDRD  |= (1<<4) | (1<<6) | (1<<7); // OUT
     PORTD &= ~((1<<4) | (1<<6) | (1<<7)); // LO
-	
+
     DDRE  |= (1<<6); // OUT
     PORTE &= ~((1<<6)); // LO
-	
+
     DDRF  |= (1<<0) | (1<<1); // OUT
     PORTF &= ~((1<<0) | (1<<1)); // LO
-	
+
     DDRC  |= (1<<7) | (1<<6); // OUT
     PORTC &= ~((1<<7) | (1<<6)); // LO
 }