From: Roland Hieber <rhi@pengutronix.de>
To: "Uwe Kleine-König" <u.kleine-koenig@baylibre.com>
Cc: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH] libpwm: New package
Date: Mon, 17 Jun 2024 11:18:03 +0200 [thread overview]
Message-ID: <20240617091803.zb2y7ojp7toikifk@pengutronix.de> (raw)
In-Reply-To: <20240614155900.799336-1-u.kleine-koenig@baylibre.com>
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 <u.kleine-koenig@baylibre.com>
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 <u.kleine-koeng@baylibre.com>
> +#
> +# 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
<https://www.gnu.org/licenses/gpl-faq.html#WhyMustIInclude>).
> +
> +#
> +# 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 <u.kleine-koeng@baylibre.com>")
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 |
next prev parent reply other threads:[~2024-06-17 9:18 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-14 15:59 Uwe Kleine-König
2024-06-17 9:18 ` Roland Hieber [this message]
2024-06-17 10:14 ` Uwe Kleine-König
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=20240617091803.zb2y7ojp7toikifk@pengutronix.de \
--to=rhi@pengutronix.de \
--cc=ptxdist@pengutronix.de \
--cc=u.kleine-koenig@baylibre.com \
/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