mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Roland Hieber <rhi@pengutronix.de>
To: Christian Melki <christian.melki@t2data.com>
Cc: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH] dump1090fa: New package. Version 6.1
Date: Tue, 16 Nov 2021 10:44:20 +0100	[thread overview]
Message-ID: <20211116094420.6jqfbutj6ixnlzah@pengutronix.de> (raw)
In-Reply-To: <20211111095249.1873275-2-christian.melki@t2data.com>

On Thu, Nov 11, 2021 at 10:52:49AM +0100, Christian Melki wrote:
> dump1090 from FlightAware is an ADS-B (Automatic Dependent Surveillance-Broadcast).
> ADS-B is used by aircrafts to signal their presence.
> Using an RTL (very cheap) based SDR, you can easily build your own ADS-B decoder.
> 
> Signed-off-by: Christian Melki <christian.melki@t2data.com>
> ---
>  .../dump1090fa-6.1/fix_cpufeatures_arch.patch | 61 ++++++++++++++
>  patches/dump1090fa-6.1/series                 |  1 +
>  rules/dump1090fa.in                           |  8 ++
>  rules/dump1090fa.make                         | 80 +++++++++++++++++++
>  4 files changed, 150 insertions(+)
>  create mode 100644 patches/dump1090fa-6.1/fix_cpufeatures_arch.patch
>  create mode 100644 patches/dump1090fa-6.1/series
>  create mode 100644 rules/dump1090fa.in
>  create mode 100644 rules/dump1090fa.make
> 
> diff --git a/patches/dump1090fa-6.1/fix_cpufeatures_arch.patch b/patches/dump1090fa-6.1/fix_cpufeatures_arch.patch
> new file mode 100644
> index 000000000..270a35846
> --- /dev/null
> +++ b/patches/dump1090fa-6.1/fix_cpufeatures_arch.patch
> @@ -0,0 +1,61 @@
> +From 059e48b82beb4fb920cca977b02db1ceb48facb9 Mon Sep 17 00:00:00 2001
> +From: Oliver Jowett <oliver.jowett@flightaware.com>
> +Date: Thu, 23 Sep 2021 13:23:17 +0800
> +Subject: [PATCH] Allow env vars (not only command line overrides) to set
> + CPUFEATURES_{ARCH,UNAME}. Change CPUFEATURES_{ARCH,UNAME} defaults to use
> + {ARCH,UNAME} values.
> +
> +Fixes #158
> +---
> + Makefile             | 8 ++++++--
> + Makefile.cpufeatures | 4 ++--
> + 2 files changed, 8 insertions(+), 4 deletions(-)
> +
> +diff --git a/Makefile b/Makefile
> +index a85d154c5..9d81ed157 100644
> +--- a/Makefile
> ++++ b/Makefile
> +@@ -39,7 +39,11 @@ else
> +   LIMESDR ?= no
> + endif
> + 
> +-UNAME := $(shell uname)
> ++HOST_UNAME := $(shell uname)
> ++HOST_ARCH := $(shell uname -m)
> ++
> ++UNAME ?= $(HOST_UNAME)
> ++ARCH ?= $(HOST_ARCH)
> + 
> + ifeq ($(UNAME), Linux)
> +   CPPFLAGS += -D_DEFAULT_SOURCE
> +@@ -150,7 +154,6 @@ endif
> + ## starch (runtime DSP code selection) mix, architecture-specific
> + ##
> + 
> +-ARCH ?= $(shell uname -m)
> + ifneq ($(CPUFEATURES),yes)
> +   # need to be able to detect CPU features at runtime to enable any non-standard compiler flags
> +   STARCH_MIX := generic
> +@@ -180,6 +183,7 @@ include dsp/generated/makefile.$(STARCH_MIX)
> + showconfig:
> + 	@echo "Building with:" >&2
> + 	@echo "  Version string:  $(DUMP1090_VERSION)" >&2
> ++	@echo "  Architecture:    $(ARCH)" >&2
> + 	@echo "  DSP mix:         $(STARCH_MIX)" >&2
> + 	@echo "  RTLSDR support:  $(RTLSDR)" >&2
> + 	@echo "  BladeRF support: $(BLADERF)" >&2
> +diff --git a/Makefile.cpufeatures b/Makefile.cpufeatures
> +index 8c3b8418d..6b4289978 100644
> +--- a/Makefile.cpufeatures
> ++++ b/Makefile.cpufeatures
> +@@ -2,8 +2,8 @@
> + 
> + # cmake integration is a little tricky, so let's do this by hand for now
> + 
> +-CPUFEATURES_UNAME := $(shell uname)
> +-CPUFEATURES_ARCH := $(shell uname -m)
> ++CPUFEATURES_UNAME ?= $(UNAME)
> ++CPUFEATURES_ARCH ?= $(ARCH)
> + 
> + CPUFEATURES_OBJS := cpu_features/src/filesystem.o cpu_features/src/stack_line_reader.o cpu_features/src/string_view.o
> + CPUFEATURES_CFLAGS := -std=c99 -O -g -DSTACK_LINE_READER_BUFFER_SIZE=1024 -DNDEBUG
> diff --git a/patches/dump1090fa-6.1/series b/patches/dump1090fa-6.1/series
> new file mode 100644
> index 000000000..75a7aa6dc
> --- /dev/null
> +++ b/patches/dump1090fa-6.1/series
> @@ -0,0 +1 @@
> +fix_cpufeatures_arch.patch
> diff --git a/rules/dump1090fa.in b/rules/dump1090fa.in
> new file mode 100644
> index 000000000..5738ba17e
> --- /dev/null
> +++ b/rules/dump1090fa.in
> @@ -0,0 +1,8 @@
> +## SECTION=networking
> +
> +menuconfig DUMP1090FA
> +	tristate
> +	prompt "dump1090fa		      "
> +	select LIBRTLSDR
> +	help
> +	  FlightAware MODE-S/ADS-B decoder.
> \ No newline at end of file
> diff --git a/rules/dump1090fa.make b/rules/dump1090fa.make
> new file mode 100644
> index 000000000..53d44791b
> --- /dev/null
> +++ b/rules/dump1090fa.make
> @@ -0,0 +1,80 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2021 by Christian Melki <christian.melki@t2data.com>
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_DUMP1090FA) += dump1090fa
> +
> +#
> +# Paths and names
> +#
> +DUMP1090FA_VERSION	:= 6.1
> +DUMP1090FA_MD5		:= a7cb661bc23974f8bf7650ce26b6e0be
> +DUMP1090FA		:= dump1090fa-$(DUMP1090FA_VERSION)
> +DUMP1090FA_SUFFIX	:= tar.gz
> +DUMP1090FA_URL		:= https://github.com/flightaware/dump1090/archive/refs/tags/v$(DUMP1090FA_VERSION).$(DUMP1090FA_SUFFIX)
> +DUMP1090FA_SOURCE	:= $(SRCDIR)/$(DUMP1090FA).$(DUMP1090FA_SUFFIX)
> +DUMP1090FA_DIR		:= $(BUILDDIR)/$(DUMP1090FA)
> +DUMP1090FA_LICENSE	:= GPL-2.0-only

Also please add a <PKG>_LICENSE_FILES with md5 sums for both packages
please.

 -  Roland

> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +ifdef PTXCONF_ARCH_X86
> +DUMP1090FA_ARCH		:= x86
> +ifdef PTXCONF_ARCH_X86_64
> +DUMP1090FA_ARCH		:= x86_64
> +endif
> +endif
> +ifdef PTXCONF_ARCH_ARM
> +DUMP1090FA_ARCH		:= arm
> +endif
> +ifdef PTXCONF_ARCH_ARM64
> +DUMP1090FA_ARCH		:= aarch64
> +endif
> +
> +DUMP1090FA_CONF_TOOL := NO
> +DUMP1090FA_MAKE_ENV := \
> +	$(CROSS_ENV) \
> +	BLADERF=no \
> +	LIMESDR=no \
> +	HACKRF=no \
> +	RTLSDR=yes \
> +	ARCH=$(DUMP1090FA_ARCH) \
> +	CPUFEATURES_ARCH=$(DUMP1090FA_ARCH) \
> +	CPUFEATURES_UNAME="Linux"
> +
> +# ----------------------------------------------------------------------------
> +# Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/dump1090fa.install:
> +	@$(call targetinfo)
> +	@install -vD -m 755 $(DUMP1090FA_DIR)/dump1090 $(DUMP1090FA_PKGDIR)/usr/bin/dump1090-fa
> +	$(call touch)
> +
> +# ----------------------------------------------------------------------------
> +# Target-Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/dump1090fa.targetinstall:
> +	@$(call targetinfo)
> +	@$(call install_init, dump1090fa)
> +	@$(call install_fixup, dump1090fa,PRIORITY,optional)
> +	@$(call install_fixup, dump1090fa,SECTION,base)
> +	@$(call install_fixup, dump1090fa,AUTHOR,"Christian Melki <christian.melki@t2data.com>")
> +	@$(call install_fixup, dump1090fa,DESCRIPTION,missing)
> +	@$(call install_copy, dump1090fa, 0, 0, 0755, -, /usr/bin/dump1090-fa)
> +
> +	@$(call install_finish, dump1090fa)
> +
> +	@$(call touch)
> +
> +# vim: syntax=make
> -- 
> 2.30.2
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://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


      parent reply	other threads:[~2021-11-16  9:45 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-11-11  9:52 [ptxdist] [PATCH] librtlsdr: New package. Version 0.8 Christian Melki
2021-11-11  9:52 ` [ptxdist] [PATCH] dump1090fa: New package. Version 6.1 Christian Melki
2021-11-12  9:08   ` Michael Olbrich
2021-11-16  9:44   ` Roland Hieber [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211116094420.6jqfbutj6ixnlzah@pengutronix.de \
    --to=rhi@pengutronix.de \
    --cc=christian.melki@t2data.com \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox