Message ID | 20221104142452.117135-1-jacopo@jmondi.org (mailing list archive) |
---|---|
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1oqxgL-00BRLe-BK; Fri, 04 Nov 2022 14:28:29 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232111AbiKDO2Z (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Fri, 4 Nov 2022 10:28:25 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42410 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232411AbiKDO12 (ORCPT <rfc822;linux-media@vger.kernel.org>); Fri, 4 Nov 2022 10:27:28 -0400 Received: from relay11.mail.gandi.net (relay11.mail.gandi.net [IPv6:2001:4b98:dc4:8::231]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 79A3E317E1 for <linux-media@vger.kernel.org>; Fri, 4 Nov 2022 07:25:10 -0700 (PDT) Received: (Authenticated sender: jacopo@jmondi.org) by mail.gandi.net (Postfix) with ESMTPSA id 2BCC5100013; Fri, 4 Nov 2022 14:25:06 +0000 (UTC) From: Jacopo Mondi <jacopo@jmondi.org> To: =?utf-8?q?Krzysztof_Ha=C5=82asa?= <khalasa@piap.pl>, Mauro Carvalho Chehab <mchehab@kernel.org>, Sakari Ailus <sakari.ailus@iki.fi>, Laurent Pinchart <laurent.pinchart@ideasonboard.com> Cc: Jacopo Mondi <jacopo@jmondi.org>, linux-media@vger.kernel.org, Dave Stevenson <dave.stevenson@raspberrypi.com> Subject: [PATCH v3 00/10] media: ar0521: Add analog gain, rework clock tree Date: Fri, 4 Nov 2022 15:24:43 +0100 Message-Id: <20221104142452.117135-1-jacopo@jmondi.org> X-Mailer: git-send-email 2.38.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_LOW, SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.4 (--) X-LSpam-Report: No, score=-2.4 required=5.0 tests=BAYES_00=-1.9,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1 autolearn=ham autolearn_force=no |
Series |
media: ar0521: Add analog gain, rework clock tree
|
|
Message
Jacopo Mondi
Nov. 4, 2022, 2:24 p.m. UTC
Hello, this series adds a few features to the ar0521 driver to enable its usage with libcamera. In particular: - enum_frame_sizes - global analog gain control - LINK_FREQ - Rework blanking handligs v3 (Dave) - Check __v4l2_ctrl_modify_range() return value - Make LINK_FREQ readonly to avoid additional churn in s_ctrl - Fix trivial early return in s_ctrl - Use exposure's default value when modifying the controls' limits - Change the exposure default to 0x70 to match the register default value v2: - I have dropped the most controverse part that allows to change the link frequency to obtain 60FPS. It can be eventually be applied on top. - Use register 0x3028 to control analog gain not to overwrite the global digital gain. - Fix the HBLANK/VBLANK max by using the values read from on-chip registers. - Fix handling of LINK_FREQ in s_cltr (but do not make the control read only). - Fix errors reported by 0-days: - use do_div() for 64-bit division - declare variables in function scope and not in case scope in s_ctrl Jacopo Mondi (10): media: ar0521: Implement enum_frame_sizes media: ar0521: Add V4L2_CID_ANALOG_GAIN media: ar0521: Set maximum resolution to 2592x1944 media: ar0521: Rework PLL computation media: ar0521: Refuse unsupported controls media: ar0521: Add LINK_FREQ control media: ar0521: Adjust exposure and blankings limits media: ar0521: Setup controls at s_stream time media: ar0521: Rework startup sequence media: ar0521: Tab-align definitions drivers/media/i2c/ar0521.c | 352 ++++++++++++++++++++++++++----------- 1 file changed, 250 insertions(+), 102 deletions(-) -- 2.38.1
Comments
I just noticed patch 10/10 was missing. Now sent in reply to this thread. While at it, gentle ping to see if this can make it for 6.2 ? Thanks j On Fri, Nov 04, 2022 at 03:24:43PM +0100, Jacopo Mondi wrote: > Hello, > this series adds a few features to the ar0521 driver to enable its usage > with libcamera. > > In particular: > - enum_frame_sizes > - global analog gain control > - LINK_FREQ > - Rework blanking handligs > > v3 (Dave) > - Check __v4l2_ctrl_modify_range() return value > - Make LINK_FREQ readonly to avoid additional churn in s_ctrl > - Fix trivial early return in s_ctrl > - Use exposure's default value when modifying the controls' limits > - Change the exposure default to 0x70 to match the register default value > > v2: > - I have dropped the most controverse part that allows to change the link > frequency to obtain 60FPS. It can be eventually be applied on top. > - Use register 0x3028 to control analog gain not to overwrite the global digital > gain. > - Fix the HBLANK/VBLANK max by using the values read from on-chip registers. > - Fix handling of LINK_FREQ in s_cltr (but do not make the control read only). > - Fix errors reported by 0-days: > - use do_div() for 64-bit division > - declare variables in function scope and not in case scope in s_ctrl > > Jacopo Mondi (10): > media: ar0521: Implement enum_frame_sizes > media: ar0521: Add V4L2_CID_ANALOG_GAIN > media: ar0521: Set maximum resolution to 2592x1944 > media: ar0521: Rework PLL computation > media: ar0521: Refuse unsupported controls > media: ar0521: Add LINK_FREQ control > media: ar0521: Adjust exposure and blankings limits > media: ar0521: Setup controls at s_stream time > media: ar0521: Rework startup sequence > media: ar0521: Tab-align definitions > > drivers/media/i2c/ar0521.c | 352 ++++++++++++++++++++++++++----------- > 1 file changed, 250 insertions(+), 102 deletions(-) > > -- > 2.38.1 >
Hi Jacopo, On Mon, Nov 21, 2022 at 06:49:33PM +0100, Jacopo Mondi wrote: > I just noticed patch 10/10 was missing. > > Now sent in reply to this thread. > > While at it, gentle ping to see if this can make it for 6.2 ? I'm drowning in reviews, so you'll need another volunteer. > On Fri, Nov 04, 2022 at 03:24:43PM +0100, Jacopo Mondi wrote: > > Hello, > > this series adds a few features to the ar0521 driver to enable its usage > > with libcamera. > > > > In particular: > > - enum_frame_sizes > > - global analog gain control > > - LINK_FREQ > > - Rework blanking handligs > > > > v3 (Dave) > > - Check __v4l2_ctrl_modify_range() return value > > - Make LINK_FREQ readonly to avoid additional churn in s_ctrl > > - Fix trivial early return in s_ctrl > > - Use exposure's default value when modifying the controls' limits > > - Change the exposure default to 0x70 to match the register default value > > > > v2: > > - I have dropped the most controverse part that allows to change the link > > frequency to obtain 60FPS. It can be eventually be applied on top. > > - Use register 0x3028 to control analog gain not to overwrite the global digital > > gain. > > - Fix the HBLANK/VBLANK max by using the values read from on-chip registers. > > - Fix handling of LINK_FREQ in s_cltr (but do not make the control read only). > > - Fix errors reported by 0-days: > > - use do_div() for 64-bit division > > - declare variables in function scope and not in case scope in s_ctrl > > > > Jacopo Mondi (10): > > media: ar0521: Implement enum_frame_sizes > > media: ar0521: Add V4L2_CID_ANALOG_GAIN > > media: ar0521: Set maximum resolution to 2592x1944 > > media: ar0521: Rework PLL computation > > media: ar0521: Refuse unsupported controls > > media: ar0521: Add LINK_FREQ control > > media: ar0521: Adjust exposure and blankings limits > > media: ar0521: Setup controls at s_stream time > > media: ar0521: Rework startup sequence > > media: ar0521: Tab-align definitions > > > > drivers/media/i2c/ar0521.c | 352 ++++++++++++++++++++++++++----------- > > 1 file changed, 250 insertions(+), 102 deletions(-) > >
Hi Laurent, Jacopo, On Tue, Nov 22, 2022 at 01:38:41AM +0200, Laurent Pinchart wrote: > Hi Jacopo, > > On Mon, Nov 21, 2022 at 06:49:33PM +0100, Jacopo Mondi wrote: > > I just noticed patch 10/10 was missing. > > > > Now sent in reply to this thread. > > > > While at it, gentle ping to see if this can make it for 6.2 ? > > I'm drowning in reviews, so you'll need another volunteer. I rather thought the set is good as there are no comments for almost three weeks. It's in my tree now, I expect to send a PR soon...
Hi Jacopo and Co., Jacopo Mondi <jacopo@jmondi.org> writes: > I just noticed patch 10/10 was missing. > > Now sent in reply to this thread. > > While at it, gentle ping to see if this can make it for 6.2 ? I understand this has been tested on actual hw. I'd like to check how does it work in my setup as well, but it is currently out of question. This will eventually happen, though. For now I guess I can Acked-by: Krzysztof Hałasa <khalasa@piap.pl>