All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v4l-utils] Move sync-with-kernel into dedicated shell script ./sync-with-kernel.sh
@ 2020-06-20 10:08 Sean Young
  2020-06-22 17:37 ` Ariel D'Alessandro
  0 siblings, 1 reply; 3+ messages in thread
From: Sean Young @ 2020-06-20 10:08 UTC (permalink / raw)
  To: linux-media
  Cc: hverkuil, p.zabel, laurent.pinchart, ezequiel, nicolas,
	kieran.bingham, ariel, gjasny

The sync-with-kernel target is a collection of shell fragments scattered
over the tree. Collect all of them into a single shell script. This makes
it is easier to see what sync-with-kernel does, and makes the move away
from autotools to meson easier.

Signed-off-by: Sean Young <sean@mess.org>
---
 Makefile.am                         |  59 +--------------
 README                              |   4 +-
 contrib/freebsd/Makefile.am         |  14 ----
 contrib/test/Makefile.am            |   9 ---
 contrib/test/gen_ioctl_list.pl      |   2 +-
 contrib/test/ioctl-test.h           |   2 +-
 contrib/xc3028-firmware/Makefile.am |   9 ---
 lib/libdvbv5/Makefile.am            |   3 -
 sync-with-kernel.sh                 | 113 ++++++++++++++++++++++++++++
 utils/keytable/Makefile.am          |  13 ----
 10 files changed, 121 insertions(+), 107 deletions(-)
 create mode 100755 sync-with-kernel.sh

diff --git a/Makefile.am b/Makefile.am
index 3c3d3ce6..3c0ceb49 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -7,60 +7,9 @@ if WITH_V4LUTILS
 SUBDIRS += utils contrib
 endif
 
-EXTRA_DIST = android-config.h bootstrap.sh doxygen_libdvbv5.cfg include COPYING.libv4l \
-        COPYING.libdvbv5 README.libv4l README.lib-multi-threading TODO.libdvbv5 \
-        doc/libdvbv5-index.doc
+EXTRA_DIST = android-config.h sync-with-kernel.sh bootstrap.sh \
+	doxygen_libdvbv5.cfg include COPYING.libv4l \
+	COPYING.libdvbv5 README.libv4l README.lib-multi-threading \
+	TODO.libdvbv5 doc/libdvbv5-index.doc
 
 include $(top_srcdir)/aminclude.am
-
-# custom targets
-
-sync-with-kernel:
-	@if [ ! -f $(KERNEL_DIR)/usr/include/linux/videodev2.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/fb.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-controls.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-common.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-subdev.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-mediabus.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/ivtv.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/dvb/frontend.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/dvb/dmx.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/lirc.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/bpf.h -o \
-	      ! -f $(KERNEL_DIR)/usr/include/linux/bpf_common.h ]; then \
-	  echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"; \
-	  echo "and run 'make headers_install' in \$$KERNEL_DIR."; \
-	  exit 1; \
-	fi
-	cp -a $(KERNEL_DIR)/usr/include/linux/videodev2.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/fb.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-controls.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-common.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-subdev.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-mediabus.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/media-bus-format.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/media.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/ivtv.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/dvb/frontend.h $(top_srcdir)/include/linux/dvb
-	cp $(top_srcdir)/include/linux/dvb/frontend.h $(top_srcdir)/lib/include/libdvbv5/dvb-frontend.h
-	cp -a $(KERNEL_DIR)/usr/include/linux/dvb/dmx.h $(top_srcdir)/include/linux/dvb
-	cp -a $(KERNEL_DIR)/usr/include/linux/lirc.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/bpf.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/bpf_common.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/cec.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/usr/include/linux/cec-funcs.h $(top_srcdir)/include/linux
-	cp -a $(KERNEL_DIR)/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c $(top_srcdir)/utils/common
-	cp -a $(KERNEL_DIR)/drivers/media/common/v4l2-tpg/v4l2-tpg-colors.c $(top_srcdir)/utils/common
-	cp -a $(KERNEL_DIR)/include/media/tpg/v4l2-tpg.h $(top_srcdir)/utils/common
-	cp -a $(KERNEL_DIR)/include/media/fwht-ctrls.h $(top_srcdir)/utils/common
-	patch -d $(top_srcdir) --no-backup-if-mismatch -p0 <$(top_srcdir)/utils/common/v4l2-tpg.patch
-	cp -a $(KERNEL_DIR)/drivers/media/test-drivers/vicodec/codec-fwht.[ch] $(top_srcdir)/utils/common/
-	cp -a $(KERNEL_DIR)/drivers/media/test-drivers/vicodec/codec-v4l2-fwht.[ch] $(top_srcdir)/utils/common/
-	patch -d $(top_srcdir) --no-backup-if-mismatch -p1 <$(top_srcdir)/utils/common/codec-fwht.patch
-	grep V4L2_.*_FMT.*descr $(KERNEL_DIR)/drivers/media/v4l2-core/v4l2-ioctl.c | grep -v V4L2_PIX_FMT_H264_SLICE | grep -v V4L2_PIX_FMT_VP8_FRAME | grep -v V4L2_PIX_FMT_HEVC_SLICE | perl -pe 's/.*V4L2_(.*)_FMT/\tcase V4L2_\1_FMT/; s/:.*descr = /: return /; s/;.*/;/;' >$(top_srcdir)/utils/common/v4l2-pix-formats.h
-
-	$(MAKE) -C utils/keytable $@
-	$(MAKE) -C lib/libdvbv5 $@
-	$(MAKE) -C contrib/freebsd $@
-	$(MAKE) -C contrib/test $@
-	$(MAKE) -C contrib/xc3028-firmware $@
diff --git a/README b/README
index e344dee1..48b33713 100644
--- a/README
+++ b/README
@@ -264,11 +264,11 @@ Be sure that you have installed both glibc development packages for
 
 step 3) at v4l-utils tree:
 
-	$ export KERNEL_DIR=location/of/the/kernel/tree make sync-with-kernel
+	$ export KERNEL_DIR=location/of/the/kernel/tree ./sync-with-kernel.sh
 
 Alternatively, steps 1 to 3 can be replaced with:
 
-export KERNEL_DIR=location/of/the/kernel/tree && (cd $KERNEL_DIR && make headers_install INSTALL_HDR_PATH=usr/) && make sync-with-kernel
+export KERNEL_DIR=location/of/the/kernel/tree && (cd $KERNEL_DIR && make headers_install INSTALL_HDR_PATH=usr/) && ./sync-with-kernel.sh
 
 step 4)
 
diff --git a/contrib/freebsd/Makefile.am b/contrib/freebsd/Makefile.am
index ca322749..abf0d57b 100644
--- a/contrib/freebsd/Makefile.am
+++ b/contrib/freebsd/Makefile.am
@@ -1,17 +1,3 @@
 EXTRA_DIST = \
 	include \
 	patches
-
-sync-with-kernel:
-	rm -rf .pc
-
-	for i in input.h input-event-codes.h ivtv.h uinput.h videodev2.h v4l2-controls.h v4l2-common.h; do \
-	    mkdir -p include/linux/$$(dirname $$i); \
-	    cp $(KERNEL_DIR)/usr/include/linux/$$i include/linux/$$i; \
-	done
-
-	for i in ivtv.h uinput.h videodev2.h v4l2-common.h; do \
-	    sed -e 's/__u8/uint8_t/g' -e 's/__u16/uint16_t/g' -e 's/__u32/uint32_t/g' -e 's/__u64/uint64_t/g' -e 's/__s8/int8_t/g' -e 's/__s16/int16_t/g' -e 's/__s32/int32_t/g' -e 's/__s64/int64_t/g' -e 's/__le32/uint32_t/g' -e 's/__user//g' -i include/linux/$$i; \
-	done
-
-	quilt push -a
diff --git a/contrib/test/Makefile.am b/contrib/test/Makefile.am
index ded3c1f4..6650cd27 100644
--- a/contrib/test/Makefile.am
+++ b/contrib/test/Makefile.am
@@ -57,15 +57,6 @@ capture_example_SOURCES = capture-example.c
 
 ioctl-test.c: ioctl-test.h
 
-sync-with-kernel:
-	./gen_ioctl_list.pl --gen_ioctl_numbers >.tmp_ioctl.c
-	gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl32 -m32
-	gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl64 -m64
-	./tmp_ioctl32 32 >ioctl_32.h
-	./tmp_ioctl64 64 >ioctl_64.h
-	rm .tmp_ioctl.c tmp_ioctl32 tmp_ioctl64
-	./gen_ioctl_list.pl >ioctl-test.h
-
 EXTRA_DIST = \
 	gen_ioctl_list.pl \
 	test-media \
diff --git a/contrib/test/gen_ioctl_list.pl b/contrib/test/gen_ioctl_list.pl
index fca51778..82168b39 100755
--- a/contrib/test/gen_ioctl_list.pl
+++ b/contrib/test/gen_ioctl_list.pl
@@ -69,7 +69,7 @@ sub print_ioc()
 	printf "#define S_IOCTLS sizeof(ioctls)/sizeof(ioctls[0])\n";
 }
 
-printf "/* This file is auto-generated by make sync-with-kernel */\n";
+printf "/* This file is auto-generated by sync-with-kernel.sh */\n";
 printf "#include <linux/compiler.h>\n";
 
 foreach my $h (sort keys %headers) {
diff --git a/contrib/test/ioctl-test.h b/contrib/test/ioctl-test.h
index 32084c52..03b53078 100644
--- a/contrib/test/ioctl-test.h
+++ b/contrib/test/ioctl-test.h
@@ -1,4 +1,4 @@
-/* This file is auto-generated by make sync-with-kernel */
+/* This file is auto-generated by sync-with-kernel.sh */
 #include <linux/compiler.h>
 #include "linux/dvb/dmx.h"
 #include "linux/dvb/frontend.h"
diff --git a/contrib/xc3028-firmware/Makefile.am b/contrib/xc3028-firmware/Makefile.am
index 43ff8b12..8b23d1e1 100644
--- a/contrib/xc3028-firmware/Makefile.am
+++ b/contrib/xc3028-firmware/Makefile.am
@@ -3,12 +3,3 @@ noinst_PROGRAMS = firmware-tool
 firmware_tool_SOURCES = firmware-tool.c standards.c extract_head.h standards.h tuner-xc2028-types.h
 
 EXTRA_DIST = README
-
-# custom targets
-
-sync-with-kernel:
-	@if [ ! -f $(KERNEL_DIR)/drivers/media/tuners/tuner-xc2028-types.h ]; then \
-	  echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"; \
-	  exit 1; \
-	fi
-	cp -a $(KERNEL_DIR)/drivers/media/tuners/tuner-xc2028-types.h $(srcdir)/
diff --git a/lib/libdvbv5/Makefile.am b/lib/libdvbv5/Makefile.am
index e65066e9..09c261be 100644
--- a/lib/libdvbv5/Makefile.am
+++ b/lib/libdvbv5/Makefile.am
@@ -122,6 +122,3 @@ libdvbv5_la_LDFLAGS = $(LIBDVBV5_VERSION) $(ENFORCE_LIBDVBV5_STATIC) $(LIBUDEV_L
 libdvbv5_la_LIBADD = $(LTLIBICONV)
 
 EXTRA_DIST = README gen_dvb_structs.pl
-
-sync-with-kernel:
-	./gen_dvb_structs.pl $(KERNEL_DIR)/usr/include/
diff --git a/sync-with-kernel.sh b/sync-with-kernel.sh
new file mode 100755
index 00000000..ab15cb8a
--- /dev/null
+++ b/sync-with-kernel.sh
@@ -0,0 +1,113 @@
+#!/bin/bash
+
+TOPSRCDIR="$( cd "$( dirname "$0" )" && pwd )"
+
+if [ -z "${KERNEL_DIR}" -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/videodev2.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/fb.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-controls.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-common.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-subdev.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-mediabus.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/ivtv.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/dvb/frontend.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/dvb/dmx.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/lirc.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/bpf.h -o \
+     ! -f ${KERNEL_DIR}/usr/include/linux/bpf_common.h ]; then
+  echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"
+  echo "and run 'make headers_install' in \$KERNEL_DIR."
+  exit 1
+fi
+
+cp -a ${KERNEL_DIR}/usr/include/linux/videodev2.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/fb.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-controls.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-common.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-subdev.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-mediabus.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/media-bus-format.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/media.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/ivtv.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/dvb/frontend.h ${TOPSRCDIR}/include/linux/dvb
+cp ${TOPSRCDIR}/include/linux/dvb/frontend.h ${TOPSRCDIR}/lib/include/libdvbv5/dvb-frontend.h
+cp -a ${KERNEL_DIR}/usr/include/linux/dvb/dmx.h ${TOPSRCDIR}/include/linux/dvb
+cp -a ${KERNEL_DIR}/usr/include/linux/lirc.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/bpf.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/bpf_common.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/cec.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/usr/include/linux/cec-funcs.h ${TOPSRCDIR}/include/linux
+cp -a ${KERNEL_DIR}/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c ${TOPSRCDIR}/utils/common
+cp -a ${KERNEL_DIR}/drivers/media/common/v4l2-tpg/v4l2-tpg-colors.c ${TOPSRCDIR}/utils/common
+cp -a ${KERNEL_DIR}/include/media/tpg/v4l2-tpg.h ${TOPSRCDIR}/utils/common
+cp -a ${KERNEL_DIR}/include/media/fwht-ctrls.h ${TOPSRCDIR}/utils/common
+patch -d ${TOPSRCDIR} --no-backup-if-mismatch -p0 <${TOPSRCDIR}/utils/common/v4l2-tpg.patch
+cp -a ${KERNEL_DIR}/drivers/media/test-drivers/vicodec/codec-fwht.[ch] ${TOPSRCDIR}/utils/common/
+cp -a ${KERNEL_DIR}/drivers/media/test-drivers/vicodec/codec-v4l2-fwht.[ch] ${TOPSRCDIR}/utils/common/
+patch -d ${TOPSRCDIR} --no-backup-if-mismatch -p1 <${TOPSRCDIR}/utils/common/codec-fwht.patch
+grep V4L2_.*_FMT.*descr ${KERNEL_DIR}/drivers/media/v4l2-core/v4l2-ioctl.c | grep -v V4L2_PIX_FMT_H264_SLICE | grep -v V4L2_PIX_FMT_VP8_FRAME | grep -v V4L2_PIX_FMT_HEVC_SLICE | perl -pe 's/.*V4L2_(.*)_FMT/\tcase V4L2_\1_FMT/; s/:.*descr = /: return /; s/;.*/;/;' >${TOPSRCDIR}/utils/common/v4l2-pix-formats.h
+
+function keytable {
+	SRCDIR=${TOPSRCDIR}/utils/keytable
+
+	cd ${SRCDIR}
+	echo generating ${SRCDIR}/parse.h
+	./gen_input_events.pl < ${KERNEL_DIR}/usr/include/linux/input-event-codes.h  > ${SRCDIR}/parse.h
+	mkdir -p ${SRCDIR}/rc_keymaps
+	rm -f ${SRCDIR}/rc_keymaps/*
+	echo storing existing keymaps at ${SRCDIR}/rc_keymaps/
+	./gen_keytables.pl ${KERNEL_DIR};
+	cp ${SRCDIR}/rc_keymaps_userspace/* ${SRCDIR}/rc_keymaps/
+}
+
+function libdvbv5 {
+	SRCDIR=${TOPSRCDIR}/lib/libdvbv5
+
+	cd ${SRCDIR}
+	./gen_dvb_structs.pl ${KERNEL_DIR}/usr/include/
+}
+
+function freebsd {
+	SRCDIR=${TOPSRCDIR}/contrib/freebsd
+
+	cp -a ${KERNEL_DIR}/drivers/media/tuners/tuner-xc2028-types.h ${SRCDIR}/
+}
+
+function ioctl-test {
+	SRCDIR=${TOPSRCDIR}/contrib/test
+
+	cd ${SRCDIR}
+
+	./gen_ioctl_list.pl --gen_ioctl_numbers >.tmp_ioctl.c
+	gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl32 -m32
+	gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl64 -m64
+	./tmp_ioctl32 32 >ioctl_32.h
+	./tmp_ioctl64 64 >ioctl_64.h
+	rm .tmp_ioctl.c tmp_ioctl32 tmp_ioctl64
+	./gen_ioctl_list.pl >ioctl-test.h
+}
+
+function xc3028-firmware {
+	SRCDIR=${TOPSRCDIR}/contrib/xc3028-firmware
+
+	cd ${SRCDIR}
+
+	rm -rf .pc
+
+	for i in input.h input-event-codes.h ivtv.h uinput.h videodev2.h v4l2-controls.h v4l2-common.h; do
+	    mkdir -p include/linux/$(dirname $i)
+	    cp ${KERNEL_DIR}/usr/include/linux/$i include/linux/$i
+	done
+
+	for i in ivtv.h uinput.h videodev2.h v4l2-common.h; do
+	    sed -e 's/__u8/uint8_t/g' -e 's/__u16/uint16_t/g' -e 's/__u32/uint32_t/g' -e 's/__u64/uint64_t/g' -e 's/__s8/int8_t/g' -e 's/__s16/int16_t/g' -e 's/__s32/int32_t/g' -e 's/__s64/int64_t/g' -e 's/__le32/uint32_t/g' -e 's/__user//g' -i include/linux/$i
+	done
+
+	quilt push -a
+}
+
+keytable
+libdvbv5
+freebsd
+ioctl-test
+xc3028-firmware
diff --git a/utils/keytable/Makefile.am b/utils/keytable/Makefile.am
index 4724897e..a057eb89 100644
--- a/utils/keytable/Makefile.am
+++ b/utils/keytable/Makefile.am
@@ -29,16 +29,3 @@ check:
 # custom target
 install-data-local:
 	$(install_sh) -d "$(DESTDIR)$(keytableuserdir)"
-
-sync-with-kernel:
-	@if [ ! -f $(KERNEL_DIR)/usr/include/linux/input-event-codes.h ]; then \
-	  echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"; \
-	  exit 1; \
-	fi
-	@echo generating $(srcdir)/parse.h
-	@./gen_input_events.pl < $(KERNEL_DIR)/usr/include/linux/input-event-codes.h  > $(srcdir)/parse.h
-	@-mkdir -p $(srcdir)/rc_keymaps
-	@-rm $(srcdir)/rc_keymaps/*
-	@echo storing existing keymaps at $(srcdir)/rc_keymaps/
-	@cd $(srcdir) && ./gen_keytables.pl $(KERNEL_DIR);
-	@-cp $(srcdir)/rc_keymaps_userspace/* $(srcdir)/rc_keymaps/
-- 
2.26.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v4l-utils] Move sync-with-kernel into dedicated shell script ./sync-with-kernel.sh
  2020-06-20 10:08 [PATCH v4l-utils] Move sync-with-kernel into dedicated shell script ./sync-with-kernel.sh Sean Young
@ 2020-06-22 17:37 ` Ariel D'Alessandro
  2020-06-22 18:58   ` Sean Young
  0 siblings, 1 reply; 3+ messages in thread
From: Ariel D'Alessandro @ 2020-06-22 17:37 UTC (permalink / raw)
  To: Sean Young
  Cc: linux-media, Hans Verkuil, Philipp Zabel, Laurent Pinchart,
	Ezequiel Garcia, Nicolas Dufresne, Kieran Bingham, Gregor Jasny

Hi Sean,

Great! Thanks for picking this up so quickly. A few comments below.

On Sat, Jun 20, 2020 at 7:08 AM Sean Young <sean@mess.org> wrote:
>
> The sync-with-kernel target is a collection of shell fragments scattered
> over the tree. Collect all of them into a single shell script. This makes
> it is easier to see what sync-with-kernel does, and makes the move away
> from autotools to meson easier.
>
> Signed-off-by: Sean Young <sean@mess.org>
> ---
>  Makefile.am                         |  59 +--------------
>  README                              |   4 +-
>  contrib/freebsd/Makefile.am         |  14 ----
>  contrib/test/Makefile.am            |   9 ---
>  contrib/test/gen_ioctl_list.pl      |   2 +-
>  contrib/test/ioctl-test.h           |   2 +-
>  contrib/xc3028-firmware/Makefile.am |   9 ---
>  lib/libdvbv5/Makefile.am            |   3 -
>  sync-with-kernel.sh                 | 113 ++++++++++++++++++++++++++++
>  utils/keytable/Makefile.am          |  13 ----
>  10 files changed, 121 insertions(+), 107 deletions(-)
>  create mode 100755 sync-with-kernel.sh
>
> diff --git a/Makefile.am b/Makefile.am
> index 3c3d3ce6..3c0ceb49 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -7,60 +7,9 @@ if WITH_V4LUTILS
>  SUBDIRS += utils contrib
>  endif
>
> -EXTRA_DIST = android-config.h bootstrap.sh doxygen_libdvbv5.cfg include COPYING.libv4l \
> -        COPYING.libdvbv5 README.libv4l README.lib-multi-threading TODO.libdvbv5 \
> -        doc/libdvbv5-index.doc
> +EXTRA_DIST = android-config.h sync-with-kernel.sh bootstrap.sh \
> +       doxygen_libdvbv5.cfg include COPYING.libv4l \
> +       COPYING.libdvbv5 README.libv4l README.lib-multi-threading \
> +       TODO.libdvbv5 doc/libdvbv5-index.doc
>
>  include $(top_srcdir)/aminclude.am
> -
> -# custom targets
> -
> -sync-with-kernel:
> -       @if [ ! -f $(KERNEL_DIR)/usr/include/linux/videodev2.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/fb.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-controls.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-common.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-subdev.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-mediabus.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/ivtv.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/dvb/frontend.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/dvb/dmx.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/lirc.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/bpf.h -o \
> -             ! -f $(KERNEL_DIR)/usr/include/linux/bpf_common.h ]; then \
> -         echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"; \
> -         echo "and run 'make headers_install' in \$$KERNEL_DIR."; \
> -         exit 1; \
> -       fi
> -       cp -a $(KERNEL_DIR)/usr/include/linux/videodev2.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/fb.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-controls.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-common.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-subdev.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-mediabus.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/media-bus-format.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/media.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/ivtv.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/dvb/frontend.h $(top_srcdir)/include/linux/dvb
> -       cp $(top_srcdir)/include/linux/dvb/frontend.h $(top_srcdir)/lib/include/libdvbv5/dvb-frontend.h
> -       cp -a $(KERNEL_DIR)/usr/include/linux/dvb/dmx.h $(top_srcdir)/include/linux/dvb
> -       cp -a $(KERNEL_DIR)/usr/include/linux/lirc.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/bpf.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/bpf_common.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/cec.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/usr/include/linux/cec-funcs.h $(top_srcdir)/include/linux
> -       cp -a $(KERNEL_DIR)/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c $(top_srcdir)/utils/common
> -       cp -a $(KERNEL_DIR)/drivers/media/common/v4l2-tpg/v4l2-tpg-colors.c $(top_srcdir)/utils/common
> -       cp -a $(KERNEL_DIR)/include/media/tpg/v4l2-tpg.h $(top_srcdir)/utils/common
> -       cp -a $(KERNEL_DIR)/include/media/fwht-ctrls.h $(top_srcdir)/utils/common
> -       patch -d $(top_srcdir) --no-backup-if-mismatch -p0 <$(top_srcdir)/utils/common/v4l2-tpg.patch
> -       cp -a $(KERNEL_DIR)/drivers/media/test-drivers/vicodec/codec-fwht.[ch] $(top_srcdir)/utils/common/
> -       cp -a $(KERNEL_DIR)/drivers/media/test-drivers/vicodec/codec-v4l2-fwht.[ch] $(top_srcdir)/utils/common/
> -       patch -d $(top_srcdir) --no-backup-if-mismatch -p1 <$(top_srcdir)/utils/common/codec-fwht.patch
> -       grep V4L2_.*_FMT.*descr $(KERNEL_DIR)/drivers/media/v4l2-core/v4l2-ioctl.c | grep -v V4L2_PIX_FMT_H264_SLICE | grep -v V4L2_PIX_FMT_VP8_FRAME | grep -v V4L2_PIX_FMT_HEVC_SLICE | perl -pe 's/.*V4L2_(.*)_FMT/\tcase V4L2_\1_FMT/; s/:.*descr = /: return /; s/;.*/;/;' >$(top_srcdir)/utils/common/v4l2-pix-formats.h
> -
> -       $(MAKE) -C utils/keytable $@
> -       $(MAKE) -C lib/libdvbv5 $@
> -       $(MAKE) -C contrib/freebsd $@
> -       $(MAKE) -C contrib/test $@
> -       $(MAKE) -C contrib/xc3028-firmware $@
> diff --git a/README b/README
> index e344dee1..48b33713 100644
> --- a/README
> +++ b/README
> @@ -264,11 +264,11 @@ Be sure that you have installed both glibc development packages for
>
>  step 3) at v4l-utils tree:
>
> -       $ export KERNEL_DIR=location/of/the/kernel/tree make sync-with-kernel
> +       $ export KERNEL_DIR=location/of/the/kernel/tree ./sync-with-kernel.sh
>
>  Alternatively, steps 1 to 3 can be replaced with:
>
> -export KERNEL_DIR=location/of/the/kernel/tree && (cd $KERNEL_DIR && make headers_install INSTALL_HDR_PATH=usr/) && make sync-with-kernel
> +export KERNEL_DIR=location/of/the/kernel/tree && (cd $KERNEL_DIR && make headers_install INSTALL_HDR_PATH=usr/) && ./sync-with-kernel.sh
>
>  step 4)
>
> diff --git a/contrib/freebsd/Makefile.am b/contrib/freebsd/Makefile.am
> index ca322749..abf0d57b 100644
> --- a/contrib/freebsd/Makefile.am
> +++ b/contrib/freebsd/Makefile.am
> @@ -1,17 +1,3 @@
>  EXTRA_DIST = \
>         include \
>         patches
> -
> -sync-with-kernel:
> -       rm -rf .pc
> -
> -       for i in input.h input-event-codes.h ivtv.h uinput.h videodev2.h v4l2-controls.h v4l2-common.h; do \
> -           mkdir -p include/linux/$$(dirname $$i); \
> -           cp $(KERNEL_DIR)/usr/include/linux/$$i include/linux/$$i; \
> -       done
> -
> -       for i in ivtv.h uinput.h videodev2.h v4l2-common.h; do \
> -           sed -e 's/__u8/uint8_t/g' -e 's/__u16/uint16_t/g' -e 's/__u32/uint32_t/g' -e 's/__u64/uint64_t/g' -e 's/__s8/int8_t/g' -e 's/__s16/int16_t/g' -e 's/__s32/int32_t/g' -e 's/__s64/int64_t/g' -e 's/__le32/uint32_t/g' -e 's/__user//g' -i include/linux/$$i; \
> -       done
> -
> -       quilt push -a
> diff --git a/contrib/test/Makefile.am b/contrib/test/Makefile.am
> index ded3c1f4..6650cd27 100644
> --- a/contrib/test/Makefile.am
> +++ b/contrib/test/Makefile.am
> @@ -57,15 +57,6 @@ capture_example_SOURCES = capture-example.c
>
>  ioctl-test.c: ioctl-test.h
>
> -sync-with-kernel:
> -       ./gen_ioctl_list.pl --gen_ioctl_numbers >.tmp_ioctl.c
> -       gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl32 -m32
> -       gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl64 -m64
> -       ./tmp_ioctl32 32 >ioctl_32.h
> -       ./tmp_ioctl64 64 >ioctl_64.h
> -       rm .tmp_ioctl.c tmp_ioctl32 tmp_ioctl64
> -       ./gen_ioctl_list.pl >ioctl-test.h
> -
>  EXTRA_DIST = \
>         gen_ioctl_list.pl \
>         test-media \
> diff --git a/contrib/test/gen_ioctl_list.pl b/contrib/test/gen_ioctl_list.pl
> index fca51778..82168b39 100755
> --- a/contrib/test/gen_ioctl_list.pl
> +++ b/contrib/test/gen_ioctl_list.pl
> @@ -69,7 +69,7 @@ sub print_ioc()
>         printf "#define S_IOCTLS sizeof(ioctls)/sizeof(ioctls[0])\n";
>  }
>
> -printf "/* This file is auto-generated by make sync-with-kernel */\n";
> +printf "/* This file is auto-generated by sync-with-kernel.sh */\n";
>  printf "#include <linux/compiler.h>\n";
>
>  foreach my $h (sort keys %headers) {
> diff --git a/contrib/test/ioctl-test.h b/contrib/test/ioctl-test.h
> index 32084c52..03b53078 100644
> --- a/contrib/test/ioctl-test.h
> +++ b/contrib/test/ioctl-test.h
> @@ -1,4 +1,4 @@
> -/* This file is auto-generated by make sync-with-kernel */
> +/* This file is auto-generated by sync-with-kernel.sh */
>  #include <linux/compiler.h>
>  #include "linux/dvb/dmx.h"
>  #include "linux/dvb/frontend.h"
> diff --git a/contrib/xc3028-firmware/Makefile.am b/contrib/xc3028-firmware/Makefile.am
> index 43ff8b12..8b23d1e1 100644
> --- a/contrib/xc3028-firmware/Makefile.am
> +++ b/contrib/xc3028-firmware/Makefile.am
> @@ -3,12 +3,3 @@ noinst_PROGRAMS = firmware-tool
>  firmware_tool_SOURCES = firmware-tool.c standards.c extract_head.h standards.h tuner-xc2028-types.h
>
>  EXTRA_DIST = README
> -
> -# custom targets
> -
> -sync-with-kernel:
> -       @if [ ! -f $(KERNEL_DIR)/drivers/media/tuners/tuner-xc2028-types.h ]; then \
> -         echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"; \
> -         exit 1; \
> -       fi

Note that sync-with-kernel.sh script is not checking for the above file anymore.

> -       cp -a $(KERNEL_DIR)/drivers/media/tuners/tuner-xc2028-types.h $(srcdir)/
> diff --git a/lib/libdvbv5/Makefile.am b/lib/libdvbv5/Makefile.am
> index e65066e9..09c261be 100644
> --- a/lib/libdvbv5/Makefile.am
> +++ b/lib/libdvbv5/Makefile.am
> @@ -122,6 +122,3 @@ libdvbv5_la_LDFLAGS = $(LIBDVBV5_VERSION) $(ENFORCE_LIBDVBV5_STATIC) $(LIBUDEV_L
>  libdvbv5_la_LIBADD = $(LTLIBICONV)
>
>  EXTRA_DIST = README gen_dvb_structs.pl
> -
> -sync-with-kernel:
> -       ./gen_dvb_structs.pl $(KERNEL_DIR)/usr/include/
> diff --git a/sync-with-kernel.sh b/sync-with-kernel.sh
> new file mode 100755
> index 00000000..ab15cb8a
> --- /dev/null
> +++ b/sync-with-kernel.sh
> @@ -0,0 +1,113 @@
> +#!/bin/bash
> +
> +TOPSRCDIR="$( cd "$( dirname "$0" )" && pwd )"
> +
> +if [ -z "${KERNEL_DIR}" -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/videodev2.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/fb.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-controls.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-common.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-subdev.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-mediabus.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/ivtv.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/dvb/frontend.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/dvb/dmx.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/lirc.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/bpf.h -o \
> +     ! -f ${KERNEL_DIR}/usr/include/linux/bpf_common.h ]; then
> +  echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"
> +  echo "and run 'make headers_install' in \$KERNEL_DIR."
> +  exit 1
> +fi
> +
> +cp -a ${KERNEL_DIR}/usr/include/linux/videodev2.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/fb.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-controls.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-common.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-subdev.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-mediabus.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/media-bus-format.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/media.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/ivtv.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/dvb/frontend.h ${TOPSRCDIR}/include/linux/dvb
> +cp ${TOPSRCDIR}/include/linux/dvb/frontend.h ${TOPSRCDIR}/lib/include/libdvbv5/dvb-frontend.h
> +cp -a ${KERNEL_DIR}/usr/include/linux/dvb/dmx.h ${TOPSRCDIR}/include/linux/dvb
> +cp -a ${KERNEL_DIR}/usr/include/linux/lirc.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/bpf.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/bpf_common.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/cec.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/usr/include/linux/cec-funcs.h ${TOPSRCDIR}/include/linux
> +cp -a ${KERNEL_DIR}/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c ${TOPSRCDIR}/utils/common
> +cp -a ${KERNEL_DIR}/drivers/media/common/v4l2-tpg/v4l2-tpg-colors.c ${TOPSRCDIR}/utils/common
> +cp -a ${KERNEL_DIR}/include/media/tpg/v4l2-tpg.h ${TOPSRCDIR}/utils/common
> +cp -a ${KERNEL_DIR}/include/media/fwht-ctrls.h ${TOPSRCDIR}/utils/common
> +patch -d ${TOPSRCDIR} --no-backup-if-mismatch -p0 <${TOPSRCDIR}/utils/common/v4l2-tpg.patch
> +cp -a ${KERNEL_DIR}/drivers/media/test-drivers/vicodec/codec-fwht.[ch] ${TOPSRCDIR}/utils/common/
> +cp -a ${KERNEL_DIR}/drivers/media/test-drivers/vicodec/codec-v4l2-fwht.[ch] ${TOPSRCDIR}/utils/common/
> +patch -d ${TOPSRCDIR} --no-backup-if-mismatch -p1 <${TOPSRCDIR}/utils/common/codec-fwht.patch
> +grep V4L2_.*_FMT.*descr ${KERNEL_DIR}/drivers/media/v4l2-core/v4l2-ioctl.c | grep -v V4L2_PIX_FMT_H264_SLICE | grep -v V4L2_PIX_FMT_VP8_FRAME | grep -v V4L2_PIX_FMT_HEVC_SLICE | perl -pe 's/.*V4L2_(.*)_FMT/\tcase V4L2_\1_FMT/; s/:.*descr = /: return /; s/;.*/;/;' >${TOPSRCDIR}/utils/common/v4l2-pix-formats.h
> +
> +function keytable {
> +       SRCDIR=${TOPSRCDIR}/utils/keytable
> +
> +       cd ${SRCDIR}
> +       echo generating ${SRCDIR}/parse.h
> +       ./gen_input_events.pl < ${KERNEL_DIR}/usr/include/linux/input-event-codes.h  > ${SRCDIR}/parse.h
> +       mkdir -p ${SRCDIR}/rc_keymaps
> +       rm -f ${SRCDIR}/rc_keymaps/*
> +       echo storing existing keymaps at ${SRCDIR}/rc_keymaps/
> +       ./gen_keytables.pl ${KERNEL_DIR};
> +       cp ${SRCDIR}/rc_keymaps_userspace/* ${SRCDIR}/rc_keymaps/
> +}
> +
> +function libdvbv5 {
> +       SRCDIR=${TOPSRCDIR}/lib/libdvbv5
> +
> +       cd ${SRCDIR}
> +       ./gen_dvb_structs.pl ${KERNEL_DIR}/usr/include/
> +}
> +
> +function freebsd {
> +       SRCDIR=${TOPSRCDIR}/contrib/freebsd
> +
> +       cp -a ${KERNEL_DIR}/drivers/media/tuners/tuner-xc2028-types.h ${SRCDIR}/
> +}
> +
> +function ioctl-test {
> +       SRCDIR=${TOPSRCDIR}/contrib/test
> +
> +       cd ${SRCDIR}
> +
> +       ./gen_ioctl_list.pl --gen_ioctl_numbers >.tmp_ioctl.c
> +       gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl32 -m32
> +       gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl64 -m64
> +       ./tmp_ioctl32 32 >ioctl_32.h
> +       ./tmp_ioctl64 64 >ioctl_64.h
> +       rm .tmp_ioctl.c tmp_ioctl32 tmp_ioctl64
> +       ./gen_ioctl_list.pl >ioctl-test.h
> +}
> +
> +function xc3028-firmware {
> +       SRCDIR=${TOPSRCDIR}/contrib/xc3028-firmware
> +
> +       cd ${SRCDIR}
> +
> +       rm -rf .pc
> +
> +       for i in input.h input-event-codes.h ivtv.h uinput.h videodev2.h v4l2-controls.h v4l2-common.h; do
> +           mkdir -p include/linux/$(dirname $i)
> +           cp ${KERNEL_DIR}/usr/include/linux/$i include/linux/$i
> +       done
> +
> +       for i in ivtv.h uinput.h videodev2.h v4l2-common.h; do
> +           sed -e 's/__u8/uint8_t/g' -e 's/__u16/uint16_t/g' -e 's/__u32/uint32_t/g' -e 's/__u64/uint64_t/g' -e 's/__s8/int8_t/g' -e 's/__s16/int16_t/g' -e 's/__s32/int32_t/g' -e 's/__s64/int64_t/g' -e 's/__le32/uint32_t/g' -e 's/__user//g' -i include/linux/$i
> +       done
> +
> +       quilt push -a
> +}

Functions freebsd and xc3028-firmware seem to be swapped.

> +
> +keytable
> +libdvbv5
> +freebsd
> +ioctl-test
> +xc3028-firmware

In the above script, I'd choose using spaces or tabs for indentation.
There's a mixing of both right now.

> diff --git a/utils/keytable/Makefile.am b/utils/keytable/Makefile.am
> index 4724897e..a057eb89 100644
> --- a/utils/keytable/Makefile.am
> +++ b/utils/keytable/Makefile.am
> @@ -29,16 +29,3 @@ check:
>  # custom target
>  install-data-local:
>         $(install_sh) -d "$(DESTDIR)$(keytableuserdir)"
> -
> -sync-with-kernel:
> -       @if [ ! -f $(KERNEL_DIR)/usr/include/linux/input-event-codes.h ]; then \
> -         echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"; \
> -         exit 1; \
> -       fi

Same as before, sync-with-kernel.sh script is not checking for the
above file anymore.

> -       @echo generating $(srcdir)/parse.h
> -       @./gen_input_events.pl < $(KERNEL_DIR)/usr/include/linux/input-event-codes.h  > $(srcdir)/parse.h
> -       @-mkdir -p $(srcdir)/rc_keymaps
> -       @-rm $(srcdir)/rc_keymaps/*
> -       @echo storing existing keymaps at $(srcdir)/rc_keymaps/
> -       @cd $(srcdir) && ./gen_keytables.pl $(KERNEL_DIR);
> -       @-cp $(srcdir)/rc_keymaps_userspace/* $(srcdir)/rc_keymaps/
> --
> 2.26.2
>

Being pedantic, there are a few more occurrences of `make
sync-with-kernel` that should be changed to `sync-with-kernel.sh`.

$ git grep "sync-with-kernel[^\.]" -- ':^ChangeLog'
    configure.ac:# sync-with-kernel and keytable install could make use of them
    contrib/test/gen_ioctl_list.pl: printf "\tprintf(\"/* This file is
auto-generated by make sync-with-kernel */\\n\\n\");\n";
    contrib/test/ioctl_32.h:/* This file is auto-generated by make
sync-with-kernel */
    contrib/test/ioctl_64.h:/* This file is auto-generated by make
sync-with-kernel */

Thanks again!
Ariel D'Alessandro

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v4l-utils] Move sync-with-kernel into dedicated shell script ./sync-with-kernel.sh
  2020-06-22 17:37 ` Ariel D'Alessandro
