[PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Anand Balagopalakrishnan]

Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
---
 ...v-seat-restrict-udev-enumeration-to-card0.patch |   36 ++++++++++++++++++++
 .../recipes-graphics/wayland/weston_1.9.0.bb       |    1 +
 2 files changed, 37 insertions(+)
 create mode 100644 meta-arago-extras/recipes-graphics/wayland/weston/0001-udev-seat-restrict-udev-enumeration-to-card0.patch

diff --git a/meta-arago-extras/recipes-graphics/wayland/weston/0001-udev-seat-restrict-udev-enumeration-to-card0.patch b/meta-arago-extras/recipes-graphics/wayland/weston/0001-udev-seat-restrict-udev-enumeration-to-card0.patch
new file mode 100644
index 0000000..9503428
--- /dev/null
+++ b/meta-arago-extras/recipes-graphics/wayland/weston/0001-udev-seat-restrict-udev-enumeration-to-card0.patch
@@ -0,0 +1,36 @@
+From e8e7a9f7dfa164a75fdbdca87622a2e13334478a Mon Sep 17 00:00:00 2001
+From: Anand Balagopalakrishnan <anandb@ti.com>
+Date: Sat, 23 Jan 2016 22:48:07 +0530
+Subject: [PATCH 1/1] udev-seat: restrict udev enumeration to card0
+
+In case of separate GPU and Display devices as found in embedded systems, we
+could have modeset node and render node controlled by different drivers.
+There is a distinct possibility that udev enumeration returns the DRM device
+corresponding to render node as the primary DRM device.
+
+Obviously, modeset operations cannot be done on the GPU DRM device.
+
+Restrict the udev enumeration to card0 and ensure that DRM device corresponding
+to display is returned as the primary DRM device.
+
+Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
+---
+ src/compositor-drm.c |    2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/compositor-drm.c b/src/compositor-drm.c
+index 6777bf8..59c2cc5 100644
+--- a/src/compositor-drm.c
++++ b/src/compositor-drm.c
+@@ -2827,7 +2827,7 @@ find_primary_gpu(struct drm_backend *b, const char *seat)
+ 
+ 	e = udev_enumerate_new(b->udev);
+ 	udev_enumerate_add_match_subsystem(e, "drm");
+-	udev_enumerate_add_match_sysname(e, "card[0-9]*");
++	udev_enumerate_add_match_sysname(e, "card0");
+ 
+ 	udev_enumerate_scan_devices(e);
+ 	drm_device = NULL;
+-- 
+1.7.9.5
+
diff --git a/meta-arago-extras/recipes-graphics/wayland/weston_1.9.0.bb b/meta-arago-extras/recipes-graphics/wayland/weston_1.9.0.bb
index 439b279..a444590 100644
--- a/meta-arago-extras/recipes-graphics/wayland/weston_1.9.0.bb
+++ b/meta-arago-extras/recipes-graphics/wayland/weston_1.9.0.bb
@@ -13,6 +13,7 @@ SRC_URI = "http://wayland.freedesktop.org/releases/${BPN}-${PV}.tar.xz \
            file://0001-make-error-portable.patch \
            file://libsystemd.patch \
            file://explicit-enable-disable-systemd.patch \
+	   file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
 "
 SRC_URI[md5sum] = "66bbba12f546570b4d97f676bc79a28e"
 SRC_URI[sha256sum] = "9c1b03f3184fa0b0dfdf67e215048085156e1a2ca344af6613fed36794ac48cf"
-- 
1.7.9.5

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Denys Dmytriyenko]

Anand,

Is this patch meant to be upstreamed? Or is this rather a hack for our 
own setup? If latter, please add the patch to weston_1.9.0.bbappend in 
meta-arago-distro instead of the main recipe. Thanks.

-- 
Denys


On Tue, Feb 02, 2016 at 08:13:15PM +0530, Anand Balagopalakrishnan wrote:
> Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> ---
>  ...v-seat-restrict-udev-enumeration-to-card0.patch |   36 ++++++++++++++++++++
>  .../recipes-graphics/wayland/weston_1.9.0.bb       |    1 +
>  2 files changed, 37 insertions(+)
>  create mode 100644 meta-arago-extras/recipes-graphics/wayland/weston/0001-udev-seat-restrict-udev-enumeration-to-card0.patch
> 
> diff --git a/meta-arago-extras/recipes-graphics/wayland/weston/0001-udev-seat-restrict-udev-enumeration-to-card0.patch b/meta-arago-extras/recipes-graphics/wayland/weston/0001-udev-seat-restrict-udev-enumeration-to-card0.patch
> new file mode 100644
> index 0000000..9503428
> --- /dev/null
> +++ b/meta-arago-extras/recipes-graphics/wayland/weston/0001-udev-seat-restrict-udev-enumeration-to-card0.patch
> @@ -0,0 +1,36 @@
> +From e8e7a9f7dfa164a75fdbdca87622a2e13334478a Mon Sep 17 00:00:00 2001
> +From: Anand Balagopalakrishnan <anandb@ti.com>
> +Date: Sat, 23 Jan 2016 22:48:07 +0530
> +Subject: [PATCH 1/1] udev-seat: restrict udev enumeration to card0
> +
> +In case of separate GPU and Display devices as found in embedded systems, we
> +could have modeset node and render node controlled by different drivers.
> +There is a distinct possibility that udev enumeration returns the DRM device
> +corresponding to render node as the primary DRM device.
> +
> +Obviously, modeset operations cannot be done on the GPU DRM device.
> +
> +Restrict the udev enumeration to card0 and ensure that DRM device corresponding
> +to display is returned as the primary DRM device.
> +
> +Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> +---
> + src/compositor-drm.c |    2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/src/compositor-drm.c b/src/compositor-drm.c
> +index 6777bf8..59c2cc5 100644
> +--- a/src/compositor-drm.c
> ++++ b/src/compositor-drm.c
> +@@ -2827,7 +2827,7 @@ find_primary_gpu(struct drm_backend *b, const char *seat)
> + 
> + 	e = udev_enumerate_new(b->udev);
> + 	udev_enumerate_add_match_subsystem(e, "drm");
> +-	udev_enumerate_add_match_sysname(e, "card[0-9]*");
> ++	udev_enumerate_add_match_sysname(e, "card0");
> + 
> + 	udev_enumerate_scan_devices(e);
> + 	drm_device = NULL;
> +-- 
> +1.7.9.5
> +
> diff --git a/meta-arago-extras/recipes-graphics/wayland/weston_1.9.0.bb b/meta-arago-extras/recipes-graphics/wayland/weston_1.9.0.bb
> index 439b279..a444590 100644
> --- a/meta-arago-extras/recipes-graphics/wayland/weston_1.9.0.bb
> +++ b/meta-arago-extras/recipes-graphics/wayland/weston_1.9.0.bb
> @@ -13,6 +13,7 @@ SRC_URI = "http://wayland.freedesktop.org/releases/${BPN}-${PV}.tar.xz \
>             file://0001-make-error-portable.patch \
>             file://libsystemd.patch \
>             file://explicit-enable-disable-systemd.patch \
> +	   file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
>  "
>  SRC_URI[md5sum] = "66bbba12f546570b4d97f676bc79a28e"
>  SRC_URI[sha256sum] = "9c1b03f3184fa0b0dfdf67e215048085156e1a2ca344af6613fed36794ac48cf"
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Anand Balagopalakrishnan]

Denys,

I plan to push the embedded patch to wayland-devel. We could have a 
rework of this patch based on comments.

Nonetheless, I have no doubt that this patch is indeed required. The 
current Weston code is not factoring in the embedded SoC configurations.

Regards,
Anand

On Friday 05 February 2016 12:05 AM, Denys Dmytriyenko wrote:
> Anand,
>
> Is this patch meant to be upstreamed? Or is this rather a hack for our
> own setup? If latter, please add the patch to weston_1.9.0.bbappend in
> meta-arago-distro instead of the main recipe. Thanks.
>

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Denys Dmytriyenko]

Anand,

Thanks. Then we need to add this line to the patch for tracking:

Upstream-Status: Pending

-- 
Denys


On Fri, Feb 05, 2016 at 12:39:18PM +0530, Anand Balagopalakrishnan wrote:
> Denys,
> 
> I plan to push the embedded patch to wayland-devel. We could have a
> rework of this patch based on comments.
> 
> Nonetheless, I have no doubt that this patch is indeed required. The
> current Weston code is not factoring in the embedded SoC
> configurations.
> 
> Regards,
> Anand
> 
> On Friday 05 February 2016 12:05 AM, Denys Dmytriyenko wrote:
> >Anand,
> >
> >Is this patch meant to be upstreamed? Or is this rather a hack for our
> >own setup? If latter, please add the patch to weston_1.9.0.bbappend in
> >meta-arago-distro instead of the main recipe. Thanks.
> >
> 

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Balagopalakrishnan, Anand]

Denys,

I posted v2 of the patch.

Regards,
Anand

-----Original Message-----
From: Dmytriyenko, Denys 
Sent: Friday, February 05, 2016 9:23 PM
To: Balagopalakrishnan, Anand
Cc: meta-arago@arago-project.org
Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

Anand,

Thanks. Then we need to add this line to the patch for tracking:

Upstream-Status: Pending

-- 
Denys


On Fri, Feb 05, 2016 at 12:39:18PM +0530, Anand Balagopalakrishnan wrote:
> Denys,
> 
> I plan to push the embedded patch to wayland-devel. We could have a
> rework of this patch based on comments.
> 
> Nonetheless, I have no doubt that this patch is indeed required. The
> current Weston code is not factoring in the embedded SoC
> configurations.
> 
> Regards,
> Anand
> 
> On Friday 05 February 2016 12:05 AM, Denys Dmytriyenko wrote:
> >Anand,
> >
> >Is this patch meant to be upstreamed? Or is this rather a hack for our
> >own setup? If latter, please add the patch to weston_1.9.0.bbappend in
> >meta-arago-distro instead of the main recipe. Thanks.
> >
> 

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Denys Dmytriyenko]

If you do several changes in the same recipe/file, make sure you submit a 
proper series with patches numbered sequentially, so applying them will not 
pose a problem.

-- 
Denys


On Thu, Mar 09, 2017 at 12:43:48PM -0500, Balagopalakrishnan, Anand wrote:
>    Hi Eric,
> 
>    You need not limit to 1 patch per day. Keep separate commits as was done
>    for krogoth. All your commits will get rebased one on top of the other
>    with a bump in PR for each patch.
> 
>    You can submit all the patches in 1 go. Denys can merge them individually.
>    We will get all the patches in quickly. Also, we have traceability.
> 
>    I follow the same approach for SGX um.
> 
>    Regards,
>    Anand
> 
>    Sent from my Android phone
> 
>    -----Original Message-----
>    From: Ruei, Eric [e-ruei1@ti.com]
>    Received: Thursday, 09 Mar 2017, 22:51
>    To: Balagopalakrishnan, Anand [anandb@ti.com]; Dmytriyenko, Denys
>    [denys@ti.com]
>    CC: meta-arago@arago-project.org [meta-arago@arago-project.org]
>    Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to
>    /dev/dri/card0
> 
>    Hi, Anand and Denys:
> 
>    Yes, there is another critical one just getting Weston up and running.
>    0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> 
>    And there are a few other import ones which should be added immediately.
>    0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
>    0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
>    0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
> 
>    The following two should be in, but not urgent:
>    0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
>    0001-Add-soc-performance-monitor-utilites.patch
> 
>    I do not know how important Karthik's patch is, but it is still a valid
>    patch:
>    0001-weston-Enabling-DRM-backend-with-multiple-displays.patch
> 
>    I agree with Anand's idea in principle, but it may limit to one patch per
>    day even though we know all those patches are required.
> 
>    Best regards,
> 
>    Eric
> 
>    -----Original Message-----
>    From: Balagopalakrishnan, Anand
>    Sent: Thursday, March 09, 2017 11:27 AM
>    To: Dmytriyenko, Denys
>    Cc: Ruei, Eric; meta-arago@arago-project.org
>    Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to
>    /dev/dri/card0
> 
>    If Eric posts another patch with visual-id, Weston startup should be
>    mostly resolved.
> 
>    Yes, I would like separate patches from future maintenance perspective.
> 
>    Regards,
>    Anand
> 
>    -----Original Message-----
>    From: Dmytriyenko, Denys
>    Sent: Thursday, March 9, 2017 9:53 PM
>    To: Balagopalakrishnan, Anand
>    Cc: Ruei, Eric; meta-arago@arago-project.org
>    Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to
>    /dev/dri/card0
> 
>    I can go either way. But we need to clean this up and keep on maintaining
>    it in the future. For now, the most urgent matter is to fix Weston startup
>    issues.
> 
>    --
>    Denys
> 
>    On Thu, Mar 09, 2017 at 05:29:09AM -0500, Balagopalakrishnan, Anand wrote:
>    > Hi Eric,
>    >
>    > I would prefer a recipe update to address a specific issue/feature
>    > rather than taking in all patches at once. This will help in better
>    > traceability when we move to next Yocto version. For example, when we
>    > move to a later Weston version, few patches may not be relevant. If we
>    > separate each feature into a separate recipe commit, rebasing becomes
>    easier.
>    >
>    > Ideally, we should have a morty patch for each patch on Krogoth. In
>    > specific cases, we may want to squash. Hence, we should logically
>    > segregate the patches to be applied and have specific recipe updates
>    > targeting each of them. We also want to change the name of this patch
>    > file removing 1.9.0 from the file name.
>    >
>    > Can you rebase this change on top of my v2 patch? We will need similar
>    > updates for other patches as well - let's plan for respective patch
>    > owners take care of that.
>    >
>    > On a related note, just like kernel commits, each patch owner should
>    > be tracking recipe commits as well when moving from one LTS to
>    > another. This got missed out this time.
>    >
>    > Regards,
>    > Anand
>    >
>    > -----Original Message-----
>    > From: Ruei, Eric
>    > Sent: Thursday, March 9, 2017 3:10 AM
>    > To: Dmytriyenko, Denys
>    > Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
>    > Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev
>    > enumeration to /dev/dri/card0
>    >
>    > Hi, Denys:
>    >
>    > Anand's patch will fix one problem and we shall need the following
>    > additional patch to get Weston up and running.
>    >
>    > 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable,
>    > resolve the EGL config problem (LCPD-9399)
>    >
>    > Best regards,
>    >
>    > Eric
>    >
>    >
>    > -----Original Message-----
>    > From: Dmytriyenko, Denys
>    > Sent: Wednesday, March 08, 2017 4:17 PM
>    > To: Ruei, Eric
>    > Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
>    > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev
>    > enumeration to /dev/dri/card0
>    >
>    > Eric,
>    >
>    > Thanks for reviewing all the patches and identifying their status!
>    >
>    > I can probably merge Anand's patch right away, which would supposed to
>    > unblock further Weston testing. And that will give you a bit more time
>    > to prepare a followup patch with the rest of them, in case there are
>    > any updates needed.
>    >
>    > Please let me know otherwise. Thanks.
>    >
>    > --
>    > Denys
>    >
>    >
>    > On Wed, Mar 08, 2017 at 03:56:13PM -0500, Ruei, Eric wrote:
>    > > Hi, Anand and Denys:
>    > >
>    > > I have confirmed that all other 7 Weston patches are still applicable.
>    > >
>    > > 0001-weston-Enabling-DRM-backend-with-multiple-displays.patch :
>    > > still applicable, need a minor update
>    > > 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable,
>    > > resolve the EGL config problem (LCPD-9399)
>    > > 0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch:
>    > > still applicable
>    > > 0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch: still
>    > > applicable
>    > > 0001-udev-seat-restrict-udev-enumeration-to-card0.patch: still
>    applicable, it is what Anand just re-enabled.
>    > > 0001-Add-soc-performance-monitor-utilites.patch : still applicable,
>    TI-specific changes
>    > > 0001-compositor-drm-support-RGB565-with-pixman-renderer.patch: still
>    applicable, Tomi has submitted this one  a while ago, but it is
>    not                                                                                                                                     
>    available at Weston 1.11.0.
>    > > 0001-compositor-drm-fix-hotplug-weston-termination-proble.patch:
>    > > still applicable, good enough as long as the connecter id/crtc id
>    > > <64
>    > >
>    > > I did submit this patch to trigger discussions, but none has
>    > > provided a better solution.
>    > >
>    > > Should I submit another patch with all other patches after Anand's
>    > > patch is applied? Or should we replace Anand's patch with this one
>    > > which covers all Weston 1.9 patches?
>    > >
>    > > Best regards,
>    > >
>    > > Eric
>    >
>    >                                                                                                                                          
>    > >
>    > > -----Original Message-----
>    > > From: meta-arago-bounces@arago-project.org
>    > > [mailto:meta-arago-bounces@arago-project.org] On Behalf Of
>    > > Balagopalakrishnan, Anand
>    > > Sent: Wednesday, March 08, 2017 12:22 PM
>    > > To: Dmytriyenko, Denys
>    > > Cc: meta-arago@arago-project.org
>    > > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev
>    > > enumeration to /dev/dri/card0
>    > >
>    > > Denys,
>    > >
>    > > We need to review whether the other patches are required or not. This
>    action will happen over the next 1-2 weeks by Karthik and Eric. Till that
>    time, I would like to keep the patches in.
>    > >
>    > > In the meantime, the commented out lines can be removed from the
>    bbappend file itself. If a patch is deemed to be required, we can submit a
>    new recipe update later for SRC_URL. Is this fine? If so, I can submit v2
>    with this proposal.
>    > >
>    > > Regards,
>    > > Anand
>    > > ________________________________________
>    > > From: Dmytriyenko, Denys
>    > > Sent: Wednesday, March 08, 2017 8:06 PM
>    > > To: Balagopalakrishnan, Anand
>    > > Cc: meta-arago@arago-project.org
>    > > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev
>    > > enumeration to /dev/dri/card0
>    > >
>    > > On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan
>    wrote:
>    > > > Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
>    > > > ---
>    > > >  .../wayland/weston_1.11.0.bbappend                 | 23
>    +++++++++++-----------
>    > > >  1 file changed, 12 insertions(+), 11 deletions(-)
>    > > >
>    > > > diff --git
>    > > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
>    > > > d
>    > > > b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
>    > > > d
>    > > > index d0691ad..36585bd 100644
>    > > > ---
>    > > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
>    > > > d
>    > > > +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bba
>    > > > +++ pp
>    > > > +++ en
>    > > > +++ d
>    > > > @@ -2,19 +2,20 @@
>    > > >  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor
>    WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev
>    mtdev"
>    > > >  PACKAGECONFIG[kms] =
>    "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
>    > > >
>    > > > -PR_append = ".arago16"
>    > > > +PR_append = ".arago17"
>    > > >
>    > > >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>    > > >
>    > > >  RDEPENDS_${PN} += "weston-conf"
>    > > >
>    > > > -#SRC_URI += " \
>    > > > -#   
>    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
>    > > > -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
>    > > > -#   
>    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
>    > > > -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
>    > > > -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch
>    \
>    > > > -#    file://0001-Add-soc-performance-monitor-utilites.patch \
>    > > > -#   
>    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
>    > > > -#   
>    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
>    > > > -#"
>    > > > +
>    > > > +SRC_URI += " \
>    > > > +    
>    > > > +file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch
>    > > > +\ "
>    > > > +#   
>    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
>    > > > +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
>    > > > +#   
>    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
>    > > > +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
>    > > > +#    file://0001-Add-soc-performance-monitor-utilites.patch
>    > > > +#   
>    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
>    > > > +#   
>    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
>    > >
>    > > Anand,
>    > >
>    > > What about the rest of the commented out patches? Do we still need
>    them?
>    > >
>    > > If we need them, will you be looking at up-porting them?
>    > > If we don't need them, can you please remove the corresponding
>    comments from the recipe.
>    > >
>    > > Thanks.
>    > >
>    > > --
>    > > Denys
>    > > _______________________________________________
>    > > meta-arago mailing list
>    > > meta-arago@arago-project.org
>    > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Balagopalakrishnan, Anand]

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

