<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 16 September 2015 at 02:25, Rob Herring <span dir="ltr"><<a href="mailto:robh@kernel.org" target="_blank">robh@kernel.org</a>></span> wrote:</div><div class="gmail_quote">Hi Rob, thanks a lot for reply:-)</div><div class="gmail_quote"><br><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"><span>On 09/15/2015 04:37 AM, Xinwei Kong wrote:<br>
> If you config DRM_HISI_FBDEV optional, this patch will only support fbdev<br>
> mode while also supporting double buffer.<br>
<br>
</span>This is a lot of duplicated code from CMA fbdev. Is double buffering the<br>
only reason why CMA fbdev can't be used or are there some other<br>
constraints? </blockquote><div>Yes, double buffering is the main reason we rewrite our own fbdev.</div><div>CMA fbdev only create one buffer. But we need at least  double buffer for running Android with fbdev.</div><div><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">Double buffering in fbdev has always been a hack, so I'm<br>
guessing that is not a feature that should be added here.<br>
<span><font color="#888888"><br></font></span></blockquote><div>If so, I think it is hard to be accepted for my cma fbdev patch to support multi buffer.</div><div>This early week, I have sent a cma fbdev patch for supporting this. The subject is </div><div>"[PATCH] drm/cma-helper: Add multi buffer support for cma fbdev".</div><div>We do have a strong will to support this feature. I described the reason in the patch. Please take a look for me. Thank you very much.</div><div><br></div><div>-Xinliang</div><div><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"><span><font color="#888888">
Rob<br>
</font></span><div><div><br>
> Signed-off-by: Xinliang Liu <<a href="mailto:xinliang.liu@linaro.org" target="_blank">xinliang.liu@linaro.org</a>><br>
> Signed-off-by: Xinwei Kong <<a href="mailto:kong.kongxinwei@hisilicon.com" target="_blank">kong.kongxinwei@hisilicon.com</a>><br>
> Signed-off-by: Andy Green <<a href="mailto:andy.green@linaro.org" target="_blank">andy.green@linaro.org</a>><br>
> Signed-off-by: Jiwen Qi <<a href="mailto:qijiwen@hisilicon.com" target="_blank">qijiwen@hisilicon.com</a>><br>
> Signed-off-by: Yu Gong <<a href="mailto:gongyu@hisilicon.com" target="_blank">gongyu@hisilicon.com</a>><br>
> ---<br>
>  drivers/gpu/drm/hisilicon/Kconfig              |  13 +<br>
>  drivers/gpu/drm/hisilicon/Makefile             |   3 +-<br>
>  drivers/gpu/drm/hisilicon/hisi_drm_connector.c |   4 +<br>
>  drivers/gpu/drm/hisilicon/hisi_drm_drv.c       |   9 +<br>
>  drivers/gpu/drm/hisilicon/hisi_drm_dsi.c       |  15 +<br>
>  drivers/gpu/drm/hisilicon/hisi_drm_fb.h        |   5 +<br>
>  drivers/gpu/drm/hisilicon/hisi_drm_fbdev.c     | 395 +++++++++++++++++++++++++<br>
>  drivers/gpu/drm/hisilicon/hisi_drm_fbdev.h     |  24 ++<br>
>  8 files changed, 467 insertions(+), 1 deletion(-)<br>
>  create mode 100644 drivers/gpu/drm/hisilicon/hisi_drm_fbdev.c<br>
>  create mode 100644 drivers/gpu/drm/hisilicon/hisi_drm_fbdev.h<br>
<br>
</div></div></blockquote></div><br></div></div>