From patchwork Sun Dec 5 19:18:34 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sean Young X-Patchwork-Id: 79156 X-Patchwork-Delegate: sean@mess.org Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from ) id 1mtx24-007xJg-JU; Sun, 05 Dec 2021 19:18:44 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237870AbhLETWG (ORCPT + 1 other); Sun, 5 Dec 2021 14:22:06 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42152 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237264AbhLETWG (ORCPT ); Sun, 5 Dec 2021 14:22:06 -0500 Received: from gofer.mess.org (gofer.mess.org [IPv6:2a02:8011:d000:212::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6EB86C061751 for ; Sun, 5 Dec 2021 11:18:38 -0800 (PST) Received: by gofer.mess.org (Postfix, from userid 1000) id 3DDEBC63C2; Sun, 5 Dec 2021 19:18:35 +0000 (GMT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=mess.org; s=2020; t=1638731915; bh=SLVSiL/DXKFM+LKl0lLgz1wxSqmzRmMxKigJDU56mZk=; h=From:To:Subject:Date:From; b=YNmPSuAsCnLmZvdPxs3AHacFuSVgNEl1rm9IZlrYygyKrCQif/vQpdWzwKspkkGve Z396UTH+jCWZKYKEfrf8jFxWiI087S1yJHvDPKuUEzaceYQS+XopD1vOKXgRtp4kzm qLiw6/EpdWBB6bX7mnAyboioOixuu6Gs7Z3HHi/jINa7MIKa+HKiPkWrtwDUy73uFD COo24Ea8xHvL1C5bzTYFMdKJNSjQEekrWlz63z0UqfG34024BYYavkJpUL116fRUfm 1av1+rN7YM0WPPiBB645oLVfo0goUEX4MKnibqEjWshD6oN50P0pnpo3bV4YoVN/Rt MXUGorAGmse9g== From: Sean Young To: linux-media@vger.kernel.org Subject: [PATCH 1/2] media: streamzap: remove unnecessary ir_raw_event_reset and handle Date: Sun, 5 Dec 2021 19:18:34 +0000 Message-Id: <8a5f192d46f6bf631feeec91f6942b568bac8c05.1638731886.git.sean@mess.org> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.3 (--) X-LSpam-Report: No, score=-2.3 required=5.0 tests=BAYES_00=-1.9,DKIM_SIGNED=0.1,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1,T_DKIM_INVALID=0.01 autolearn=ham autolearn_force=no There is no reason to have a reset after an IR timeout. Calling ir_raw_event_handle() twice for the same interrupt has no affect. Fixes: 56b0ec30c4bc ("[media] rc/streamzap: fix reporting response times") Signed-off-by: Sean Young --- drivers/media/rc/streamzap.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/media/rc/streamzap.c b/drivers/media/rc/streamzap.c index 1cc5ebb85b6c..76f5c72501c4 100644 --- a/drivers/media/rc/streamzap.c +++ b/drivers/media/rc/streamzap.c @@ -244,8 +244,6 @@ static void streamzap_callback(struct urb *urb) sz->idle = true; if (sz->timeout_enabled) sz_push(sz, rawir); - ir_raw_event_handle(sz->rdev); - ir_raw_event_reset(sz->rdev); } else { sz_push_full_space(sz, sz->buf_in[i]); }