<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0cm;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
span.EmailStyle17
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-family:"Calibri","sans-serif";}
@page WordSection1
        {size:612.0pt 792.0pt;
        margin:70.85pt 70.85pt 2.0cm 70.85pt;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Hi Matt,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Thank you for your support. It was our main motivation for these changes to be able to use existing programs/frameworks with ivi-shell.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Because it is not feasible to upstream ivi-shell support for all these existing programs.<o:p></o:p></span></p>
<p class="MsoNormal"><a name="_MailEndCompose"><o:p> </o:p></a></p>
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> wayland-devel [mailto:wayland-devel-bounces@lists.freedesktop.org]
<b>On Behalf Of </b>Matt Hoosier<br>
<b>Sent:</b> Dienstag, 24. Oktober 2017 16:09<br>
<b>To:</b> Ucan, Emre (ADITG/ESB)<br>
<b>Cc:</b> wayland-devel@lists.freedesktop.org; Teyfel, Michael (ADITG/ESB)<br>
<b>Subject:</b> Re: [PATCH weston 00/14] Desktop Protocol Support for IVI-Shell<o:p></o:p></span></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<div>
<p class="MsoNormal">I'm not at all familiar with the internal implementation of ivi-shell, so I can't give much meaningful review. But I am very much in favor of this patch series. Without wl_shell and xdg_shell support, I've never been able to really give
 ivi-shell serious consideration on my products. The ability to use generic client Wayland programs is very important.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">-Matt<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">On Tue, Oct 17, 2017 at 5:51 AM, Ucan, Emre (ADITG/ESB) <<a href="mailto:eucan@de.adit-jv.com" target="_blank">eucan@de.adit-jv.com</a>> wrote:<o:p></o:p></p>
<p class="MsoNormal">Hi,<br>
<br>
I already reviewed the patches before Michael sent:<br>
Reviewed-by: Emre Ucan <<a href="mailto:eucan@de.adit-jv.com">eucan@de.adit-jv.com</a>><br>
<br>
Best regards<br>
<br>
Emre Ucan<br>
Engineering Software Base (ADITG/ESB)<br>
<br>
Tel. <a href="tel:%2B49%205121%2049%206937">+49 5121 49 6937</a><o:p></o:p></p>
<div>
<div>
<p class="MsoNormal"><br>
> -----Original Message-----<br>
> From: wayland-devel [mailto:<a href="mailto:wayland-devel-">wayland-devel-</a><br>
> <a href="mailto:bounces@lists.freedesktop.org">bounces@lists.freedesktop.org</a>] On Behalf Of Michael Teyfel<br>
> Sent: Dienstag, 17. Oktober 2017 12:02<br>
> To: <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
> Subject: [PATCH weston 00/14] Desktop Protocol Support for IVI-Shell<br>
><br>
> Hello all,<br>
><br>
> since some time I’m working on ivi-shell to add xdg-protocol support by<br>
> means of libweston-desktop. Due to my changes both xdg-protocol<br>
> applications and ivi-shell / ivi-application-protocol applications are supported<br>
> within ivi-shell now. The known functionality is preserved and just extended<br>
> by a further protocol. The advantage is that client applications do not need to<br>
> be edited to generate an id and are also not limited to use the custom ivi-<br>
> application protocol anymore, since the ids are handled by an id agent inside<br>
> of weston now.<br>
><br>
> As a preparation for the changes the goto labels in ivi-shell have been<br>
> reworked to avoid memory leaks. In ivi-layout I added an interface<br>
> (ivi_layout_surface_set_id) to set the surface-id of an ivi-layout-surface. It<br>
> can be done once after being created by an xdg-protocol application to assign<br>
> a numeric id by means of an id agent for example. Additionally I introduced a<br>
> new event to notify about a desktop surface being configured<br>
> (desktop_surface_configured). An id agent can register to this event and<br>
> react to this accordingly by assigning an id by means of<br>
> ivi_layout_surface_set_id. As a result I also changed the test client<br>
> applications in the Weston repository and removed the ivi-application<br>
> protocol support since that has only been used, if xdg protocol is not<br>
> supported. Finally hmi-controller has been edited to accept desktop<br>
> surfaces.<br>
><br>
> There are some things that can be done in the future:  At first it would be<br>
> diligent, if hmi-controller would also use xdg protocol for the GUI itself. Then<br>
> also the surface_configure event could be removed from hmi-controller.<br>
> Secondly the weston unit tests should also test the interface changes for<br>
> surface_set_id and also should stop using the ivi-application protocol.<br>
><br>
> Thanks for reading and questions are very welcome.<br>
><br>
><br>
> Best regards<br>
><br>
> Michael Teyfel<br>
> Advanced Driver Information Technology GmbH<br>
> Engineering Software Base (ADITG/ESB)<br>
> Robert-Bosch-Str. 200<br>
> 31139 Hildesheim<br>
> Germany<br>
> Tel. <a href="tel:%2B49%205121%2049%206932">+49 5121 49 6932</a><br>
> Fax <a href="tel:%2B49%205121%2049%206999">+49 5121 49 6999</a><br>
> <a href="mailto:mteyfel@de.adit-jv.com">mteyfel@de.adit-jv.com</a><br>
> ADIT is a joint venture company of Robert Bosch GmbH/Robert Bosch Car<br>
> Multimedia GmbH and DENSO Corporation<br>
> Sitz: Hildesheim, Registergericht: Amtsgericht Hildesheim HRB 3438<br>
> Geschäftsführung: Wilhelm Grabow, Ken Yaguchi<br>
><br>
><br>
> Michael Teyfel (14):<br>
>   ivi-shell: rework goto labels to avoid memory leaks<br>
>   ivi-shell: removed assert<br>
>   ivi-shell: introduction of IVI_INVALID_ID<br>
>   layout-interface: added interface to change surface id<br>
>   ivi-layout: introduced configure_desktop_changed<br>
>   ivi-layout: introduced surface create and configure<br>
>   ivi-shell: linked libweston-desktop and added structs<br>
>   ivi-shell: added libweston-desktop-api implementation<br>
>   ivi-shell: remove surface_destroy_listener<br>
>   ivi-shell: create weston_desktop in wet_shell_init<br>
>   hmi-controller: register for desktop_surface_configured<br>
>   simple-egl: remove ivi-application support<br>
>   simple-shm: remove ivi-application support<br>
>   window client: remove ivi-application support<br>
><br>
>  Makefile.am                    |  11 +--<br>
>  clients/simple-egl.c           |  86 +++-------------<br>
>  clients/simple-shm.c           |  40 --------<br>
>  clients/window.c               |  44 +--------<br>
>  ivi-shell/hmi-controller.c     |  70 +++++++------<br>
>  ivi-shell/ivi-layout-export.h  |  18 ++++<br>
>  ivi-shell/ivi-layout-private.h |   3 +<br>
>  ivi-shell/ivi-layout-shell.h   |   8 ++<br>
>  ivi-shell/ivi-layout.c         | 142 ++++++++++++++++++++++-----<br>
>  ivi-shell/ivi-shell.c          | 218<br>
> ++++++++++++++++++++++++++++++++++++++---<br>
>  ivi-shell/ivi-shell.h          |   2 +<br>
>  11 files changed, 407 insertions(+), 235 deletions(-)<br>
><br>
> --<br>
> 2.7.4<br>
><br>
> _______________________________________________<br>
> wayland-devel mailing list<br>
> <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">
https://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
_______________________________________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">https://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><o:p></o:p></p>
</div>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</body>
</html>