From patchwork Tue Jan 17 10:06:00 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 13104444 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id D6317C3DA78 for ; Tue, 17 Jan 2023 10:06:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236321AbjAQKGU (ORCPT ); Tue, 17 Jan 2023 05:06:20 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53172 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236148AbjAQKGR (ORCPT ); Tue, 17 Jan 2023 05:06:17 -0500 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A73CB3C1D for ; Tue, 17 Jan 2023 02:06:16 -0800 (PST) Received: from uno.LocalDomain (93-61-96-190.ip145.fastwebnet.it [93.61.96.190]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 239FF10C; Tue, 17 Jan 2023 11:06:14 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1673949974; bh=pb3wRTv3sp84GATXwl39TeWyAD9ucGQdE+Af62iP3VA=; h=From:To:Cc:Subject:Date:From; b=BUF8V8l0uBSYvYqEkaqQ07cyLzS5AcnufWqmSgZi9PtzvGVfHp9qJ+OteebDb2U9w s1G+N6KqOrw56hbA42s7b+rYKbVOLsOcaTaKdPmklto4OwHVSHkzqRT5iw8R1iVLYU pdNTJcpxlMy/nq8O8x8fuoFcmdxNpYzbRQXfpbog= From: Jacopo Mondi To: Robert Mader , Sakari Ailus , Dave Stevenson , Laurent Pinchart Cc: Jacopo Mondi , linux-media@vger.kernel.org Subject: [PATCH 0/3] media: imx258: Remove rotation=<80 requirement Date: Tue, 17 Jan 2023 11:06:00 +0100 Message-Id: <20230117100603.51631-1-jacopo.mondi@ideasonboard.com> X-Mailer: git-send-email 2.39.0 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Currently the imx258 driver requires to have the 'rotation' device node property specified in DTS with a fixed value of 180 degrees. The "rotation" fwnode device property is intended to allow specify the sensor's physical mounting rotation, so that it can be exposed through the read-only V4L2_CID_CAMERA_SENSOR_ROTATION control and applications can decide how to compensate for that. The imx258 driver has read-only VFLIP and HFLIP enabled, resulting in a 180 degrees image rotation being produced by the sensor. But this doesn't imply that the physical mounting rotation should match the driver's implementation. I took into the series Robert's patch that register device node properties and on top of that register flips controls, in order to remove the hard requirement of the 180 degrees rotation property presence. Jacopo Mondi (2): media: imx258: Register H/V flip controls media: imx258: Remove mandatory 180 degrees rotation Robert Mader (1): media: imx258: Parse and register properties drivers/media/i2c/imx258.c | 33 ++++++++++++++++++++++++--------- 1 file changed, 24 insertions(+), 9 deletions(-) --- 2.39.0