All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3] linuxptp: Add recipe for linux ptp
@ 2014-02-12 18:26 Sam Nelson
  2014-02-12 18:38 ` Denys Dmytriyenko
  0 siblings, 1 reply; 5+ messages in thread
From: Sam Nelson @ 2014-02-12 18:26 UTC (permalink / raw)
  To: meta-arago

- linuxptp is an implementation of precision time protocol (PTP)

Signed-off-by: Sam Nelson <sam.nelson@ti.com>

---
Change from previous patch
* Moved to recipes-connectivity directory
* Updated to use tarball from release
* Rename recipe to 1.3 version
* Added SRC_URI checksum
---
---
 .../recipes-connectivity/linuxptp/linuxptp_1.3.bb  |   22 ++++++++++++++++++++
 1 file changed, 22 insertions(+)
 create mode 100755 meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb

diff --git a/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
new file mode 100755
index 0000000..33ddb06
--- /dev/null
+++ b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
@@ -0,0 +1,22 @@
+DESCRIPTION = "This software is an implementation of the Precision Time Protocol (PTP) \
+according to IEEE standard 1588 for Linux"
+HOMEPAGE = "http://linuxptp.sourceforge.net/"
+SECTION = "console/network"
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
+DEPENDS += "virtual/kernel"
+PR = "r0"
+
+SRC_URI = "http://iweb.dl.sourceforge.net/project/linuxptp/v${PV}/linuxptp-${PV}.tgz"
+SRC_URI[md5sum] = "664706a86fe1413391d4536080548501"
+SRC_URI[sha256sum] = "b1651186689ef06e56add506bd9bb55a700ca56e3f92ba71d08f3ad12fad3085"
+
+inherit autotools 
+
+EXTRA_OEMAKE = "KBUILD_OUTPUT=${STAGING_KERNEL_DIR} CROSS_COMPILE=${TARGET_PREFIX} ARCH=${TARGET_ARCH}"
+
+do_install() {
+	install -d ${D}${bindir}/
+	install -c -m 755 ${S}/ptp4l ${D}${bindir}/
+}
+
-- 
1.7.9.5



^ permalink raw reply related	[flat|nested] 5+ messages in thread
* Re: [PATCH v3] linuxptp: Add recipe for linux ptp
  2014-02-12 18:26 [PATCH v3] linuxptp: Add recipe for linux ptp Sam Nelson
@ 2014-02-12 18:38 ` Denys Dmytriyenko
  2014-02-28 11:51   ` Nelson, Sam
  0 siblings, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2014-02-12 18:38 UTC (permalink / raw)
  To: Sam Nelson; +Cc: meta-arago

On Wed, Feb 12, 2014 at 01:26:04PM -0500, Sam Nelson wrote:
> - linuxptp is an implementation of precision time protocol (PTP)
> 
> Signed-off-by: Sam Nelson <sam.nelson@ti.com>

Ack, will push soon.

> ---
> Change from previous patch
> * Moved to recipes-connectivity directory
> * Updated to use tarball from release
> * Rename recipe to 1.3 version
> * Added SRC_URI checksum
> ---
> ---
>  .../recipes-connectivity/linuxptp/linuxptp_1.3.bb  |   22 ++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>  create mode 100755 meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
> 
> diff --git a/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
> new file mode 100755
> index 0000000..33ddb06
> --- /dev/null
> +++ b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
> @@ -0,0 +1,22 @@
> +DESCRIPTION = "This software is an implementation of the Precision Time Protocol (PTP) \
> +according to IEEE standard 1588 for Linux"
> +HOMEPAGE = "http://linuxptp.sourceforge.net/"
> +SECTION = "console/network"
> +LICENSE = "GPLv2"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> +DEPENDS += "virtual/kernel"
> +PR = "r0"
> +
> +SRC_URI = "http://iweb.dl.sourceforge.net/project/linuxptp/v${PV}/linuxptp-${PV}.tgz"
> +SRC_URI[md5sum] = "664706a86fe1413391d4536080548501"
> +SRC_URI[sha256sum] = "b1651186689ef06e56add506bd9bb55a700ca56e3f92ba71d08f3ad12fad3085"
> +
> +inherit autotools 
> +
> +EXTRA_OEMAKE = "KBUILD_OUTPUT=${STAGING_KERNEL_DIR} CROSS_COMPILE=${TARGET_PREFIX} ARCH=${TARGET_ARCH}"
> +
> +do_install() {
> +	install -d ${D}${bindir}/
> +	install -c -m 755 ${S}/ptp4l ${D}${bindir}/
> +}
> +
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH v3] linuxptp: Add recipe for linux ptp
  2014-02-12 18:38 ` Denys Dmytriyenko
@ 2014-02-28 11:51   ` Nelson, Sam
  2014-02-28 18:17     ` Denys Dmytriyenko
  0 siblings, 1 reply; 5+ messages in thread
From: Nelson, Sam @ 2014-02-28 11:51 UTC (permalink / raw)
  To: Dmytriyenko, Denys; +Cc: meta-arago

Considering removing dependency of Kernel see my following comments inline.

> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Wednesday, February 12, 2014 1:38 PM
> To: Nelson, Sam
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH v3] linuxptp: Add recipe for linux ptp
> 
> On Wed, Feb 12, 2014 at 01:26:04PM -0500, Sam Nelson wrote:
> > - linuxptp is an implementation of precision time protocol (PTP)
> >
> > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> 
> Ack, will push soon.
> 
> > ---
> > Change from previous patch
> > * Moved to recipes-connectivity directory
> > * Updated to use tarball from release
> > * Rename recipe to 1.3 version
> > * Added SRC_URI checksum
> > ---
> > ---
> >  .../recipes-connectivity/linuxptp/linuxptp_1.3.bb  |   22
> ++++++++++++++++++++
> >  1 file changed, 22 insertions(+)
> >  create mode 100755 meta-arago-extras/recipes-
> connectivity/linuxptp/linuxptp_1.3.bb
> >
> > diff --git a/meta-arago-extras/recipes-
> connectivity/linuxptp/linuxptp_1.3.bb b/meta-arago-extras/recipes-
> connectivity/linuxptp/linuxptp_1.3.bb
> > new file mode 100755
> > index 0000000..33ddb06
> > --- /dev/null
> > +++ b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
> > @@ -0,0 +1,22 @@
> > +DESCRIPTION = "This software is an implementation of the Precision Time
> Protocol (PTP) \
> > +according to IEEE standard 1588 for Linux"
> > +HOMEPAGE = "http://linuxptp.sourceforge.net/"
> > +SECTION = "console/network"
> > +LICENSE = "GPLv2"
> > +LIC_FILES_CHKSUM =
> "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> > +DEPENDS += "virtual/kernel"
[Sam] I am assuming this can be removed? Do I setup dependency on libc-linux headers? How can I do that.
> > +PR = "r0"
> > +
> > +SRC_URI =
> "http://iweb.dl.sourceforge.net/project/linuxptp/v${PV}/linuxptp-${PV}.tgz"
> > +SRC_URI[md5sum] = "664706a86fe1413391d4536080548501"
> > +SRC_URI[sha256sum] =
> "b1651186689ef06e56add506bd9bb55a700ca56e3f92ba71d08f3ad12fad30
> 85"
> > +
> > +inherit autotools
> > +
> > +EXTRA_OEMAKE = "KBUILD_OUTPUT=${STAGING_KERNEL_DIR}
[Sam] How can I remove this dependency and point to the libc-linux-headers? Especially the linux kernel root directory is used here. ( Can I use STAGING_DIR_TARGET? ). Appreciate your comments.
> CROSS_COMPILE=${TARGET_PREFIX} ARCH=${TARGET_ARCH}"
> > +
> > +do_install() {
> > +	install -d ${D}${bindir}/
> > +	install -c -m 755 ${S}/ptp4l ${D}${bindir}/
> > +}
> > +
> > --
> > 1.7.9.5
> >
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH v3] linuxptp: Add recipe for linux ptp
  2014-02-28 11:51   ` Nelson, Sam
@ 2014-02-28 18:17     ` Denys Dmytriyenko
  2014-03-02 21:51       ` Nelson, Sam
  0 siblings, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2014-02-28 18:17 UTC (permalink / raw)
  To: Nelson, Sam; +Cc: meta-arago

On Fri, Feb 28, 2014 at 06:51:41AM -0500, Nelson, Sam wrote:
> Considering removing dependency of Kernel see my following comments inline.
> 
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Wednesday, February 12, 2014 1:38 PM
> > To: Nelson, Sam
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH v3] linuxptp: Add recipe for linux ptp
> > 
> > On Wed, Feb 12, 2014 at 01:26:04PM -0500, Sam Nelson wrote:
> > > - linuxptp is an implementation of precision time protocol (PTP)
> > >
> > > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > 
> > Ack, will push soon.
> > 
> > > ---
> > > Change from previous patch
> > > * Moved to recipes-connectivity directory
> > > * Updated to use tarball from release
> > > * Rename recipe to 1.3 version
> > > * Added SRC_URI checksum
> > > ---
> > > ---
> > >  .../recipes-connectivity/linuxptp/linuxptp_1.3.bb  |   22
> > ++++++++++++++++++++
> > >  1 file changed, 22 insertions(+)
> > >  create mode 100755 meta-arago-extras/recipes-
> > connectivity/linuxptp/linuxptp_1.3.bb
> > >
> > > diff --git a/meta-arago-extras/recipes-
> > connectivity/linuxptp/linuxptp_1.3.bb b/meta-arago-extras/recipes-
> > connectivity/linuxptp/linuxptp_1.3.bb
> > > new file mode 100755
> > > index 0000000..33ddb06
> > > --- /dev/null
> > > +++ b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
> > > @@ -0,0 +1,22 @@
> > > +DESCRIPTION = "This software is an implementation of the Precision Time
> > Protocol (PTP) \
> > > +according to IEEE standard 1588 for Linux"
> > > +HOMEPAGE = "http://linuxptp.sourceforge.net/"
> > > +SECTION = "console/network"
> > > +LICENSE = "GPLv2"
> > > +LIC_FILES_CHKSUM =
> > "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> > > +DEPENDS += "virtual/kernel"
> [Sam] I am assuming this can be removed? Do I setup dependency on libc-linux 
> headers? How can I do that.

Yes, drop the DEPENDS completely.
Dependency on linux-libc-headers is implicit, as those are part of the system 
headers. Similar to eglibc, gcc and everything else you need for compiling the 
code - all those dependencies are taken care of automatically.


> > > +PR = "r0"
> > > +
> > > +SRC_URI =
> > "http://iweb.dl.sourceforge.net/project/linuxptp/v${PV}/linuxptp-${PV}.tgz"
> > > +SRC_URI[md5sum] = "664706a86fe1413391d4536080548501"
> > > +SRC_URI[sha256sum] =
> > "b1651186689ef06e56add506bd9bb55a700ca56e3f92ba71d08f3ad12fad30
> > 85"
> > > +
> > > +inherit autotools
> > > +
> > > +EXTRA_OEMAKE = "KBUILD_OUTPUT=${STAGING_KERNEL_DIR}

> [Sam] How can I remove this dependency and point to the libc-linux-headers? 
> Especially the linux kernel root directory is used here. ( Can I use 
> STAGING_DIR_TARGET? ). Appreciate your comments.

Ok, so I ended up looking into their Makefile and what they try to do there 
with KBUILD_OUTPUT... Besides few errors in their script, what they try to do 
is to check couple header files for availability of specific features and at 
the end pass corresponding defines to their source code. When crosscompiling, 
your target kernel is different from host kernel, hence they try to locate the 
correct /usr/include/linux/net_tstamp.h just to see if it contains definition 
for HWTSTAMP_TX_ONESTEP_SYNC. But that header file hasn't changed in at least 
5 years... They also check time.h to have clock_adjtime.
Anyway, it should be possible to short-cut those checks, but at the end they 
need to define these: -D_GNU_SOURCE -DHAVE_CLOCK_ADJTIME -DHAVE_ONESTEP_SYNC
We can pass those to EXTRA_CFLAGS directly:

EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX} ARCH=${TARGET_ARCH} \
		EXTRA_CFLAGS=\"-D_GNU_SOURCE -DHAVE_CLOCK_ADJTIME \
		-DHAVE_ONESTEP_SYNC\""


> > CROSS_COMPILE=${TARGET_PREFIX} ARCH=${TARGET_ARCH}"
> > > +
> > > +do_install() {
> > > +	install -d ${D}${bindir}/
> > > +	install -c -m 755 ${S}/ptp4l ${D}${bindir}/
> > > +}
> > > +
> > > --
> > > 1.7.9.5
> > >
> > > _______________________________________________
> > > meta-arago mailing list
> > > meta-arago@arago-project.org
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH v3] linuxptp: Add recipe for linux ptp
  2014-02-28 18:17     ` Denys Dmytriyenko
