From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Mon, 17 Jun 2024 11:18:24 +0200 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1sJ8VM-007VWL-1E for lore@lore.pengutronix.de; Mon, 17 Jun 2024 11:18:24 +0200 Received: from localhost ([127.0.0.1] helo=metis.whiteo.stw.pengutronix.de) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1sJ8VM-0002kX-0H; Mon, 17 Jun 2024 11:18:24 +0200 Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1sJ8V2-0002kL-2l; Mon, 17 Jun 2024 11:18:04 +0200 Received: from [2a0a:edc0:2:b01:1d::c5] (helo=pty.whiteo.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1sJ8V1-002wPd-JS; Mon, 17 Jun 2024 11:18:03 +0200 Received: from rhi by pty.whiteo.stw.pengutronix.de with local (Exim 4.96) (envelope-from ) id 1sJ8V1-00G5YJ-1e; Mon, 17 Jun 2024 11:18:03 +0200 Date: Mon, 17 Jun 2024 11:18:03 +0200 From: Roland Hieber To: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= Message-ID: <20240617091803.zb2y7ojp7toikifk@pengutronix.de> References: <20240614155900.799336-1-u.kleine-koenig@baylibre.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20240614155900.799336-1-u.kleine-koenig@baylibre.com> Subject: Re: [ptxdist] [PATCH] libpwm: New package X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Cc: ptxdist@pengutronix.de Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false On Fri, Jun 14, 2024 at 05:59:00PM +0200, Uwe Kleine-König wrote: > libpwm is a thin abstract layer to drive PWM devices under Linux. > > Signed-off-by: Uwe Kleine-König Thanks for your first contribution, u.kleine-koenig@baylibre.com! (No, seriously: nice to read you again! Do you want to add yourself to .mailmap?) > --- > patches/libpwm-f7a7496fcb9d/autogen.sh | 1 + > rules/libpwm.in | 8 ++++ > rules/libpwm.make | 57 ++++++++++++++++++++++++++ > 3 files changed, 66 insertions(+) > create mode 120000 patches/libpwm-f7a7496fcb9d/autogen.sh > create mode 100644 rules/libpwm.in > create mode 100644 rules/libpwm.make > > diff --git a/patches/libpwm-f7a7496fcb9d/autogen.sh b/patches/libpwm-f7a7496fcb9d/autogen.sh > new file mode 120000 > index 000000000000..9f8a4cb7ddcb > --- /dev/null > +++ b/patches/libpwm-f7a7496fcb9d/autogen.sh > @@ -0,0 +1 @@ > +../autogen.sh > \ No newline at end of file > diff --git a/rules/libpwm.in b/rules/libpwm.in > new file mode 100644 > index 000000000000..6d07e056a3e5 > --- /dev/null > +++ b/rules/libpwm.in > @@ -0,0 +1,8 @@ > +## SECTION=system_libraries > + > +config LIBPWM > + bool > + prompt "libpwm" > + select KERNEL_HEADER > + help > + drive Linux PWM devices from userspace > diff --git a/rules/libpwm.make b/rules/libpwm.make > new file mode 100644 > index 000000000000..b7c1810951e5 > --- /dev/null > +++ b/rules/libpwm.make > @@ -0,0 +1,57 @@ > +# -*-makefile-*- > +# > +# Copyright (C) 2024 by Uwe Kleine-Koenig > +# > +# For further information about the PTXdist project and license conditions > +# see the README file. > +# > + > +# > +# We provide this package > +# > +PACKAGES-$(PTXCONF_LIBPWM) += libpwm > + > +# > +# Paths and names > +# > +LIBPWM_VERSION := f7a7496fcb9d I think it makes sense here to have a pseudo-git-describe string containing the date of the commit, so that future versions stay sortable, e.g. 2024-04-09-gf7a7496fcb9d. (You'll have to rename the patch folder too). > +LIBPWM_MD5 := 8ed77baaad0328359cbaf075517b0025 > +LIBPWM := libpwm-$(LIBPWM_VERSION) > +LIBPWM_SUFFIX := tar.gz > +LIBPWM_URL := https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/libpwm.git/snapshot/$(LIBPWM_VERSION).$(LIBPWM_SUFFIX) > +LIBPWM_SOURCE := $(SRCDIR)/$(LIBPWM).$(LIBPWM_SUFFIX) > +LIBPWM_DIR := $(BUILDDIR)/$(LIBPWM) > +LIBPWM_BUILD_OOT := YES > +LIBPWM_LICENSE := LGPL-2.1 AND 0BSD Use LGPL-2.0-only here (SPDX 2.0), as the copyright dedication doesn't say anything about "or later". Are the "test programs" mentioned in the license the ones that you targetinstall at /usr/bin/pwmtest and /usr/bin/pwmtestperf ? > +LIBPWM_LICENSE_FILES := file://COPYING;md5=41d65f310284fe1f2945ea9c57f297c7 Note: COPYING does not include the LGPL license text; since you are the upstream maintainer, consider adding it to the repository (see ). > + > +# > +# autoconf > +# > +LIBPWM_CONF_TOOL := autoconf > +LIBPWM_CONF_OPT := $(CROSS_AUTOCONF_USR) > + > +LIBPWM_CPPFLAGS := -isystem $(KERNEL_HEADERS_INCLUDE_DIR) > + > +# ---------------------------------------------------------------------------- > +# Target-Install > +# ---------------------------------------------------------------------------- > + > +$(STATEDIR)/libpwm.targetinstall: > + @$(call targetinfo) > + > + @$(call install_init, libpwm) > + @$(call install_fixup, libpwm, PRIORITY, optional) > + @$(call install_fixup, libpwm, SECTION, base) > + @$(call install_fixup, libpwm, AUTHOR, "Uwe Kleine-Koenig ") This is a different e-mail address than what you wrote from. Other than that, looks good to me. - Roland > + @$(call install_fixup, libpwm, DESCRIPTION, missing) > + > + @$(call install_lib, libpwm, 0, 0, 0644, libpwm) > + @$(call install_copy, libpwm, 0, 0, 0755, -, /usr/bin/pwmtest) > + @$(call install_copy, libpwm, 0, 0, 0755, -, /usr/bin/pwmtestperf) > + > + @$(call install_finish, libpwm) > + > + @$(call touch) > + > +# vim: syntax=make > > base-commit: 7f9c1cc0688da418cc9d921e35a5dc8a4d3acdff > -- > 2.43.0 -- 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 |