[ptxdist] [PATCH v2 1/3] host-gengetopt: add package

[ptxdist] [PATCH v2 1/3] host-gengetopt: add package

[Marian Cichy]

this package is a dependency for bayer2rgb3,
which is a dependency for gst-bayer2rgb-neon
---
 configs/ptxconfig         |  1 +
 rules/host-gengetopt.in   |  9 +++++++++
 rules/host-gengetopt.make | 26 ++++++++++++++++++++++++++
 3 files changed, 36 insertions(+)
 create mode 100644 rules/host-gengetopt.in
 create mode 100644 rules/host-gengetopt.make

diff --git a/configs/ptxconfig b/configs/ptxconfig
index 972e7615..67200af3 100644
--- a/configs/ptxconfig
+++ b/configs/ptxconfig
@@ -105,6 +105,7 @@ PTXCONF__ptxconfig_MAGIC__=y
 PTXCONF_HOST_BSDIFF=y
 # PTXCONF_HOST_DFU_UTIL is not set
 PTXCONF_HOST_FILE_STACK=y
+# PTXCONF_HOST_GENGETOPT is not set
 # PTXCONF_HOST_OPENOCD is not set
 # PTXCONF_HOST_QEMU is not set
 # PTXCONF_HOST_TESTFRAMEWORK is not set
diff --git a/rules/host-gengetopt.in b/rules/host-gengetopt.in
new file mode 100644
index 00000000..f6855ab5
--- /dev/null
+++ b/rules/host-gengetopt.in
@@ -0,0 +1,9 @@
+## SECTION=hosttools
+
+config HOST_GENGETOPT
+	tristate
+	prompt "gengetopt"
+	help
+	  Gengetopt is a tool to write command line option
+	  parsing code for C programs.
+
diff --git a/rules/host-gengetopt.make b/rules/host-gengetopt.make
new file mode 100644
index 00000000..2c1b3bb0
--- /dev/null
+++ b/rules/host-gengetopt.make
@@ -0,0 +1,26 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2020 by Marian Cichy <m.cichy@pengutronix.de>
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+#
+# We provide this package
+#
+HOST_PACKAGES-$(PTXCONF_HOST_GENGETOPT) += host-gengetopt
+
+#
+# Paths and names
+#
+HOST_GENGETOPT_VERSION	:= 2.23
+HOST_GENGETOPT_MD5	:= ea165d618640dbdecb42ae4491337965
+HOST_GENGETOPT		:= gengetopt-$(HOST_GENGETOPT_VERSION)
+HOST_GENGETOPT_SUFFIX	:= tar.xz
+HOST_GENGETOPT_URL	:= https://ftp.gnu.org/gnu/gengetopt/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
+HOST_GENGETOPT_SOURCE	:= $(SRCDIR)/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
+HOST_GENGETOPT_DIR	:= $(HOST_BUILDDIR)/$(HOST_GENGETOPT)
+HOST_GENGETOPT_LICENSE	:= GPL-3.0-only
+HOST_GENGETOPT_LICENSE_FILES := file://COPYING;md5=ff95bfe019feaf92f524b73dd79e76eb
+# vim: syntax=make
-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

[ptxdist] [PATCH v2 2/3] bayer2rgb3: add package

[Marian Cichy]

this package is a dependency for gst-bayer2rgb-neon
---
 configs/ptxconfig                   |  1 +
 patches/bayer2rgb3-0.4.1/autogen.sh |  1 +
 rules/bayer2rgb3.in                 | 18 +++++++++++
 rules/bayer2rgb3.make               | 49 +++++++++++++++++++++++++++++
 4 files changed, 69 insertions(+)
 create mode 120000 patches/bayer2rgb3-0.4.1/autogen.sh
 create mode 100644 rules/bayer2rgb3.in
 create mode 100644 rules/bayer2rgb3.make

diff --git a/configs/ptxconfig b/configs/ptxconfig
index 67200af3..16db79a0 100644
--- a/configs/ptxconfig
+++ b/configs/ptxconfig
@@ -2,6 +2,7 @@
 # Automatically generated file; DO NOT EDIT.
 # PTXdist 2020.02.0
 #
+# PTXCONF_BAYER2RGB3 is not set
 PTXCONF_BOOTCOUNT=y
 PTXCONF_BOOTCOUNT_SYSTEMD_UNIT=y
 PTXCONF_COMPOSE_INITRD=y
