From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 096EDC88E4D for ; Fri, 11 Sep 2026 17:35:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=PgOxBvIYMhCiqcjRKAs1NE2i0HsOlBqtwvluUUrsVNA=; b=1JX66W0/dVlQ+W sIcxwfUaSMqBsCbhNv7G4sIvQ7x0expzFpexSr1PxH//H8Bnr0lmQ7M/wh8MInQd296vKr54OKDHv g8hkqw+wVO56/rkujDl6hxObDNvsmzTPu696zluLI9HG7tdmRRhjNbD7IlCCpOjAKDZlZvEJZDdKV Oqs0o3rtF63ClA6qvXp5Ez6BG+wlwZmqY+9a9jdpLGclOdqCUHpquFe3Fu+iPWDXkgI7eXRcwSehK KLc7gyvwYzQP9n2ALK64g54UnRAT4p60TtTQmmVP3Qex6VSKkZmsstJmguLSNloQ6aCqXXI376HDH xvS24PkS/BIqSAy6xHEA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x559j-0000000HMi2-31Nl; Fri, 11 Sep 2026 17:35:19 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x559f-0000000HMgl-1PdB; Fri, 11 Sep 2026 17:35:17 +0000 Received: from killaraus.ideasonboard.com (2001-14ba-70f3-e800--a06.rev.dnainternet.fi [IPv6:2001:14ba:70f3:e800::a06]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 5A6B756F; Fri, 11 Sep 2026 19:33:32 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1789148012; bh=kN+PV2iOq3JJVRTjxSU6DGs6/edFIKhfuQcDCarufaM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=oLgUx8i+zFrsl3PNAFYR+UY96C8AxETKgkFa43f5G2dsiQlRtt3jNbUoZ7ZljlSEV QpbLQ765WCKzy/jPiKBccIXmyLy5PuTGKOo5pgYtV+zHDYwP/YMApCB0kTnTrIzNpD /5wyKnc+qpx+Wes0SQJ0CZrf0+PwjFTV9WNpqn+w= Date: Fri, 11 Sep 2026 20:35:08 +0300 From: Laurent Pinchart To: Jacopo Mondi Cc: Kieran Bingham , Dafna Hirschfeld , Heiko Stuebner , Mauro Carvalho Chehab , Paul Elder , Sakari Ailus , linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: Re: [PATCH] media: rkisp1: Fix Bayer demosaicing bypass Message-ID: <20260911173508.GW1892234@killaraus.ideasonboard.com> References: <20260911-imx8mp-demosaicing-bypass-v1-1-5568a7a560a6@ideasonboard.com> <178913687012.1723501.3167773155478207003@ping.linuxembedded.co.uk> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260911_103515_538825_7EDA80BB X-CRM114-Status: GOOD ( 27.97 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org On Fri, Sep 11, 2026 at 04:43:32PM +0200, Jacopo Mondi wrote: > On Fri, Sep 11, 2026 at 03:27:50PM +0100, Kieran Bingham wrote: > > Quoting Jacopo Mondi (2026-09-11 14:56:19) > > > The RKISP1_CIF_ISP_DEMOSAIC_BYPASS bit, when set, bypasses the > > > demosaicing block on the RkISP1 ISP. > > > > > > The current implementation however clears the bit when demosaicing > > > have to be bypassed and sets it when demosaicing has to be enabled, > > > effectively inverting the bypass bit handling logic. > > > > Ouch. > > > > > Fix this by setting the bypass bit when disabling the demosaicing block, > > > and by clearing it instead when demosaicing has to be performed. > > > > > > The issue never manifested itself as libcamera hasn't an algorithm > > > to control Bayer demosaicing bypass yet. > > > > > > Fixes: 6c53a7b68c5d ("media: rkisp1: Implement extensible params support") > > > Cc: stable@vger.kernel.org > > > Signed-off-by: Jacopo Mondi > > > --- > > > media: rkisp1: Fix demosaicing bypass > > > --- > > > drivers/media/platform/rockchip/rkisp1/rkisp1-params.c | 8 ++++---- > > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-params.c b/drivers/media/platform/rockchip/rkisp1/rkisp1-params.c > > > index 042b759eba62..496381962f1b 100644 > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-params.c > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-params.c > > > @@ -1854,8 +1854,8 @@ rkisp1_ext_params_bdm(struct rkisp1_params *params, > > > const struct rkisp1_ext_params_bdm_config *bdm = &block->bdm; > > > > > > if (bdm->header.flags & RKISP1_EXT_PARAMS_FL_BLOCK_DISABLE) { > > > - rkisp1_param_clear_bits(params, RKISP1_CIF_ISP_DEMOSAIC, > > > - RKISP1_CIF_ISP_DEMOSAIC_BYPASS); > > > + rkisp1_param_set_bits(params, RKISP1_CIF_ISP_DEMOSAIC, > > > + RKISP1_CIF_ISP_DEMOSAIC_BYPASS); > > > return; > > > } > > > > > > @@ -1863,8 +1863,8 @@ rkisp1_ext_params_bdm(struct rkisp1_params *params, > > > > > > if ((bdm->header.flags & RKISP1_EXT_PARAMS_FL_BLOCK_ENABLE) && > > > !(params->enabled_blocks & BIT(bdm->header.type))) > > > - rkisp1_param_set_bits(params, RKISP1_CIF_ISP_DEMOSAIC, > > > - RKISP1_CIF_ISP_DEMOSAIC_BYPASS); > > > + rkisp1_param_clear_bits(params, RKISP1_CIF_ISP_DEMOSAIC, > > > + RKISP1_CIF_ISP_DEMOSAIC_BYPASS); > > > > I think this also opens us up to add mono formats as explicitly > > supported by the ISP and potentially set the demosaic defaulting to off > > in that instance? > > mmm, I think userspace is in a better position to decide when to > bypass debayer instead of relying on auto-configuration of the ISP ? > > I guess we'll discuss this when support for luma-only formats will be > added to the driver While I understand why auto-configuration is tempting, I've found that more often than not it makes the life of both the driver and userspace more difficult. Look for instance at the colourspace handling code in rkisp1. > > Anyway, Looks sane to me in this order. > > > > Reviewed-by: Kieran Bingham Reviewed-by: Laurent Pinchart > thanks > > > > } > > > > > > static void > > > > > > --- > > > base-commit: 27953c044974baf7e24dee3e9342fe0103dea80c > > > change-id: 20260911-imx8mp-demosaicing-bypass-bb284d9cca60 -- Regards, Laurent Pinchart _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip