On Mon, Jun 13, 2022 at 09:51:27PM +0200, Hans de Goede wrote: > highmem is always false, drop it. ... > /* > - * allocate/free physical pages for the bo. will try to alloc mem > - * from highmem if from_highmem is set, and type indicate that the > + * allocate/free physical pages for the bo. type indicate that the Since you are touching this anyway, can be capitalized Allocate/Free ... > * pages will be allocated by using video driver (for share buffer) > * or by ISP driver itself. > */ -- With Best Regards, Andy Shevchenko