[PATCH v2 04/11] drm/tegra: Set fbdev FBINFO_VIRTFB flag
Thomas Zimmermann
tzimmermann at suse.de
Thu Jul 6 07:47:02 PDT 2023
Hi
Am 06.07.23 um 16:31 schrieb Thierry Reding:
> On Thu, Jul 06, 2023 at 02:46:42PM +0200, Thomas Zimmermann wrote:
>> Mark the framebuffer with FBINFO_VIRTFB. The framebuffer range is
>> in DMA-able memory and should be accessed with the CPU's regular
>> memory ops.
>>
>> v2:
>> * drop FBINFO_DEFAULT
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
>> Reviewed-by: Javier Martinez Canillas <javierm at redhat.com>
>> Acked-by: Maxime Ripard <mripard at kernel.org>
>> Cc: Thierry Reding <thierry.reding at gmail.com>
>> Cc: Mikko Perttunen <mperttunen at nvidia.com>
>> ---
>> drivers/gpu/drm/tegra/fbdev.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/gpu/drm/tegra/fbdev.c b/drivers/gpu/drm/tegra/fbdev.c
>> index 82577b7c88da..d8460c5dc91e 100644
>> --- a/drivers/gpu/drm/tegra/fbdev.c
>> +++ b/drivers/gpu/drm/tegra/fbdev.c
>> @@ -132,6 +132,7 @@ static int tegra_fbdev_probe(struct drm_fb_helper *helper,
>> }
>> }
>>
>> + info->flags |= FBINFO_VIRTFB;
>> info->screen_base = (void __iomem *)bo->vaddr + offset;
>
> As part of this, do we also need to set info->screen_buffer instead of
> info->screen_base? The drm_fbdev_dma_helper functions do that.
Indeed, good point. I'll update this in the next iteration.
Best regards
Thomas
>
> Thierry
>
>> info->screen_size = size;
>> info->fix.smem_start = (unsigned long)(bo->iova + offset);
>> --
>> 2.41.0
>>
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20230706/b521a71a/attachment-0001.sig>
More information about the linux-arm-kernel
mailing list