Re: [PATCH RFC 6/8] drm: hisilicon: Add support for fbdev

From: Xinwei Kong
Date: Tue Sep 15 2015 - 23:44:35 EST


hi rob

On 2015/9/16 2:25, Rob Herring wrote:
> On 09/15/2015 04:37 AM, Xinwei Kong wrote:
>> If you config DRM_HISI_FBDEV optional, this patch will only support fbdev
>> mode while also supporting double buffer.
>
> This is a lot of duplicated code from CMA fbdev. Is double buffering the
> only reason why CMA fbdev can't be used or are there some other
> constraints? Double buffering in fbdev has always been a hack, so I'm
> guessing that is not a feature that should be added here.
>
I will drop it.

xinwei
> Rob
>
>> Signed-off-by: Xinliang Liu <xinliang.liu@xxxxxxxxxx>
>> Signed-off-by: Xinwei Kong <kong.kongxinwei@xxxxxxxxxxxxx>
>> Signed-off-by: Andy Green <andy.green@xxxxxxxxxx>
>> Signed-off-by: Jiwen Qi <qijiwen@xxxxxxxxxxxxx>
>> Signed-off-by: Yu Gong <gongyu@xxxxxxxxxxxxx>
>> ---
>> drivers/gpu/drm/hisilicon/Kconfig | 13 +
>> drivers/gpu/drm/hisilicon/Makefile | 3 +-
>> drivers/gpu/drm/hisilicon/hisi_drm_connector.c | 4 +
>> drivers/gpu/drm/hisilicon/hisi_drm_drv.c | 9 +
>> drivers/gpu/drm/hisilicon/hisi_drm_dsi.c | 15 +
>> drivers/gpu/drm/hisilicon/hisi_drm_fb.h | 5 +
>> drivers/gpu/drm/hisilicon/hisi_drm_fbdev.c | 395 +++++++++++++++++++++++++
>> drivers/gpu/drm/hisilicon/hisi_drm_fbdev.h | 24 ++
>> 8 files changed, 467 insertions(+), 1 deletion(-)
>> create mode 100644 drivers/gpu/drm/hisilicon/hisi_drm_fbdev.c
>> create mode 100644 drivers/gpu/drm/hisilicon/hisi_drm_fbdev.h
>
>
> .
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/