> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Monday, March 03, 2014 9:54 PM
> To: Nelson, Sam
> Cc: [email protected]
> Subject: Re: [meta-arago] [PATCH v4] linuxptp: Add recipe for linux ptp
>
> On Sun, Mar 02, 2014 at 04:50:46PM -0500, Sam Nelson wrote:
> > - linuxptp is an implementation of precision time protocol (PTP)
>
> Much better! Sorry, couple of comments I missed the last time.
>
>
> > Signed-off-by: Sam Nelson <[email protected]>
> >
> > ---
> > Change from previous patch
> > * Removed dependency on kernel
> > * Added flags for CLOCKADJTIME & ONESTEPSYNC
> > ---
> > ---
> > .../recipes-connectivity/linuxptp/linuxptp_1.3.bb | 23
> ++++++++++++++++++++
> > 1 file changed, 23 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..0a36631
> > --- /dev/null
> > +++ b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
>
> The latest version is 1.4 and you are adding 1.3 - I only checked 1.4, but
> don't think it changed that much. Do you specifically need 1.3 or 1.4 will
> also work?
[Sam] Currently system tested with 1.3. Probably will consider 1.4 later. Is
that Ok?
>
>
> > @@ -0,0 +1,23 @@
> > +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"
> > +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
>
> I didn't see any autotools file in the package - no configure script, no
> Makefile.am, no acinclude.m4 etc., etc.
[Sam] I guess this can be removed.
>
>
> > +EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX}
> ARCH=${TARGET_ARCH} \
> > + EXTRA_CFLAGS=\"-D_GNU_SOURCE -
> DHAVE_CLOCK_ADJTIME \
> > + -DHAVE_ONESTEP_SYNC\""
> > +
> > +do_install() {
> > + install -d ${D}${bindir}/
> > + install -c -m 755 ${S}/ptp4l ${D}${bindir}/
> > +}
>
> Other than that, looks much cleaner. If you are tired re-iterating it, just
> answer my 2 questions above and I'll make the amends myself when
> applying your
> patch...
>
> --
> Denys
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
