upstream: [media] vb2: fix num_buffers calculation if req->count > VIDEO_MAX_FRAMES
authorPhilipp Zabel <p.zabel@pengutronix.de>
Fri, 9 May 2014 15:32:10 +0000 (12:32 -0300)
committerChanho Park <chanho61.park@samsung.com>
Tue, 18 Nov 2014 02:59:21 +0000 (11:59 +0900)
commitf6bf29bffa50e33da4a3a4e5653db9dae9463622
treeb41b31c1419b8fb65d258e8365b13bddc655bde5
parent0f8d19273906ee5ff9fa3672744a387e86984178
upstream: [media] vb2: fix num_buffers calculation if req->count > VIDEO_MAX_FRAMES

num_buffers can't be bigger than VIDEO_MAX_FRAME. This is assured by:
  num_buffers = min_t(unsigned int, req->count, VIDEO_MAX_FRAME);

However, this value is overriden by:
  num_buffers = max_t(unsigned int, req->count, q->min_buffers_needed);

It should, instead, use the previously calculated value as an input
to max_t:
  num_buffers = max_t(unsigned int, num_buffers, q->min_buffers_needed);

Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Reviewed-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/v4l2-core/videobuf2-core.c