Hi Eric,

You need not limit to 1 patch per day. Keep separate commits as was done for krogoth. All your commits will get rebased one on top of the other with a bump in PR for each patch.

You can submit all the patches in 1 go. Denys can merge them individually. We will get all the patches in quickly. Also, we have traceability.

I follow the same approach for SGX um.

Regards,
Anand

Sent from my Android phone

-----Original Message-----
From: Ruei, Eric [e-ruei1@ti.com]
Received: Thursday, 09 Mar 2017, 22:51
To: Balagopalakrishnan, Anand [anandb@ti.com]; Dmytriyenko, Denys [denys@ti.com]
CC: meta-arago@arago-project.org [meta-arago@arago-project.org]
Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

Hi, Anand and Denys:

Yes, there is another critical one just getting Weston up and running.
0002-Weston1.9.0-Allow-visual_id-to-be-0.patch

And there are a few other import ones which should be added immediately.
0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
0001-compositor-drm-fix-hotplug-weston-termination-proble.patch

The following two should be in, but not urgent:
0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
0001-Add-soc-performance-monitor-utilites.patch

I do not know how important Karthik's patch is, but it is still a valid patch:
0001-weston-Enabling-DRM-backend-with-multiple-displays.patch

I agree with Anand's idea in principle, but it may limit to one patch per day even though we know all those patches are required.

Best regards,

Eric





-----Original Message-----
From: Balagopalakrishnan, Anand
Sent: Thursday, March 09, 2017 11:27 AM
To: Dmytriyenko, Denys
Cc: Ruei, Eric; meta-arago@arago-project.org
Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

If Eric posts another patch with visual-id, Weston startup should be mostly resolved.

Yes, I would like separate patches from future maintenance perspective.

Regards,
Anand

-----Original Message-----
From: Dmytriyenko, Denys
Sent: Thursday, March 9, 2017 9:53 PM
To: Balagopalakrishnan, Anand
Cc: Ruei, Eric; meta-arago@arago-project.org
Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

I can go either way. But we need to clean this up and keep on maintaining it in the future. For now, the most urgent matter is to fix Weston startup issues.

--
Denys


On Thu, Mar 09, 2017 at 05:29:09AM -0500, Balagopalakrishnan, Anand wrote:
> Hi Eric,
>
> I would prefer a recipe update to address a specific issue/feature
> rather than taking in all patches at once. This will help in better
> traceability when we move to next Yocto version. For example, when we
> move to a later Weston version, few patches may not be relevant. If we
> separate each feature into a separate recipe commit, rebasing becomes easier.
>
> Ideally, we should have a morty patch for each patch on Krogoth. In
> specific cases, we may want to squash. Hence, we should logically
> segregate the patches to be applied and have specific recipe updates
> targeting each of them. We also want to change the name of this patch
> file removing 1.9.0 from the file name.
>
> Can you rebase this change on top of my v2 patch? We will need similar
> updates for other patches as well - let's plan for respective patch
> owners take care of that.
>
> On a related note, just like kernel commits, each patch owner should
> be tracking recipe commits as well when moving from one LTS to
> another. This got missed out this time.
>
> Regards,
> Anand
>
> -----Original Message-----
> From: Ruei, Eric
> Sent: Thursday, March 9, 2017 3:10 AM
> To: Dmytriyenko, Denys
> Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
> Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev
> enumeration to /dev/dri/card0
>
> Hi, Denys:
>
> Anand's patch will fix one problem and we shall need the following
> additional patch to get Weston up and running.
>
> 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable,
> resolve the EGL config problem (LCPD-9399)
>
> Best regards,
>
> Eric
>
>
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Wednesday, March 08, 2017 4:17 PM
> To: Ruei, Eric
> Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev
> enumeration to /dev/dri/card0
>
> Eric,
>
> Thanks for reviewing all the patches and identifying their status!
>
> I can probably merge Anand's patch right away, which would supposed to
> unblock further Weston testing. And that will give you a bit more time
> to prepare a followup patch with the rest of them, in case there are
> any updates needed.
>
> Please let me know otherwise. Thanks.
>
> --
> Denys
>
>
> On Wed, Mar 08, 2017 at 03:56:13PM -0500, Ruei, Eric wrote:
> > Hi, Anand and Denys:
> >
> > I have confirmed that all other 7 Weston patches are still applicable.
> >
> > 0001-weston-Enabling-DRM-backend-with-multiple-displays.patch :
> > still applicable, need a minor update
> > 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable,
> > resolve the EGL config problem (LCPD-9399)
> > 0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch:
> > still applicable
> > 0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch: still
> > applicable
> > 0001-udev-seat-restrict-udev-enumeration-to-card0.patch: still applicable, it is what Anand just re-enabled.
> > 0001-Add-soc-performance-monitor-utilites.patch : still applicable, TI-specific changes
> > 0001-compositor-drm-support-RGB565-with-pixman-renderer.patch: still applicable, Tomi has submitted this one  a while ago, but it is not                                                                                                                                      available at Weston 1.11.0.
> > 0001-compositor-drm-fix-hotplug-weston-termination-proble.patch:
> > still applicable, good enough as long as the connecter id/crtc id
> > <64
> >
> > I did submit this patch to trigger discussions, but none has
> > provided a better solution.
> >
> > Should I submit another patch with all other patches after Anand's
> > patch is applied? Or should we replace Anand's patch with this one
> > which covers all Weston 1.9 patches?
> >
> > Best regards,
> >
> > Eric
> >
> >
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org
> > [mailto:meta-arago-bounces@arago-project.org] On Behalf Of
> > Balagopalakrishnan, Anand
> > Sent: Wednesday, March 08, 2017 12:22 PM
> > To: Dmytriyenko, Denys
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev
> > enumeration to /dev/dri/card0
> >
> > Denys,
> >
> > We need to review whether the other patches are required or not. This action will happen over the next 1-2 weeks by Karthik and Eric. Till that time, I would like to keep the patches in.
> >
> > In the meantime, the commented out lines can be removed from the bbappend file itself. If a patch is deemed to be required, we can submit a new recipe update later for SRC_URL. Is this fine? If so, I can submit v2 with this proposal.
> >
> > Regards,
> > Anand
> > ________________________________________
> > From: Dmytriyenko, Denys
> > Sent: Wednesday, March 08, 2017 8:06 PM
> > To: Balagopalakrishnan, Anand
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev
> > enumeration to /dev/dri/card0
> >
> > On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> > > Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> > > ---
> > >  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
> > >  1 file changed, 12 insertions(+), 11 deletions(-)
> > >
> > > diff --git
> > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
> > > d
> > > b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
> > > d
> > > index d0691ad..36585bd 100644
> > > ---
> > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
> > > d
> > > +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bba
> > > +++ pp
> > > +++ en
> > > +++ d
> > > @@ -2,19 +2,20 @@
> > >  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
> > >  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
> > >
> > > -PR_append = ".arago16"
> > > +PR_append = ".arago17"
> > >
> > >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> > >
> > >  RDEPENDS_${PN} += "weston-conf"
> > >
> > > -#SRC_URI += " \
> > > -#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> > > -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> > > -#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> > > -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> > > -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> > > -#    file://0001-Add-soc-performance-monitor-utilites.patch \
> > > -#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> > > -#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> > > -#"
> > > +
> > > +SRC_URI += " \
> > > +
> > > +file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch
> > > +\ "
> > > +#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> > > +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> > > +#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> > > +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> > > +#    file://0001-Add-soc-performance-monitor-utilites.patch
> > > +#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> > > +#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
> >
> > Anand,
> >
> > What about the rest of the commented out patches? Do we still need them?
> >
> > If we need them, will you be looking at up-porting them?
> > If we don't need them, can you please remove the corresponding comments from the recipe.
> >
> > Thanks.
> >
> > --
> > Denys
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

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

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Ruei, Eric]

Hi, Anand and Denys:

Yes, there is another critical one just getting Weston up and running.
0002-Weston1.9.0-Allow-visual_id-to-be-0.patch

And there are a few other import ones which should be added immediately.
0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
0001-compositor-drm-fix-hotplug-weston-termination-proble.patch

The following two should be in, but not urgent:
0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
0001-Add-soc-performance-monitor-utilites.patch

I do not know how important Karthik's patch is, but it is still a valid patch:
0001-weston-Enabling-DRM-backend-with-multiple-displays.patch

I agree with Anand's idea in principle, but it may limit to one patch per day even though we know all those patches are required.

Best regards,

Eric





-----Original Message-----
From: Balagopalakrishnan, Anand 
Sent: Thursday, March 09, 2017 11:27 AM
To: Dmytriyenko, Denys
Cc: Ruei, Eric; meta-arago@arago-project.org
Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

If Eric posts another patch with visual-id, Weston startup should be mostly resolved. 

Yes, I would like separate patches from future maintenance perspective.

Regards,
Anand

-----Original Message-----
From: Dmytriyenko, Denys
Sent: Thursday, March 9, 2017 9:53 PM
To: Balagopalakrishnan, Anand
Cc: Ruei, Eric; meta-arago@arago-project.org
Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

I can go either way. But we need to clean this up and keep on maintaining it in the future. For now, the most urgent matter is to fix Weston startup issues.

--
Denys


On Thu, Mar 09, 2017 at 05:29:09AM -0500, Balagopalakrishnan, Anand wrote:
> Hi Eric,
> 
> I would prefer a recipe update to address a specific issue/feature 
> rather than taking in all patches at once. This will help in better 
> traceability when we move to next Yocto version. For example, when we 
> move to a later Weston version, few patches may not be relevant. If we 
> separate each feature into a separate recipe commit, rebasing becomes easier.
> 
> Ideally, we should have a morty patch for each patch on Krogoth. In 
> specific cases, we may want to squash. Hence, we should logically 
> segregate the patches to be applied and have specific recipe updates 
> targeting each of them. We also want to change the name of this patch 
> file removing 1.9.0 from the file name.
> 
> Can you rebase this change on top of my v2 patch? We will need similar 
> updates for other patches as well - let's plan for respective patch 
> owners take care of that.
> 
> On a related note, just like kernel commits, each patch owner should 
> be tracking recipe commits as well when moving from one LTS to 
> another. This got missed out this time.
> 
> Regards,
> Anand
> 
> -----Original Message-----
> From: Ruei, Eric
> Sent: Thursday, March 9, 2017 3:10 AM
> To: Dmytriyenko, Denys
> Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
> Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev 
> enumeration to /dev/dri/card0
> 
> Hi, Denys:
> 
> Anand's patch will fix one problem and we shall need the following 
> additional patch to get Weston up and running.
> 
> 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, 
> resolve the EGL config problem (LCPD-9399)
> 
> Best regards,
> 
> Eric
> 
> 
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Wednesday, March 08, 2017 4:17 PM
> To: Ruei, Eric
> Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev 
> enumeration to /dev/dri/card0
> 
> Eric,
> 
> Thanks for reviewing all the patches and identifying their status!
> 
> I can probably merge Anand's patch right away, which would supposed to 
> unblock further Weston testing. And that will give you a bit more time 
> to prepare a followup patch with the rest of them, in case there are 
> any updates needed.
> 
> Please let me know otherwise. Thanks.
> 
> --
> Denys
> 
> 
> On Wed, Mar 08, 2017 at 03:56:13PM -0500, Ruei, Eric wrote:
> > Hi, Anand and Denys:
> > 
> > I have confirmed that all other 7 Weston patches are still applicable.
> > 
> > 0001-weston-Enabling-DRM-backend-with-multiple-displays.patch : 
> > still applicable, need a minor update
> > 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, 
> > resolve the EGL config problem (LCPD-9399)
> > 0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch: 
> > still applicable
> > 0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch: still 
> > applicable
> > 0001-udev-seat-restrict-udev-enumeration-to-card0.patch: still applicable, it is what Anand just re-enabled.
> > 0001-Add-soc-performance-monitor-utilites.patch : still applicable, TI-specific changes
> > 0001-compositor-drm-support-RGB565-with-pixman-renderer.patch: still applicable, Tomi has submitted this one  a while ago, but it is not                                                                                                                                      available at Weston 1.11.0.
> > 0001-compositor-drm-fix-hotplug-weston-termination-proble.patch: 
> > still applicable, good enough as long as the connecter id/crtc id 
> > <64
> > 
> > I did submit this patch to trigger discussions, but none has 
> > provided a better solution.
> > 
> > Should I submit another patch with all other patches after Anand's 
> > patch is applied? Or should we replace Anand's patch with this one 
> > which covers all Weston 1.9 patches?
> > 
> > Best regards,
> > 
> > Eric
> >                                                                                                                                           
> > 
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org
> > [mailto:meta-arago-bounces@arago-project.org] On Behalf Of 
> > Balagopalakrishnan, Anand
> > Sent: Wednesday, March 08, 2017 12:22 PM
> > To: Dmytriyenko, Denys
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev 
> > enumeration to /dev/dri/card0
> > 
> > Denys,
> > 
> > We need to review whether the other patches are required or not. This action will happen over the next 1-2 weeks by Karthik and Eric. Till that time, I would like to keep the patches in. 
> > 
> > In the meantime, the commented out lines can be removed from the bbappend file itself. If a patch is deemed to be required, we can submit a new recipe update later for SRC_URL. Is this fine? If so, I can submit v2 with this proposal.
> > 
> > Regards,
> > Anand
> > ________________________________________
> > From: Dmytriyenko, Denys
> > Sent: Wednesday, March 08, 2017 8:06 PM
> > To: Balagopalakrishnan, Anand
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev 
> > enumeration to /dev/dri/card0
> > 
> > On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> > > Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> > > ---
> > >  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
> > >  1 file changed, 12 insertions(+), 11 deletions(-)
> > >
> > > diff --git
> > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
> > > d 
> > > b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
> > > d
> > > index d0691ad..36585bd 100644
> > > ---
> > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
> > > d
> > > +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bba
> > > +++ pp
> > > +++ en
> > > +++ d
> > > @@ -2,19 +2,20 @@
> > >  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
> > >  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
> > >
> > > -PR_append = ".arago16"
> > > +PR_append = ".arago17"
> > >
> > >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> > >
> > >  RDEPENDS_${PN} += "weston-conf"
> > >
> > > -#SRC_URI += " \
> > > -#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> > > -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> > > -#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> > > -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> > > -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> > > -#    file://0001-Add-soc-performance-monitor-utilites.patch \
> > > -#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> > > -#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> > > -#"
> > > +
> > > +SRC_URI += " \
> > > +     
> > > +file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch
> > > +\ "
> > > +#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> > > +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> > > +#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> > > +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> > > +#    file://0001-Add-soc-performance-monitor-utilites.patch
> > > +#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> > > +#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
> > 
> > Anand,
> > 
> > What about the rest of the commented out patches? Do we still need them?
> > 
> > If we need them, will you be looking at up-porting them?
> > If we don't need them, can you please remove the corresponding comments from the recipe.
> > 
> > Thanks.
> > 
> > --
> > Denys
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Balagopalakrishnan, Anand]

