mmccoyd / hillside

Family of split ergonomic keyboards with three rows of five or six keys, aggressive column stagger, generous thumb arc and optional bottom utility keys
MIT License
309 stars 41 forks source link

QMK: compiling directly with 'qmk compile' gives errors #2

Closed jurgen-kluft closed 2 years ago

jurgen-kluft commented 2 years ago

qmk_compile_hillside_errors.log

These errors can be fixed by adding the following into config.h, however this does have qmk spew out warnings.

define VENDOR_ID 0xFEED

define PRODUCT_ID 0x67C0

define DEVICE_VER 0x0001

define DIODE_DIRECTION COL2ROW

define RGBLED_NUM 5

define RGB_DI_PIN D3

define SOFT_SERIAL_PIN D2

Furthermore if the following is not defined in config.h then no key is registered:

define MATRIX_ROW_PINS \

{ D7, E6, B4, B5 }

define MATRIX_COL_PINS \

{ F6, F7, B1, B3, B2, B6 }

By adding all of the above in 'config.h' my own firmware/keymap works, can be compiled and flashed and registers all keys.

mmccoyd commented 2 years ago

Jurgen, Thanks for the report.

It looks as if you are running a much older version of QMK. Your error log lists QMK Firmware 0.12.50, while 0.16.0 started in February. Recent changes have more more config into info.json, which is likely not read correctly by prior versions.

jurgen-kluft commented 2 years ago

Hi @mmccoyd,

Yes you are right, after upgrading QMK (correctly this time) it does indeed not complain anymore.