From nobody Wed Apr 1 13:57:24 2026 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B354F3F87FA; Tue, 31 Mar 2026 10:28:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.167.242.64 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774952899; cv=none; b=LLN8usN+Wj5u0YgayuetVTkC98+O+yrC6uyNW1Qefj64/EFdRQEY6YIDsi7WwW48+X6vQQYq1MmlnoVBX5K4qB1Nhf7UpCx/8oej3tjbDlZLhcsCOe7w9ahqB34m+LKezTXFIDuMbSKtTAi2LufRxl0uSb040gCcCa+LOZIOxdo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774952899; c=relaxed/simple; bh=B7Z0qz1KWQYocWsu5+Ifr/zBqKWMWpNbiEUfrqDJ1y0=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=HMWij8hgVOHJOCaNGCefxWkQJyxa2jX0Wq0pknzMEIMlEGz4fX7HGrtpdyGR9CfAvh//eOKih4SCnsFAWcjCo504qhRcV902eiGXufa4Tn/QHdDC/f/JsnihTWVo4oufkw3AHYDO2/HH/kvlRpk6VDjfshJsrWqbCSXx3m4YXfI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com; spf=pass smtp.mailfrom=ideasonboard.com; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b=uGHuLUkK; arc=none smtp.client-ip=213.167.242.64 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="uGHuLUkK" Received: from [100.93.44.16] (net-93-65-100-155.cust.vodafonedsl.it [93.65.100.155]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 86FEC1E5A; Tue, 31 Mar 2026 12:26:42 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1774952803; bh=B7Z0qz1KWQYocWsu5+Ifr/zBqKWMWpNbiEUfrqDJ1y0=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=uGHuLUkKxTsTnGtpiiYF1BWIvURXijGXYVPjpChmh3J2a4oybz+s26EIj6RGUI9qH Q4zIj53OdP2fcqKtH/juUbN0+o6ZjtDuH1Kg5xQ0zd4qbiOR7RfzlZvWaY1qNrc5Uo SBAVvZL/iXoDwX/k705wYK2KlSDiAjmLKkfPzVcc= From: Jacopo Mondi Date: Tue, 31 Mar 2026 12:27:45 +0200 Subject: [PATCH v2 15/15] media: rzg2l-cru: Simplify irq return value handling Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260331-b4-cru-rework-v2-15-f94b238b35d4@ideasonboard.com> References: <20260331-b4-cru-rework-v2-0-f94b238b35d4@ideasonboard.com> In-Reply-To: <20260331-b4-cru-rework-v2-0-f94b238b35d4@ideasonboard.com> To: Mauro Carvalho Chehab , Laurent Pinchart , Biju Das , Hans Verkuil , Sakari Ailus , Tommaso Merciai Cc: Daniel Scally , =?utf-8?q?Barnab=C3=A1s_P=C5=91cze?= , Lad Prabhakar , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Jacopo Mondi , Jacopo Mondi X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=1486; i=jacopo.mondi@ideasonboard.com; h=from:subject:message-id; bh=gRFkpLLIbDTTNqHG9UI0gnxHk9cmv8yF4XICpOXdbIg=; b=owEBbQKS/ZANAwAKAXI0Bo8WoVY8AcsmYgBpy6GptX0NXeWwkVsBdoPnNxbbQihNQVJ2k/bNd ocdYo15KGaJAjMEAAEKAB0WIQS1xD1IgJogio9YOMByNAaPFqFWPAUCacuhqQAKCRByNAaPFqFW PHF9D/9EG/1bQHgZrFVDNvKZA5OJlGK/QvlYAQziYV+ZWqXQ5QYLObYxht8QPLC4Pd6flhQjG/e cJxcNXfm9fHZv5tkOzZG9GaFgKR8sh6TbhUBYrB6vZyPk1ICMpB44WNRHDNhRSawwCl7UditUW5 S3ld2ExZW5oEqNO37TFmsGWF6VnTN06PVBqbvl3W0f2WPJFZMgAZ+tu1KvY3TrTLZtawuj50xgc N8TUGbMLqS6m8f0kCkcAa9yZjw/wB9Wtjfv+QNV8nhp4vClw9+dJsq4GUVwCo5zKBNwbYsLMHxR FkryAnKi70dfvrKO+M64BxWrg9LaqrOUiONuvIJV4opes8zFtIxw8i6Y7Do7JeARAHS6bJ4bTYm o8h4dqq6tdrFo545P7O1U+dylKNTWX1PglnhsTSPgpsgatoeJ7lHNWC3xcDtZKG+3kmdHReZacs rzg2rhKDnZhVu02vpQYw/wBpAyQefOCKki4fdgEoIW0ImcLLFZfm226EBPTXBTJFs0Yg6rsAEzb Yv1ep0O5oGFUfhwT1N8iy1uNspnWCsc++ALE594DgHNQP5KzkScbMTbKn/cIHiWbGm7GHSVg82r akmmM70EHnVKLXyeR8724T8AFt/bL92otYPSOZWtpUZ/Kq6+7sp9ijL0JbRPsJJqJsNMNTsK5ks B83MFJxzcL2i0Mw== X-Developer-Key: i=jacopo.mondi@ideasonboard.com; a=openpgp; fpr=72392EDC88144A65C701EA9BA5826A2587AD026B From: Jacopo Mondi The rzg2l_cru_irq() irq handler uses a local variable to store the handler return value. Simplify it by using IRQ_NONE and IRQ_HANDLED. Signed-off-by: Jacopo Mondi Reviewed-by: Tommaso Merciai Reviewed-by: Daniel Scally --- drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c b/drive= rs/media/platform/renesas/rzg2l-cru/rzg2l-video.c index e2073cc1afc6..3390e08cd868 100644 --- a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c +++ b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c @@ -591,16 +591,13 @@ static void rzg2l_cru_stop_streaming(struct rzg2l_cru= _dev *cru) irqreturn_t rzg2l_cru_irq(int irq, void *data) { struct rzg2l_cru_dev *cru =3D data; - unsigned int handled =3D 0; u32 irq_status; u32 amnmbs; int slot; =20 irq_status =3D rzg2l_cru_read(cru, CRUnINTS); if (!irq_status) - return IRQ_RETVAL(handled); - - handled =3D 1; + return IRQ_NONE; =20 rzg2l_cru_write(cru, CRUnINTS, rzg2l_cru_read(cru, CRUnINTS)); =20 @@ -634,7 +631,7 @@ irqreturn_t rzg2l_cru_irq(int irq, void *data) /* Prepare for next frame */ rzg2l_cru_fill_hw_slot(cru, slot); =20 - return IRQ_RETVAL(handled); + return IRQ_HANDLED; } =20 irqreturn_t rzg3e_cru_irq(int irq, void *data) --=20 2.53.0