@ 2014-03-02 21:51       ` Nelson, Sam
  0 siblings, 0 replies; 5+ messages in thread
From: Nelson, Sam @ 2014-03-02 21:51 UTC (permalink / raw)
  To: Dmytriyenko, Denys; +Cc: meta-arago

Thanks Denys. I have sent an updated patch.

> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Friday, February 28, 2014 1:18 PM
> To: Nelson, Sam
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH v3] linuxptp: Add recipe for linux ptp
> 
> On Fri, Feb 28, 2014 at 06:51:41AM -0500, Nelson, Sam wrote:
> > Considering removing dependency of Kernel see my following comments
> inline.
> >
> > > -----Original Message-----
> > > From: Dmytriyenko, Denys
> > > Sent: Wednesday, February 12, 2014 1:38 PM
> > > To: Nelson, Sam
> > > Cc: meta-arago@arago-project.org
> > > Subject: Re: [meta-arago] [PATCH v3] linuxptp: Add recipe for linux ptp
> > >
> > > On Wed, Feb 12, 2014 at 01:26:04PM -0500, Sam Nelson wrote:
> > > > - linuxptp is an implementation of precision time protocol (PTP)
> > > >
> > > > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > >
> > > Ack, will push soon.
> > >
> > > > ---
> > > > Change from previous patch
> > > > * Moved to recipes-connectivity directory
> > > > * Updated to use tarball from release
> > > > * Rename recipe to 1.3 version
> > > > * Added SRC_URI checksum
> > > > ---
> > > > ---
> > > >  .../recipes-connectivity/linuxptp/linuxptp_1.3.bb  |   22
> > > ++++++++++++++++++++
> > > >  1 file changed, 22 insertions(+)
> > > >  create mode 100755 meta-arago-extras/recipes-
> > > connectivity/linuxptp/linuxptp_1.3.bb
> > > >
> > > > diff --git a/meta-arago-extras/recipes-
> > > connectivity/linuxptp/linuxptp_1.3.bb b/meta-arago-extras/recipes-
> > > connectivity/linuxptp/linuxptp_1.3.bb
> > > > new file mode 100755
> > > > index 0000000..33ddb06
> > > > --- /dev/null
> > > > +++ b/meta-arago-extras/recipes-
> connectivity/linuxptp/linuxptp_1.3.bb
> > > > @@ -0,0 +1,22 @@
> > > > +DESCRIPTION = "This software is an implementation of the Precision
> Time
> > > Protocol (PTP) \
> > > > +according to IEEE standard 1588 for Linux"
> > > > +HOMEPAGE = "http://linuxptp.sourceforge.net/"
> > > > +SECTION = "console/network"
> > > > +LICENSE = "GPLv2"
> > > > +LIC_FILES_CHKSUM =
> > > "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> > > > +DEPENDS += "virtual/kernel"
> > [Sam] I am assuming this can be removed? Do I setup dependency on libc-
> linux
> > headers? How can I do that.
> 
> Yes, drop the DEPENDS completely.
> Dependency on linux-libc-headers is implicit, as those are part of the system
> headers. Similar to eglibc, gcc and everything else you need for compiling
> the
> code - all those dependencies are taken care of automatically.
> 
> 
> > > > +PR = "r0"
> > > > +
> > > > +SRC_URI =
> > > "http://iweb.dl.sourceforge.net/project/linuxptp/v${PV}/linuxptp-
> ${PV}.tgz"
> > > > +SRC_URI[md5sum] = "664706a86fe1413391d4536080548501"
> > > > +SRC_URI[sha256sum] =
> > >
> "b1651186689ef06e56add506bd9bb55a700ca56e3f92ba71d08f3ad12fad30
> > > 85"
> > > > +
> > > > +inherit autotools
> > > > +
> > > > +EXTRA_OEMAKE = "KBUILD_OUTPUT=${STAGING_KERNEL_DIR}
> 
> > [Sam] How can I remove this dependency and point to the libc-linux-
> headers?
> > Especially the linux kernel root directory is used here. ( Can I use
> > STAGING_DIR_TARGET? ). Appreciate your comments.
> 
> Ok, so I ended up looking into their Makefile and what they try to do there
> with KBUILD_OUTPUT... Besides few errors in their script, what they try to do
> is to check couple header files for availability of specific features and at
> the end pass corresponding defines to their source code. When
> crosscompiling,
> your target kernel is different from host kernel, hence they try to locate the
> correct /usr/include/linux/net_tstamp.h just to see if it contains definition
> for HWTSTAMP_TX_ONESTEP_SYNC. But that header file hasn't changed in
> at least
> 5 years... They also check time.h to have clock_adjtime.
> Anyway, it should be possible to short-cut those checks, but at the end they
> need to define these: -D_GNU_SOURCE -DHAVE_CLOCK_ADJTIME -
> DHAVE_ONESTEP_SYNC
> We can pass those to EXTRA_CFLAGS directly:
> 
> EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX}
> ARCH=${TARGET_ARCH} \
> 		EXTRA_CFLAGS=\"-D_GNU_SOURCE -
> DHAVE_CLOCK_ADJTIME \
> 		-DHAVE_ONESTEP_SYNC\""
> 
> 
> > > CROSS_COMPILE=${TARGET_PREFIX} ARCH=${TARGET_ARCH}"
> > > > +
> > > > +do_install() {
> > > > +	install -d ${D}${bindir}/
> > > > +	install -c -m 755 ${S}/ptp4l ${D}${bindir}/
> > > > +}
> > > > +
> > > > --
> > > > 1.7.9.5
> > > >
> > > > _______________________________________________
> > > > meta-arago mailing list
> > > > meta-arago@arago-project.org
> > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-03-02 21:51 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-12 18:26 [PATCH v3] linuxptp: Add recipe for linux ptp Sam Nelson
2014-02-12 18:38 ` Denys Dmytriyenko
2014-02-28 11:51   ` Nelson, Sam
2014-02-28 18:17     ` Denys Dmytriyenko
2014-03-02 21:51       ` Nelson, Sam

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.