Skip to content

Commit db464d1

Browse files
author
Jyri Sarha
committed
drm/tilcdc: Remove tilcdc_verify_fb()
commit c72cc66 upstream. Remove tilcdc_verify_fb(). The tilcdc_verify_fb() function is not needed because the same checks are implemented in tilcdc_plane_atomic_check(). Signed-off-by: Jyri Sarha <jsarha@ti.com>
1 parent ebbd1c9 commit db464d1

File tree

1 file changed

+0
-21
lines changed

1 file changed

+0
-21
lines changed

drivers/gpu/drm/tilcdc/tilcdc_crtc.c

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -151,36 +151,15 @@ static void tilcdc_crtc_destroy(struct drm_crtc *crtc)
151151
drm_flip_work_cleanup(&tilcdc_crtc->unref_work);
152152
}
153153

154-
static int tilcdc_verify_fb(struct drm_crtc *crtc, struct drm_framebuffer *fb)
155-
{
156-
struct drm_device *dev = crtc->dev;
157-
unsigned int depth, bpp;
158-
159-
drm_fb_get_bpp_depth(fb->pixel_format, &depth, &bpp);
160-
161-
if (fb->pitches[0] != crtc->mode.hdisplay * bpp / 8) {
162-
dev_err(dev->dev,
163-
"Invalid pitch: fb and crtc widths must be the same");
164-
return -EINVAL;
165-
}
166-
167-
return 0;
168-
}
169-
170154
int tilcdc_crtc_page_flip(struct drm_crtc *crtc,
171155
struct drm_framebuffer *fb,
172156
struct drm_pending_vblank_event *event,
173157
uint32_t page_flip_flags)
174158
{
175159
struct tilcdc_crtc *tilcdc_crtc = to_tilcdc_crtc(crtc);
176160
struct drm_device *dev = crtc->dev;
177-
int r;
178161
unsigned long flags;
179162

180-
r = tilcdc_verify_fb(crtc, fb);
181-
if (r)
182-
return r;
183-
184163
if (tilcdc_crtc->event) {
185164
dev_err(dev->dev, "already pending page flip!\n");
186165
return -EBUSY;

0 commit comments

Comments
 (0)