pi-blaster: Add recipe

PATCH In review Alex J Lennon 2 participants 7 messages May 20, 2014 – Jun 11, 2014 1 message not in the archive

lore

Share & tools

lore

Alex J Lennon

author

2014-05-20 15:34 UTC #0 lore raw

cover pi-blaster: Add recipe

Please see following patch for details.

The following changes since commit f3a8693f08f99893453fd1fe282515b2f222c080:

omxplayer: Update to remote's HEAD (2014-05-09 14:56:59 +0300)

are available in the git repository at:

git://github.com/DynamicDevices/meta-raspberrypi ajl/pi-blaster
https://github.com/DynamicDevices/meta-raspberrypi/tree/master

Alex J Lennon (1):
pi-blaster: Added recipe

 recipes-devtools/pi-blaster/files/initscript.patch | 71 ++++++++++++++++++++++
 recipes-devtools/pi-blaster/pi-blaster.inc         | 36 +++++++++++
 recipes-devtools/pi-blaster/pi-blaster_git.bb      |  3 +
 3 files changed, 110 insertions(+)
 create mode 100644 recipes-devtools/pi-blaster/files/initscript.patch
 create mode 100644 recipes-devtools/pi-blaster/pi-blaster.inc
 create mode 100644 recipes-devtools/pi-blaster/pi-blaster_git.bb

Signature · 1 line

1.9.3

Alex J Lennon

author

2014-05-20 14:48 UTC #1 lore raw

PATCH 1/1 pi-blaster: Added recipe

This project enables PWM on the GPIO pins you request of a Raspberry Pi. The technique used is extremely efficient: does not use the CPU and gives very stable pulses.

Signed-off-by Alex J Lennon <[email protected]>

---
 recipes-devtools/pi-blaster/files/initscript.patch | 71 ++++++++++++++++++++++
 recipes-devtools/pi-blaster/pi-blaster.inc         | 36 +++++++++++
 recipes-devtools/pi-blaster/pi-blaster_git.bb      |  3 +
 3 files changed, 110 insertions(+)
 create mode 100644 recipes-devtools/pi-blaster/files/initscript.patch
 create mode 100644 recipes-devtools/pi-blaster/pi-blaster.inc
 create mode 100644 recipes-devtools/pi-blaster/pi-blaster_git.bb

recipes-devtools/pi-blaster/files/initscript.patch new file +71 -0

new file mode 100644
index 0000000..8545546
@@ -0,0 +1,71 @@
+diff -ur git.org/pi-blaster.boot.sh git/pi-blaster.boot.sh
+--- git.org/pi-blaster.boot.sh	2014-05-20 14:49:44.378582168 +0100
++++ git/pi-blaster.boot.sh	2014-05-20 14:51:08.330582386 +0100
+@@ -28,12 +28,12 @@
+ [ -r /etc/default/$NAME ] && . /etc/default/$NAME
+ 
+ # Load the VERBOSE setting and other rcS variables
+-. /lib/init/vars.sh
++#. /lib/init/vars.sh
+ 
+ # Define LSB log_* functions.
+ # Depend on lsb-base (>= 3.2-14) to ensure that this file is present
+ # and status_of_proc is working.
+-. /lib/lsb/init-functions
++#. /lib/lsb/init-functions
+ 
+ #
+ # Function that starts the daemon/service
+@@ -77,48 +77,23 @@
+ 
+ case "$1" in
+   start)
+-	[ "$VERBOSE" != no ] && log_daemon_msg "Starting $DESC" "$NAME"
++	[ "$VERBOSE" != no ] && echo "Starting $DESC" "$NAME"
+ 	do_start
+-	case "$?" in
+-		0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
+-		2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
+-	esac
+ 	;;
+   stop)
+-	[ "$VERBOSE" != no ] && log_daemon_msg "Stopping $DESC" "$NAME"
++	[ "$VERBOSE" != no ] && echo "Stopping $DESC" "$NAME"
+ 	do_stop
+-	case "$?" in
+-		0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
+-		2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
+-	esac
+-	;;
+-  status)
+-	status_of_proc "$DAEMON" "$NAME" && exit 0 || exit $?
+ 	;;
+   restart|force-reload)
+ 	#
+ 	# If the "reload" option is implemented then remove the
+ 	# 'force-reload' alias
+ 	#
+-	log_daemon_msg "Restarting $DESC" "$NAME"
+ 	do_stop
+-	case "$?" in
+-	  0|1)
+-		do_start
+-		case "$?" in
+-			0) log_end_msg 0 ;;
+-			1) log_end_msg 1 ;; # Old process is still running
+-			*) log_end_msg 1 ;; # Failed to start
+-		esac
+-		;;
+-	  *)
+-		# Failed to stop
+-		log_end_msg 1
+-		;;
+-	esac
++	do_start
+ 	;;
+   *)
+-	echo "Usage: $SCRIPTNAME {start|stop|status|restart|force-reload}" >&2
++	echo "Usage: $SCRIPTNAME {start|stop|restart}" >&2
+ 	exit 3
+ 	;;
+ esac