diff --git a/patches/bayer2rgb3-0.4.1/autogen.sh b/patches/bayer2rgb3-0.4.1/autogen.sh
new file mode 120000
index 00000000..9f8a4cb7
--- /dev/null
+++ b/patches/bayer2rgb3-0.4.1/autogen.sh
@@ -0,0 +1 @@
+../autogen.sh
\ No newline at end of file
diff --git a/rules/bayer2rgb3.in b/rules/bayer2rgb3.in
new file mode 100644
index 00000000..7e72a5b7
--- /dev/null
+++ b/rules/bayer2rgb3.in
@@ -0,0 +1,18 @@
+## SECTION=multimedia_tools
+
+config BAYER2RGB3
+	tristate
+	prompt "bayer2rgb3"
+	select HOST_GENGETOPT
+	help
+	  bayer2rgb will convert naked (no header) bayer grid data into rgb data.
+
+if BAYER2RGB3
+config BAYER2RGB3_COMMAND_TOOL
+	bool
+	default n
+	prompt "bayer2rgb3 command line tool"
+	help
+	  install additional command line tool along the lib
+
+endif
diff --git a/rules/bayer2rgb3.make b/rules/bayer2rgb3.make
new file mode 100644
index 00000000..8833e18b
--- /dev/null
+++ b/rules/bayer2rgb3.make
@@ -0,0 +1,49 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2020 by Marian Cichy <m.cichy@pengutronix.de>
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+#
+# We provide this package
+#
+PACKAGES-$(PTXCONF_BAYER2RGB3) += bayer2rgb3
+
+#
+# Paths and names
+#
+BAYER2RGB3_VERSION	:= 0.4.1
+BAYER2RGB3_MD5		:= 5362a93d4251eb012d38c1409af0182e
+BAYER2RGB3		:= bayer2rgb3-$(BAYER2RGB3_VERSION)
+BAYER2RGB3_SUFFIX	:= zip
+BAYER2RGB3_URL		:= https://gitlab-ext.sigma-chemnitz.de/ensc/bayer2rgb/-/archive/master/bayer2rgb-master.zip
+BAYER2RGB3_SOURCE	:= $(SRCDIR)/$(BAYER2RGB3).$(BAYER2RGB3_SUFFIX)
+BAYER2RGB3_DIR		:= $(BUILDDIR)/$(BAYER2RGB3)
+BAYER2RGB3_LICENSE	:= GPL-3.0-only
+BAYER2RGB3_LICENSE_FILES := files://COPYING;md5sum=d32239bcb673463ab874e80d47fae504
+
+# ----------------------------------------------------------------------------
+# Target-Install
+# ----------------------------------------------------------------------------
+
+$(STATEDIR)/bayer2rgb3.targetinstall:
+	@$(call targetinfo)
+
+	@$(call install_init, bayer2rgb3)
+	@$(call install_fixup, bayer2rgb3,PRIORITY,optional)
+	@$(call install_fixup, bayer2rgb3,SECTION,base)
+	@$(call install_fixup, bayer2rgb3,AUTHOR,"Marian Cichy <m.cichy@pengutronix.de>")
+	@$(call install_fixup, bayer2rgb3,DESCRIPTION,missing)
+
+ifdef PTXCONF_BAYER2RGB3_COMMAND_TOOL
+	@$(call install_copy, bayer2rgb3, 0, 0, 0755, -, /usr/bin/bayer2rgb)
+endif
+	@$(call install_lib, bayer2rgb3, 0, 0, 0644, libbayer2rgb3)
+
+	@$(call install_finish, bayer2rgb3)
+
+	@$(call touch)
+
+# vim: syntax=make
-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

[ptxdist] [PATCH v2 3/3] gst-bayer2rgb-neon: add package

[Marian Cichy]

---
 configs/ptxconfig                         |  1 +
 patches/gst-bayer2rgb-neon-0.3/autogen.sh |  1 +
 rules/gst-bayer2rgb-neon.in               | 10 ++++
 rules/gst-bayer2rgb-neon.make             | 56 +++++++++++++++++++++++
 4 files changed, 68 insertions(+)
 create mode 120000 patches/gst-bayer2rgb-neon-0.3/autogen.sh
 create mode 100644 rules/gst-bayer2rgb-neon.in
 create mode 100644 rules/gst-bayer2rgb-neon.make

diff --git a/configs/ptxconfig b/configs/ptxconfig
index 16db79a0..d1e896ca 100644
--- a/configs/ptxconfig
+++ b/configs/ptxconfig
@@ -13,6 +13,7 @@ PTXCONF_FIRMWARE_IMX=y
 # PTXCONF_FIRMWARE_IMX_VPU_IMX6DL is not set
 PTXCONF_FIRMWARE_IMX_VPU_IMX6Q=y
 PTXCONF_FIX_SYSTEMD=y
+# PTXCONF_GST_BAYER2RGB_NEON is not set
 PTXCONF_MTD_HANDLING=y
 PTXCONF_MTD_HANDLING_UDEV_RULES=y
 PTXCONF_MTD_HANDLING_MTDDATA_ATTACH_CREATE_MOUNT=y
diff --git a/patches/gst-bayer2rgb-neon-0.3/autogen.sh b/patches/gst-bayer2rgb-neon-0.3/autogen.sh
new file mode 120000
index 00000000..9f8a4cb7
--- /dev/null
+++ b/patches/gst-bayer2rgb-neon-0.3/autogen.sh
@@ -0,0 +1 @@
+../autogen.sh
\ No newline at end of file
diff --git a/rules/gst-bayer2rgb-neon.in b/rules/gst-bayer2rgb-neon.in
new file mode 100644
index 00000000..653a65b5
--- /dev/null
+++ b/rules/gst-bayer2rgb-neon.in
@@ -0,0 +1,10 @@
+## SECTION=multimedia_gstreamer1
+
+config GST_BAYER2RGB_NEON
+	tristate
+	prompt "gst-bayer2rgb-neon"
+	select BAYER2RGB3
+	select GSTREAMER1
+	select GST_PLUGINS_BASE1
+	help
+	  Inofficial Gstreamer-plugin that utilizes NEON-extension to convert video/x-bayer to RGB.
diff --git a/rules/gst-bayer2rgb-neon.make b/rules/gst-bayer2rgb-neon.make
new file mode 100644
index 00000000..eea13ca7
--- /dev/null
+++ b/rules/gst-bayer2rgb-neon.make
@@ -0,0 +1,56 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2020 by Marian Cichy <m.cichy@pengutronix.de>
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+#
+# We provide this package
+#
+PACKAGES-$(PTXCONF_GST_BAYER2RGB_NEON) += gst-bayer2rgb-neon
+
+#
+# Paths and names
+#
+GST_BAYER2RGB_NEON_VERSION	:= 0.3
+GST_BAYER2RGB_NEON_MD5		:= 73f05f8504e29c03d6b1164ee37b4fb9
+GST_BAYER2RGB_NEON		:= gst-bayer2rgb-neon-$(GST_BAYER2RGB_NEON_VERSION)
+GST_BAYER2RGB_NEON_SUFFIX	:= tar.bz2
+GST_BAYER2RGB_NEON_URL		:= https://git.phytec.de/gst-bayer2rgb-neon/snapshot/$(GST_BAYER2RGB_NEON).$(GST_BAYER2RGB_NEON_SUFFIX)
+GST_BAYER2RGB_NEON_SOURCE	:= $(SRCDIR)/$(GST_BAYER2RGB_NEON).$(GST_BAYER2RGB_NEON_SUFFIX)
+GST_BAYER2RGB_NEON_DIR		:= $(BUILDDIR)/$(GST_BAYER2RGB_NEON)
+GST_BAYER2RGB_NEON_LICENSE	:= GPL-3.0-only
+GST_BAYER2RGB_NEON_LICENSE_FILES:= file://COPYING;md5sum=d32239bcb673463ab874e80d47fae504
+
+# ----------------------------------------------------------------------------
+# Prepare
+# ----------------------------------------------------------------------------
+
+GST_BAYER2RGB_NEON_CONF_ENV	:= $(CROSS_ENV)
+
+GST_BAYER2RGB_NEON_CONF_TOOL	:= autoconf
+GST_BAYER2RGB_NEON_CONF_OPT	:= \
+	$(CROSS_AUTOCONF_USR) \
+	--with-plugindir=/usr/lib/gstreamer-1.0
+
+# ----------------------------------------------------------------------------
+# Target-Install
+# ----------------------------------------------------------------------------
+
+$(STATEDIR)/gst-bayer2rgb-neon.targetinstall:
+	@$(call targetinfo)
+
+	@$(call install_init, gst-bayer2rgb-neon)
+	@$(call install_fixup, gst-bayer2rgb-neon,PRIORITY,optional)
+	@$(call install_fixup, gst-bayer2rgb-neon,SECTION,base)
+	@$(call install_fixup, gst-bayer2rgb-neon,AUTHOR,"Marian Cichy <m.cichy@pengutronix.de>")
+	@$(call install_fixup, gst-bayer2rgb-neon,DESCRIPTION,missing)
+
+	@$(call install_lib, gst-bayer2rgb-neon, 0, 0, 644, gstreamer-1.0/gstbayer2rgbneon)
+	@$(call install_finish, gst-bayer2rgb-neon)
+
+	@$(call touch)
+
+# vim: syntax=make
-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] [PATCH v2 1/3] host-gengetopt: add package

[Michael Olbrich]

On Mon, Jun 01, 2020 at 11:49:39AM +0200, Marian Cichy wrote:
> this package is a dependency for bayer2rgb3,
> which is a dependency for gst-bayer2rgb-neon
> ---

