<div dir="ltr">Thanks for your review, Rob. Using kms-dri would mean writing a new gralloc to basically the same thing as<br><div>gralloc.default and moving to grm_gralloc seems to be a bigger task meant for another day. But I agree we would</div><div>want to avoid introducing dependency. Would extending drisw loader func and limit gralloc dependency in egl_android<br></div><div>ok with you?</div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-01-07 2:14 GMT+08:00 Rob Herring <span dir="ltr"><<a href="mailto:robh@kernel.org" target="_blank">robh@kernel.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Jan 6, 2017 at 11:35 AM, Wu Zhen <<a href="mailto:wuzhen@jidemail.com">wuzhen@jidemail.com</a>> wrote:<br>
> From: WuZhen <<a href="mailto:wuzhen@jidemail.com">wuzhen@jidemail.com</a>><br>
><br>
> Change-Id: Ifabf40fe94007f73171a89b235450<wbr>02707817053<br>
> Reviewed-by: Mauro Rossi <<a href="mailto:issor.oruam@gmail.com">issor.oruam@gmail.com</a>><br>
> Reviewed-by: Chih-Wei Huang <<a href="mailto:cwhuang@linux.org.tw">cwhuang@linux.org.tw</a>><br>
> ---<br>
>  src/gallium/targets/dri/<wbr>Android.mk        |  1 +<br>
>  src/gallium/winsys/sw/dri/dri_<wbr>sw_winsys.c | 65 ++++++++++++++++++++++++++++++<wbr>+<br>
>  2 files changed, 66 insertions(+)<br>
<br>
</span>I don't think we want to add a gralloc dependency here. Either we<br>
should use kms-dri or GBM (and gbm_gralloc) here instead.<br>
<span class="HOEnZb"><font color="#888888"><br>
Rob<br>
</font></span></blockquote></div><br></div>