* [OE-core][PATCH] igt-gpu-tools: add new package @ 2020-06-15 6:00 Arthur She 2020-06-15 7:45 ` Alexander Kanavin 0 siblings, 1 reply; 9+ messages in thread From: Arthur She @ 2020-06-15 6:00 UTC (permalink / raw) To: openembedded-core; +Cc: 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 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [OE-core][PATCH] igt-gpu-tools: add new package 2020-06-15 6:00 [OE-core][PATCH] igt-gpu-tools: add new package Arthur She @ 2020-06-15 7:45 ` Alexander Kanavin 2020-06-15 7:56 ` Denys Dmytriyenko 0 siblings, 1 reply; 9+ messages in thread From: Alexander Kanavin @ 2020-06-15 7:45 UTC (permalink / raw) To: Arthur She; +Cc: openembedded-core [-- 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 --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [OE-core][PATCH] igt-gpu-tools: add new package 2020-06-15 7:45 ` Alexander Kanavin @ 2020-06-15 7:56 ` Denys Dmytriyenko 2020-06-16 4:44 ` Arthur She 0 siblings, 1 reply; 9+ messages in thread From: Denys Dmytriyenko @ 2020-06-15 7:56 UTC (permalink / raw) To: Alexander Kanavin; +Cc: Arthur She, openembedded-core 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 > > > > > > > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [OE-core][PATCH] igt-gpu-tools: add new package 2020-06-15 7:56 ` Denys Dmytriyenko @ 2020-06-16 4:44 ` Arthur She 2020-06-16 7:04 ` Alexander Kanavin 0 siblings, 1 reply; 9+ messages in thread From: Arthur She @ 2020-06-16 4:44 UTC (permalink / raw) To: Denys Dmytriyenko, Alexander Kanavin; +Cc: openembedded-core, Mittal, Anuj [-- 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 --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [OE-core][PATCH] igt-gpu-tools: add new package 2020-06-16 4:44 ` Arthur She @ 2020-06-16 7:04 ` Alexander Kanavin 2020-06-16 7:16 ` Anuj Mittal 0 siblings, 1 reply; 9+ messages in thread From: Alexander Kanavin @ 2020-06-16 7:04 UTC (permalink / raw) To: Arthur She; +Cc: Denys Dmytriyenko, OE-core, Mittal, Anuj [-- 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 --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [OE-core][PATCH] igt-gpu-tools: add new package 2020-06-16 7:04 ` Alexander Kanavin @ 2020-06-16 7:16 ` Anuj Mittal 2020-06-18 5:27 ` Arthur She 0 siblings, 1 reply; 9+ messages in thread From: Anuj Mittal @ 2020-06-16 7:16 UTC (permalink / raw) To: alex.kanavin@gmail.com, arthur.she@linaro.org Cc: denis@denix.org, openembedded-core@lists.openembedded.org 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. > > > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [OE-core][PATCH] igt-gpu-tools: add new package 2020-06-16 7:16 ` Anuj Mittal @ 2020-06-18 5:27 ` Arthur She 2020-06-18 15:25 ` Alexander Kanavin 0 siblings, 1 reply; 9+ messages in thread From: Arthur She @ 2020-06-18 5:27 UTC (permalink / raw) To: Mittal, Anuj Cc: alex.kanavin@gmail.com, denis@denix.org, openembedded-core@lists.openembedded.org [-- 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 --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [OE-core][PATCH] igt-gpu-tools: add new package 2020-06-18 5:27 ` Arthur She @ 2020-06-18 15:25 ` Alexander Kanavin 2020-06-19 2:07 ` Arthur She 0 siblings, 1 reply; 9+ messages in thread From: Alexander Kanavin @ 2020-06-18 15:25 UTC (permalink / raw) To: Arthur She Cc: Mittal, Anuj, denis@denix.org, openembedded-core@lists.openembedded.org [-- 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 --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [OE-core][PATCH] igt-gpu-tools: add new package 2020-06-18 15:25 ` Alexander Kanavin @ 2020-06-19 2:07 ` Arthur She 0 siblings, 0 replies; 9+ messages in thread From: Arthur She @ 2020-06-19 2:07 UTC (permalink / raw) To: Alexander Kanavin Cc: Mittal, Anuj, denis@denix.org, openembedded-core@lists.openembedded.org [-- 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 --] ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2020-06-19 2:07 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-06-15 6:00 [OE-core][PATCH] igt-gpu-tools: add new package Arthur She 2020-06-15 7:45 ` Alexander Kanavin 2020-06-15 7:56 ` Denys Dmytriyenko 2020-06-16 4:44 ` Arthur She 2020-06-16 7:04 ` Alexander Kanavin 2020-06-16 7:16 ` Anuj Mittal 2020-06-18 5:27 ` Arthur She 2020-06-18 15:25 ` Alexander Kanavin 2020-06-19 2:07 ` Arthur She
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox