[PATCH v2 18/18] fbdev: Document that framebuffer_alloc() returns zero'ed data

Randy Dunlap rdunlap at infradead.org
Fri Jul 14 02:02:39 AEST 2023

On 7/13/23 06:21, Miguel Ojeda wrote:
> On Thu, Jul 13, 2023 at 3:03 PM Thomas Zimmermann <tzimmermann at suse.de> wrote:
>> Most fbdev drivers depend on framebuffer_alloc() to initialize the
>> allocated memory to 0. Document this guarantee.
>> Suggested-by: Miguel Ojeda <ojeda at kernel.org>
>> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
>> Cc: Helge Deller <deller at gmx.de>
> Thanks for sending this! Maybe this would be best earlier in the
> series, so that later patches make more sense (since they use the
> guarantee), but it is not a big deal.
>> + * aligned to sizeof(long). Both, the instance of struct fb_info and
>> + * the driver private data, are cleared to zero.
> I think both commas may be best omitted (but I am not a native speaker).

Yes, it would be better to omit them.

> Reviewed-by: Miguel Ojeda <ojeda at kernel.org>
> Cheers,
> Miguel


More information about the Linuxppc-dev mailing list