mirror of
				https://github.com/mfulz/qmk_firmware.git
				synced 2025-10-31 21:32:31 +01:00 
			
		
		
		
	Reorder logic within common_features.mk (#8517)
* Reorder logic within common_features.mk * Revert haptic logic * Add back path to make tests happy * Update common_features.mk Co-Authored-By: Ryan <fauxpark@gmail.com> Co-authored-by: Ryan <fauxpark@gmail.com>
This commit is contained in:
		
							parent
							
								
									147bc6ec43
								
							
						
					
					
						commit
						5f9f62fb8c
					
				| @ -13,26 +13,24 @@ | |||||||
| # You should have received a copy of the GNU General Public License
 | # You should have received a copy of the GNU General Public License
 | ||||||
| # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 | # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 | ||||||
| 
 | 
 | ||||||
| SERIAL_DIR := $(QUANTUM_DIR)/serial_link |  | ||||||
| SERIAL_PATH := $(QUANTUM_PATH)/serial_link | SERIAL_PATH := $(QUANTUM_PATH)/serial_link | ||||||
| SERIAL_SRC := $(wildcard $(SERIAL_PATH)/protocol/*.c) | 
 | ||||||
| SERIAL_SRC += $(wildcard $(SERIAL_PATH)/system/*.c) | QUANTUM_SRC += \
 | ||||||
| SERIAL_DEFS += -DSERIAL_LINK_ENABLE |     $(QUANTUM_DIR)/quantum.c \
 | ||||||
| COMMON_VPATH += $(SERIAL_PATH) |     $(QUANTUM_DIR)/keymap_common.c \
 | ||||||
|  |     $(QUANTUM_DIR)/keycode_config.c | ||||||
| 
 | 
 | ||||||
| ifeq ($(strip $(API_SYSEX_ENABLE)), yes) | ifeq ($(strip $(API_SYSEX_ENABLE)), yes) | ||||||
|     OPT_DEFS += -DAPI_SYSEX_ENABLE |     OPT_DEFS += -DAPI_SYSEX_ENABLE | ||||||
|     SRC += $(QUANTUM_DIR)/api/api_sysex.c |  | ||||||
|     OPT_DEFS += -DAPI_ENABLE |     OPT_DEFS += -DAPI_ENABLE | ||||||
|     SRC += $(QUANTUM_DIR)/api.c |  | ||||||
|     MIDI_ENABLE=yes |     MIDI_ENABLE=yes | ||||||
|  |     SRC += $(QUANTUM_DIR)/api/api_sysex.c | ||||||
|  |     SRC += $(QUANTUM_DIR)/api.c | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
| MUSIC_ENABLE := 0 |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(AUDIO_ENABLE)), yes) | ifeq ($(strip $(AUDIO_ENABLE)), yes) | ||||||
|     OPT_DEFS += -DAUDIO_ENABLE |     OPT_DEFS += -DAUDIO_ENABLE | ||||||
|     MUSIC_ENABLE := 1 |     MUSIC_ENABLE = yes | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_audio.c |     SRC += $(QUANTUM_DIR)/process_keycode/process_audio.c | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_clicky.c |     SRC += $(QUANTUM_DIR)/process_keycode/process_clicky.c | ||||||
|     SRC += $(QUANTUM_DIR)/audio/audio_$(PLATFORM_KEY).c |     SRC += $(QUANTUM_DIR)/audio/audio_$(PLATFORM_KEY).c | ||||||
| @ -42,19 +40,15 @@ endif | |||||||
| 
 | 
 | ||||||
| ifeq ($(strip $(MIDI_ENABLE)), yes) | ifeq ($(strip $(MIDI_ENABLE)), yes) | ||||||
|     OPT_DEFS += -DMIDI_ENABLE |     OPT_DEFS += -DMIDI_ENABLE | ||||||
|     MUSIC_ENABLE := 1 |     MUSIC_ENABLE = yes | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_midi.c |     SRC += $(QUANTUM_DIR)/process_keycode/process_midi.c | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
| ifeq ($(MUSIC_ENABLE), 1) | MUSIC_ENABLE ?= no | ||||||
|  | ifeq ($(MUSIC_ENABLE), yes) | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_music.c |     SRC += $(QUANTUM_DIR)/process_keycode/process_music.c | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
| ifeq ($(strip $(COMBO_ENABLE)), yes) |  | ||||||
|     OPT_DEFS += -DCOMBO_ENABLE |  | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_combo.c |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(STENO_ENABLE)), yes) | ifeq ($(strip $(STENO_ENABLE)), yes) | ||||||
|     OPT_DEFS += -DSTENO_ENABLE |     OPT_DEFS += -DSTENO_ENABLE | ||||||
|     VIRTSER_ENABLE ?= yes |     VIRTSER_ENABLE ?= yes | ||||||
| @ -76,28 +70,6 @@ ifeq ($(strip $(POINTING_DEVICE_ENABLE)), yes) | |||||||
|     SRC += $(QUANTUM_DIR)/pointing_device.c |     SRC += $(QUANTUM_DIR)/pointing_device.c | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
| ifeq ($(strip $(UCIS_ENABLE)), yes) |  | ||||||
|     OPT_DEFS += -DUCIS_ENABLE |  | ||||||
|     UNICODE_COMMON := yes |  | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_ucis.c |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(UNICODEMAP_ENABLE)), yes) |  | ||||||
|     OPT_DEFS += -DUNICODEMAP_ENABLE |  | ||||||
|     UNICODE_COMMON := yes |  | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_unicodemap.c |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(UNICODE_ENABLE)), yes) |  | ||||||
|     OPT_DEFS += -DUNICODE_ENABLE |  | ||||||
|     UNICODE_COMMON := yes |  | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_unicode.c |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(UNICODE_COMMON)), yes) |  | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_unicode_common.c |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| VALID_EEPROM_DRIVER_TYPES := vendor custom transient i2c | VALID_EEPROM_DRIVER_TYPES := vendor custom transient i2c | ||||||
| EEPROM_DRIVER ?= vendor | EEPROM_DRIVER ?= vendor | ||||||
| ifeq ($(filter $(EEPROM_DRIVER),$(VALID_EEPROM_DRIVER_TYPES)),) | ifeq ($(filter $(EEPROM_DRIVER),$(VALID_EEPROM_DRIVER_TYPES)),) | ||||||
| @ -245,31 +217,18 @@ ifeq ($(strip $(RGB_KEYCODES_ENABLE)), yes) | |||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_rgb.c |     SRC += $(QUANTUM_DIR)/process_keycode/process_rgb.c | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
| ifeq ($(strip $(TAP_DANCE_ENABLE)), yes) |  | ||||||
|     OPT_DEFS += -DTAP_DANCE_ENABLE |  | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_tap_dance.c |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(KEY_LOCK_ENABLE)), yes) |  | ||||||
|     OPT_DEFS += -DKEY_LOCK_ENABLE |  | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_key_lock.c |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(PRINTING_ENABLE)), yes) | ifeq ($(strip $(PRINTING_ENABLE)), yes) | ||||||
|     OPT_DEFS += -DPRINTING_ENABLE |     OPT_DEFS += -DPRINTING_ENABLE | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_printer.c |     SRC += $(QUANTUM_DIR)/process_keycode/process_printer.c | ||||||
|     SRC += $(TMK_DIR)/protocol/serial_uart.c |     SRC += $(TMK_DIR)/protocol/serial_uart.c | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
| ifeq ($(strip $(AUTO_SHIFT_ENABLE)), yes) |  | ||||||
|     OPT_DEFS += -DAUTO_SHIFT_ENABLE |  | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_auto_shift.c |  | ||||||
|     ifeq ($(strip $(AUTO_SHIFT_MODIFIERS)), yes) |  | ||||||
|         OPT_DEFS += -DAUTO_SHIFT_MODIFIERS |  | ||||||
|     endif |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(SERIAL_LINK_ENABLE)), yes) | ifeq ($(strip $(SERIAL_LINK_ENABLE)), yes) | ||||||
|  |     SERIAL_SRC := $(wildcard $(SERIAL_PATH)/protocol/*.c) | ||||||
|  |     SERIAL_SRC += $(wildcard $(SERIAL_PATH)/system/*.c) | ||||||
|  |     SERIAL_DEFS += -DSERIAL_LINK_ENABLE | ||||||
|  |     COMMON_VPATH += $(SERIAL_PATH) | ||||||
|  | 
 | ||||||
|     SRC += $(patsubst $(QUANTUM_PATH)/%,%,$(SERIAL_SRC)) |     SRC += $(patsubst $(QUANTUM_PATH)/%,%,$(SERIAL_SRC)) | ||||||
|     OPT_DEFS += $(SERIAL_DEFS) |     OPT_DEFS += $(SERIAL_DEFS) | ||||||
|     VAPTH += $(SERIAL_PATH) |     VAPTH += $(SERIAL_PATH) | ||||||
| @ -368,29 +327,6 @@ ifeq ($(strip $(ENCODER_ENABLE)), yes) | |||||||
|     OPT_DEFS += -DENCODER_ENABLE |     OPT_DEFS += -DENCODER_ENABLE | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
| HAPTIC_ENABLE ?= no |  | ||||||
| ifneq ($(strip $(HAPTIC_ENABLE)),no) |  | ||||||
| 	COMMON_VPATH += $(DRIVER_PATH)/haptic |  | ||||||
| 	SRC += haptic.c |  | ||||||
| 	OPT_DEFS += -DHAPTIC_ENABLE |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifneq ($(filter DRV2605L, $(HAPTIC_ENABLE)), ) |  | ||||||
|     SRC += DRV2605L.c |  | ||||||
|     QUANTUM_LIB_SRC += i2c_master.c |  | ||||||
|     OPT_DEFS += -DDRV2605L |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifneq ($(filter SOLENOID, $(HAPTIC_ENABLE)), ) |  | ||||||
|     SRC += solenoid.c |  | ||||||
|     OPT_DEFS += -DSOLENOID_ENABLE |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(HD44780_ENABLE)), yes) |  | ||||||
|     SRC += drivers/avr/hd44780.c |  | ||||||
|     OPT_DEFS += -DHD44780_ENABLE |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(VELOCIKEY_ENABLE)), yes) | ifeq ($(strip $(VELOCIKEY_ENABLE)), yes) | ||||||
|     OPT_DEFS += -DVELOCIKEY_ENABLE |     OPT_DEFS += -DVELOCIKEY_ENABLE | ||||||
|     SRC += $(QUANTUM_DIR)/velocikey.c |     SRC += $(QUANTUM_DIR)/velocikey.c | ||||||
| @ -409,26 +345,11 @@ ifeq ($(strip $(DYNAMIC_KEYMAP_ENABLE)), yes) | |||||||
|     SRC += $(QUANTUM_DIR)/dynamic_keymap.c |     SRC += $(QUANTUM_DIR)/dynamic_keymap.c | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
| ifeq ($(strip $(LEADER_ENABLE)), yes) |  | ||||||
|   SRC += $(QUANTUM_DIR)/process_keycode/process_leader.c |  | ||||||
|   OPT_DEFS += -DLEADER_ENABLE |  | ||||||
| endif |  | ||||||
| 
 |  | ||||||
| 
 |  | ||||||
| ifeq ($(strip $(DIP_SWITCH_ENABLE)), yes) | ifeq ($(strip $(DIP_SWITCH_ENABLE)), yes) | ||||||
|   SRC += $(QUANTUM_DIR)/dip_switch.c |  | ||||||
|     OPT_DEFS += -DDIP_SWITCH_ENABLE |     OPT_DEFS += -DDIP_SWITCH_ENABLE | ||||||
|  |     SRC += $(QUANTUM_DIR)/dip_switch.c | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
| include $(DRIVER_PATH)/qwiic/qwiic.mk |  | ||||||
| 
 |  | ||||||
| QUANTUM_SRC:= \
 |  | ||||||
|     $(QUANTUM_DIR)/quantum.c \
 |  | ||||||
|     $(QUANTUM_DIR)/keymap_common.c \
 |  | ||||||
|     $(QUANTUM_DIR)/keycode_config.c |  | ||||||
| 
 |  | ||||||
| 
 |  | ||||||
| 
 |  | ||||||
| VALID_CUSTOM_MATRIX_TYPES:= yes lite no | VALID_CUSTOM_MATRIX_TYPES:= yes lite no | ||||||
| 
 | 
 | ||||||
| CUSTOM_MATRIX ?= no | CUSTOM_MATRIX ?= no | ||||||
| @ -486,6 +407,29 @@ ifeq ($(strip $(SPLIT_KEYBOARD)), yes) | |||||||
|     COMMON_VPATH += $(QUANTUM_PATH)/split_common |     COMMON_VPATH += $(QUANTUM_PATH)/split_common | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
|  | HAPTIC_ENABLE ?= no | ||||||
|  | ifneq ($(strip $(HAPTIC_ENABLE)),no) | ||||||
|  |     COMMON_VPATH += $(DRIVER_PATH)/haptic | ||||||
|  |     SRC += haptic.c | ||||||
|  |     OPT_DEFS += -DHAPTIC_ENABLE | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifneq ($(filter DRV2605L, $(HAPTIC_ENABLE)), ) | ||||||
|  |     SRC += DRV2605L.c | ||||||
|  |     QUANTUM_LIB_SRC += i2c_master.c | ||||||
|  |     OPT_DEFS += -DDRV2605L | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifneq ($(filter SOLENOID, $(HAPTIC_ENABLE)), ) | ||||||
|  |     SRC += solenoid.c | ||||||
|  |     OPT_DEFS += -DSOLENOID_ENABLE | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(HD44780_ENABLE)), yes) | ||||||
|  |     SRC += drivers/avr/hd44780.c | ||||||
|  |     OPT_DEFS += -DHD44780_ENABLE | ||||||
|  | endif | ||||||
|  | 
 | ||||||
| ifeq ($(strip $(OLED_DRIVER_ENABLE)), yes) | ifeq ($(strip $(OLED_DRIVER_ENABLE)), yes) | ||||||
|     OPT_DEFS += -DOLED_DRIVER_ENABLE |     OPT_DEFS += -DOLED_DRIVER_ENABLE | ||||||
|     COMMON_VPATH += $(DRIVER_PATH)/oled |     COMMON_VPATH += $(DRIVER_PATH)/oled | ||||||
| @ -493,6 +437,30 @@ ifeq ($(strip $(OLED_DRIVER_ENABLE)), yes) | |||||||
|     SRC += oled_driver.c |     SRC += oled_driver.c | ||||||
| endif | endif | ||||||
| 
 | 
 | ||||||
|  | include $(DRIVER_PATH)/qwiic/qwiic.mk | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(UCIS_ENABLE)), yes) | ||||||
|  |     OPT_DEFS += -DUCIS_ENABLE | ||||||
|  |     UNICODE_COMMON := yes | ||||||
|  |     SRC += $(QUANTUM_DIR)/process_keycode/process_ucis.c | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(UNICODEMAP_ENABLE)), yes) | ||||||
|  |     OPT_DEFS += -DUNICODEMAP_ENABLE | ||||||
|  |     UNICODE_COMMON := yes | ||||||
|  |     SRC += $(QUANTUM_DIR)/process_keycode/process_unicodemap.c | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(UNICODE_ENABLE)), yes) | ||||||
|  |     OPT_DEFS += -DUNICODE_ENABLE | ||||||
|  |     UNICODE_COMMON := yes | ||||||
|  |     SRC += $(QUANTUM_DIR)/process_keycode/process_unicode.c | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(UNICODE_COMMON)), yes) | ||||||
|  |     SRC += $(QUANTUM_DIR)/process_keycode/process_unicode_common.c | ||||||
|  | endif | ||||||
|  | 
 | ||||||
| SPACE_CADET_ENABLE ?= yes | SPACE_CADET_ENABLE ?= yes | ||||||
| ifeq ($(strip $(SPACE_CADET_ENABLE)), yes) | ifeq ($(strip $(SPACE_CADET_ENABLE)), yes) | ||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_space_cadet.c |     SRC += $(QUANTUM_DIR)/process_keycode/process_space_cadet.c | ||||||
| @ -515,3 +483,31 @@ ifeq ($(strip $(DYNAMIC_MACRO_ENABLE)), yes) | |||||||
|     SRC += $(QUANTUM_DIR)/process_keycode/process_dynamic_macro.c |     SRC += $(QUANTUM_DIR)/process_keycode/process_dynamic_macro.c | ||||||
|     OPT_DEFS += -DDYNAMIC_MACRO_ENABLE |     OPT_DEFS += -DDYNAMIC_MACRO_ENABLE | ||||||
| endif | endif | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(COMBO_ENABLE)), yes) | ||||||
|  |     SRC += $(QUANTUM_DIR)/process_keycode/process_combo.c | ||||||
|  |     OPT_DEFS += -DCOMBO_ENABLE | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(TAP_DANCE_ENABLE)), yes) | ||||||
|  |     SRC += $(QUANTUM_DIR)/process_keycode/process_tap_dance.c | ||||||
|  |     OPT_DEFS += -DTAP_DANCE_ENABLE | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(KEY_LOCK_ENABLE)), yes) | ||||||
|  |     SRC += $(QUANTUM_DIR)/process_keycode/process_key_lock.c | ||||||
|  |     OPT_DEFS += -DKEY_LOCK_ENABLE | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(LEADER_ENABLE)), yes) | ||||||
|  |     SRC += $(QUANTUM_DIR)/process_keycode/process_leader.c | ||||||
|  |     OPT_DEFS += -DLEADER_ENABLE | ||||||
|  | endif | ||||||
|  | 
 | ||||||
|  | ifeq ($(strip $(AUTO_SHIFT_ENABLE)), yes) | ||||||
|  |     SRC += $(QUANTUM_DIR)/process_keycode/process_auto_shift.c | ||||||
|  |     OPT_DEFS += -DAUTO_SHIFT_ENABLE | ||||||
|  |     ifeq ($(strip $(AUTO_SHIFT_MODIFIERS)), yes) | ||||||
|  |         OPT_DEFS += -DAUTO_SHIFT_MODIFIERS | ||||||
|  |     endif | ||||||
|  | endif | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user
	 Joel Challis
						Joel Challis