If Eric posts another patch with visual-id, Weston startup should be mostly resolved. 

Yes, I would like separate patches from future maintenance perspective.

Regards,
Anand

-----Original Message-----
From: Dmytriyenko, Denys 
Sent: Thursday, March 9, 2017 9:53 PM
To: Balagopalakrishnan, Anand
Cc: Ruei, Eric; meta-arago@arago-project.org
Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

I can go either way. But we need to clean this up and keep on maintaining it in the future. For now, the most urgent matter is to fix Weston startup issues.

--
Denys


On Thu, Mar 09, 2017 at 05:29:09AM -0500, Balagopalakrishnan, Anand wrote:
> Hi Eric,
> 
> I would prefer a recipe update to address a specific issue/feature rather 
> than taking in all patches at once. This will help in better traceability 
> when we move to next Yocto version. For example, when we move to a later 
> Weston version, few patches may not be relevant. If we separate each feature 
> into a separate recipe commit, rebasing becomes easier.
> 
> Ideally, we should have a morty patch for each patch on Krogoth. In specific 
> cases, we may want to squash. Hence, we should logically segregate the 
> patches to be applied and have specific recipe updates targeting each of 
> them. We also want to change the name of this patch file removing 1.9.0 from 
> the file name.
> 
> Can you rebase this change on top of my v2 patch? We will need similar 
> updates for other patches as well - let's plan for respective patch owners 
> take care of that.
> 
> On a related note, just like kernel commits, each patch owner should be 
> tracking recipe commits as well when moving from one LTS to another. This 
> got missed out this time.
> 
> Regards,
> Anand
> 
> -----Original Message-----
> From: Ruei, Eric 
> Sent: Thursday, March 9, 2017 3:10 AM
> To: Dmytriyenko, Denys
> Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
> Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0
> 
> Hi, Denys:
> 
> Anand's patch will fix one problem and we shall need the following 
> additional patch to get Weston up and running.
> 
> 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, resolve the EGL config problem (LCPD-9399)
> 
> Best regards,
> 
> Eric
> 
> 
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Wednesday, March 08, 2017 4:17 PM
> To: Ruei, Eric
> Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0
> 
> Eric,
> 
> Thanks for reviewing all the patches and identifying their status!
> 
> I can probably merge Anand's patch right away, which would supposed to 
> unblock further Weston testing. And that will give you a bit more time to 
> prepare a followup patch with the rest of them, in case there are any 
> updates needed.
> 
> Please let me know otherwise. Thanks.
> 
> --
> Denys
> 
> 
> On Wed, Mar 08, 2017 at 03:56:13PM -0500, Ruei, Eric wrote:
> > Hi, Anand and Denys:
> > 
> > I have confirmed that all other 7 Weston patches are still applicable.
> > 
> > 0001-weston-Enabling-DRM-backend-with-multiple-displays.patch : still 
> > applicable, need a minor update
> > 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, 
> > resolve the EGL config problem (LCPD-9399)
> > 0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch: still 
> > applicable
> > 0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch: still applicable
> > 0001-udev-seat-restrict-udev-enumeration-to-card0.patch: still applicable, it is what Anand just re-enabled.
> > 0001-Add-soc-performance-monitor-utilites.patch : still applicable, TI-specific changes
> > 0001-compositor-drm-support-RGB565-with-pixman-renderer.patch: still applicable, Tomi has submitted this one  a while ago, but it is not                                                                                                                                      available at Weston 1.11.0.
> > 0001-compositor-drm-fix-hotplug-weston-termination-proble.patch: still 
> > applicable, good enough as long as the connecter id/crtc id <64
> > 
> > I did submit this patch to trigger discussions, but none has provided 
> > a better solution.
> > 
> > Should I submit another patch with all other patches after Anand's 
> > patch is applied? Or should we replace Anand's patch with this one 
> > which covers all Weston 1.9 patches?
> > 
> > Best regards,
> > 
> > Eric
> >                                                                                                                                           
> > 
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org 
> > [mailto:meta-arago-bounces@arago-project.org] On Behalf Of 
> > Balagopalakrishnan, Anand
> > Sent: Wednesday, March 08, 2017 12:22 PM
> > To: Dmytriyenko, Denys
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev 
> > enumeration to /dev/dri/card0
> > 
> > Denys,
> > 
> > We need to review whether the other patches are required or not. This action will happen over the next 1-2 weeks by Karthik and Eric. Till that time, I would like to keep the patches in. 
> > 
> > In the meantime, the commented out lines can be removed from the bbappend file itself. If a patch is deemed to be required, we can submit a new recipe update later for SRC_URL. Is this fine? If so, I can submit v2 with this proposal.
> > 
> > Regards,
> > Anand
> > ________________________________________
> > From: Dmytriyenko, Denys
> > Sent: Wednesday, March 08, 2017 8:06 PM
> > To: Balagopalakrishnan, Anand
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev 
> > enumeration to /dev/dri/card0
> > 
> > On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> > > Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> > > ---
> > >  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
> > >  1 file changed, 12 insertions(+), 11 deletions(-)
> > >
> > > diff --git
> > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > > b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > > index d0691ad..36585bd 100644
> > > ---
> > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > > +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbapp
> > > +++ en
> > > +++ d
> > > @@ -2,19 +2,20 @@
> > >  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
> > >  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
> > >
> > > -PR_append = ".arago16"
> > > +PR_append = ".arago17"
> > >
> > >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> > >
> > >  RDEPENDS_${PN} += "weston-conf"
> > >
> > > -#SRC_URI += " \
> > > -#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> > > -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> > > -#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> > > -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> > > -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> > > -#    file://0001-Add-soc-performance-monitor-utilites.patch \
> > > -#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> > > -#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> > > -#"
> > > +
> > > +SRC_URI += " \
> > > +     file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch 
> > > +\ "
> > > +#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> > > +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> > > +#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> > > +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> > > +#    file://0001-Add-soc-performance-monitor-utilites.patch
> > > +#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> > > +#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
> > 
> > Anand,
> > 
> > What about the rest of the commented out patches? Do we still need them?
> > 
> > If we need them, will you be looking at up-porting them?
> > If we don't need them, can you please remove the corresponding comments from the recipe.
> > 
> > Thanks.
> > 
> > --
> > Denys
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Denys Dmytriyenko]

I can go either way. But we need to clean this up and keep on maintaining it 
in the future. For now, the most urgent matter is to fix Weston startup 
issues.

-- 
Denys


On Thu, Mar 09, 2017 at 05:29:09AM -0500, Balagopalakrishnan, Anand wrote:
> Hi Eric,
> 
> I would prefer a recipe update to address a specific issue/feature rather 
> than taking in all patches at once. This will help in better traceability 
> when we move to next Yocto version. For example, when we move to a later 
> Weston version, few patches may not be relevant. If we separate each feature 
> into a separate recipe commit, rebasing becomes easier.
> 
> Ideally, we should have a morty patch for each patch on Krogoth. In specific 
> cases, we may want to squash. Hence, we should logically segregate the 
> patches to be applied and have specific recipe updates targeting each of 
> them. We also want to change the name of this patch file removing 1.9.0 from 
> the file name.
> 
> Can you rebase this change on top of my v2 patch? We will need similar 
> updates for other patches as well - let's plan for respective patch owners 
> take care of that.
> 
> On a related note, just like kernel commits, each patch owner should be 
> tracking recipe commits as well when moving from one LTS to another. This 
> got missed out this time.
> 
> Regards,
> Anand
> 
> -----Original Message-----
> From: Ruei, Eric 
> Sent: Thursday, March 9, 2017 3:10 AM
> To: Dmytriyenko, Denys
> Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
> Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0
> 
> Hi, Denys:
> 
> Anand's patch will fix one problem and we shall need the following 
> additional patch to get Weston up and running.
> 
> 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, resolve the EGL config problem (LCPD-9399)
> 
> Best regards,
> 
> Eric
> 
> 
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Wednesday, March 08, 2017 4:17 PM
> To: Ruei, Eric
> Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0
> 
> Eric,
> 
> Thanks for reviewing all the patches and identifying their status!
> 
> I can probably merge Anand's patch right away, which would supposed to 
> unblock further Weston testing. And that will give you a bit more time to 
> prepare a followup patch with the rest of them, in case there are any 
> updates needed.
> 
> Please let me know otherwise. Thanks.
> 
> --
> Denys
> 
> 
> On Wed, Mar 08, 2017 at 03:56:13PM -0500, Ruei, Eric wrote:
> > Hi, Anand and Denys:
> > 
> > I have confirmed that all other 7 Weston patches are still applicable.
> > 
> > 0001-weston-Enabling-DRM-backend-with-multiple-displays.patch : still 
> > applicable, need a minor update
> > 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, 
> > resolve the EGL config problem (LCPD-9399)
> > 0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch: still 
> > applicable
> > 0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch: still applicable
> > 0001-udev-seat-restrict-udev-enumeration-to-card0.patch: still applicable, it is what Anand just re-enabled.
> > 0001-Add-soc-performance-monitor-utilites.patch : still applicable, TI-specific changes
> > 0001-compositor-drm-support-RGB565-with-pixman-renderer.patch: still applicable, Tomi has submitted this one  a while ago, but it is not                                                                                                                                      available at Weston 1.11.0.
> > 0001-compositor-drm-fix-hotplug-weston-termination-proble.patch: still 
> > applicable, good enough as long as the connecter id/crtc id <64
> > 
> > I did submit this patch to trigger discussions, but none has provided 
> > a better solution.
> > 
> > Should I submit another patch with all other patches after Anand's 
> > patch is applied? Or should we replace Anand's patch with this one 
> > which covers all Weston 1.9 patches?
> > 
> > Best regards,
> > 
> > Eric
> >                                                                                                                                           
> > 
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org 
> > [mailto:meta-arago-bounces@arago-project.org] On Behalf Of 
> > Balagopalakrishnan, Anand
> > Sent: Wednesday, March 08, 2017 12:22 PM
> > To: Dmytriyenko, Denys
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev 
> > enumeration to /dev/dri/card0
> > 
> > Denys,
> > 
> > We need to review whether the other patches are required or not. This action will happen over the next 1-2 weeks by Karthik and Eric. Till that time, I would like to keep the patches in. 
> > 
> > In the meantime, the commented out lines can be removed from the bbappend file itself. If a patch is deemed to be required, we can submit a new recipe update later for SRC_URL. Is this fine? If so, I can submit v2 with this proposal.
> > 
> > Regards,
> > Anand
> > ________________________________________
> > From: Dmytriyenko, Denys
> > Sent: Wednesday, March 08, 2017 8:06 PM
> > To: Balagopalakrishnan, Anand
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev 
> > enumeration to /dev/dri/card0
> > 
> > On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> > > Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> > > ---
> > >  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
> > >  1 file changed, 12 insertions(+), 11 deletions(-)
> > >
> > > diff --git
> > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > > b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > > index d0691ad..36585bd 100644
> > > ---
> > > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > > +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbapp
> > > +++ en
> > > +++ d
> > > @@ -2,19 +2,20 @@
> > >  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
> > >  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
> > >
> > > -PR_append = ".arago16"
> > > +PR_append = ".arago17"
> > >
> > >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> > >
> > >  RDEPENDS_${PN} += "weston-conf"
> > >
> > > -#SRC_URI += " \
> > > -#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> > > -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> > > -#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> > > -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> > > -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> > > -#    file://0001-Add-soc-performance-monitor-utilites.patch \
> > > -#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> > > -#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> > > -#"
> > > +
> > > +SRC_URI += " \
> > > +     file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch 
> > > +\ "
> > > +#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> > > +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> > > +#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> > > +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> > > +#    file://0001-Add-soc-performance-monitor-utilites.patch
> > > +#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> > > +#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
> > 
> > Anand,
> > 
> > What about the rest of the commented out patches? Do we still need them?
> > 
> > If we need them, will you be looking at up-porting them?
> > If we don't need them, can you please remove the corresponding comments from the recipe.
> > 
> > Thanks.
> > 
> > --
> > Denys
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Balagopalakrishnan, Anand]

Hi Eric,

I would prefer a recipe update to address a specific issue/feature rather than taking in all patches at once. This will help in better traceability when we move to next Yocto version. For example, when we move to a later Weston version, few patches may not be relevant. If we separate each feature into a separate recipe commit, rebasing becomes easier.

Ideally, we should have a morty patch for each patch on Krogoth. In specific cases, we may want to squash. Hence, we should logically segregate the patches to be applied and have specific recipe updates targeting each of them. We also want to change the name of this patch file removing 1.9.0 from the file name.

Can you rebase this change on top of my v2 patch? We will need similar updates for other patches as well - let's plan for respective patch owners take care of that.

On a related note, just like kernel commits, each patch owner should be tracking recipe commits as well when moving from one LTS to another. This got missed out this time. 

Regards,
Anand

-----Original Message-----
From: Ruei, Eric 
Sent: Thursday, March 9, 2017 3:10 AM
To: Dmytriyenko, Denys
Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
Subject: RE: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

Hi, Denys:

Anand's  patch will fix one problem  and we shall need the following additional patch to get Weston up and running.

0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, resolve the EGL config problem (LCPD-9399)

Best regards,

Eric


-----Original Message-----
From: Dmytriyenko, Denys
Sent: Wednesday, March 08, 2017 4:17 PM
To: Ruei, Eric
Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

Eric,

Thanks for reviewing all the patches and identifying their status!

I can probably merge Anand's patch right away, which would supposed to unblock further Weston testing. And that will give you a bit more time to prepare a followup patch with the rest of them, in case there are any updates needed.

Please let me know otherwise. Thanks.

--
Denys


