[OE-core][PATCH] igt-gpu-tools: add new package

[OE-core][PATCH] igt-gpu-tools: add new package

[Arthur She]

igt-gpu-tools is a collection of tools for development and testing of the DRM
drivers. (https://gitlab.freedesktop.org/drm/igt-gpu-tools)
This recipe is originally copied from meta-intel and added the following
modifications
1. Enable ARM architecture
2. Enable igt-runner and Chamelium
3. Build git HEAD code instead of a specific version
The original recipe in the meta-intel will be removed and replaced by
this one once it get merged.

Signed-off-by: Arthur She <arthur.she@linaro.org>
---
 .../igt-gpu-tools/igt-gpu-tools_git.bb        | 47 +++++++++++++++++++
 1 file changed, 47 insertions(+)
 create mode 100644 meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb

diff --git a/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
new file mode 100644
index 0000000000..dc9537c9e3
--- /dev/null
+++ b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
@@ -0,0 +1,47 @@
+SUMMARY = "IGT GPU Tools"
+DESCRIPTION = "IGT GPU Tools is a collection of tools for development and testing of the DRM drivers"
+
+LIC_FILES_CHKSUM = "file://COPYING;md5=67bfee4df38fa6ecbe3a675c552d4c08"
+
+LICENSE = "MIT"
+
+inherit meson
+
+SRCREV = "d16ad07e7f2a028e14d61f570931c87fa5ce404c"
+PV = "1.25+git${SRCPV}"
+
+SRC_URI = "git://gitlab.freedesktop.org/drm/igt-gpu-tools.git;protocol=https"
+
+S = "${WORKDIR}/git"
+
+DEPENDS += "libdrm libpciaccess cairo udev glib-2.0 procps libunwind kmod openssl xmlrpc-c gsl elfutils alsa-lib json-c bison-native"
+RDEPENDS_${PN} += "bash"
+RDEPENDS_${PN}-tests += "bash"
+
+PACKAGE_BEFORE_PN = "${PN}-benchmarks ${PN}-tests"
+
+EXTRA_OEMESON = "-Ddocs=disabled -Drunner=enabled -Dchamelium=enabled"
+COMPATIBLE_HOST = "(x86_64.*|i.86.*|arm.*|aarch64).*-linux"
+COMPATIBLE_HOST_libc-musl_class-target = "null"
+SECURITY_LDFLAGS = "${SECURITY_X_LDFLAGS}"
+
+gputools_sysroot_preprocess() {
+	rm -f ${SYSROOT_DESTDIR}${libdir}/pkgconfig/intel-gen4asm.pc
+}
+SYSROOT_PREPROCESS_FUNCS += "gputools_sysroot_preprocess"
+
+do_install_append() {
+    install -d ${D}/usr/share/${BPN}/scripts
+    install ${S}/scripts/run-tests.sh ${D}/usr/share/${BPN}/scripts
+    install -d ${D}/usr/share/${BPN}/runner
+    install -D ${B}/runner/igt_runner ${D}/usr/share/${BPN}/runner
+    install -D ${B}/runner/igt_resume ${D}/usr/share/${BPN}/runner
+}
+
+FILES_${PN}-benchmarks += "${libexecdir}/${BPN}/benchmarks"
+FILES_${PN}-tests += "\
+        ${libexecdir}/${BPN}/*\
+        ${datadir}/${BPN}/1080p-right.png\
+        ${datadir}/${BPN}/1080p-left.png\
+        ${datadir}/${BPN}/pass.png\
+        ${datadir}/${BPN}/test-list.txt"
-- 
2.25.1

Re: [OE-core][PATCH] igt-gpu-tools: add new package

[Alexander Kanavin]

[-- Attachment #1: Type: text/plain, Size: 3013 bytes --]

This should go to meta-oe, rather than core?

Alex

On Mon 15. Jun 2020 at 8.00, Arthur She <arthur.she@linaro.org> wrote:

> igt-gpu-tools is a collection of tools for development and testing of the
> DRM
> drivers. (https://gitlab.freedesktop.org/drm/igt-gpu-tools)
> This recipe is originally copied from meta-intel and added the following
> modifications
> 1. Enable ARM architecture
> 2. Enable igt-runner and Chamelium
> 3. Build git HEAD code instead of a specific version
> The original recipe in the meta-intel will be removed and replaced by
> this one once it get merged.
>
> Signed-off-by: Arthur She <arthur.she@linaro.org>
> ---
>  .../igt-gpu-tools/igt-gpu-tools_git.bb        | 47 +++++++++++++++++++
>  1 file changed, 47 insertions(+)
>  create mode 100644 meta/recipes-graphics/igt-gpu-tools/
> igt-gpu-tools_git.bb
>
> diff --git a/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
> b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
> new file mode 100644
> index 0000000000..dc9537c9e3
> --- /dev/null
> +++ b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
> @@ -0,0 +1,47 @@
> +SUMMARY = "IGT GPU Tools"
> +DESCRIPTION = "IGT GPU Tools is a collection of tools for development and
> testing of the DRM drivers"
> +
> +LIC_FILES_CHKSUM = "file://COPYING;md5=67bfee4df38fa6ecbe3a675c552d4c08"
> +
> +LICENSE = "MIT"
> +
> +inherit meson
> +
> +SRCREV = "d16ad07e7f2a028e14d61f570931c87fa5ce404c"
> +PV = "1.25+git${SRCPV}"
> +
> +SRC_URI = "git://
> gitlab.freedesktop.org/drm/igt-gpu-tools.git;protocol=https"
> +
> +S = "${WORKDIR}/git"
> +
> +DEPENDS += "libdrm libpciaccess cairo udev glib-2.0 procps libunwind kmod
> openssl xmlrpc-c gsl elfutils alsa-lib json-c bison-native"
> +RDEPENDS_${PN} += "bash"
> +RDEPENDS_${PN}-tests += "bash"
> +
> +PACKAGE_BEFORE_PN = "${PN}-benchmarks ${PN}-tests"
> +
> +EXTRA_OEMESON = "-Ddocs=disabled -Drunner=enabled -Dchamelium=enabled"
> +COMPATIBLE_HOST = "(x86_64.*|i.86.*|arm.*|aarch64).*-linux"
> +COMPATIBLE_HOST_libc-musl_class-target = "null"
> +SECURITY_LDFLAGS = "${SECURITY_X_LDFLAGS}"
> +
> +gputools_sysroot_preprocess() {
> +       rm -f ${SYSROOT_DESTDIR}${libdir}/pkgconfig/intel-gen4asm.pc
> +}
> +SYSROOT_PREPROCESS_FUNCS += "gputools_sysroot_preprocess"
> +
> +do_install_append() {
> +    install -d ${D}/usr/share/${BPN}/scripts
> +    install ${S}/scripts/run-tests.sh ${D}/usr/share/${BPN}/scripts
> +    install -d ${D}/usr/share/${BPN}/runner
> +    install -D ${B}/runner/igt_runner ${D}/usr/share/${BPN}/runner
> +    install -D ${B}/runner/igt_resume ${D}/usr/share/${BPN}/runner
> +}
> +
> +FILES_${PN}-benchmarks += "${libexecdir}/${BPN}/benchmarks"
> +FILES_${PN}-tests += "\
> +        ${libexecdir}/${BPN}/*\
> +        ${datadir}/${BPN}/1080p-right.png\
> +        ${datadir}/${BPN}/1080p-left.png\
> +        ${datadir}/${BPN}/pass.png\
> +        ${datadir}/${BPN}/test-list.txt"
> --
> 2.25.1
>
> 
>

[-- Attachment #2: Type: text/html, Size: 4407 bytes --]

Re: [OE-core][PATCH] igt-gpu-tools: add new package

[Denys Dmytriyenko]

https://lists.openembedded.org/g/openembedded-devel/topic/74775294

But I tend to agree.

-- 
Denys


On Mon, Jun 15, 2020 at 09:45:25AM +0200, Alexander Kanavin wrote:
> This should go to meta-oe, rather than core?
> 
> Alex
> 
> On Mon 15. Jun 2020 at 8.00, Arthur She <arthur.she@linaro.org> wrote:
> 
> > igt-gpu-tools is a collection of tools for development and testing of the
> > DRM
> > drivers. (https://gitlab.freedesktop.org/drm/igt-gpu-tools)
> > This recipe is originally copied from meta-intel and added the following
> > modifications
> > 1. Enable ARM architecture
> > 2. Enable igt-runner and Chamelium
> > 3. Build git HEAD code instead of a specific version
> > The original recipe in the meta-intel will be removed and replaced by
> > this one once it get merged.
> >
> > Signed-off-by: Arthur She <arthur.she@linaro.org>
> > ---
> >  .../igt-gpu-tools/igt-gpu-tools_git.bb        | 47 +++++++++++++++++++
> >  1 file changed, 47 insertions(+)
> >  create mode 100644 meta/recipes-graphics/igt-gpu-tools/
> > igt-gpu-tools_git.bb
> >
> > diff --git a/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
> > b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
> > new file mode 100644
> > index 0000000000..dc9537c9e3
> > --- /dev/null
> > +++ b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
> > @@ -0,0 +1,47 @@
> > +SUMMARY = "IGT GPU Tools"
> > +DESCRIPTION = "IGT GPU Tools is a collection of tools for development and
> > testing of the DRM drivers"
> > +
> > +LIC_FILES_CHKSUM = "file://COPYING;md5=67bfee4df38fa6ecbe3a675c552d4c08"
> > +
> > +LICENSE = "MIT"
> > +
> > +inherit meson
> > +
> > +SRCREV = "d16ad07e7f2a028e14d61f570931c87fa5ce404c"
> > +PV = "1.25+git${SRCPV}"
> > +
> > +SRC_URI = "git://
> > gitlab.freedesktop.org/drm/igt-gpu-tools.git;protocol=https"
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +DEPENDS += "libdrm libpciaccess cairo udev glib-2.0 procps libunwind kmod
> > openssl xmlrpc-c gsl elfutils alsa-lib json-c bison-native"
> > +RDEPENDS_${PN} += "bash"
> > +RDEPENDS_${PN}-tests += "bash"
> > +
> > +PACKAGE_BEFORE_PN = "${PN}-benchmarks ${PN}-tests"
> > +
> > +EXTRA_OEMESON = "-Ddocs=disabled -Drunner=enabled -Dchamelium=enabled"
> > +COMPATIBLE_HOST = "(x86_64.*|i.86.*|arm.*|aarch64).*-linux"
> > +COMPATIBLE_HOST_libc-musl_class-target = "null"
> > +SECURITY_LDFLAGS = "${SECURITY_X_LDFLAGS}"
> > +
> > +gputools_sysroot_preprocess() {
> > +       rm -f ${SYSROOT_DESTDIR}${libdir}/pkgconfig/intel-gen4asm.pc
> > +}
> > +SYSROOT_PREPROCESS_FUNCS += "gputools_sysroot_preprocess"
> > +
> > +do_install_append() {
> > +    install -d ${D}/usr/share/${BPN}/scripts
> > +    install ${S}/scripts/run-tests.sh ${D}/usr/share/${BPN}/scripts
> > +    install -d ${D}/usr/share/${BPN}/runner
> > +    install -D ${B}/runner/igt_runner ${D}/usr/share/${BPN}/runner
> > +    install -D ${B}/runner/igt_resume ${D}/usr/share/${BPN}/runner
> > +}
> > +
> > +FILES_${PN}-benchmarks += "${libexecdir}/${BPN}/benchmarks"
> > +FILES_${PN}-tests += "\
> > +        ${libexecdir}/${BPN}/*\
> > +        ${datadir}/${BPN}/1080p-right.png\
> > +        ${datadir}/${BPN}/1080p-left.png\
> > +        ${datadir}/${BPN}/pass.png\
> > +        ${datadir}/${BPN}/test-list.txt"
> > --
> > 2.25.1
> >
> > 
> >

> 

Re: [OE-core][PATCH] igt-gpu-tools: add new package

[Arthur She]

[-- Attachment #1: Type: text/plain, Size: 3905 bytes --]

Hi Alexander and Denys,
Thanks for your feedback.
I have no problem submitting it to meta-oe.
Since the meta-intel maintainer requested to submit it to oe-core, do you
mind sharing some thought about the reason?

Thank you so much,
Arthur

On Mon, Jun 15, 2020 at 12:56 AM Denys Dmytriyenko <denis@denix.org> wrote:

> https://lists.openembedded.org/g/openembedded-devel/topic/74775294
>
> But I tend to agree.
>
> --
> Denys
>
>
> On Mon, Jun 15, 2020 at 09:45:25AM +0200, Alexander Kanavin wrote:
> > This should go to meta-oe, rather than core?
> >
> > Alex
> >
> > On Mon 15. Jun 2020 at 8.00, Arthur She <arthur.she@linaro.org> wrote:
> >
> > > igt-gpu-tools is a collection of tools for development and testing of
> the
> > > DRM
> > > drivers. (https://gitlab.freedesktop.org/drm/igt-gpu-tools)
> > > This recipe is originally copied from meta-intel and added the
> following
> > > modifications
> > > 1. Enable ARM architecture
> > > 2. Enable igt-runner and Chamelium
> > > 3. Build git HEAD code instead of a specific version
> > > The original recipe in the meta-intel will be removed and replaced by
> > > this one once it get merged.
> > >
> > > Signed-off-by: Arthur She <arthur.she@linaro.org>
> > > ---
> > >  .../igt-gpu-tools/igt-gpu-tools_git.bb        | 47
> +++++++++++++++++++
> > >  1 file changed, 47 insertions(+)
> > >  create mode 100644 meta/recipes-graphics/igt-gpu-tools/
> > > igt-gpu-tools_git.bb
> > >
> > > diff --git a/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
> > > b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
> > > new file mode 100644
> > > index 0000000000..dc9537c9e3
> > > --- /dev/null
> > > +++ b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
> > > @@ -0,0 +1,47 @@
> > > +SUMMARY = "IGT GPU Tools"
> > > +DESCRIPTION = "IGT GPU Tools is a collection of tools for development
> and
> > > testing of the DRM drivers"
> > > +
> > > +LIC_FILES_CHKSUM =
> "file://COPYING;md5=67bfee4df38fa6ecbe3a675c552d4c08"
> > > +
> > > +LICENSE = "MIT"
> > > +
> > > +inherit meson
> > > +
> > > +SRCREV = "d16ad07e7f2a028e14d61f570931c87fa5ce404c"
> > > +PV = "1.25+git${SRCPV}"
> > > +
> > > +SRC_URI = "git://
> > > gitlab.freedesktop.org/drm/igt-gpu-tools.git;protocol=https"
> > > +
> > > +S = "${WORKDIR}/git"
> > > +
> > > +DEPENDS += "libdrm libpciaccess cairo udev glib-2.0 procps libunwind
> kmod
> > > openssl xmlrpc-c gsl elfutils alsa-lib json-c bison-native"
> > > +RDEPENDS_${PN} += "bash"
> > > +RDEPENDS_${PN}-tests += "bash"
> > > +
> > > +PACKAGE_BEFORE_PN = "${PN}-benchmarks ${PN}-tests"
> > > +
> > > +EXTRA_OEMESON = "-Ddocs=disabled -Drunner=enabled -Dchamelium=enabled"
> > > +COMPATIBLE_HOST = "(x86_64.*|i.86.*|arm.*|aarch64).*-linux"
> > > +COMPATIBLE_HOST_libc-musl_class-target = "null"
> > > +SECURITY_LDFLAGS = "${SECURITY_X_LDFLAGS}"
> > > +
> > > +gputools_sysroot_preprocess() {
> > > +       rm -f ${SYSROOT_DESTDIR}${libdir}/pkgconfig/intel-gen4asm.pc
> > > +}
> > > +SYSROOT_PREPROCESS_FUNCS += "gputools_sysroot_preprocess"
> > > +
> > > +do_install_append() {
> > > +    install -d ${D}/usr/share/${BPN}/scripts
> > > +    install ${S}/scripts/run-tests.sh ${D}/usr/share/${BPN}/scripts
> > > +    install -d ${D}/usr/share/${BPN}/runner
> > > +    install -D ${B}/runner/igt_runner ${D}/usr/share/${BPN}/runner
> > > +    install -D ${B}/runner/igt_resume ${D}/usr/share/${BPN}/runner
> > > +}
> > > +
> > > +FILES_${PN}-benchmarks += "${libexecdir}/${BPN}/benchmarks"
> > > +FILES_${PN}-tests += "\
> > > +        ${libexecdir}/${BPN}/*\
> > > +        ${datadir}/${BPN}/1080p-right.png\
> > > +        ${datadir}/${BPN}/1080p-left.png\
> > > +        ${datadir}/${BPN}/pass.png\
> > > +        ${datadir}/${BPN}/test-list.txt"
> > > --
> > > 2.25.1
> > >
> > >
> > >
>
> > 
>
>

[-- Attachment #2: Type: text/html, Size: 5980 bytes --]

Re: [OE-core][PATCH] igt-gpu-tools: add new package

[Alexander Kanavin]

[-- Attachment #1: Type: text/plain, Size: 4207 bytes --]

Usually it's the other way around - what is the reason to have this recipe
in core?

Alex

On Tue, 16 Jun 2020 at 06:45, Arthur She <arthur.she@linaro.org> wrote:

> Hi Alexander and Denys,
> Thanks for your feedback.
> I have no problem submitting it to meta-oe.
> Since the meta-intel maintainer requested to submit it to oe-core, do you
> mind sharing some thought about the reason?
>
> Thank you so much,
> Arthur
>
> On Mon, Jun 15, 2020 at 12:56 AM Denys Dmytriyenko <denis@denix.org>
> wrote:
>
>> https://lists.openembedded.org/g/openembedded-devel/topic/74775294
>>
>> But I tend to agree.
>>
>> --
>> Denys
>>
>>
>> On Mon, Jun 15, 2020 at 09:45:25AM +0200, Alexander Kanavin wrote:
>> > This should go to meta-oe, rather than core?
>> >
>> > Alex
>> >
>> > On Mon 15. Jun 2020 at 8.00, Arthur She <arthur.she@linaro.org> wrote:
>> >
>> > > igt-gpu-tools is a collection of tools for development and testing of
>> the
>> > > DRM
>> > > drivers. (https://gitlab.freedesktop.org/drm/igt-gpu-tools)
>> > > This recipe is originally copied from meta-intel and added the
>> following
>> > > modifications
>> > > 1. Enable ARM architecture
>> > > 2. Enable igt-runner and Chamelium
>> > > 3. Build git HEAD code instead of a specific version
>> > > The original recipe in the meta-intel will be removed and replaced by
>> > > this one once it get merged.
>> > >
>> > > Signed-off-by: Arthur She <arthur.she@linaro.org>
>> > > ---
>> > >  .../igt-gpu-tools/igt-gpu-tools_git.bb        | 47
>> +++++++++++++++++++
>> > >  1 file changed, 47 insertions(+)
>> > >  create mode 100644 meta/recipes-graphics/igt-gpu-tools/
>> > > igt-gpu-tools_git.bb
>> > >
>> > > diff --git a/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
>> > > b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
>> > > new file mode 100644
>> > > index 0000000000..dc9537c9e3
>> > > --- /dev/null
>> > > +++ b/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb
>> > > @@ -0,0 +1,47 @@
>> > > +SUMMARY = "IGT GPU Tools"
>> > > +DESCRIPTION = "IGT GPU Tools is a collection of tools for
>> development and
>> > > testing of the DRM drivers"
>> > > +
>> > > +LIC_FILES_CHKSUM =
>> "file://COPYING;md5=67bfee4df38fa6ecbe3a675c552d4c08"
>> > > +
>> > > +LICENSE = "MIT"
>> > > +
>> > > +inherit meson
>> > > +
>> > > +SRCREV = "d16ad07e7f2a028e14d61f570931c87fa5ce404c"
>> > > +PV = "1.25+git${SRCPV}"
>> > > +
>> > > +SRC_URI = "git://
>> > > gitlab.freedesktop.org/drm/igt-gpu-tools.git;protocol=https"
>> > > +
>> > > +S = "${WORKDIR}/git"
>> > > +
>> > > +DEPENDS += "libdrm libpciaccess cairo udev glib-2.0 procps libunwind
>> kmod
>> > > openssl xmlrpc-c gsl elfutils alsa-lib json-c bison-native"
>> > > +RDEPENDS_${PN} += "bash"
>> > > +RDEPENDS_${PN}-tests += "bash"
>> > > +
>> > > +PACKAGE_BEFORE_PN = "${PN}-benchmarks ${PN}-tests"
>> > > +
>> > > +EXTRA_OEMESON = "-Ddocs=disabled -Drunner=enabled
>> -Dchamelium=enabled"
>> > > +COMPATIBLE_HOST = "(x86_64.*|i.86.*|arm.*|aarch64).*-linux"
>> > > +COMPATIBLE_HOST_libc-musl_class-target = "null"
>> > > +SECURITY_LDFLAGS = "${SECURITY_X_LDFLAGS}"
>> > > +
>> > > +gputools_sysroot_preprocess() {
>> > > +       rm -f ${SYSROOT_DESTDIR}${libdir}/pkgconfig/intel-gen4asm.pc
>> > > +}
>> > > +SYSROOT_PREPROCESS_FUNCS += "gputools_sysroot_preprocess"
>> > > +
>> > > +do_install_append() {
>> > > +    install -d ${D}/usr/share/${BPN}/scripts
>> > > +    install ${S}/scripts/run-tests.sh ${D}/usr/share/${BPN}/scripts
>> > > +    install -d ${D}/usr/share/${BPN}/runner
>> > > +    install -D ${B}/runner/igt_runner ${D}/usr/share/${BPN}/runner
>> > > +    install -D ${B}/runner/igt_resume ${D}/usr/share/${BPN}/runner
>> > > +}
>> > > +
>> > > +FILES_${PN}-benchmarks += "${libexecdir}/${BPN}/benchmarks"
>> > > +FILES_${PN}-tests += "\
>> > > +        ${libexecdir}/${BPN}/*\
>> > > +        ${datadir}/${BPN}/1080p-right.png\
>> > > +        ${datadir}/${BPN}/1080p-left.png\
>> > > +        ${datadir}/${BPN}/pass.png\
>> > > +        ${datadir}/${BPN}/test-list.txt"
>> > > --
>> > > 2.25.1
>> > >
>> > >
>> > >
>>
>> > 
>>
>>

[-- Attachment #2: Type: text/html, Size: 6484 bytes --]

Re: [OE-core][PATCH] igt-gpu-tools: add new package

[Anuj Mittal]

Hi Alex

On Tue, 2020-06-16 at 09:04 +0200, Alexander Kanavin wrote:
> Usually it's the other way around - what is the reason to have this
> recipe in core?
> 

I had requested Arthur to send this to oe-core since this provides
a core validation functionality that is needed by all providing DRM
drivers and is applicable not just to a single arch.

It's a well maintained project and I can keep the recipe updated and
working. If it can be in oe-core, I can remove it from meta-intel layer
and avoid having a duplicate recipe.

Thanks,

Anuj

> Alex
> 
> On Tue, 16 Jun 2020 at 06:45, Arthur She <arthur.she@linaro.org>
> wrote:
> > Hi Alexander and Denys,
> > Thanks for your feedback.
> > I have no problem submitting it to meta-oe.
> > Since the meta-intel maintainer requested to submit it to oe-core,
> > do you mind sharing some thought about the reason?
> > 
> > Thank you so much,
> > Arthur
> > 
> > On Mon, Jun 15, 2020 at 12:56 AM Denys Dmytriyenko <denis@denix.org
> > > wrote:
> > > https://lists.openembedded.org/g/openembedded-devel/topic/74775294
> > > 
> > > But I tend to agree.
> > > 

Re: [OE-core][PATCH] igt-gpu-tools: add new package

[Arthur She]

[-- Attachment #1: Type: text/plain, Size: 1423 bytes --]

Hi Alex,
Would it be a sufficient reason for doing that?
Or any thought could be shared from you?

Thank you so much,
Arthur

On Tue, Jun 16, 2020 at 12:16 AM Mittal, Anuj <anuj.mittal@intel.com> wrote:

> Hi Alex
>
> On Tue, 2020-06-16 at 09:04 +0200, Alexander Kanavin wrote:
> > Usually it's the other way around - what is the reason to have this
> > recipe in core?
> >
>
> I had requested Arthur to send this to oe-core since this provides
> a core validation functionality that is needed by all providing DRM
> drivers and is applicable not just to a single arch.
>
> It's a well maintained project and I can keep the recipe updated and
> working. If it can be in oe-core, I can remove it from meta-intel layer
> and avoid having a duplicate recipe.
>
> Thanks,
>
> Anuj
>
> > Alex
> >
> > On Tue, 16 Jun 2020 at 06:45, Arthur She <arthur.she@linaro.org>
> > wrote:
> > > Hi Alexander and Denys,
> > > Thanks for your feedback.
> > > I have no problem submitting it to meta-oe.
> > > Since the meta-intel maintainer requested to submit it to oe-core,
> > > do you mind sharing some thought about the reason?
> > >
> > > Thank you so much,
> > > Arthur
> > >
> > > On Mon, Jun 15, 2020 at 12:56 AM Denys Dmytriyenko <denis@denix.org
> > > > wrote:
> > > > https://lists.openembedded.org/g/openembedded-devel/topic/74775294
> > > >
> > > > But I tend to agree.
> > > >
>

[-- Attachment #2: Type: text/html, Size: 2219 bytes --]

Re: [OE-core][PATCH] igt-gpu-tools: add new package

[Alexander Kanavin]

[-- Attachment #1: Type: text/plain, Size: 1661 bytes --]

I am fine with that, please resend with Anuj's rationale included into the
commit message.

Alex

On Thu, 18 Jun 2020 at 07:27, Arthur She <arthur.she@linaro.org> wrote:

> Hi Alex,
> Would it be a sufficient reason for doing that?
> Or any thought could be shared from you?
>
> Thank you so much,
> Arthur
>
> On Tue, Jun 16, 2020 at 12:16 AM Mittal, Anuj <anuj.mittal@intel.com>
> wrote:
>
>> Hi Alex
>>
>> On Tue, 2020-06-16 at 09:04 +0200, Alexander Kanavin wrote:
>> > Usually it's the other way around - what is the reason to have this
>> > recipe in core?
>> >
>>
>> I had requested Arthur to send this to oe-core since this provides
>> a core validation functionality that is needed by all providing DRM
>> drivers and is applicable not just to a single arch.
>>
>> It's a well maintained project and I can keep the recipe updated and
>> working. If it can be in oe-core, I can remove it from meta-intel layer
>> and avoid having a duplicate recipe.
>>
>> Thanks,
>>
>> Anuj
>>
>> > Alex
>> >
>> > On Tue, 16 Jun 2020 at 06:45, Arthur She <arthur.she@linaro.org>
>> > wrote:
>> > > Hi Alexander and Denys,
>> > > Thanks for your feedback.
>> > > I have no problem submitting it to meta-oe.
>> > > Since the meta-intel maintainer requested to submit it to oe-core,
>> > > do you mind sharing some thought about the reason?
>> > >
>> > > Thank you so much,
>> > > Arthur
>> > >
>> > > On Mon, Jun 15, 2020 at 12:56 AM Denys Dmytriyenko <denis@denix.org
>> > > > wrote:
>> > > > https://lists.openembedded.org/g/openembedded-devel/topic/74775294
>> > > >
>> > > > But I tend to agree.
>> > > >
>>
>

[-- Attachment #2: Type: text/html, Size: 2714 bytes --]

Re: [OE-core][PATCH] igt-gpu-tools: add new package

[Arthur She]

[-- Attachment #1: Type: text/plain, Size: 1920 bytes --]

Hi Alex,
Thank you so much. I will update the commit message and send the patch v2
out.

Thanks,
Arthur

On Thu, Jun 18, 2020 at 8:25 AM Alexander Kanavin <alex.kanavin@gmail.com>
wrote:

> I am fine with that, please resend with Anuj's rationale included into the
> commit message.
>
> Alex
>
> On Thu, 18 Jun 2020 at 07:27, Arthur She <arthur.she@linaro.org> wrote:
>
>> Hi Alex,
>> Would it be a sufficient reason for doing that?
>> Or any thought could be shared from you?
>>
>> Thank you so much,
>> Arthur
>>
>> On Tue, Jun 16, 2020 at 12:16 AM Mittal, Anuj <anuj.mittal@intel.com>
>> wrote:
>>
>>> Hi Alex
>>>
>>> On Tue, 2020-06-16 at 09:04 +0200, Alexander Kanavin wrote:
>>> > Usually it's the other way around - what is the reason to have this
>>> > recipe in core?
>>> >
>>>
>>> I had requested Arthur to send this to oe-core since this provides
>>> a core validation functionality that is needed by all providing DRM
>>> drivers and is applicable not just to a single arch.
>>>
>>> It's a well maintained project and I can keep the recipe updated and
>>> working. If it can be in oe-core, I can remove it from meta-intel layer
>>> and avoid having a duplicate recipe.
>>>
>>> Thanks,
>>>
>>> Anuj
>>>
>>> > Alex
>>> >
>>> > On Tue, 16 Jun 2020 at 06:45, Arthur She <arthur.she@linaro.org>
>>> > wrote:
>>> > > Hi Alexander and Denys,
>>> > > Thanks for your feedback.
>>> > > I have no problem submitting it to meta-oe.
>>> > > Since the meta-intel maintainer requested to submit it to oe-core,
>>> > > do you mind sharing some thought about the reason?
>>> > >
>>> > > Thank you so much,
>>> > > Arthur
>>> > >
>>> > > On Mon, Jun 15, 2020 at 12:56 AM Denys Dmytriyenko <denis@denix.org
>>> > > > wrote:
>>> > > > https://lists.openembedded.org/g/openembedded-devel/topic/74775294
>>> > > >
>>> > > > But I tend to agree.
>>> > > >
>>>
>>

[-- Attachment #2: Type: text/html, Size: 3244 bytes --]