Commit 6a5228fd authored by Tomi Valkeinen's avatar Tomi Valkeinen

drm/omap: remove align_pitch()

The previous commit removed aligning the pitch to SGX's pitch
requirement from align_pitch(). What's left is effectively a function
that returns width * bytespp.

To clean up the driver, we can remove the function and have the
calculation inline in the two places which call align_pitch().
Signed-off-by: default avatarTomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
parent 63871f71
...@@ -239,14 +239,6 @@ struct dma_buf *omap_gem_prime_export(struct drm_device *dev, ...@@ -239,14 +239,6 @@ struct dma_buf *omap_gem_prime_export(struct drm_device *dev,
struct drm_gem_object *omap_gem_prime_import(struct drm_device *dev, struct drm_gem_object *omap_gem_prime_import(struct drm_device *dev,
struct dma_buf *buffer); struct dma_buf *buffer);
static inline int align_pitch(int pitch, int width, int bpp)
{
int bytespp = (bpp + 7) / 8;
/* in case someone tries to feed us a completely bogus stride: */
pitch = max(pitch, width * bytespp);
return pitch;
}
/* map crtc to vblank mask */ /* map crtc to vblank mask */
uint32_t pipe2vbl(struct drm_crtc *crtc); uint32_t pipe2vbl(struct drm_crtc *crtc);
struct omap_dss_device *omap_encoder_get_dssdev(struct drm_encoder *encoder); struct omap_dss_device *omap_encoder_get_dssdev(struct drm_encoder *encoder);
......
...@@ -125,9 +125,8 @@ static int omap_fbdev_create(struct drm_fb_helper *helper, ...@@ -125,9 +125,8 @@ static int omap_fbdev_create(struct drm_fb_helper *helper,
mode_cmd.width = sizes->surface_width; mode_cmd.width = sizes->surface_width;
mode_cmd.height = sizes->surface_height; mode_cmd.height = sizes->surface_height;
mode_cmd.pitches[0] = align_pitch( mode_cmd.pitches[0] = mode_cmd.width *
mode_cmd.width * ((sizes->surface_bpp + 7) / 8), DIV_ROUND_UP(sizes->surface_bpp, 8);
mode_cmd.width, sizes->surface_bpp);
fbdev->ywrap_enabled = priv->has_dmm && ywrap_enabled; fbdev->ywrap_enabled = priv->has_dmm && ywrap_enabled;
if (fbdev->ywrap_enabled) { if (fbdev->ywrap_enabled) {
......
...@@ -661,7 +661,8 @@ int omap_gem_dumb_create(struct drm_file *file, struct drm_device *dev, ...@@ -661,7 +661,8 @@ int omap_gem_dumb_create(struct drm_file *file, struct drm_device *dev,
{ {
union omap_gem_size gsize; union omap_gem_size gsize;
args->pitch = align_pitch(0, args->width, args->bpp); args->pitch = args->width * DIV_ROUND_UP(args->bpp, 8);
args->size = PAGE_ALIGN(args->pitch * args->height); args->size = PAGE_ALIGN(args->pitch * args->height);
gsize = (union omap_gem_size){ gsize = (union omap_gem_size){
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment