[PATCH] video/aperture: fix typos
Javier Martinez Canillas
javierm at redhat.com
Tue Apr 4 11:03:00 UTC 2023
Javier Martinez Canillas <javierm at redhat.com> writes:
[...]
>>> /*
>>> * Remove the device from the device hierarchy. This is the right thing
>>> - * to do for firmware-based DRM drivers, such as EFI, VESA or VGA. After
>>> + * to do for firmware-based fb drivers, such as EFI, VESA or VGA. After
>>
>> That sentences is not well phrased. Maybe say 'This is required for
>> firmware-provided graphics, such as EFI, VESA or VGA.'
>>
>
> Graphic drivers or display drivers would indeed be more accurate here. But
> I think that "fb drivers" is still well pharsed since the are other places
> where either fbdev or DRM drivers for firmware-provided framebuffers are
> named like that.
>
Sui,
Maybe you could post a follow-up patch to improve the comment as suggested
by Thomas?
--
Best regards,
Javier Martinez Canillas
Core Platforms
Red Hat
More information about the dri-devel
mailing list