From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Hanyuan Zhao <hanyuan-z@qq.com>
Cc: Samuel Martin <s.martin49@gmail.com>, buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 1/1 RESEND] package/wiringX: add GPIO library wiringX
Date: Sat, 18 Nov 2023 16:11:27 +0100 [thread overview]
Message-ID: <20231118151127.GF996011@scaer> (raw)
In-Reply-To: <tencent_757F73E3FFB48495C31F20C1B8F0CBDBD205@qq.com>
Hanyuan, All,
On 2023-11-18 22:53 +0800, Hanyuan Zhao via buildroot spake thusly:
> wiringX is a cross-platform GPIO interface that allows
> developers to control the GPIO of various platforms with
> generic and uniform functions. By using wiringX, the same
> code will run on all platforms supported by wiringX.
>
> https://github.com/wiringX/wiringX
>
> Signed-off-by: Hanyuan Zhao <hanyuan-z@qq.com>
There is no need to resend, as the patch is still pending for review on
our patchowrk:
https://patchwork.ozlabs.org/project/buildroot/list/
https://patchwork.ozlabs.org/project/buildroot/patch/tencent_0D8E30000E63F8AF28CBDFCD8FFFBE122007@qq.com/
However, if this new iteration has changes from the previous, then;
- it should be marked v2, not RESEND
- the changes fromn the previous iteration should be (briefly) listed
after the three-dash line [0]
> ---
[0] here.
As far as I could see, there was no change between the two iterations,
so I marked this one as not-applicable, and left the initial one open.
Regards,
Yann E. MORIN.
> package/Config.in | 1 +
> ...ed-time-type-error-when-compiling-by.patch | 47 +++++++++++++++++++
> package/wiringx/Config.in | 18 +++++++
> package/wiringx/wiringx.hash | 3 ++
> package/wiringx/wiringx.mk | 12 +++++
> 5 files changed, 81 insertions(+)
> create mode 100644 package/wiringx/0001-Fix-the-undeclared-time-type-error-when-compiling-by.patch
> create mode 100644 package/wiringx/Config.in
> create mode 100644 package/wiringx/wiringx.hash
> create mode 100644 package/wiringx/wiringx.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 9c037f2ef4..b46bb4654e 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -654,6 +654,7 @@ endmenu
> source "package/wf111/Config.in"
> source "package/wilc-driver/Config.in"
> source "package/wipe/Config.in"
> + source "package/wiringx/Config.in"
> source "package/xorriso/Config.in"
> source "package/xr819-xradio/Config.in"
> endmenu
> diff --git a/package/wiringx/0001-Fix-the-undeclared-time-type-error-when-compiling-by.patch b/package/wiringx/0001-Fix-the-undeclared-time-type-error-when-compiling-by.patch
> new file mode 100644
> index 0000000000..fe0fd56f82
> --- /dev/null
> +++ b/package/wiringx/0001-Fix-the-undeclared-time-type-error-when-compiling-by.patch
> @@ -0,0 +1,47 @@
> +From 3185311331f8fb90ce5dd9376e8b35bff4bb083f Mon Sep 17 00:00:00 2001
> +From: Hanyuan Zhao <hanyuan-z@qq.com>
> +Date: Sat, 14 Oct 2023 14:46:15 +0800
> +Subject: [PATCH] Fix the undeclared time type error when compiling by
> + buildroot
> +
> +Signed-off-by: Hanyuan Zhao <hanyuan-z@qq.com>
> +---
> + src/wiringx.c | 8 ++++----
> + 1 file changed, 4 insertions(+), 4 deletions(-)
> +
> +diff --git a/src/wiringx.c b/src/wiringx.c
> +index e831caa..b424c5b 100644
> +--- a/src/wiringx.c
> ++++ b/src/wiringx.c
> +@@ -119,8 +119,8 @@ static void delayMicrosecondsHard(unsigned int howLong) {
> + tLong.tv_sec = howLong / 1000000;
> + tLong.tv_usec = howLong % 1000000;
> + #else
> +- tLong.tv_sec = (__time_t)howLong / 1000000;
> +- tLong.tv_usec = (__suseconds_t)howLong % 1000000;
> ++ tLong.tv_sec = (time_t)howLong / 1000000;
> ++ tLong.tv_usec = (suseconds_t)howLong % 1000000;
> + #endif
> + timeradd(&tNow, &tLong, &tEnd);
> +
> +@@ -135,7 +135,7 @@ EXPORT void delayMicroseconds(unsigned int howLong) {
> + long int uSecs = howLong % 1000000;
> + unsigned int wSecs = howLong / 1000000;
> + #else
> +- long int uSecs = (__time_t)howLong % 1000000;
> ++ long int uSecs = (time_t)howLong % 1000000;
> + unsigned int wSecs = howLong / 1000000;
> + #endif
> +
> +@@ -147,7 +147,7 @@ EXPORT void delayMicroseconds(unsigned int howLong) {
> + #ifdef _WIN32
> + sleeper.tv_sec = wSecs;
> + #else
> +- sleeper.tv_sec = (__time_t)wSecs;
> ++ sleeper.tv_sec = (time_t)wSecs;
> + #endif
> + sleeper.tv_nsec = (long)(uSecs * 1000L);
> + nanosleep(&sleeper, NULL);
> +--
> +2.34.1
> +
> diff --git a/package/wiringx/Config.in b/package/wiringx/Config.in
> new file mode 100644
> index 0000000000..04b944b217
> --- /dev/null
> +++ b/package/wiringx/Config.in
> @@ -0,0 +1,18 @@
> +config BR2_PACKAGE_WIRINGX
> + bool "wiringX"
> + help
> + wiringX is a cross-platform GPIO interface that allows
> + developers to control the GPIO of various platforms with
> + generic and uniform functions. By using wiringX, the same
> + code will run on all platforms supported by wiringX.
> +
> + Supported vendors:
> +
> + allwinner = <a10, a31s, h3, h5>
> + amlogic = <s805, s905>
> + broadcom = <bcm2711, bcm2835, bcm2836>
> + nXP = <imx6dqrm, imx6sdlrm>
> + rockchip = <rk3399>
> + samsung = <exynos5422>
> +
> + https://wiringx.org/
> diff --git a/package/wiringx/wiringx.hash b/package/wiringx/wiringx.hash
> new file mode 100644
> index 0000000000..b98e707326
> --- /dev/null
> +++ b/package/wiringx/wiringx.hash
> @@ -0,0 +1,3 @@
> +# Locally computed:
> +sha256 62a0bae8f749ad934f1309cdb9df1dbc79a6ce2abc8c6da11040673564c8fb0b wiringx-54030cd9b68ef895439e1444f418524e16cbb4c1.tar.gz
> +sha256 fab3dd6bdab226f1c08630b1dd917e11fcb4ec5e1e020e2c16f83a0a13863e85 LICENSE
> diff --git a/package/wiringx/wiringx.mk b/package/wiringx/wiringx.mk
> new file mode 100644
> index 0000000000..b72046ab60
> --- /dev/null
> +++ b/package/wiringx/wiringx.mk
> @@ -0,0 +1,12 @@
> +################################################################################
> +#
> +# wiringX
> +#
> +################################################################################
> +
> +WIRINGX_VERSION = 54030cd9b68ef895439e1444f418524e16cbb4c1
> +WIRINGX_SITE = $(call github,wiringX,wiringX,$(WIRINGX_VERSION))
> +WIRINGX_LICENSE = MPL-2.0
> +WIRINGX_LICENSE_FILES = LICENSE
> +
> +$(eval $(cmake-package))
> --
> 2.39.3 (Apple Git-145)
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2023-11-18 15:11 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-18 14:53 [Buildroot] [PATCH 1/1 RESEND] package/wiringX: add GPIO library wiringX Hanyuan Zhao via buildroot
2023-11-18 15:11 ` Yann E. MORIN [this message]
2023-11-18 15:29 ` hanyuan via buildroot
-- strict thread matches above, loose matches on Subject: below --
2023-11-18 15:31 hanyuan-z via buildroot
2023-11-18 16:20 ` Yann E. MORIN
2023-11-18 16:27 ` hanyuan-z via buildroot
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=20231118151127.GF996011@scaer \
--to=yann.morin.1998@free.fr \
--cc=buildroot@buildroot.org \
--cc=hanyuan-z@qq.com \
--cc=s.martin49@gmail.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