mirror of git://git.yoctoproject.org/linux-yocto.git synced 2025-08-21 08:21:58 +02:00

Masahiro Yamada 5c8166419a kbuild: replace $(if A,A,B) with $(or A,B)

$(or ...) is available since GNU Make 3.81, and useful to shorten the
code in some places.

Covert as follows:

  $(if A,A,B)  -->  $(or A,B)

This patch also converts:

  $(if A, A, B) --> $(or A, B)

Strictly speaking, the latter is not an equivalent conversion because
GNU Make keeps spaces after commas; if A is not empty, $(if A, A, B)
expands to " A", while $(or A, B) expands to "A".

Anyway, preceding spaces are not significant in the code hunks I touched.

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Reviewed-by: Nicolas Schier <nicolas@fjasle.eu>

2022-02-15 12:25:56 +09:00

1.7 KiB

Raw Permalink Blame History

SPDX-License-Identifier: GPL-2.0-only

include ../scripts/Makefile.include

bindir ?= /usr/bin

ifeq ($(srctree),) srctree := $(patsubst %/,%,$(dir $(CURDIR))) srctree := $(patsubst %/,%,$(dir $(srctree))) endif

Do not use make's built-in rules

(this improves performance and avoids hard-to-debug behaviour);

MAKEFLAGS += -r

CFLAGS += -O2 -Wall -g -D_GNU_SOURCE -I$(OUTPUT)include

ALL_TARGETS := spidev_test spidev_fdx ALL_PROGRAMS := $(patsubst %,$(OUTPUT)%,$(ALL_TARGETS))

all: $(ALL_PROGRAMS)

export srctree OUTPUT CC LD CFLAGS include $(srctree)/tools/build/Makefile.include

We need the following to be outside of kernel tree

$(OUTPUT)include/linux/spi: ../../include/uapi/linux/spi mkdir -p $(OUTPUT)include/linux/spi 2>&1 || true ln -sf $(CURDIR)/../../include/uapi/linux/spi/spidev.h $@ ln -sf $(CURDIR)/../../include/uapi/linux/spi/spi.h $@

prepare: $(OUTPUT)include/linux/spi

spidev_test

SPIDEV_TEST_IN := $(OUTPUT)spidev_test-in.o $(SPIDEV_TEST_IN): prepare FORCE $(Q)$(MAKE) $(build)=spidev_test $(OUTPUT)spidev_test: $(SPIDEV_TEST_IN) $(QUIET_LINK)$(CC) $(CFLAGS) $(LDFLAGS) $< -o $@

spidev_fdx

SPIDEV_FDX_IN := $(OUTPUT)spidev_fdx-in.o $(SPIDEV_FDX_IN): prepare FORCE $(Q)$(MAKE) $(build)=spidev_fdx $(OUTPUT)spidev_fdx: $(SPIDEV_FDX_IN) $(QUIET_LINK)$(CC) $(CFLAGS) $(LDFLAGS) $< -o $@

clean: rm -f $(ALL_PROGRAMS) rm -rf $(OUTPUT)include/ find $(or $(OUTPUT),.) -name '.o' -delete find $(or $(OUTPUT),.) -name '..o.d' -delete find $(or $(OUTPUT),.) -name '.*.o.cmd' -delete

install: $(ALL_PROGRAMS) install -d -m 755 $(DESTDIR)$(bindir);
for program in $(ALL_PROGRAMS); do
install $$program $(DESTDIR)$(bindir);
done

FORCE:

.PHONY: all install clean FORCE prepare

1.7 KiB Raw Permalink Blame History