On Wed, Mar 08, 2017 at 03:56:13PM -0500, Ruei, Eric wrote:
> Hi, Anand and Denys:
> 
> I have confirmed that all other 7 Weston patches are still applicable.
> 
> 0001-weston-Enabling-DRM-backend-with-multiple-displays.patch : still 
> applicable, need a minor update
> 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, 
> resolve the EGL config problem (LCPD-9399)
> 0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch: still 
> applicable
> 0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch: still applicable
> 0001-udev-seat-restrict-udev-enumeration-to-card0.patch: still applicable, it is what Anand just re-enabled.
> 0001-Add-soc-performance-monitor-utilites.patch : still applicable, TI-specific changes
> 0001-compositor-drm-support-RGB565-with-pixman-renderer.patch: still applicable, Tomi has submitted this one  a while ago, but it is not                                                                                                                                      available at Weston 1.11.0.
> 0001-compositor-drm-fix-hotplug-weston-termination-proble.patch: still 
> applicable, good enough as long as the connecter id/crtc id <64
> 
> I did submit this patch to trigger discussions, but none has provided 
> a better solution.
> 
> Should I submit another patch with all other patches after Anand's 
> patch is applied? Or should we replace Anand's patch with this one 
> which covers all Weston 1.9 patches?
> 
> Best regards,
> 
> Eric
>                                                                                                                                           
> 
> -----Original Message-----
> From: meta-arago-bounces@arago-project.org 
> [mailto:meta-arago-bounces@arago-project.org] On Behalf Of 
> Balagopalakrishnan, Anand
> Sent: Wednesday, March 08, 2017 12:22 PM
> To: Dmytriyenko, Denys
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev 
> enumeration to /dev/dri/card0
> 
> Denys,
> 
> We need to review whether the other patches are required or not. This action will happen over the next 1-2 weeks by Karthik and Eric. Till that time, I would like to keep the patches in. 
> 
> In the meantime, the commented out lines can be removed from the bbappend file itself. If a patch is deemed to be required, we can submit a new recipe update later for SRC_URL. Is this fine? If so, I can submit v2 with this proposal.
> 
> Regards,
> Anand
> ________________________________________
> From: Dmytriyenko, Denys
> Sent: Wednesday, March 08, 2017 8:06 PM
> To: Balagopalakrishnan, Anand
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev 
> enumeration to /dev/dri/card0
> 
> On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> > Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> > ---
> >  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
> >  1 file changed, 12 insertions(+), 11 deletions(-)
> >
> > diff --git
> > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > index d0691ad..36585bd 100644
> > ---
> > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbapp
> > +++ en
> > +++ d
> > @@ -2,19 +2,20 @@
> >  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
> >  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
> >
> > -PR_append = ".arago16"
> > +PR_append = ".arago17"
> >
> >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> >
> >  RDEPENDS_${PN} += "weston-conf"
> >
> > -#SRC_URI += " \
> > -#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> > -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> > -#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> > -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> > -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> > -#    file://0001-Add-soc-performance-monitor-utilites.patch \
> > -#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> > -#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> > -#"
> > +
> > +SRC_URI += " \
> > +     file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch 
> > +\ "
> > +#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> > +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> > +#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> > +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> > +#    file://0001-Add-soc-performance-monitor-utilites.patch
> > +#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> > +#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
> 
> Anand,
> 
> What about the rest of the commented out patches? Do we still need them?
> 
> If we need them, will you be looking at up-porting them?
> If we don't need them, can you please remove the corresponding comments from the recipe.
> 
> Thanks.
> 
> --
> Denys
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Ruei, Eric]

Hi, Denys:

Anand's  patch will fix one problem  and we shall need the following additional patch to get Weston up and running.

0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, resolve the EGL config problem (LCPD-9399)

Best regards,

Eric


-----Original Message-----
From: Dmytriyenko, Denys 
Sent: Wednesday, March 08, 2017 4:17 PM
To: Ruei, Eric
Cc: Balagopalakrishnan, Anand; meta-arago@arago-project.org
Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

Eric,

Thanks for reviewing all the patches and identifying their status!

I can probably merge Anand's patch right away, which would supposed to unblock further Weston testing. And that will give you a bit more time to prepare a followup patch with the rest of them, in case there are any updates needed.

Please let me know otherwise. Thanks.

--
Denys


On Wed, Mar 08, 2017 at 03:56:13PM -0500, Ruei, Eric wrote:
> Hi, Anand and Denys:
> 
> I have confirmed that all other 7 Weston patches are still applicable.
> 
> 0001-weston-Enabling-DRM-backend-with-multiple-displays.patch : still applicable, need a minor update
> 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, resolve the EGL config problem (LCPD-9399)
> 0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch: still applicable
> 0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch: still applicable
> 0001-udev-seat-restrict-udev-enumeration-to-card0.patch: still applicable, it is what Anand just re-enabled.
> 0001-Add-soc-performance-monitor-utilites.patch : still applicable, TI-specific changes
> 0001-compositor-drm-support-RGB565-with-pixman-renderer.patch: still applicable, Tomi has submitted this one  a while ago, but it is not                                                                                                                                      available at Weston 1.11.0.
> 0001-compositor-drm-fix-hotplug-weston-termination-proble.patch: still applicable, good enough as long as the connecter id/crtc id <64
> 
> I did submit this patch to trigger discussions, but none has provided a 
> better solution.
> 
> Should I submit another patch with all other patches after Anand's patch is 
> applied? Or should we replace Anand's patch with this one which covers all 
> Weston 1.9 patches?
> 
> Best regards,
> 
> Eric
>                                                                                                                                           
> 
> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-bounces@arago-project.org] On Behalf Of Balagopalakrishnan, Anand
> Sent: Wednesday, March 08, 2017 12:22 PM
> To: Dmytriyenko, Denys
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0
> 
> Denys,
> 
> We need to review whether the other patches are required or not. This action will happen over the next 1-2 weeks by Karthik and Eric. Till that time, I would like to keep the patches in. 
> 
> In the meantime, the commented out lines can be removed from the bbappend file itself. If a patch is deemed to be required, we can submit a new recipe update later for SRC_URL. Is this fine? If so, I can submit v2 with this proposal.
> 
> Regards,
> Anand
> ________________________________________
> From: Dmytriyenko, Denys
> Sent: Wednesday, March 08, 2017 8:06 PM
> To: Balagopalakrishnan, Anand
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0
> 
> On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> > Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> > ---
> >  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
> >  1 file changed, 12 insertions(+), 11 deletions(-)
> >
> > diff --git 
> > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend 
> > b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > index d0691ad..36585bd 100644
> > --- 
> > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
> > +++ d
> > @@ -2,19 +2,20 @@
> >  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
> >  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
> >
> > -PR_append = ".arago16"
> > +PR_append = ".arago17"
> >
> >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> >
> >  RDEPENDS_${PN} += "weston-conf"
> >
> > -#SRC_URI += " \
> > -#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> > -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> > -#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> > -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> > -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> > -#    file://0001-Add-soc-performance-monitor-utilites.patch \
> > -#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> > -#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> > -#"
> > +
> > +SRC_URI += " \
> > +     file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \ 
> > +"
> > +#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> > +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> > +#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> > +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> > +#    file://0001-Add-soc-performance-monitor-utilites.patch
> > +#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> > +#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
> 
> Anand,
> 
> What about the rest of the commented out patches? Do we still need them?
> 
> If we need them, will you be looking at up-porting them?
> If we don't need them, can you please remove the corresponding comments from the recipe.
> 
> Thanks.
> 
> --
> Denys
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Denys Dmytriyenko]

Eric,

Thanks for reviewing all the patches and identifying their status!

I can probably merge Anand's patch right away, which would supposed to unblock 
further Weston testing. And that will give you a bit more time to prepare a 
followup patch with the rest of them, in case there are any updates needed.

Please let me know otherwise. Thanks.

-- 
Denys


On Wed, Mar 08, 2017 at 03:56:13PM -0500, Ruei, Eric wrote:
> Hi, Anand and Denys:
> 
> I have confirmed that all other 7 Weston patches are still applicable.
> 
> 0001-weston-Enabling-DRM-backend-with-multiple-displays.patch : still applicable, need a minor update
> 0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, resolve the EGL config problem (LCPD-9399)
> 0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch: still applicable
> 0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch: still applicable
> 0001-udev-seat-restrict-udev-enumeration-to-card0.patch: still applicable, it is what Anand just re-enabled.
> 0001-Add-soc-performance-monitor-utilites.patch : still applicable, TI-specific changes
> 0001-compositor-drm-support-RGB565-with-pixman-renderer.patch: still applicable, Tomi has submitted this one  a while ago, but it is not                                                                                                                                      available at Weston 1.11.0.
> 0001-compositor-drm-fix-hotplug-weston-termination-proble.patch: still applicable, good enough as long as the connecter id/crtc id <64
> 
> I did submit this patch to trigger discussions, but none has provided a 
> better solution.
> 
> Should I submit another patch with all other patches after Anand's patch is 
> applied? Or should we replace Anand's patch with this one which covers all 
> Weston 1.9 patches?
> 
> Best regards,
> 
> Eric
>                                                                                                                                           
> 
> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-bounces@arago-project.org] On Behalf Of Balagopalakrishnan, Anand
> Sent: Wednesday, March 08, 2017 12:22 PM
> To: Dmytriyenko, Denys
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0
> 
> Denys,
> 
> We need to review whether the other patches are required or not. This action will happen over the next 1-2 weeks by Karthik and Eric. Till that time, I would like to keep the patches in. 
> 
> In the meantime, the commented out lines can be removed from the bbappend file itself. If a patch is deemed to be required, we can submit a new recipe update later for SRC_URL. Is this fine? If so, I can submit v2 with this proposal.
> 
> Regards,
> Anand
> ________________________________________
> From: Dmytriyenko, Denys
> Sent: Wednesday, March 08, 2017 8:06 PM
> To: Balagopalakrishnan, Anand
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0
> 
> On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> > Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> > ---
> >  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
> >  1 file changed, 12 insertions(+), 11 deletions(-)
> >
> > diff --git 
> > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend 
> > b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > index d0691ad..36585bd 100644
> > --- 
> > a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> > +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
> > +++ d
> > @@ -2,19 +2,20 @@
> >  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
> >  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
> >
> > -PR_append = ".arago16"
> > +PR_append = ".arago17"
> >
> >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> >
> >  RDEPENDS_${PN} += "weston-conf"
> >
> > -#SRC_URI += " \
> > -#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> > -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> > -#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> > -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> > -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> > -#    file://0001-Add-soc-performance-monitor-utilites.patch \
> > -#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> > -#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> > -#"
> > +
> > +SRC_URI += " \
> > +     file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \ 
> > +"
> > +#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> > +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> > +#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> > +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> > +#    file://0001-Add-soc-performance-monitor-utilites.patch
> > +#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> > +#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
> 
> Anand,
> 
> What about the rest of the commented out patches? Do we still need them?
> 
> If we need them, will you be looking at up-porting them?
> If we don't need them, can you please remove the corresponding comments from the recipe.
> 
> Thanks.
> 
> --
> Denys
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Ruei, Eric]

Hi, Anand and Denys:

I have confirmed that all other 7 Weston patches are still applicable.

0001-weston-Enabling-DRM-backend-with-multiple-displays.patch : still applicable, need a minor update
0002-Weston1.9.0-Allow-visual_id-to-be-0.patch: still applicable, resolve the EGL config problem (LCPD-9399)
0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch: still applicable
0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch: still applicable
0001-udev-seat-restrict-udev-enumeration-to-card0.patch: still applicable, it is what Anand just re-enabled.
0001-Add-soc-performance-monitor-utilites.patch : still applicable, TI-specific changes
0001-compositor-drm-support-RGB565-with-pixman-renderer.patch: still applicable, Tomi has submitted this one  a while ago, but it is not                                                                                                                                      available at Weston 1.11.0.
0001-compositor-drm-fix-hotplug-weston-termination-proble.patch: still applicable, good enough as long as the connecter id/crtc id <64
I did submit this patch to trigger discussions, but none has provided a better solution.

Should I submit another patch with all other patches after Anand's patch is applied? Or should we replace Anand's patch with this one which covers all Weston 1.9 patches?

Best regards,

Eric
                                                                                                                                          

-----Original Message-----
From: meta-arago-bounces@arago-project.org [mailto:meta-arago-bounces@arago-project.org] On Behalf Of Balagopalakrishnan, Anand
Sent: Wednesday, March 08, 2017 12:22 PM
To: Dmytriyenko, Denys
Cc: meta-arago@arago-project.org
Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

Denys,

We need to review whether the other patches are required or not. This action will happen over the next 1-2 weeks by Karthik and Eric. Till that time, I would like to keep the patches in. 

In the meantime, the commented out lines can be removed from the bbappend file itself. If a patch is deemed to be required, we can submit a new recipe update later for SRC_URL. Is this fine? If so, I can submit v2 with this proposal.

Regards,
Anand
________________________________________
From: Dmytriyenko, Denys
Sent: Wednesday, March 08, 2017 8:06 PM
To: Balagopalakrishnan, Anand
Cc: meta-arago@arago-project.org
Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> ---
>  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
>  1 file changed, 12 insertions(+), 11 deletions(-)
>
> diff --git 
> a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend 
> b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> index d0691ad..36585bd 100644
> --- 
> a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappen
> +++ d
> @@ -2,19 +2,20 @@
>  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
>  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
>
> -PR_append = ".arago16"
> +PR_append = ".arago17"
>
>  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>
>  RDEPENDS_${PN} += "weston-conf"
>
> -#SRC_URI += " \
> -#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> -#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> -#    file://0001-Add-soc-performance-monitor-utilites.patch \
> -#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> -#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> -#"
> +
> +SRC_URI += " \
> +     file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \ 
> +"
> +#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> +#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> +#    file://0001-Add-soc-performance-monitor-utilites.patch
> +#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> +#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch

Anand,

What about the rest of the commented out patches? Do we still need them?

If we need them, will you be looking at up-porting them?
If we don't need them, can you please remove the corresponding comments from the recipe.

Thanks.

--
Denys
_______________________________________________
meta-arago mailing list
meta-arago@arago-project.org
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Balagopalakrishnan, Anand]

Denys,

We need to review whether the other patches are required or not. This action will happen over the next 1-2 weeks by Karthik and Eric. Till that time, I would like to keep the patches in. 

In the meantime, the commented out lines can be removed from the bbappend file itself. If a patch is deemed to be required, we can submit a new recipe update later for SRC_URL. Is this fine? If so, I can submit v2 with this proposal.

Regards,
Anand
________________________________________
From: Dmytriyenko, Denys
Sent: Wednesday, March 08, 2017 8:06 PM
To: Balagopalakrishnan, Anand
Cc: meta-arago@arago-project.org
Subject: Re: [meta-arago] [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> ---
>  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
>  1 file changed, 12 insertions(+), 11 deletions(-)
>
> diff --git a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> index d0691ad..36585bd 100644
> --- a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> @@ -2,19 +2,20 @@
>  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
>  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
>
> -PR_append = ".arago16"
> +PR_append = ".arago17"
>
>  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>
>  RDEPENDS_${PN} += "weston-conf"
>
> -#SRC_URI += " \
> -#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> -#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> -#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> -#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> -#    file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> -#    file://0001-Add-soc-performance-monitor-utilites.patch \
> -#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> -#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> -#"
> +
> +SRC_URI += " \
> +     file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> +"
> +#    file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> +#    file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> +#    file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> +#    file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> +#    file://0001-Add-soc-performance-monitor-utilites.patch
> +#    file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> +#    file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch

Anand,

What about the rest of the commented out patches? Do we still need them?

If we need them, will you be looking at up-porting them?
If we don't need them, can you please remove the corresponding comments from
the recipe.

Thanks.

--
Denys

Re: [PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Denys Dmytriyenko]

On Wed, Mar 08, 2017 at 01:28:25PM +0530, Anand Balagopalakrishnan wrote:
> Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
> ---
>  .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
>  1 file changed, 12 insertions(+), 11 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> index d0691ad..36585bd 100644
> --- a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> @@ -2,19 +2,20 @@
>  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
>  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
>  
> -PR_append = ".arago16"
> +PR_append = ".arago17"
>  
>  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>  
>  RDEPENDS_${PN} += "weston-conf"
>  
> -#SRC_URI += " \
> -#	file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
> -#	file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
> -#	file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
> -#	file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
> -#	file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> -#	file://0001-Add-soc-performance-monitor-utilites.patch \
> -#	file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
> -#	file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
> -#"
> +
> +SRC_URI += " \
> +	file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
> +"
> +#	file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
> +#	file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
> +#	file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
> +#	file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
> +#	file://0001-Add-soc-performance-monitor-utilites.patch
> +#	file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
> +#	file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch

Anand,

What about the rest of the commented out patches? Do we still need them?

If we need them, will you be looking at up-porting them?
If we don't need them, can you please remove the corresponding comments from 
the recipe.

Thanks.

-- 
Denys

[PATCH 1/1] weston: restrict udev enumeration to /dev/dri/card0

[Anand Balagopalakrishnan]

Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
---
 .../wayland/weston_1.11.0.bbappend                 | 23 +++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
index d0691ad..36585bd 100644
--- a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
+++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
@@ -2,19 +2,20 @@
 PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
 PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
 
-PR_append = ".arago16"
+PR_append = ".arago17"
 
 FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
 
 RDEPENDS_${PN} += "weston-conf"
 
-#SRC_URI += " \
-#	file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch \
-#	file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch \
-#	file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch \
-#	file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch \
-#	file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
-#	file://0001-Add-soc-performance-monitor-utilites.patch \
-#	file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
-#	file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
-#"
+
+SRC_URI += " \
+	file://0001-udev-seat-restrict-udev-enumeration-to-card0.patch \
+"
+#	file://0001-weston1.9.0-Enabling-DRM-backend-with-multiple-displ.patch
+#	file://0002-Weston1.9.0-Allow-visual_id-to-be-0.patch
+#	file://0003-Weston1.9.0-Fix-virtual-keyboard-display-issue-for-Q.patch
+#	file://0004-Weston1.9.0-Fix-touch-screen-crash-issue.patch
+#	file://0001-Add-soc-performance-monitor-utilites.patch
+#	file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch
+#	file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch
-- 
1.9.1