make: Specify source files to include
Replace use of wildcard with list of source files to include. The `-y` suffix is added, but has no significance since conditional compilation has not been added yet. Signed-off-by: Tim Crawford <tcrawford@system76.com>
This commit is contained in:
		
				
					committed by
					
						 Tim Crawford
						Tim Crawford
					
				
			
			
				
	
			
			
			
						parent
						
							ce66685c70
						
					
				
				
					commit
					ac9631f948
				
			
							
								
								
									
										10
									
								
								Makefile
									
									
									
									
									
								
							
							
						
						
									
										10
									
								
								Makefile
									
									
									
									
									
								
							| @@ -25,33 +25,35 @@ all: $(BUILD)/ec.rom | ||||
|  | ||||
| # Include common source | ||||
| COMMON_DIR=src/common | ||||
| SRC=$(wildcard $(COMMON_DIR)/*.c) | ||||
| INCLUDE=$(wildcard $(COMMON_DIR)/include/common/*.h) $(COMMON_DIR)/common.mk | ||||
| CFLAGS=-I$(COMMON_DIR)/include -D__FIRMWARE_VERSION__=$(VERSION) | ||||
| include $(COMMON_DIR)/common.mk | ||||
| SRC += $(foreach src, $(common-y), $(COMMON_DIR)/$(src)) | ||||
|  | ||||
| # Include the board's source | ||||
| BOARD_DIR=src/board/$(BOARD) | ||||
| SRC+=$(wildcard $(BOARD_DIR)/*.c) | ||||
| INCLUDE+=$(wildcard $(BOARD_DIR)/include/board/*.h) $(BOARD_DIR)/board.mk | ||||
| CFLAGS+=-I$(BOARD_DIR)/include -D__BOARD__=$(BOARD) | ||||
| include $(BOARD_DIR)/board.mk | ||||
| SRC += $(foreach src, $(board-y), $(BOARD_DIR)/$(src)) | ||||
| SRC += $(foreach src, $(board-common-y), $(SYSTEM76_COMMON_DIR)/$(src)) | ||||
| SRC += $(foreach src, $(keyboard-y), $(KEYBOARD_DIR)/$(src)) | ||||
|  | ||||
| # The board will define the embedded controller | ||||
| # Include the embedded controller's source | ||||
| EC_DIR=src/ec/$(EC) | ||||
| SRC+=$(wildcard $(EC_DIR)/*.c) | ||||
| INCLUDE+=$(wildcard $(EC_DIR)/include/ec/*.h) $(EC_DIR)/ec.mk | ||||
| CFLAGS+=-I$(EC_DIR)/include -D__EC__=$(EC_VARIANT) -D$(EC_VARIANT) | ||||
| include $(EC_DIR)/ec.mk | ||||
| SRC += $(foreach src, $(ec-y), $(EC_DIR)/$(src)) | ||||
|  | ||||
| # The EC will define the architecture | ||||
| # Include the architecture's source | ||||
| ARCH_DIR=src/arch/$(ARCH) | ||||
| SRC+=$(wildcard $(ARCH_DIR)/*.c) | ||||
| INCLUDE+=$(wildcard $(ARCH_DIR)/include/arch/*.h) $(ARCH_DIR)/arch.mk | ||||
| CFLAGS+=-I$(ARCH_DIR)/include -D__ARCH__=$(ARCH) | ||||
| include $(ARCH_DIR)/arch.mk | ||||
| SRC += $(foreach src, $(arch-y), $(ARCH_DIR)/$(src)) | ||||
|  | ||||
| include $(ARCH_DIR)/toolchain.mk | ||||
|  | ||||
|   | ||||
| @@ -0,0 +1,5 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| arch-y += arch.c | ||||
| arch-y += delay.c | ||||
| arch-y += time.c | ||||
|   | ||||
| @@ -0,0 +1,6 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| arch-y += gpio.c | ||||
| arch-y += i2c.c | ||||
| arch-y += i2c_slave.c | ||||
| arch-y += uart.c | ||||
|   | ||||
| @@ -1,5 +1,10 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += battery.c | ||||
| board-y += i2c.c | ||||
| board-y += main.c | ||||
| board-y += parallel.c | ||||
|  | ||||
| EC=atmega | ||||
| EC_VARIANT=atmega2560 | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,7 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += main.c | ||||
|  | ||||
| EC=atmega | ||||
| EC_VARIANT=atmega32u4 | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,10 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += battery.c | ||||
| board-y += i2c.c | ||||
| board-y += main.c | ||||
| board-y += parallel.c | ||||
|  | ||||
| EC=atmega | ||||
| EC_VARIANT=atmega328p | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it8587e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,31 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-common-y += acpi.c | ||||
| board-common-y += battery.c | ||||
| board-common-y += config.c | ||||
| board-common-y += dgpu.c | ||||
| board-common-y += ecpm.c | ||||
| board-common-y += espi.c | ||||
| board-common-y += fan.c | ||||
| board-common-y += gctrl.c | ||||
| board-common-y += kbc.c | ||||
| board-common-y += kbled.c | ||||
| board-common-y += kbscan.c | ||||
| board-common-y += keymap.c | ||||
| board-common-y += lid.c | ||||
| board-common-y += main.c | ||||
| board-common-y += parallel.c | ||||
| board-common-y += peci.c | ||||
| board-common-y += pmc.c | ||||
| board-common-y += pnp.c | ||||
| board-common-y += power.c | ||||
| board-common-y += ps2.c | ||||
| board-common-y += pwm.c | ||||
| board-common-y += scratch.c | ||||
| board-common-y += smbus.c | ||||
| board-common-y += smfi.c | ||||
| board-common-y += stdio.c | ||||
|  | ||||
| # Set log level | ||||
| # 0 - NONE | ||||
| # 1 - ERROR | ||||
| @@ -20,7 +46,6 @@ PROGRAMMER=$(wildcard /dev/serial/by-id/usb-Arduino*) | ||||
|  | ||||
| # Include system76 common source | ||||
| SYSTEM76_COMMON_DIR=src/board/system76/common | ||||
| SRC+=$(wildcard $(SYSTEM76_COMMON_DIR)/*.c) | ||||
| INCLUDE+=$(wildcard $(SYSTEM76_COMMON_DIR)/include/board/*.h) $(SYSTEM76_COMMON_DIR)/common.mk | ||||
| CFLAGS+=-I$(SYSTEM76_COMMON_DIR)/include | ||||
|  | ||||
| @@ -34,7 +59,7 @@ CFLAGS+=\ | ||||
|  | ||||
| # Add charger | ||||
| CHARGER?=bq24780s | ||||
| SRC+=$(SYSTEM76_COMMON_DIR)/charger/$(CHARGER).c | ||||
| board-common-y += charger/$(CHARGER).c | ||||
|  | ||||
| # Add keyboard | ||||
| ifndef KEYBOARD | ||||
| @@ -45,13 +70,14 @@ include $(KEYBOARD_DIR)/keyboard.mk | ||||
|  | ||||
| # Add kbled | ||||
| KBLED?=none | ||||
| SRC+=$(SYSTEM76_COMMON_DIR)/kbled/$(KBLED).c | ||||
| board-common-y += kbled/$(KBLED).c | ||||
|  | ||||
| # Add scratch ROM | ||||
| include $(SYSTEM76_COMMON_DIR)/scratch/scratch.mk | ||||
|  | ||||
| # Add scratch ROM for flash access | ||||
| include $(SYSTEM76_COMMON_DIR)/flash/flash.mk | ||||
| board-common-y += flash/wrapper.c | ||||
|  | ||||
| console_internal: | ||||
| 	cargo build --manifest-path tool/Cargo.toml --release | ||||
|   | ||||
| @@ -1,5 +1,7 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-or-later | ||||
|  | ||||
| flash-y += main.c | ||||
|  | ||||
| # Set flash ROM parameters | ||||
| FLASH_OFFSET=2048 | ||||
| FLASH_SIZE=1024 | ||||
| @@ -12,9 +14,9 @@ FLASH_CFLAGS=$(CFLAGS) | ||||
| # Include flash source. | ||||
| FLASH_DIR=$(SYSTEM76_COMMON_DIR)/flash | ||||
| # Note: main.c *must* be first to ensure that flash_start is at the correct address | ||||
| FLASH_SRC=$(FLASH_DIR)/main.c | ||||
| FLASH_INCLUDE+=$(wildcard $(FLASH_DIR)/include/flash/*.h) $(FLASH_DIR)/flash.mk | ||||
| FLASH_CFLAGS+=-I$(FLASH_DIR)/include -D__FLASH__ | ||||
| FLASH_SRC += $(foreach src, $(flash-y), $(FLASH_DIR)/$(src)) | ||||
|  | ||||
| FLASH_BUILD=$(BUILD)/flash | ||||
| FLASH_OBJ=$(sort $(patsubst src/%.c,$(FLASH_BUILD)/%.rel,$(FLASH_SRC))) | ||||
| @@ -52,4 +54,3 @@ $(FLASH_OBJ): $(FLASH_BUILD)/%.rel: src/%.c $(FLASH_INCLUDE) | ||||
| CFLAGS+=-I$(BUILD)/include | ||||
| LDFLAGS+=-Wl -g_flash_entry=$(FLASH_OFFSET) | ||||
| INCLUDE+=$(BUILD)/include/flash.h | ||||
| SRC+=$(FLASH_DIR)/wrapper.c | ||||
|   | ||||
| @@ -1,5 +1,10 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| scratch-y += main.c | ||||
| scratch-y += stdio.c | ||||
| # TODO: Use relative path; generates different binary | ||||
| SCRATCH_SRC += $(SYSTEM76_COMMON_DIR)/smfi.c | ||||
|  | ||||
| # Set scratch ROM parameters | ||||
| SCRATCH_OFFSET=1024 | ||||
| SCRATCH_SIZE=1024 | ||||
| @@ -9,15 +14,11 @@ CFLAGS+=-DSCRATCH_OFFSET=$(SCRATCH_OFFSET) -DSCRATCH_SIZE=$(SCRATCH_SIZE) | ||||
| SCRATCH_INCLUDE=$(INCLUDE) | ||||
| SCRATCH_CFLAGS=$(CFLAGS) | ||||
|  | ||||
| #Include scratch source | ||||
| # Include scratch source | ||||
| SCRATCH_DIR=$(SYSTEM76_COMMON_DIR)/scratch | ||||
| SCRATCH_SRC=$(wildcard $(SCRATCH_DIR)/*.c) | ||||
| SCRATCH_INCLUDE+=$(wildcard $(SCRATCH_DIR)/include/scratch/*.h) $(SCRATCH_DIR)/scratch.mk | ||||
| SCRATCH_CFLAGS+=-I$(SCRATCH_DIR)/include -D__SCRATCH__ | ||||
|  | ||||
| # Add minimal source from other directories | ||||
| SCRATCH_SRC+=\ | ||||
| 	$(SYSTEM76_COMMON_DIR)/smfi.c | ||||
| SCRATCH_SRC += $(foreach src, $(scratch-y), $(SCRATCH_DIR)/$(src)) | ||||
|  | ||||
| SCRATCH_BUILD=$(BUILD)/scratch | ||||
| SCRATCH_OBJ=$(sort $(patsubst src/%.c,$(SCRATCH_BUILD)/%.rel,$(SCRATCH_SRC))) | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it8587e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it8587e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it8587e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -1,5 +1,8 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| board-y += board.c | ||||
| board-y += gpio.c | ||||
|  | ||||
| EC=ite | ||||
| EC_VARIANT=it5570e | ||||
|  | ||||
|   | ||||
| @@ -0,0 +1,5 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| common-y += i2c.c | ||||
| common-y += keymap.c | ||||
| common-y += version.c | ||||
|   | ||||
| @@ -1,5 +1,14 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| ec-y += ec.c | ||||
| ec-y += espi.c | ||||
| ec-y += gpio.c | ||||
| ec-y += i2c.c | ||||
| ec-y += kbc.c | ||||
| ec-y += pmc.c | ||||
| ec-y += ps2.c | ||||
| ec-y += signature.c | ||||
|  | ||||
| ifeq ($(EC_VARIANT), it8587e) | ||||
| # SRAM is 4096 bytes, but SRAM at address 2048 is used for scratch ROM | ||||
| SRAM_SIZE=2048 | ||||
|   | ||||
| @@ -1,7 +1,7 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| # Include keyboard source | ||||
| KEYMAP?=default | ||||
| SRC+=$(KEYBOARD_DIR)/keymap/$(KEYMAP).c | ||||
| keyboard-y += keymap/$(KEYMAP).c | ||||
|  | ||||
| INCLUDE+=$(wildcard $(KEYBOARD_DIR)/include/board/*.h) $(KEYBOARD_DIR)/keyboard.mk | ||||
| CFLAGS+=-I$(KEYBOARD_DIR)/include | ||||
|   | ||||
| @@ -1,7 +1,7 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| # Include keyboard source | ||||
| KEYMAP?=default | ||||
| SRC+=$(KEYBOARD_DIR)/keymap/$(KEYMAP).c | ||||
| keyboard-y += keymap/$(KEYMAP).c | ||||
|  | ||||
| INCLUDE+=$(wildcard $(KEYBOARD_DIR)/include/board/*.h) $(KEYBOARD_DIR)/keyboard.mk | ||||
| CFLAGS+=-I$(KEYBOARD_DIR)/include | ||||
|   | ||||
| @@ -1,7 +1,7 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| # Include keyboard source | ||||
| KEYMAP?=default | ||||
| SRC+=$(KEYBOARD_DIR)/keymap/$(KEYMAP).c | ||||
| keyboard-y += keymap/$(KEYMAP).c | ||||
|  | ||||
| INCLUDE+=$(wildcard $(KEYBOARD_DIR)/include/board/*.h) $(KEYBOARD_DIR)/keyboard.mk | ||||
| CFLAGS+=-I$(KEYBOARD_DIR)/include | ||||
|   | ||||
| @@ -1,7 +1,7 @@ | ||||
| # SPDX-License-Identifier: GPL-3.0-only | ||||
|  | ||||
| # Include keyboard source | ||||
| KEYMAP?=default | ||||
| SRC+=$(KEYBOARD_DIR)/keymap/$(KEYMAP).c | ||||
| keyboard-y += keymap/$(KEYMAP).c | ||||
|  | ||||
| INCLUDE+=$(wildcard $(KEYBOARD_DIR)/include/board/*.h) $(KEYBOARD_DIR)/keyboard.mk | ||||
| CFLAGS+=-I$(KEYBOARD_DIR)/include | ||||
|   | ||||
		Reference in New Issue
	
	Block a user