[oe] [meta-oe][PATCH v2] Add analyze-suspend

Sören Brinkmann soren.brinkmann at xilinx.com
Fri Aug 28 15:59:47 UTC 2015


On Fri, 2015-08-28 at 10:03AM +0200, Martin Jansa wrote:
> On Thu, Aug 27, 2015 at 08:44:05PM -0700, Khem Raj wrote:
> > On Thu, Aug 27, 2015 at 2:50 PM, Soren Brinkmann
> > <soren.brinkmann at xilinx.com> wrote:
> > > Signed-off-by: Soren Brinkmann <soren.brinkmann at xilinx.com>
> > > ---
> > > v2:
> > >  - replace 'boot'->'suspend' in commit message
> > >  - document kernel dependencies in recipe
> > >  - pin git commit
> > > ---
> > >  .../analyze-suspend/analyze-suspend_git.bb         | 28 ++++++++++++++++++++++
> > >  1 file changed, 28 insertions(+)
> > >  create mode 100644 meta-oe/recipes-benchmark/analyze-suspend/analyze-suspend_git.bb
> > >
> > > diff --git a/meta-oe/recipes-benchmark/analyze-suspend/analyze-suspend_git.bb b/meta-oe/recipes-benchmark/analyze-suspend/analyze-suspend_git.bb
> > > new file mode 100644
> > > index 000000000000..6932e9fadfc4
> > > --- /dev/null
> > > +++ b/meta-oe/recipes-benchmark/analyze-suspend/analyze-suspend_git.bb
> > > @@ -0,0 +1,28 @@
> > > +SUMMARY = "Analyze Suspend"
> > 
> > If you add  DESCRIPTION with more details it could help folks trying to use this
> > 
> > and please define PV = "3.2+gitr${SRCPV}"
> > 
> > > +HOMEPAGE = "https://01.org/suspendresume"
> > > +LICENSE = "GPLv2"
> > > +LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e"
> > > +
> > > +# Apart from the listed RDEPENDS, analyze-suspend depends on some features
> > > +# provided by the kernel. These options are:
> > > +#   - CONFIG_PM_DEBUG=y
> > > +#   - CONFIG_PM_SLEEP_DEBUG=y
> > > +#   - CONFIG_FTRACE=y
> > > +#   - CONFIG_FUNCTION_TRACER=y
> > > +#   - CONFIG_FUNCTION_GRAPH_TRACER=y
> > > +
> > > +RDEPENDS_${PN} += "python-core python-datetime python-stringold"
> > > +
> > > +SRCREV = "bce3cbec43bc2ce7a8c79b210314dd9d9ac1010b"
> > > +SRC_URI = "git://github.com/01org/suspendresume.git;protocol=https"
> > > +S = "${WORKDIR}/git"
> > > +
> > > +do_install() {
> > > +       install -d -m 0755 ${D}${bindir}
> > > +       install -m 0755 analyze_suspend.py ${D}${bindir}
> > 
> > both these could be squashed into
> > 
> > install -Dm 0755 analyze_suspend.py ${D}${bindir}/analyze_suspend.py
> > 
> > > +
> > > +       install -d -m 0755 ${D}${docdir}/analyze-suspend
> > > +       install -m 0755 README ${D}${docdir}/analyze-suspend
> > 
> > same as above
> 
> and README file shouldn't be executable, use 0644

Good catch. Just fixed that up too together with all the other things
Khem pointed out. I'll send out v3 a little later today.

	Thanks,
	Sören



More information about the Openembedded-devel mailing list