<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, Feb 6, 2025 at 2:57 AM Thomas Zimmermann <<a href="mailto:tzimmermann@suse.de">tzimmermann@suse.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi<br>
<br>
<br>
Am 05.02.25 um 21:08 schrieb Anusha Srivatsa:<br>
> Update the Documentation to be more precise.<br>
><br>
> v2: Update for clarity<br>
><br>
> Cc: Thomas Zimmermann <<a href="mailto:tzimmermann@suse.de" target="_blank">tzimmermann@suse.de</a>><br>
> Signed-off-by: Anusha Srivatsa <<a href="mailto:asrivats@redhat.com" target="_blank">asrivats@redhat.com</a>><br>
> ---<br>
>   Documentation/gpu/todo.rst | 12 ++++++------<br>
>   1 file changed, 6 insertions(+), 6 deletions(-)<br>
><br>
> diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst<br>
> index 256d0d1cb2164bd94f9b610a751b907834d96a21..03fcd086889acbd1294669b260292ffc79e97e2f 100644<br>
> --- a/Documentation/gpu/todo.rst<br>
> +++ b/Documentation/gpu/todo.rst<br>
> @@ -441,13 +441,13 @@ Contact: Thomas Zimmermann <<a href="mailto:tzimmermann@suse.de" target="_blank">tzimmermann@suse.de</a>><br>
>   <br>
>   Level: Intermediate<br>
>   <br>
> -Request memory regions in all drivers<br>
> --------------------------------------<br>
> +Request memory regions in all fbdev drivers<br>
> +--------------------------------------------<br>
>   <br>
> -Go through all drivers and add code to request the memory regions that the<br>
> -driver uses. This requires adding calls to request_mem_region(),<br>
> -pci_request_region() or similar functions. Use helpers for managed cleanup<br>
> -where possible.<br>
> +Old/ancient fbdev drivers do not request their memory properly.<br>
> +Go through these drivers and add code to request the memory regions that the<br>
> +driver uses. Problematic areas include hardware that has exclusive ranges like<br>
> +VGA. VGA16fb does not request the range as it is expected.<br>
<br>
I'd keep both, old and new text, and turn them into a single paragraph. <br>
The old text explains what to do and the new text gives examples.<br>
<br></blockquote><div><br></div><div>Sure. Waiting for one final patch to be reviewed(patch 11). With that, I will make the changes you suggested and re-roll the series. <br></div><div><br></div><div>Thanks for the feedback!</div><div><br></div><div>Anusha<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Best regards<br>
Thomas<br>
>   <br>
>   Drivers are pretty bad at doing this and there used to be conflicts among<br>
>   DRM and fbdev drivers. Still, it's the correct thing to do.<br>
><br>
<br>
-- <br>
--<br>
Thomas Zimmermann<br>
Graphics Driver Developer<br>
SUSE Software Solutions Germany GmbH<br>
Frankenstrasse 146, 90461 Nuernberg, Germany<br>
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman<br>
HRB 36809 (AG Nuernberg)<br>
<br>
</blockquote></div></div>