<div dir="ltr"><div class="im"><div>> You should execute the ioctl in fbdev_set_screen_info.</div></div><div>fbdev_set_screen_info
 is used only when the fb device is reenabled. The colors map has to be 
initialized at the beginning in all cases.<br>
</div><div>If we use this function we will call it in all cases and it's not required.<div class="im"><br><br>>I don't see the codepath which results in cmap.transp being not set in the test for it.<br></div></div>
<div>Yes... I forgot => new patch<div class="im"><br>
<br>>You don't initialize cmap.red so the test for it gives an undefined result.<br></div></div><div>Yes... error => new patch<div class="im"><br><br>>Don't define macros in the middle of code with a literal that's only used once.<br>

</div></div><div>In fact that comes from the cmap.transp which are not 
always initialized. In some cases transparency is useless. But I will 
repair => new patch<br><br></div><div>New patch<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/7 John Kåre Alsaker <span dir="ltr"><<a href="mailto:john.kare.alsaker@gmail.com" target="_blank">john.kare.alsaker@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>You should execute the ioctl in fbdev_set_screen_info.</div><div><br></div><div>I don't see the codepath which results in cmap.transp being not set in the test for it.<br>
</div><div class="gmail_extra">
<br></div><div class="gmail_extra">You don't initialize cmap.red so the test for it gives an undefined result.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Don't define macros in the middle of code with a literal that's only used once.<br>

<br><div class="gmail_quote"><div class="im">On Fri, Jun 7, 2013 at 12:18 PM, Marc Chalain <span dir="ltr"><<a href="mailto:marc.chalain@gmail.com" target="_blank">marc.chalain@gmail.com</a>></span> wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im"><div dir="ltr">this patch allow to use direct color and set the colors map with default value.<br>

<br></div>
<br></div>_______________________________________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
<br></blockquote></div><br></div></div>
</blockquote></div><br></div>