This is a v2 patch, so it should be marked as such.


>  configs/ptxconfig         |  1 +
>  rules/host-gengetopt.in   |  9 +++++++++
>  rules/host-gengetopt.make | 26 ++++++++++++++++++++++++++
>  3 files changed, 36 insertions(+)
>  create mode 100644 rules/host-gengetopt.in
>  create mode 100644 rules/host-gengetopt.make
> 
> diff --git a/configs/ptxconfig b/configs/ptxconfig
> index 972e7615..67200af3 100644
> --- a/configs/ptxconfig
> +++ b/configs/ptxconfig

This file does not exist. I guess you created the patch from a BSP?

Michael

> @@ -105,6 +105,7 @@ PTXCONF__ptxconfig_MAGIC__=y
>  PTXCONF_HOST_BSDIFF=y
>  # PTXCONF_HOST_DFU_UTIL is not set
>  PTXCONF_HOST_FILE_STACK=y
> +# PTXCONF_HOST_GENGETOPT is not set
>  # PTXCONF_HOST_OPENOCD is not set
>  # PTXCONF_HOST_QEMU is not set
>  # PTXCONF_HOST_TESTFRAMEWORK is not set
> diff --git a/rules/host-gengetopt.in b/rules/host-gengetopt.in
> new file mode 100644
> index 00000000..f6855ab5
> --- /dev/null
> +++ b/rules/host-gengetopt.in
> @@ -0,0 +1,9 @@
> +## SECTION=hosttools
> +
> +config HOST_GENGETOPT
> +	tristate
> +	prompt "gengetopt"
> +	help
> +	  Gengetopt is a tool to write command line option
> +	  parsing code for C programs.
> +
> diff --git a/rules/host-gengetopt.make b/rules/host-gengetopt.make
> new file mode 100644
> index 00000000..2c1b3bb0
> --- /dev/null
> +++ b/rules/host-gengetopt.make
> @@ -0,0 +1,26 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2020 by Marian Cichy <m.cichy@pengutronix.de>
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +HOST_PACKAGES-$(PTXCONF_HOST_GENGETOPT) += host-gengetopt
> +
> +#
> +# Paths and names
> +#
> +HOST_GENGETOPT_VERSION	:= 2.23
> +HOST_GENGETOPT_MD5	:= ea165d618640dbdecb42ae4491337965
> +HOST_GENGETOPT		:= gengetopt-$(HOST_GENGETOPT_VERSION)
> +HOST_GENGETOPT_SUFFIX	:= tar.xz
> +HOST_GENGETOPT_URL	:= https://ftp.gnu.org/gnu/gengetopt/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
> +HOST_GENGETOPT_SOURCE	:= $(SRCDIR)/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
> +HOST_GENGETOPT_DIR	:= $(HOST_BUILDDIR)/$(HOST_GENGETOPT)
> +HOST_GENGETOPT_LICENSE	:= GPL-3.0-only
> +HOST_GENGETOPT_LICENSE_FILES := file://COPYING;md5=ff95bfe019feaf92f524b73dd79e76eb
> +# vim: syntax=make
> -- 
> 2.20.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] [PATCH v2 2/3] bayer2rgb3: add package

[Michael Olbrich]

On Mon, Jun 01, 2020 at 11:49:40AM +0200, Marian Cichy wrote:
> this package is a dependency for gst-bayer2rgb-neon
> ---

Also, v2. And changes to the last version should be noted here below the
'---'. That makes review easier.

>  configs/ptxconfig                   |  1 +
>  patches/bayer2rgb3-0.4.1/autogen.sh |  1 +
>  rules/bayer2rgb3.in                 | 18 +++++++++++
>  rules/bayer2rgb3.make               | 49 +++++++++++++++++++++++++++++
>  4 files changed, 69 insertions(+)
>  create mode 120000 patches/bayer2rgb3-0.4.1/autogen.sh
>  create mode 100644 rules/bayer2rgb3.in
>  create mode 100644 rules/bayer2rgb3.make
> 
> diff --git a/configs/ptxconfig b/configs/ptxconfig
> index 67200af3..16db79a0 100644
> --- a/configs/ptxconfig
> +++ b/configs/ptxconfig

Still no ptxconfig.. :-)

> @@ -2,6 +2,7 @@
>  # Automatically generated file; DO NOT EDIT.
>  # PTXdist 2020.02.0
>  #
> +# PTXCONF_BAYER2RGB3 is not set
>  PTXCONF_BOOTCOUNT=y
>  PTXCONF_BOOTCOUNT_SYSTEMD_UNIT=y
>  PTXCONF_COMPOSE_INITRD=y
> diff --git a/patches/bayer2rgb3-0.4.1/autogen.sh b/patches/bayer2rgb3-0.4.1/autogen.sh
> new file mode 120000
> index 00000000..9f8a4cb7
> --- /dev/null
> +++ b/patches/bayer2rgb3-0.4.1/autogen.sh
> @@ -0,0 +1 @@
> +../autogen.sh
> \ No newline at end of file
> diff --git a/rules/bayer2rgb3.in b/rules/bayer2rgb3.in
> new file mode 100644
> index 00000000..7e72a5b7
> --- /dev/null
> +++ b/rules/bayer2rgb3.in
> @@ -0,0 +1,18 @@
> +## SECTION=multimedia_tools
> +
> +config BAYER2RGB3

Use menuconfig here.

> +	tristate
> +	prompt "bayer2rgb3"

add spaces at the end to align the '-->' in menuconfig. Take a look at
other packages to see what I mean.

> +	select HOST_GENGETOPT
> +	help
> +	  bayer2rgb will convert naked (no header) bayer grid data into rgb data.
> +
> +if BAYER2RGB3

blank line.

> +config BAYER2RGB3_COMMAND_TOOL
> +	bool
> +	default n

Drop this line, it's the default.

Michael

> +	prompt "bayer2rgb3 command line tool"
> +	help
> +	  install additional command line tool along the lib
> +
> +endif
> diff --git a/rules/bayer2rgb3.make b/rules/bayer2rgb3.make
> new file mode 100644
> index 00000000..8833e18b
> --- /dev/null
> +++ b/rules/bayer2rgb3.make
> @@ -0,0 +1,49 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2020 by Marian Cichy <m.cichy@pengutronix.de>
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_BAYER2RGB3) += bayer2rgb3
> +
> +#
> +# Paths and names
> +#
> +BAYER2RGB3_VERSION	:= 0.4.1
> +BAYER2RGB3_MD5		:= 5362a93d4251eb012d38c1409af0182e
> +BAYER2RGB3		:= bayer2rgb3-$(BAYER2RGB3_VERSION)
> +BAYER2RGB3_SUFFIX	:= zip
> +BAYER2RGB3_URL		:= https://gitlab-ext.sigma-chemnitz.de/ensc/bayer2rgb/-/archive/master/bayer2rgb-master.zip
> +BAYER2RGB3_SOURCE	:= $(SRCDIR)/$(BAYER2RGB3).$(BAYER2RGB3_SUFFIX)
> +BAYER2RGB3_DIR		:= $(BUILDDIR)/$(BAYER2RGB3)
> +BAYER2RGB3_LICENSE	:= GPL-3.0-only
> +BAYER2RGB3_LICENSE_FILES := files://COPYING;md5sum=d32239bcb673463ab874e80d47fae504
> +
> +# ----------------------------------------------------------------------------
> +# Target-Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/bayer2rgb3.targetinstall:
> +	@$(call targetinfo)
> +
> +	@$(call install_init, bayer2rgb3)
> +	@$(call install_fixup, bayer2rgb3,PRIORITY,optional)
> +	@$(call install_fixup, bayer2rgb3,SECTION,base)
> +	@$(call install_fixup, bayer2rgb3,AUTHOR,"Marian Cichy <m.cichy@pengutronix.de>")
> +	@$(call install_fixup, bayer2rgb3,DESCRIPTION,missing)
> +
> +ifdef PTXCONF_BAYER2RGB3_COMMAND_TOOL
> +	@$(call install_copy, bayer2rgb3, 0, 0, 0755, -, /usr/bin/bayer2rgb)
> +endif
> +	@$(call install_lib, bayer2rgb3, 0, 0, 0644, libbayer2rgb3)
> +
> +	@$(call install_finish, bayer2rgb3)
> +
> +	@$(call touch)
> +
> +# vim: syntax=make
> -- 
> 2.20.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] [PATCH v2 3/3] gst-bayer2rgb-neon: add package

[Michael Olbrich]

On Mon, Jun 01, 2020 at 11:49:41AM +0200, Marian Cichy wrote:
> ---
>  configs/ptxconfig                         |  1 +
>  patches/gst-bayer2rgb-neon-0.3/autogen.sh |  1 +
>  rules/gst-bayer2rgb-neon.in               | 10 ++++
>  rules/gst-bayer2rgb-neon.make             | 56 +++++++++++++++++++++++
>  4 files changed, 68 insertions(+)
>  create mode 120000 patches/gst-bayer2rgb-neon-0.3/autogen.sh
>  create mode 100644 rules/gst-bayer2rgb-neon.in
>  create mode 100644 rules/gst-bayer2rgb-neon.make
> 
> diff --git a/configs/ptxconfig b/configs/ptxconfig
> index 16db79a0..d1e896ca 100644
> --- a/configs/ptxconfig
> +++ b/configs/ptxconfig

...

> @@ -13,6 +13,7 @@ PTXCONF_FIRMWARE_IMX=y
>  # PTXCONF_FIRMWARE_IMX_VPU_IMX6DL is not set
>  PTXCONF_FIRMWARE_IMX_VPU_IMX6Q=y
>  PTXCONF_FIX_SYSTEMD=y
> +# PTXCONF_GST_BAYER2RGB_NEON is not set
>  PTXCONF_MTD_HANDLING=y
>  PTXCONF_MTD_HANDLING_UDEV_RULES=y
>  PTXCONF_MTD_HANDLING_MTDDATA_ATTACH_CREATE_MOUNT=y
> diff --git a/patches/gst-bayer2rgb-neon-0.3/autogen.sh b/patches/gst-bayer2rgb-neon-0.3/autogen.sh
> new file mode 120000
> index 00000000..9f8a4cb7
> --- /dev/null
> +++ b/patches/gst-bayer2rgb-neon-0.3/autogen.sh
> @@ -0,0 +1 @@
> +../autogen.sh
> \ No newline at end of file
> diff --git a/rules/gst-bayer2rgb-neon.in b/rules/gst-bayer2rgb-neon.in
> new file mode 100644
> index 00000000..653a65b5
> --- /dev/null
> +++ b/rules/gst-bayer2rgb-neon.in
> @@ -0,0 +1,10 @@
> +## SECTION=multimedia_gstreamer1
> +
> +config GST_BAYER2RGB_NEON
> +	tristate
> +	prompt "gst-bayer2rgb-neon"
> +	select BAYER2RGB3
> +	select GSTREAMER1
> +	select GST_PLUGINS_BASE1
> +	help
> +	  Inofficial Gstreamer-plugin that utilizes NEON-extension to convert video/x-bayer to RGB.

Break this line.

Michael

> diff --git a/rules/gst-bayer2rgb-neon.make b/rules/gst-bayer2rgb-neon.make
> new file mode 100644
> index 00000000..eea13ca7
> --- /dev/null
> +++ b/rules/gst-bayer2rgb-neon.make
> @@ -0,0 +1,56 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2020 by Marian Cichy <m.cichy@pengutronix.de>
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_GST_BAYER2RGB_NEON) += gst-bayer2rgb-neon
> +
> +#
> +# Paths and names
> +#
> +GST_BAYER2RGB_NEON_VERSION	:= 0.3
> +GST_BAYER2RGB_NEON_MD5		:= 73f05f8504e29c03d6b1164ee37b4fb9
> +GST_BAYER2RGB_NEON		:= gst-bayer2rgb-neon-$(GST_BAYER2RGB_NEON_VERSION)
> +GST_BAYER2RGB_NEON_SUFFIX	:= tar.bz2
> +GST_BAYER2RGB_NEON_URL		:= https://git.phytec.de/gst-bayer2rgb-neon/snapshot/$(GST_BAYER2RGB_NEON).$(GST_BAYER2RGB_NEON_SUFFIX)
> +GST_BAYER2RGB_NEON_SOURCE	:= $(SRCDIR)/$(GST_BAYER2RGB_NEON).$(GST_BAYER2RGB_NEON_SUFFIX)
> +GST_BAYER2RGB_NEON_DIR		:= $(BUILDDIR)/$(GST_BAYER2RGB_NEON)
> +GST_BAYER2RGB_NEON_LICENSE	:= GPL-3.0-only
> +GST_BAYER2RGB_NEON_LICENSE_FILES:= file://COPYING;md5sum=d32239bcb673463ab874e80d47fae504
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +GST_BAYER2RGB_NEON_CONF_ENV	:= $(CROSS_ENV)
> +
> +GST_BAYER2RGB_NEON_CONF_TOOL	:= autoconf
> +GST_BAYER2RGB_NEON_CONF_OPT	:= \
> +	$(CROSS_AUTOCONF_USR) \
> +	--with-plugindir=/usr/lib/gstreamer-1.0
> +
> +# ----------------------------------------------------------------------------
> +# Target-Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/gst-bayer2rgb-neon.targetinstall:
> +	@$(call targetinfo)
> +
> +	@$(call install_init, gst-bayer2rgb-neon)
> +	@$(call install_fixup, gst-bayer2rgb-neon,PRIORITY,optional)
> +	@$(call install_fixup, gst-bayer2rgb-neon,SECTION,base)
> +	@$(call install_fixup, gst-bayer2rgb-neon,AUTHOR,"Marian Cichy <m.cichy@pengutronix.de>")
> +	@$(call install_fixup, gst-bayer2rgb-neon,DESCRIPTION,missing)
> +
> +	@$(call install_lib, gst-bayer2rgb-neon, 0, 0, 644, gstreamer-1.0/gstbayer2rgbneon)
> +	@$(call install_finish, gst-bayer2rgb-neon)
> +
> +	@$(call touch)
> +
> +# vim: syntax=make
> -- 
> 2.20.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] [PATCH v2 1/3] host-gengetopt: add package

[Marian Cichy]

On 6/2/20 3:18 PM, Michael Olbrich wrote:
> On Mon, Jun 01, 2020 at 11:49:39AM +0200, Marian Cichy wrote:
>> this package is a dependency for bayer2rgb3,
>> which is a dependency for gst-bayer2rgb-neon
>> ---
> 
> This is a v2 patch, so it should be marked as such.
> 
>

I did use git format-patch -v 2 to mark it. Should I also mention v2 in 
the commit message?

>>   configs/ptxconfig         |  1 +
>>   rules/host-gengetopt.in   |  9 +++++++++
>>   rules/host-gengetopt.make | 26 ++++++++++++++++++++++++++
>>   3 files changed, 36 insertions(+)
>>   create mode 100644 rules/host-gengetopt.in
>>   create mode 100644 rules/host-gengetopt.make
>>
>> diff --git a/configs/ptxconfig b/configs/ptxconfig
>> index 972e7615..67200af3 100644
>> --- a/configs/ptxconfig
>> +++ b/configs/ptxconfig
> 
> This file does not exist. I guess you created the patch from a BSP?
> 
> Michael
>

Yes. I thought that we need to update the ptxconfig entries with every 
new package.

>> @@ -105,6 +105,7 @@ PTXCONF__ptxconfig_MAGIC__=y
>>   PTXCONF_HOST_BSDIFF=y
>>   # PTXCONF_HOST_DFU_UTIL is not set
>>   PTXCONF_HOST_FILE_STACK=y
>> +# PTXCONF_HOST_GENGETOPT is not set
>>   # PTXCONF_HOST_OPENOCD is not set
>>   # PTXCONF_HOST_QEMU is not set
>>   # PTXCONF_HOST_TESTFRAMEWORK is not set
>> diff --git a/rules/host-gengetopt.in b/rules/host-gengetopt.in
>> new file mode 100644
>> index 00000000..f6855ab5
>> --- /dev/null
>> +++ b/rules/host-gengetopt.in
>> @@ -0,0 +1,9 @@
>> +## SECTION=hosttools
>> +
>> +config HOST_GENGETOPT
>> +	tristate
>> +	prompt "gengetopt"
>> +	help
>> +	  Gengetopt is a tool to write command line option
>> +	  parsing code for C programs.
>> +
>> diff --git a/rules/host-gengetopt.make b/rules/host-gengetopt.make
>> new file mode 100644
>> index 00000000..2c1b3bb0
>> --- /dev/null
>> +++ b/rules/host-gengetopt.make
>> @@ -0,0 +1,26 @@
>> +# -*-makefile-*-
>> +#
>> +# Copyright (C) 2020 by Marian Cichy <m.cichy@pengutronix.de>
>> +#
>> +# For further information about the PTXdist project and license conditions
>> +# see the README file.
>> +#
>> +
>> +#
>> +# We provide this package
>> +#
>> +HOST_PACKAGES-$(PTXCONF_HOST_GENGETOPT) += host-gengetopt
>> +
>> +#
>> +# Paths and names
>> +#
>> +HOST_GENGETOPT_VERSION	:= 2.23
>> +HOST_GENGETOPT_MD5	:= ea165d618640dbdecb42ae4491337965
>> +HOST_GENGETOPT		:= gengetopt-$(HOST_GENGETOPT_VERSION)
>> +HOST_GENGETOPT_SUFFIX	:= tar.xz
>> +HOST_GENGETOPT_URL	:= https://ftp.gnu.org/gnu/gengetopt/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
>> +HOST_GENGETOPT_SOURCE	:= $(SRCDIR)/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
>> +HOST_GENGETOPT_DIR	:= $(HOST_BUILDDIR)/$(HOST_GENGETOPT)
>> +HOST_GENGETOPT_LICENSE	:= GPL-3.0-only
>> +HOST_GENGETOPT_LICENSE_FILES := file://COPYING;md5=ff95bfe019feaf92f524b73dd79e76eb
>> +# vim: syntax=make
>> -- 
>> 2.20.1
>>
>>
>> _______________________________________________
>> ptxdist mailing list
>> ptxdist@pengutronix.de
>> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
>>
> 