recipes-devtools/pi-blaster/pi-blaster.inc new file +36 -0

new file mode 100644
index 0000000..85a6a4d
@@ -0,0 +1,36 @@
+DESCRIPTION = "This project enables PWM on the GPIO pins you request of a Raspberry Pi."
+HOMEPAGE = "https://github.com/sarfata/pi-blaster/"
+SECTION = "devel/libs"
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = "file://README.md;beginline=138;endline=151;md5=e0f0f92050a87a160939707c60940307"
+
+SRC_URI = "git://github.com/sarfata/pi-blaster \
+	   file://initscript.patch \
+"
+
+
+S = "${WORKDIR}/git"
+
+COMPATIBLE_MACHINE = "raspberrypi"
+
+inherit update-rc.d
+
+INITSCRIPT_PACKAGES = "${PN}"
+INITSCRIPT_NAME_${PN} = "pi-blaster.boot.sh"
+INITSCRIPT_PARAMS_${PN} = "defaults 15 85"
+
+do_configure() {
+  sed -e "s|gcc|${CC}|g" -i Makefile
+}
+
+do_compile() {
+  oe_runmake 
+}
+
+do_install() {
+  install -d ${D}${sbindir}
+  install -m 0755 ${S}/pi-blaster ${D}${sbindir}
+  install -d ${D}${sysconfdir}
+  install -d ${D}${sysconfdir}/init.d
+  install -m 0755 ${S}/pi-blaster.boot.sh ${D}${sysconfdir}/init.d
+}

recipes-devtools/pi-blaster/pi-blaster_git.bb new file +3 -0

new file mode 100644
index 0000000..25eb382
@@ -0,0 +1,3 @@
+require pi-blaster.inc
+
+SRCREV = "177cef81774d6df5c6284c198ff818515281db14"

Signature · 1 line

1.9.3

A message that is not in the archive (sent off-list or not yet received): CAK18fxFixNCCVRt7An58rZSukfmcziprGitd6fFywkfeiupAww@mail.gmail.com

Alex J Lennon

author

2014-06-10 23:41 UTC #3 lore raw

Re: [meta-raspberrypi][PATCH 1/1] pi-blaster: Added recipe

Quoted 5 lines · Andrei Gherzan

Hello,
Found this "guy" here in the meanwhile. Sorry for missing it.

Fair comment. I am somewhat in the dark as to the usefulness of the 0/1 covernote myself. I usually provide it as that's what seems to be required by use of the recommended create-pull-request but sometimes I feel it's overkill (?)

Cheers,

Alex

Andrei Gherzan

2014-06-11 07:39 UTC #4 lore raw

Re: [meta-raspberrypi][PATCH 1/1] pi-blaster: Added recipe

Quoted 4 lines · Alex J Lennon

Quoted 2 lines · Andrei Gherzan
Hello,

Quoted 9 lines

Quoted 3 lines
Found this "guy" here in the meanwhile. Sorry for missing it.
Fair comment. I am somewhat in the dark as to the usefulness of the 0/1 covernote myself. I usually provide it as that's what seems to be required by use of the recommended create-pull-request but sometimes I feel it's overkill (?)

Agreed. I don't use covernote generally. Only for very big pull requests that really need additional info for the whole "package". But for small pullreqs, I only send the actual patches. For me, seems enough.

Signature · 2 lines

*Andrei Gherzan*
m: +40.744.478.414 |  f: +40.31.816.28.12

Andrei Gherzan

2014-06-10 19:51 UTC #5 lore raw

In reply to Alex J Lennon

Hi Alex,

Quoted 15 lines · Alex J Lennon

Please see following patch for details.
The following changes since commit
f3a8693f08f99893453fd1fe282515b2f222c080:
omxplayer: Update to remote's HEAD (2014-05-09 14:56:59 +0300)
are available in the git repository at:
git://github.com/DynamicDevices/meta-raspberrypi ajl/pi-blaster
https://github.com/DynamicDevices/meta-raspberrypi/tree/master
Alex J Lennon (1):
pi-blaster: Added recipe

Quoted 8 lines

 recipes-devtools/pi-blaster/files/initscript.patch | 71
++++++++++++++++++++++
 recipes-devtools/pi-blaster/pi-blaster.inc         | 36 +++++++++++
 recipes-devtools/pi-blaster/pi-blaster_git.bb      |  3 +
 3 files changed, 110 insertions(+)
 create mode 100644 recipes-devtools/pi-blaster/files/initscript.patch
 create mode 100644 recipes-devtools/pi-blaster/pi-blaster.inc
 create mode 100644 recipes-devtools/pi-blaster/pi-blaster_git.bb

You missed the actual patch so I will give my feedback listed below:
1. Refactor commit log.
2. One space left after "oe_runmake".
3. Why exactly do you need oe_runmake after all?
4. I would replace the do install/configure appends with a patch on Makefile (would be useful for the project's maintainer too).
5. Please add comments to patches too. Here is an example:
http://git.yoctoproject.org/cgit/cgit.cgi/meta-raspberrypi/tree/recipes-bsp/rpi-mkimage/rpi-mkimage/open-files-relative-to-script.patch

Thanks,

Signature · 2 lines

*Andrei Gherzan*
m: +40.744.478.414 |  f: +40.31.816.28.12

Alex J Lennon

author

2014-06-11 16:53 UTC #6 lore raw

Quoted 35 lines · Andrei Gherzan

Hi Alex,
On Tue, May 20, 2014 at 5:40 PM, Alex J Lennon <[email protected] <mailto:[email protected]>> wrote:
Please see following patch for details.
The following changes since commit
f3a8693f08f99893453fd1fe282515b2f222c080:
omxplayer: Update to remote's HEAD (2014-05-09 14:56:59 +0300)
are available in the git repository at:
git://github.com/DynamicDevices/meta-raspberrypi
<http://github.com/DynamicDevices/meta-raspberrypi> ajl/pi-blaster
https://github.com/DynamicDevices/meta-raspberrypi/tree/master
Alex J Lennon (1):
pi-blaster: Added recipe
recipes-devtools/pi-blaster/files/initscript.patch | 71
++++++++++++++++++++++
recipes-devtools/pi-blaster/pi-blaster.inc | 36 +++++++++++
recipes-devtools/pi-blaster/pi-blaster_git.bb
<http://pi-blaster_git.bb> | 3 +
3 files changed, 110 insertions(+)
create mode 100644 recipes-devtools/pi-blaster/files/initscript.patch create mode 100644 recipes-devtools/pi-blaster/pi-blaster.inc create mode 100644 recipes-devtools/pi-blaster/pi-blaster_git.bb
<http://pi-blaster_git.bb>

Quoted 9 lines

You missed the actual patch so I will give my feedback listed below:
1. Refactor commit log.
2. One space left after "oe_runmake".
3. Why exactly do you need oe_runmake after all?
4. I would replace the do install/configure appends with a patch on Makefile (would be useful for the project's maintainer too).
5. Please add comments to patches too. Here is an example:
http://git.yoctoproject.org/cgit/cgit.cgi/meta-raspberrypi/tree/recipes-bsp/rpi-mkimage/rpi-mkimage/open-files-relative-to-script.patch

Thanks for the comprehensive response Andrei, and also for the link to the OpenEmbedded Commit Patch Message Guidelines - very useful.

I've reworked the pi-blaster recipe, hopefully in line with your comments, and pushed a second patch-set up to the review server.

Regards,

Alex

Andrei Gherzan

2014-06-11 16:59 UTC #7 lore raw

Quoted 48 lines · Alex J Lennon

On 10/06/2014 20:50, Andrei Gherzan wrote:
Hi Alex,
Quoted 15 lines · Alex J Lennon
Please see following patch for details.
The following changes since commit
f3a8693f08f99893453fd1fe282515b2f222c080:
omxplayer: Update to remote's HEAD (2014-05-09 14:56:59 +0300)
are available in the git repository at:
git://github.com/DynamicDevices/meta-raspberrypi ajl/pi-blaster
https://github.com/DynamicDevices/meta-raspberrypi/tree/master
Alex J Lennon (1):
pi-blaster: Added recipe
Quoted 8 lines
 recipes-devtools/pi-blaster/files/initscript.patch | 71
++++++++++++++++++++++
 recipes-devtools/pi-blaster/pi-blaster.inc         | 36 +++++++++++
 recipes-devtools/pi-blaster/pi-blaster_git.bb      |  3 +
 3 files changed, 110 insertions(+)
 create mode 100644 recipes-devtools/pi-blaster/files/initscript.patch
 create mode 100644 recipes-devtools/pi-blaster/pi-blaster.inc
 create mode 100644 recipes-devtools/pi-blaster/pi-blaster_git.bb
You missed the actual patch so I will give my feedback listed below:
1. Refactor commit log.
2. One space left after "oe_runmake".
3. Why exactly do you need oe_runmake after all?
4. I would replace the do install/configure appends with a patch on Makefile (would be useful for the project's maintainer too).
5. Please add comments to patches too. Here is an example:
http://git.yoctoproject.org/cgit/cgit.cgi/meta-raspberrypi/tree/recipes-bsp/rpi-mkimage/rpi-mkimage/open-files-relative-to-script.patch
Thanks for the comprehensive response Andrei, and also for the link to the OpenEmbedded Commit Patch Message Guidelines - very useful.

Quoted 8 lines

I've reworked the pi-blaster recipe, hopefully in line with your comments, and pushed a second patch-set up to the review server.
Regards,
Alex

Great work. Everything seems right now. Merged.

Signature · 2 lines

*Andrei Gherzan*
m: +40.744.478.414 |  f: +40.31.816.28.12