kernel/drivers/gpu/drm/omapdrm/omap_fbdev.h, branch linux-4.17.yHosts the 0x221E linux distro kernel.https://universe.0xinfinity.dev/distro/kernel/atom?h=linux-4.17.y2018-03-01T07:18:18Zdrm/omap: cleanup fbdev init/free2018-03-01T07:18:18ZTomi Valkeinentomi.valkeinen@ti.com2018-02-09T07:36:23Zurn:sha1:efd1f06be004a6a384f0482ef76c12bc202e1b8e
omap_fbdev_init() and omap_fbdev_free() use priv->fbdev directly.
However, omap_fbdev_init() returns the fbdev, and omap_drv.c also
assigns the return value to priv->fbdev. This is slightly confusing.
Clean this up by removing the omap_fbdev_init() return value, as we
don't care whether fbdev init succeeded or not. Also change omap_drv.c
to call omap_fbdev_free() always, and omap_fbdev_free() does the check
if fbdev was initialized.
While at it, rename omap_fbdev_free() to omap_fbdev_fini() to better
match the "init" counterpart.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
drm: omapdrm: Deconstruct the omap_drv.h header.2017-12-19T08:32:00ZLaurent Pinchartlaurent.pinchart@ideasonboard.com2017-10-13T14:59:05Zurn:sha1:b5561412975633049661436fb3764d96b945eed0
The number of function declarations in the omap_drv.h degrades
readability. To fix it, create new header files for each part of the
driver and move the related functions.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>