mci

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] [PATCH v2 1/3] host-gengetopt: add package

[Michael Olbrich]

On Wed, Jun 03, 2020 at 10:30:51AM +0200, Marian Cichy wrote:
> 
> 
> On 6/2/20 3:18 PM, Michael Olbrich wrote:
> > On Mon, Jun 01, 2020 at 11:49:39AM +0200, Marian Cichy wrote:
> > > this package is a dependency for bayer2rgb3,
> > > which is a dependency for gst-bayer2rgb-neon
> > > ---
> > 
> > This is a v2 patch, so it should be marked as such.
> > 
> > 
> 
> I did use git format-patch -v 2 to mark it. Should I also mention v2 in the
> commit message?

Sorry, I am blind. You did the right thing here.

Michael

> > >   configs/ptxconfig         |  1 +
> > >   rules/host-gengetopt.in   |  9 +++++++++
> > >   rules/host-gengetopt.make | 26 ++++++++++++++++++++++++++
> > >   3 files changed, 36 insertions(+)
> > >   create mode 100644 rules/host-gengetopt.in
> > >   create mode 100644 rules/host-gengetopt.make
> > > 
> > > diff --git a/configs/ptxconfig b/configs/ptxconfig
> > > index 972e7615..67200af3 100644
> > > --- a/configs/ptxconfig
> > > +++ b/configs/ptxconfig
> > 
> > This file does not exist. I guess you created the patch from a BSP?
> > 
> > Michael
> > 
> 
> Yes. I thought that we need to update the ptxconfig entries with every new
> package.
> 
> > > @@ -105,6 +105,7 @@ PTXCONF__ptxconfig_MAGIC__=y
> > >   PTXCONF_HOST_BSDIFF=y
> > >   # PTXCONF_HOST_DFU_UTIL is not set
> > >   PTXCONF_HOST_FILE_STACK=y
> > > +# PTXCONF_HOST_GENGETOPT is not set
> > >   # PTXCONF_HOST_OPENOCD is not set
> > >   # PTXCONF_HOST_QEMU is not set
> > >   # PTXCONF_HOST_TESTFRAMEWORK is not set
> > > diff --git a/rules/host-gengetopt.in b/rules/host-gengetopt.in
> > > new file mode 100644
> > > index 00000000..f6855ab5
> > > --- /dev/null
> > > +++ b/rules/host-gengetopt.in
> > > @@ -0,0 +1,9 @@
> > > +## SECTION=hosttools
> > > +
> > > +config HOST_GENGETOPT
> > > +	tristate
> > > +	prompt "gengetopt"
> > > +	help
> > > +	  Gengetopt is a tool to write command line option
> > > +	  parsing code for C programs.
> > > +
> > > diff --git a/rules/host-gengetopt.make b/rules/host-gengetopt.make
> > > new file mode 100644
> > > index 00000000..2c1b3bb0
> > > --- /dev/null
> > > +++ b/rules/host-gengetopt.make
> > > @@ -0,0 +1,26 @@
> > > +# -*-makefile-*-
> > > +#
> > > +# Copyright (C) 2020 by Marian Cichy <m.cichy@pengutronix.de>
> > > +#
> > > +# For further information about the PTXdist project and license conditions
> > > +# see the README file.
> > > +#
> > > +
> > > +#
> > > +# We provide this package
> > > +#
> > > +HOST_PACKAGES-$(PTXCONF_HOST_GENGETOPT) += host-gengetopt
> > > +
> > > +#
> > > +# Paths and names
> > > +#
> > > +HOST_GENGETOPT_VERSION	:= 2.23
> > > +HOST_GENGETOPT_MD5	:= ea165d618640dbdecb42ae4491337965
> > > +HOST_GENGETOPT		:= gengetopt-$(HOST_GENGETOPT_VERSION)
> > > +HOST_GENGETOPT_SUFFIX	:= tar.xz
> > > +HOST_GENGETOPT_URL	:= https://ftp.gnu.org/gnu/gengetopt/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
> > > +HOST_GENGETOPT_SOURCE	:= $(SRCDIR)/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
> > > +HOST_GENGETOPT_DIR	:= $(HOST_BUILDDIR)/$(HOST_GENGETOPT)
> > > +HOST_GENGETOPT_LICENSE	:= GPL-3.0-only
> > > +HOST_GENGETOPT_LICENSE_FILES := file://COPYING;md5=ff95bfe019feaf92f524b73dd79e76eb
> > > +# vim: syntax=make
> > > -- 
> > > 2.20.1
> > > 
> > > 
> > > _______________________________________________
> > > ptxdist mailing list
> > > ptxdist@pengutronix.de
> > > To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> > > 
> > 
> 
> mci
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de