build system: Remove flock calls from intermediate processing
Now that intermediate coreboot.pre manipulation is serialized within the build system, remove the flock calls. Change-Id: I8a767918aec5fcb7127ebb19ac46e58bed7967fb Signed-off-by: Patrick Georgi <pgeorgi@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/49381 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin Roth <martinroth@google.com> Reviewed-by: Julius Werner <jwerner@chromium.org>
This commit is contained in:
		@@ -21,7 +21,7 @@ ifeq ($(CONFIG_KBC1126_FIRMWARE),y)
 | 
				
			|||||||
	printf "    Building kbc1126_ec_insert.\n"
 | 
						printf "    Building kbc1126_ec_insert.\n"
 | 
				
			||||||
	$(MAKE) -C util/kbc1126
 | 
						$(MAKE) -C util/kbc1126
 | 
				
			||||||
	printf "    KBC1126    Inserting KBC1126 firmware blobs.\n"
 | 
						printf "    KBC1126    Inserting KBC1126 firmware blobs.\n"
 | 
				
			||||||
	flock $< $(KBC1126_EC_INSERT) $(obj)/coreboot.pre \
 | 
						$(KBC1126_EC_INSERT) $(obj)/coreboot.pre \
 | 
				
			||||||
		$(CONFIG_KBC1126_FW1_OFFSET) $(CONFIG_KBC1126_FW2_OFFSET)
 | 
							$(CONFIG_KBC1126_FW1_OFFSET) $(CONFIG_KBC1126_FW2_OFFSET)
 | 
				
			||||||
endif
 | 
					endif
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -170,7 +170,7 @@ STONEYRIDGE_FWM_ROM_POSITION=$(call int-add, \
 | 
				
			|||||||
$(call add_intermediate, add_amdfw, $(obj)/coreboot.pre $(obj)/amdfw.rom)
 | 
					$(call add_intermediate, add_amdfw, $(obj)/coreboot.pre $(obj)/amdfw.rom)
 | 
				
			||||||
	printf "    DD         Adding AMD Firmware at ROM offset 0x%x\n" \
 | 
						printf "    DD         Adding AMD Firmware at ROM offset 0x%x\n" \
 | 
				
			||||||
		"$(STONEYRIDGE_FWM_ROM_POSITION)"
 | 
							"$(STONEYRIDGE_FWM_ROM_POSITION)"
 | 
				
			||||||
	flock $< dd if=$(obj)/amdfw.rom \
 | 
						dd if=$(obj)/amdfw.rom \
 | 
				
			||||||
		of=$(obj)/coreboot.pre conv=notrunc bs=1 \
 | 
							of=$(obj)/coreboot.pre conv=notrunc bs=1 \
 | 
				
			||||||
		seek=$(STONEYRIDGE_FWM_ROM_POSITION) >/dev/null 2>&1
 | 
							seek=$(STONEYRIDGE_FWM_ROM_POSITION) >/dev/null 2>&1
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -159,7 +159,7 @@ $(obj)/amdfw.rom:	$(call strip_quotes, $(CONFIG_HUDSON_XHCI_FWM_FILE)) \
 | 
				
			|||||||
ifeq ($(CONFIG_AMDFW_OUTSIDE_CBFS),y)
 | 
					ifeq ($(CONFIG_AMDFW_OUTSIDE_CBFS),y)
 | 
				
			||||||
$(call add_intermediate, add_amdfw, $(obj)/coreboot.pre $(obj)/amdfw.rom)
 | 
					$(call add_intermediate, add_amdfw, $(obj)/coreboot.pre $(obj)/amdfw.rom)
 | 
				
			||||||
	printf "    DD         Adding AMD Firmware\n"
 | 
						printf "    DD         Adding AMD Firmware\n"
 | 
				
			||||||
	flock $< dd if=$(obj)/amdfw.rom \
 | 
						dd if=$(obj)/amdfw.rom \
 | 
				
			||||||
		of=$(obj)/coreboot.pre conv=notrunc bs=1 seek=131072 >/dev/null 2>&1
 | 
							of=$(obj)/coreboot.pre conv=notrunc bs=1 seek=131072 >/dev/null 2>&1
 | 
				
			||||||
 | 
					
 | 
				
			||||||
else # ifeq ($(CONFIG_AMDFW_OUTSIDE_CBFS),y)
 | 
					else # ifeq ($(CONFIG_AMDFW_OUTSIDE_CBFS),y)
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -35,7 +35,7 @@ add_intel_firmware: $(call strip_quotes,$(CONFIG_EC_BIN_PATH))
 | 
				
			|||||||
endif
 | 
					endif
 | 
				
			||||||
add_intel_firmware: $(obj)/coreboot.pre $(IFDTOOL)
 | 
					add_intel_firmware: $(obj)/coreboot.pre $(IFDTOOL)
 | 
				
			||||||
	printf "    DD         Adding Intel Firmware Descriptor\n"
 | 
						printf "    DD         Adding Intel Firmware Descriptor\n"
 | 
				
			||||||
	flock $< dd if=$(IFD_BIN_PATH) \
 | 
						dd if=$(IFD_BIN_PATH) \
 | 
				
			||||||
		of=$(obj)/coreboot.pre conv=notrunc >/dev/null 2>&1
 | 
							of=$(obj)/coreboot.pre conv=notrunc >/dev/null 2>&1
 | 
				
			||||||
ifeq ($(CONFIG_VALIDATE_INTEL_DESCRIPTOR),y)
 | 
					ifeq ($(CONFIG_VALIDATE_INTEL_DESCRIPTOR),y)
 | 
				
			||||||
	$(objutil)/ifdtool/ifdtool \
 | 
						$(objutil)/ifdtool/ifdtool \
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user