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 9AD22CCFA13 for ; Mon, 10 Nov 2025 09:26:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:References:Cc:To:Subject:From:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=w5WWXAjdY6BMcCGSBDzoSTydR63r4/SDUM2HBs1PBvc=; b=S+c+mfryHvxlWY0/A4FU6ouvSq 4PUD2WALqECenflndL1n9SL6eccZ2DEo8yr/OTRDPg1oeZKor3RrdllksxHNKcKTc7GksuPJnQr+x YeSSka/MrdocZMpkkXtyOsZmKfXUD3FnFUCcqp2zfyY5l2alPdzpLxYPn6PD74v32gwxGnUGezxs7 6/roC2INdIMGPj+qUjVMmjT2yun5U3zOjg+MFW0Jrg+ibeqZXJhtFeqPSKbIUpfYNMAZ1l2jpJhw2 LmzHXCNFdIsihunQaT6y7EBxAQ8rCbBusCIkoAGg1ffF2PNXTvkI20Loa8uEun1T3urUfkxsMEoQq ttttS9fQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vIOAj-000000055fS-1bcg; Mon, 10 Nov 2025 09:26:49 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vIOAg-000000055f1-2sQG for linux-arm-kernel@lists.infradead.org; Mon, 10 Nov 2025 09:26:48 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id CBF7441A9B; Mon, 10 Nov 2025 09:26:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7F06EC116B1; Mon, 10 Nov 2025 09:26:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1762766805; bh=b0IFXuV/9PpQKpNg/c6/i5YuNUFwarm8QI9JGvletFs=; h=Date:From:Subject:To:Cc:References:In-Reply-To:From; b=XDMsKLsK+bVR9DeEuIXtyzRVuyhrICIVTSQIMVL07RtxAGM2Y8MnHEoMVJAQ8piaC ZgoHaLn52+YX/eIS4IDhqSxZJ0jcoiNuujVU08+zY9L3oG0mwVvRIF/h/oAeLIcXa7 Oy5eQzv2EzYDCUYESJO4t0Fo8tJyNDMFfBnEl5wjYQZd+uo4uM5KSEAZbGWkECnCvR RQ7Q/5StUrhU7qDDazxIa/MV03yTX9WupBtWECFn+BpV09QaaBNXL6mB6W5jyTkQhM azn9iNXFi4uJbYEVeLrU+e34VFMFcGwS0rBF5LG9LyMg9JFTD/ft899JzkgQqKVqvp Zc47RZDbgZ2tg== Message-ID: <55a8a7f1-013e-4528-b4dd-b26ee24dbb6b@kernel.org> Date: Mon, 10 Nov 2025 10:26:40 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird From: Hans Verkuil Subject: Re: [PATCH 00/18] media: microchip-isc: Color correction and histogram stats To: Balamanikandan Gunasundar , Mauro Carvalho Chehab Cc: Eugen Hristev , Chas Williams <3chas3@gmail.com>, Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Balakrishnan Sambath , Hans Verkuil , Ricardo Ribalda , Laurent Pinchart , Jacopo Mondi , Daniel Scally , Tomi Valkeinen , linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-atm-general@lists.sourceforge.net, netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org References: <20251009155251.102472-1-balamanikandan.gunasundar@microchip.com> Content-Language: en-US, nl In-Reply-To: <20251009155251.102472-1-balamanikandan.gunasundar@microchip.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251110_012646_799825_48EF10F7 X-CRM114-Status: GOOD ( 18.02 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 09/10/2025 17:52, Balamanikandan Gunasundar wrote: > Hi, > > This patch series has a set of enhancements to the Microchip Image Sensor > Controller driver. The objective is to expand its image processing > capabilities and to improve the colors. > > This series also introduces a new stats driver that exposes the histogram > data to userspace via v4l2 controls. This allows applications such as > libcamera to access real time image statistics for advanced image > processing like automatic exposure, white balance adjustments etc. Note that bisect of this series failed: some patches seem to have dependencies on later patches, which is obviously wrong. Make sure you double check that for v2. The documentation also fails to build since there is no documentation for the new meta format, but I already commented on that. Regards, Hans > > Balakrishnan Sambath (11): > media: microchip-isc: Enable GDC and CBC module flags for RGB formats > media: microchip-isc: Improve histogram calculation with outlier > rejection > media: microchip-isc: Use channel averages for Grey World AWB > media: microchip-isc: Add range based black level correction > media: platform: microchip: Extend gamma table and control range > media: platform: microchip: Add new histogram submodule > media: microchip-isc: Register and unregister statistics device > media: microchip-isc: Always enable histogram for all RAW formats > media: microchip-isc: fix histogram state initialization order > media: microchip-isc: decouple histogram cycling from AWB mode > media: microchip-isc: enable userspace histogram statistics export > > Balamanikandan Gunasundar (7): > media: platform: microchip: set maximum resolution for sam9x7 > media: platform: microchip: Include DPC modules flags in pipeline > media: microchip-isc: expose hue and saturation as v4l2 controls > media: microchip-isc: Rename CBC to CBHS > media: microchip-isc: Store histogram data of all channels > media: videodev2.h, v4l2-ioctl: Add microchip statistics format > media: microchip-isc: expose color correction registers as v4l2 > controls > > drivers/media/platform/microchip/Kconfig | 2 + > drivers/media/platform/microchip/Makefile | 2 +- > .../platform/microchip/microchip-isc-base.c | 373 ++++++++++-- > .../platform/microchip/microchip-isc-regs.h | 3 + > .../platform/microchip/microchip-isc-stats.c | 549 ++++++++++++++++++ > .../media/platform/microchip/microchip-isc.h | 44 +- > .../microchip/microchip-sama5d2-isc.c | 2 +- > .../microchip/microchip-sama7g5-isc.c | 73 ++- > drivers/media/v4l2-core/v4l2-ioctl.c | 1 + > include/linux/atmel-isc-media.h | 13 + > include/uapi/linux/videodev2.h | 3 + > 11 files changed, 1001 insertions(+), 64 deletions(-) > create mode 100644 drivers/media/platform/microchip/microchip-isc-stats.c >