Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f51ce0e6 authored by Srinu Gorle's avatar Srinu Gorle
Browse files

Revert "msm: wfd: Restart hrtimer after forwarding it" 



This reverts commit Ia8e82a6ad327acddc135835268a755e3646d2ee8.
Calling hrtimer_forward_now() outside of the hrtimer callback context
and not restarting it seems to be a misuse of the hrt APIs.

Change-Id: Idd3f967d8037435ec43bbd843ce122d989aa838a
Signed-off-by: default avatarSrinu Gorle <sgorle@codeaurora.org>
parent 5f4d31f2

drivers/media/platform/msm/wfd/vsg-subdev.c

+1 −7
Original line number Original line Diff line number Diff line
@@ -26,12 +26,6 @@ 
#define TICKS_PER_TIMEOUT 2

#define TICKS_PER_TIMEOUT 2









static void vsg_reset_timer(struct hrtimer *timer, ktime_t time)

{

	hrtimer_forward_now(timer, time);

	hrtimer_restart(timer);

}



static int vsg_release_input_buffer(struct vsg_context *context,

static int vsg_release_input_buffer(struct vsg_context *context,

		struct vsg_buf_info *buf)

		struct vsg_buf_info *buf)

{

{
@@ -467,7 +461,7 @@ static long vsg_queue_buffer(struct v4l2_subdev *sd, void *arg) 
			 * otherwise, diff between two consecutive frames might

			 * otherwise, diff between two consecutive frames might

			 * be less than max_frame_interval (for just one sample)

			 * be less than max_frame_interval (for just one sample)

			 */

			 */

			vsg_reset_timer(&context->threshold_timer,

			hrtimer_forward_now(&context->threshold_timer,

				ns_to_ktime(context->max_frame_interval));

				ns_to_ktime(context->max_frame_interval));

		}

		}

	}

	}