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: [email protected] > 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: [email protected] > > > 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 <[email protected]> > > > > > > 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 > > > > [email protected] > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago _______________________________________________ meta-arago mailing list [email protected] http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