@ 2020-06-22 18:58   ` Sean Young
  0 siblings, 0 replies; 3+ messages in thread
From: Sean Young @ 2020-06-22 18:58 UTC (permalink / raw)
  To: Ariel D'Alessandro
  Cc: linux-media, Hans Verkuil, Philipp Zabel, Laurent Pinchart,
	Ezequiel Garcia, Nicolas Dufresne, Kieran Bingham, Gregor Jasny

Hi Ariel,

On Mon, Jun 22, 2020 at 02:37:11PM -0300, Ariel D'Alessandro wrote:
> Hi Sean,
> 
> Great! Thanks for picking this up so quickly. A few comments below.

Thank you for doing a thorough review. I'll fix all the issues you found
and send a v2.


Sean

> 
> On Sat, Jun 20, 2020 at 7:08 AM Sean Young <sean@mess.org> wrote:
> >
> > The sync-with-kernel target is a collection of shell fragments scattered
> > over the tree. Collect all of them into a single shell script. This makes
> > it is easier to see what sync-with-kernel does, and makes the move away
> > from autotools to meson easier.
> >
> > Signed-off-by: Sean Young <sean@mess.org>
> > ---
> >  Makefile.am                         |  59 +--------------
> >  README                              |   4 +-
> >  contrib/freebsd/Makefile.am         |  14 ----
> >  contrib/test/Makefile.am            |   9 ---
> >  contrib/test/gen_ioctl_list.pl      |   2 +-
> >  contrib/test/ioctl-test.h           |   2 +-
> >  contrib/xc3028-firmware/Makefile.am |   9 ---
> >  lib/libdvbv5/Makefile.am            |   3 -
> >  sync-with-kernel.sh                 | 113 ++++++++++++++++++++++++++++
> >  utils/keytable/Makefile.am          |  13 ----
> >  10 files changed, 121 insertions(+), 107 deletions(-)
> >  create mode 100755 sync-with-kernel.sh
> >
> > diff --git a/Makefile.am b/Makefile.am
> > index 3c3d3ce6..3c0ceb49 100644
> > --- a/Makefile.am
> > +++ b/Makefile.am
> > @@ -7,60 +7,9 @@ if WITH_V4LUTILS
> >  SUBDIRS += utils contrib
> >  endif
> >
> > -EXTRA_DIST = android-config.h bootstrap.sh doxygen_libdvbv5.cfg include COPYING.libv4l \
> > -        COPYING.libdvbv5 README.libv4l README.lib-multi-threading TODO.libdvbv5 \
> > -        doc/libdvbv5-index.doc
> > +EXTRA_DIST = android-config.h sync-with-kernel.sh bootstrap.sh \
> > +       doxygen_libdvbv5.cfg include COPYING.libv4l \
> > +       COPYING.libdvbv5 README.libv4l README.lib-multi-threading \
> > +       TODO.libdvbv5 doc/libdvbv5-index.doc
> >
> >  include $(top_srcdir)/aminclude.am
> > -
> > -# custom targets
> > -
> > -sync-with-kernel:
> > -       @if [ ! -f $(KERNEL_DIR)/usr/include/linux/videodev2.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/fb.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-controls.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-common.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-subdev.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/v4l2-mediabus.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/ivtv.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/dvb/frontend.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/dvb/dmx.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/lirc.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/bpf.h -o \
> > -             ! -f $(KERNEL_DIR)/usr/include/linux/bpf_common.h ]; then \
> > -         echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"; \
> > -         echo "and run 'make headers_install' in \$$KERNEL_DIR."; \
> > -         exit 1; \
> > -       fi
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/videodev2.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/fb.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-controls.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-common.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-subdev.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/v4l2-mediabus.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/media-bus-format.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/media.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/ivtv.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/dvb/frontend.h $(top_srcdir)/include/linux/dvb
> > -       cp $(top_srcdir)/include/linux/dvb/frontend.h $(top_srcdir)/lib/include/libdvbv5/dvb-frontend.h
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/dvb/dmx.h $(top_srcdir)/include/linux/dvb
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/lirc.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/bpf.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/bpf_common.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/cec.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/usr/include/linux/cec-funcs.h $(top_srcdir)/include/linux
> > -       cp -a $(KERNEL_DIR)/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c $(top_srcdir)/utils/common
> > -       cp -a $(KERNEL_DIR)/drivers/media/common/v4l2-tpg/v4l2-tpg-colors.c $(top_srcdir)/utils/common
> > -       cp -a $(KERNEL_DIR)/include/media/tpg/v4l2-tpg.h $(top_srcdir)/utils/common
> > -       cp -a $(KERNEL_DIR)/include/media/fwht-ctrls.h $(top_srcdir)/utils/common
> > -       patch -d $(top_srcdir) --no-backup-if-mismatch -p0 <$(top_srcdir)/utils/common/v4l2-tpg.patch
> > -       cp -a $(KERNEL_DIR)/drivers/media/test-drivers/vicodec/codec-fwht.[ch] $(top_srcdir)/utils/common/
> > -       cp -a $(KERNEL_DIR)/drivers/media/test-drivers/vicodec/codec-v4l2-fwht.[ch] $(top_srcdir)/utils/common/
> > -       patch -d $(top_srcdir) --no-backup-if-mismatch -p1 <$(top_srcdir)/utils/common/codec-fwht.patch
> > -       grep V4L2_.*_FMT.*descr $(KERNEL_DIR)/drivers/media/v4l2-core/v4l2-ioctl.c | grep -v V4L2_PIX_FMT_H264_SLICE | grep -v V4L2_PIX_FMT_VP8_FRAME | grep -v V4L2_PIX_FMT_HEVC_SLICE | perl -pe 's/.*V4L2_(.*)_FMT/\tcase V4L2_\1_FMT/; s/:.*descr = /: return /; s/;.*/;/;' >$(top_srcdir)/utils/common/v4l2-pix-formats.h
> > -
> > -       $(MAKE) -C utils/keytable $@
> > -       $(MAKE) -C lib/libdvbv5 $@
> > -       $(MAKE) -C contrib/freebsd $@
> > -       $(MAKE) -C contrib/test $@
> > -       $(MAKE) -C contrib/xc3028-firmware $@
> > diff --git a/README b/README
> > index e344dee1..48b33713 100644
> > --- a/README
> > +++ b/README
> > @@ -264,11 +264,11 @@ Be sure that you have installed both glibc development packages for
> >
> >  step 3) at v4l-utils tree:
> >
> > -       $ export KERNEL_DIR=location/of/the/kernel/tree make sync-with-kernel
> > +       $ export KERNEL_DIR=location/of/the/kernel/tree ./sync-with-kernel.sh
> >
> >  Alternatively, steps 1 to 3 can be replaced with:
> >
> > -export KERNEL_DIR=location/of/the/kernel/tree && (cd $KERNEL_DIR && make headers_install INSTALL_HDR_PATH=usr/) && make sync-with-kernel
> > +export KERNEL_DIR=location/of/the/kernel/tree && (cd $KERNEL_DIR && make headers_install INSTALL_HDR_PATH=usr/) && ./sync-with-kernel.sh
> >
> >  step 4)
> >
> > diff --git a/contrib/freebsd/Makefile.am b/contrib/freebsd/Makefile.am
> > index ca322749..abf0d57b 100644
> > --- a/contrib/freebsd/Makefile.am
> > +++ b/contrib/freebsd/Makefile.am
> > @@ -1,17 +1,3 @@
> >  EXTRA_DIST = \
> >         include \
> >         patches
> > -
> > -sync-with-kernel:
> > -       rm -rf .pc
> > -
> > -       for i in input.h input-event-codes.h ivtv.h uinput.h videodev2.h v4l2-controls.h v4l2-common.h; do \
> > -           mkdir -p include/linux/$$(dirname $$i); \
> > -           cp $(KERNEL_DIR)/usr/include/linux/$$i include/linux/$$i; \
> > -       done
> > -
> > -       for i in ivtv.h uinput.h videodev2.h v4l2-common.h; do \
> > -           sed -e 's/__u8/uint8_t/g' -e 's/__u16/uint16_t/g' -e 's/__u32/uint32_t/g' -e 's/__u64/uint64_t/g' -e 's/__s8/int8_t/g' -e 's/__s16/int16_t/g' -e 's/__s32/int32_t/g' -e 's/__s64/int64_t/g' -e 's/__le32/uint32_t/g' -e 's/__user//g' -i include/linux/$$i; \
> > -       done
> > -
> > -       quilt push -a
> > diff --git a/contrib/test/Makefile.am b/contrib/test/Makefile.am
> > index ded3c1f4..6650cd27 100644
> > --- a/contrib/test/Makefile.am
> > +++ b/contrib/test/Makefile.am
> > @@ -57,15 +57,6 @@ capture_example_SOURCES = capture-example.c
> >
> >  ioctl-test.c: ioctl-test.h
> >
> > -sync-with-kernel:
> > -       ./gen_ioctl_list.pl --gen_ioctl_numbers >.tmp_ioctl.c
> > -       gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl32 -m32
> > -       gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl64 -m64
> > -       ./tmp_ioctl32 32 >ioctl_32.h
> > -       ./tmp_ioctl64 64 >ioctl_64.h
> > -       rm .tmp_ioctl.c tmp_ioctl32 tmp_ioctl64
> > -       ./gen_ioctl_list.pl >ioctl-test.h
> > -
> >  EXTRA_DIST = \
> >         gen_ioctl_list.pl \
> >         test-media \
> > diff --git a/contrib/test/gen_ioctl_list.pl b/contrib/test/gen_ioctl_list.pl
> > index fca51778..82168b39 100755
> > --- a/contrib/test/gen_ioctl_list.pl
> > +++ b/contrib/test/gen_ioctl_list.pl
> > @@ -69,7 +69,7 @@ sub print_ioc()
> >         printf "#define S_IOCTLS sizeof(ioctls)/sizeof(ioctls[0])\n";
> >  }
> >
> > -printf "/* This file is auto-generated by make sync-with-kernel */\n";
> > +printf "/* This file is auto-generated by sync-with-kernel.sh */\n";
> >  printf "#include <linux/compiler.h>\n";
> >
> >  foreach my $h (sort keys %headers) {
> > diff --git a/contrib/test/ioctl-test.h b/contrib/test/ioctl-test.h
> > index 32084c52..03b53078 100644
> > --- a/contrib/test/ioctl-test.h
> > +++ b/contrib/test/ioctl-test.h
> > @@ -1,4 +1,4 @@
> > -/* This file is auto-generated by make sync-with-kernel */
> > +/* This file is auto-generated by sync-with-kernel.sh */
> >  #include <linux/compiler.h>
> >  #include "linux/dvb/dmx.h"
> >  #include "linux/dvb/frontend.h"
> > diff --git a/contrib/xc3028-firmware/Makefile.am b/contrib/xc3028-firmware/Makefile.am
> > index 43ff8b12..8b23d1e1 100644
> > --- a/contrib/xc3028-firmware/Makefile.am
> > +++ b/contrib/xc3028-firmware/Makefile.am
> > @@ -3,12 +3,3 @@ noinst_PROGRAMS = firmware-tool
> >  firmware_tool_SOURCES = firmware-tool.c standards.c extract_head.h standards.h tuner-xc2028-types.h
> >
> >  EXTRA_DIST = README
> > -
> > -# custom targets
> > -
> > -sync-with-kernel:
> > -       @if [ ! -f $(KERNEL_DIR)/drivers/media/tuners/tuner-xc2028-types.h ]; then \
> > -         echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"; \
> > -         exit 1; \
> > -       fi
> 
> Note that sync-with-kernel.sh script is not checking for the above file anymore.
> 
> > -       cp -a $(KERNEL_DIR)/drivers/media/tuners/tuner-xc2028-types.h $(srcdir)/
> > diff --git a/lib/libdvbv5/Makefile.am b/lib/libdvbv5/Makefile.am
> > index e65066e9..09c261be 100644
> > --- a/lib/libdvbv5/Makefile.am
> > +++ b/lib/libdvbv5/Makefile.am
> > @@ -122,6 +122,3 @@ libdvbv5_la_LDFLAGS = $(LIBDVBV5_VERSION) $(ENFORCE_LIBDVBV5_STATIC) $(LIBUDEV_L
> >  libdvbv5_la_LIBADD = $(LTLIBICONV)
> >
> >  EXTRA_DIST = README gen_dvb_structs.pl
> > -
> > -sync-with-kernel:
> > -       ./gen_dvb_structs.pl $(KERNEL_DIR)/usr/include/
> > diff --git a/sync-with-kernel.sh b/sync-with-kernel.sh
> > new file mode 100755
> > index 00000000..ab15cb8a
> > --- /dev/null
> > +++ b/sync-with-kernel.sh
> > @@ -0,0 +1,113 @@
> > +#!/bin/bash
> > +
> > +TOPSRCDIR="$( cd "$( dirname "$0" )" && pwd )"
> > +
> > +if [ -z "${KERNEL_DIR}" -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/videodev2.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/fb.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-controls.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-common.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-subdev.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/v4l2-mediabus.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/ivtv.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/dvb/frontend.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/dvb/dmx.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/lirc.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/bpf.h -o \
> > +     ! -f ${KERNEL_DIR}/usr/include/linux/bpf_common.h ]; then
> > +  echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"
> > +  echo "and run 'make headers_install' in \$KERNEL_DIR."
> > +  exit 1
> > +fi
> > +
> > +cp -a ${KERNEL_DIR}/usr/include/linux/videodev2.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/fb.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-controls.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-common.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-subdev.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/v4l2-mediabus.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/media-bus-format.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/media.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/ivtv.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/dvb/frontend.h ${TOPSRCDIR}/include/linux/dvb
> > +cp ${TOPSRCDIR}/include/linux/dvb/frontend.h ${TOPSRCDIR}/lib/include/libdvbv5/dvb-frontend.h
> > +cp -a ${KERNEL_DIR}/usr/include/linux/dvb/dmx.h ${TOPSRCDIR}/include/linux/dvb
> > +cp -a ${KERNEL_DIR}/usr/include/linux/lirc.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/bpf.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/bpf_common.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/cec.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/usr/include/linux/cec-funcs.h ${TOPSRCDIR}/include/linux
> > +cp -a ${KERNEL_DIR}/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c ${TOPSRCDIR}/utils/common
> > +cp -a ${KERNEL_DIR}/drivers/media/common/v4l2-tpg/v4l2-tpg-colors.c ${TOPSRCDIR}/utils/common
> > +cp -a ${KERNEL_DIR}/include/media/tpg/v4l2-tpg.h ${TOPSRCDIR}/utils/common
> > +cp -a ${KERNEL_DIR}/include/media/fwht-ctrls.h ${TOPSRCDIR}/utils/common
> > +patch -d ${TOPSRCDIR} --no-backup-if-mismatch -p0 <${TOPSRCDIR}/utils/common/v4l2-tpg.patch
> > +cp -a ${KERNEL_DIR}/drivers/media/test-drivers/vicodec/codec-fwht.[ch] ${TOPSRCDIR}/utils/common/
> > +cp -a ${KERNEL_DIR}/drivers/media/test-drivers/vicodec/codec-v4l2-fwht.[ch] ${TOPSRCDIR}/utils/common/
> > +patch -d ${TOPSRCDIR} --no-backup-if-mismatch -p1 <${TOPSRCDIR}/utils/common/codec-fwht.patch
> > +grep V4L2_.*_FMT.*descr ${KERNEL_DIR}/drivers/media/v4l2-core/v4l2-ioctl.c | grep -v V4L2_PIX_FMT_H264_SLICE | grep -v V4L2_PIX_FMT_VP8_FRAME | grep -v V4L2_PIX_FMT_HEVC_SLICE | perl -pe 's/.*V4L2_(.*)_FMT/\tcase V4L2_\1_FMT/; s/:.*descr = /: return /; s/;.*/;/;' >${TOPSRCDIR}/utils/common/v4l2-pix-formats.h
> > +
> > +function keytable {
> > +       SRCDIR=${TOPSRCDIR}/utils/keytable
> > +
> > +       cd ${SRCDIR}
> > +       echo generating ${SRCDIR}/parse.h
> > +       ./gen_input_events.pl < ${KERNEL_DIR}/usr/include/linux/input-event-codes.h  > ${SRCDIR}/parse.h
> > +       mkdir -p ${SRCDIR}/rc_keymaps
> > +       rm -f ${SRCDIR}/rc_keymaps/*
> > +       echo storing existing keymaps at ${SRCDIR}/rc_keymaps/
> > +       ./gen_keytables.pl ${KERNEL_DIR};
> > +       cp ${SRCDIR}/rc_keymaps_userspace/* ${SRCDIR}/rc_keymaps/
> > +}
> > +
> > +function libdvbv5 {
> > +       SRCDIR=${TOPSRCDIR}/lib/libdvbv5
> > +
> > +       cd ${SRCDIR}
> > +       ./gen_dvb_structs.pl ${KERNEL_DIR}/usr/include/
> > +}
> > +
> > +function freebsd {
> > +       SRCDIR=${TOPSRCDIR}/contrib/freebsd
> > +
> > +       cp -a ${KERNEL_DIR}/drivers/media/tuners/tuner-xc2028-types.h ${SRCDIR}/
> > +}
> > +
> > +function ioctl-test {
> > +       SRCDIR=${TOPSRCDIR}/contrib/test
> > +
> > +       cd ${SRCDIR}
> > +
> > +       ./gen_ioctl_list.pl --gen_ioctl_numbers >.tmp_ioctl.c
> > +       gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl32 -m32
> > +       gcc -I ../../include/ .tmp_ioctl.c -o tmp_ioctl64 -m64
> > +       ./tmp_ioctl32 32 >ioctl_32.h
> > +       ./tmp_ioctl64 64 >ioctl_64.h
> > +       rm .tmp_ioctl.c tmp_ioctl32 tmp_ioctl64
> > +       ./gen_ioctl_list.pl >ioctl-test.h
> > +}
> > +
> > +function xc3028-firmware {
> > +       SRCDIR=${TOPSRCDIR}/contrib/xc3028-firmware
> > +
> > +       cd ${SRCDIR}
> > +
> > +       rm -rf .pc
> > +
> > +       for i in input.h input-event-codes.h ivtv.h uinput.h videodev2.h v4l2-controls.h v4l2-common.h; do
> > +           mkdir -p include/linux/$(dirname $i)
> > +           cp ${KERNEL_DIR}/usr/include/linux/$i include/linux/$i
> > +       done
> > +
> > +       for i in ivtv.h uinput.h videodev2.h v4l2-common.h; do
> > +           sed -e 's/__u8/uint8_t/g' -e 's/__u16/uint16_t/g' -e 's/__u32/uint32_t/g' -e 's/__u64/uint64_t/g' -e 's/__s8/int8_t/g' -e 's/__s16/int16_t/g' -e 's/__s32/int32_t/g' -e 's/__s64/int64_t/g' -e 's/__le32/uint32_t/g' -e 's/__user//g' -i include/linux/$i
> > +       done
> > +
> > +       quilt push -a
> > +}
> 
> Functions freebsd and xc3028-firmware seem to be swapped.
> 
> > +
> > +keytable
> > +libdvbv5
> > +freebsd
> > +ioctl-test
> > +xc3028-firmware
> 
> In the above script, I'd choose using spaces or tabs for indentation.
> There's a mixing of both right now.
> 
> > diff --git a/utils/keytable/Makefile.am b/utils/keytable/Makefile.am
> > index 4724897e..a057eb89 100644
> > --- a/utils/keytable/Makefile.am
> > +++ b/utils/keytable/Makefile.am
> > @@ -29,16 +29,3 @@ check:
> >  # custom target
> >  install-data-local:
> >         $(install_sh) -d "$(DESTDIR)$(keytableuserdir)"
> > -
> > -sync-with-kernel:
> > -       @if [ ! -f $(KERNEL_DIR)/usr/include/linux/input-event-codes.h ]; then \
> > -         echo "Error you must set KERNEL_DIR to point to an extracted kernel source dir"; \
> > -         exit 1; \
> > -       fi
> 
> Same as before, sync-with-kernel.sh script is not checking for the
> above file anymore.
> 
> > -       @echo generating $(srcdir)/parse.h
> > -       @./gen_input_events.pl < $(KERNEL_DIR)/usr/include/linux/input-event-codes.h  > $(srcdir)/parse.h
> > -       @-mkdir -p $(srcdir)/rc_keymaps
> > -       @-rm $(srcdir)/rc_keymaps/*
> > -       @echo storing existing keymaps at $(srcdir)/rc_keymaps/
> > -       @cd $(srcdir) && ./gen_keytables.pl $(KERNEL_DIR);
> > -       @-cp $(srcdir)/rc_keymaps_userspace/* $(srcdir)/rc_keymaps/
> > --
> > 2.26.2
> >
> 
> Being pedantic, there are a few more occurrences of `make
> sync-with-kernel` that should be changed to `sync-with-kernel.sh`.
> 
> $ git grep "sync-with-kernel[^\.]" -- ':^ChangeLog'
>     configure.ac:# sync-with-kernel and keytable install could make use of them
>     contrib/test/gen_ioctl_list.pl: printf "\tprintf(\"/* This file is
> auto-generated by make sync-with-kernel */\\n\\n\");\n";
>     contrib/test/ioctl_32.h:/* This file is auto-generated by make
> sync-with-kernel */
>     contrib/test/ioctl_64.h:/* This file is auto-generated by make
> sync-with-kernel */
> 
> Thanks again!
> Ariel D'Alessandro

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-06-22 18:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-20 10:08 [PATCH v4l-utils] Move sync-with-kernel into dedicated shell script ./sync-with-kernel.sh Sean Young
2020-06-22 17:37 ` Ariel D'Alessandro
2020-06-22 18:58   ` Sean Young

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.