Linux Samsung SOC development
 help / color / mirror / Atom feed
From: Thomas Zimmermann <tzimmermann@suse.de>
To: Marek Szyprowski <m.szyprowski@samsung.com>,
	inki.dae@samsung.com, sw0312.kim@samsung.com,
	kyungmin.park@samsung.com, airlied@gmail.com, simona@ffwll.ch,
	dri-devel@lists.freedesktop.org
Cc: linux-samsung-soc@vger.kernel.org
Subject: Re: [PATCH 0/5] drm/exynos: fbdev: Use client buffers
Date: Fri, 8 May 2026 08:34:10 +0200	[thread overview]
Message-ID: <87ca272f-d202-4c08-b289-67fc953cde25@suse.de> (raw)
In-Reply-To: <b9765ddd-bb8a-458f-85d4-159d3cc5fe5b@samsung.com>

Hi

Am 06.05.26 um 08:35 schrieb Marek Szyprowski:
> On 23.04.2026 11:37, Thomas Zimmermann wrote:
>> A client buffer holds the DRM framebuffer for an in-kernel DRM
>> client. Until now, exynos created an internal ad-hoc framebuffer for
>> its fbdev emulation, while by-passing the regular interfaces used by
>> user-space compositors.
>>
>> Convert exynos' fbdev emulation to use client buffers. Replacing the
>> existing code with a client buffer allows for stream-lining exynos code
>> and later also the fbdev helpers. The new framebuffer will be registered
>> against the client's file and will support handles for GEM objects. It
>> is then just another framebuffer within the DRM ecosystem.
>>
>> If all driver's fbdev-emulation helpers can be converted to use client
>> buffers, the emulation's framebuffer handling as a whole can possibly be
>> moved into shared helpers.
> Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
>
> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com>

Thanks a lot!

>
>
>> Patch 1 first fixes a long-standing bug.
>>
>> Patches 2 to 4 convert exynos' fbdev emulation to client buffers. It
>> still allocates a GEM object buffer tailored towards fbdev emulation,
>> but size calculations now use common DRM helpers.
>>
>> Patch 5 cleans up symbol visibility in exynos' fb code.
>>
>> Thomas Zimmermann (5):
>>    drm/exynos: fbdev: Remove offset into screen_buffer
>>    drm/exynos: fbdev: Inline exynos_drm_fbdev_update()
>>    drm/exynos: fbdev: Calculate buffer geometry with format helpers
>>    drm/exynos: fbdev: Use a DRM client buffer
>>    drm/exynos: Make exynos_drm_framebuffer_init() an internal interface
>>
>>   drivers/gpu/drm/exynos/exynos_drm_fb.c    |   2 +-
>>   drivers/gpu/drm/exynos/exynos_drm_fb.h    |   7 --
>>   drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 100 ++++++++++------------
>>   3 files changed, 47 insertions(+), 62 deletions(-)
>>
> Best regards

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com
GF: Jochen Jaser, Andrew McDonald, Werner Knoblich, (HRB 36809, AG Nürnberg)



      reply	other threads:[~2026-05-08  6:34 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20260423095139eucas1p149a0e97f62e8bf353a65466ca26cbff9@eucas1p1.samsung.com>
2026-04-23  9:37 ` [PATCH 0/5] drm/exynos: fbdev: Use client buffers Thomas Zimmermann
2026-04-23  9:37   ` [PATCH 1/5] drm/exynos: fbdev: Remove offset into screen_buffer Thomas Zimmermann
2026-04-23  9:37   ` [PATCH 2/5] drm/exynos: fbdev: Inline exynos_drm_fbdev_update() Thomas Zimmermann
2026-04-23  9:37   ` [PATCH 3/5] drm/exynos: fbdev: Calculate buffer geometry with format helpers Thomas Zimmermann
2026-04-23  9:37   ` [PATCH 4/5] drm/exynos: fbdev: Use a DRM client buffer Thomas Zimmermann
2026-04-23  9:37   ` [PATCH 5/5] drm/exynos: Make exynos_drm_framebuffer_init() an internal interface Thomas Zimmermann
2026-05-06  6:59     ` Chen-Yu Tsai
2026-05-08  6:41       ` Thomas Zimmermann
2026-05-06  6:35   ` [PATCH 0/5] drm/exynos: fbdev: Use client buffers Marek Szyprowski
2026-05-08  6:34     ` Thomas Zimmermann [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87ca272f-d202-4c08-b289-67fc953cde25@suse.de \
    --to=tzimmermann@suse.de \
    --cc=airlied@gmail.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=inki.dae@samsung.com \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    --cc=simona@ffwll.ch \
    --cc=sw0312.kim@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox