Received-SPF: pass (google.com: domain of linux-kernel+bounces-72714-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Michael Riesch <michael.riesch@wolfvision.net>
Date: Tue, 20 Feb 2024 10:39:16 +0100
Subject: [PATCH 06/14] media: rockchip: cif: store endpoint information in
 private data
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Message-Id: <20240220-v6-8-topic-rk3568-vicap-v1-6-2680a1fa640b@wolfvision.net>
References: <20240220-v6-8-topic-rk3568-vicap-v1-0-2680a1fa640b@wolfvision.net>
In-Reply-To: <20240220-v6-8-topic-rk3568-vicap-v1-0-2680a1fa640b@wolfvision.net>
To: Mehdi Djait <mehdi.djait.k@gmail.com>, 
 =?utf-8?q?Th=C3=A9o_Lebrun?= <theo.lebrun@bootlin.com>, 
 Thomas Petazzoni <thomas.petazzoni@bootlin.com>, 
 Laurent Pinchart <laurent.pinchart@ideasonboard.com>, 
 Mauro Carvalho Chehab <mchehab@kernel.org>, 
 Rob Herring <robh+dt@kernel.org>, 
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, 
 Conor Dooley <conor+dt@kernel.org>, Heiko Stuebner <heiko@sntech.de>, 
 Sakari Ailus <sakari.ailus@linux.intel.com>
Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org, 
 linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, 
 linux-rockchip@lists.infradead.org, 
 Michael Riesch <michael.riesch@wolfvision.net>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?ZGtPV3ROVHZqWm9zbHNsZEI3L1ltWlJrazJrczJhOHMzWnFKcWVRdXZkdFZn?=
 =?utf-8?B?SFFvaTBHc1hITCtPeStMK2NxOHBMc01RMzY3WWZkS055ajlRdTJ3b000TzdX?=
 =?utf-8?B?eXZzSktnNW1jOXlHZnhCMHppU2ZxT251UnZqZFNibVJkUEJZOXpPT3VTUC9v?=
 =?utf-8?B?UjNsRHVOSFMzQ2ZhNlVVNktUU2VIZ21vZ2JydGxmeUhmQk9lSlJ5elBuUFBH?=
 =?utf-8?B?bmE4S3BJUCtaeWVpczg1Wi9ZWERLR0FRU3pJcjJjL2QyL1BuSXJqZFRxSWJp?=
 =?utf-8?B?YzlVTDdlVzFiYWdwN3pmZk02Y3NBTXpVcWFGS2xLUmlGbU1DZ3FKUm9QK1Qz?=
 =?utf-8?B?V1NuYjlEOTdHMkxYRTZDYXlMSHo3c3pXODFwUk5aclNLYUtlTmtoZUd0T2g2?=
 =?utf-8?B?c2tsRGtIaXlyNjM2VDFOT1JUeWZ2b0poMTNpK2lPaFlycHdIRlhTcVJYNWts?=
 =?utf-8?B?SS94ZzgwWkVvYjlmSFFzU3dQS2NBMGZPTnEreXNpVkdtR213TEhtMlpYa0Ix?=
 =?utf-8?B?eHRkcGNvY08xWWpuYUJMVE52ejBOZTZkYzFweTJvdGV5bWRxNVZZMS80Q2U2?=
 =?utf-8?B?aVhDcExGQnd1SExuenFTZXdmbko1bkplaFp5YW45d05PL1JBWWZwdERJM3Fr?=
 =?utf-8?B?ZzRvK0lnQ0V6eXc2UTdzR2JSNncvZEJMeUJ1Mk9FZFBXV0NJUUc4MHVJTGpv?=
 =?utf-8?B?dzNVZUQxNVJQait6R3krT1o5eFl1Y1B4Q0hmR2s4aEp2MWp5d2diRjllMEFq?=
 =?utf-8?B?Q3ZpWTNLSXlEY1FRVXZNdHhwS09nR0FZaFkyNDkvenk5VmE1QXc4UmJoeC9M?=
 =?utf-8?B?MXRhbitqSkcrV1pnWkNxcGpRT2Q2TkIzMzRvSnVnTnIrSHMvUnh6QitIaEor?=
 =?utf-8?B?QVViV0JsTTdGUG9YSEFTcXdNVXo3YzZnOUd2c2VlanQrTWxlalN1NGQxUGRM?=
 =?utf-8?B?eWJlL1F3UEZxZkFoVTQ2M21kR3FYK05jb0lHSFZOc25CU1VldGszYm9uSk9M?=
 =?utf-8?B?V1RuVWM5NGN4VEJTMzlJUEp5Q2tUeVdqaUhYVW96blA1K2t5bWd6cGZGMXpw?=
 =?utf-8?B?RHJZanlMTS9hdW8zUHV4a0w2SXd1OEVIQU1BTEhrVjVuYWI1WXlXa3B6dUJl?=
 =?utf-8?B?R2xKZVZncW1sYmlEcmJFWkl1TTM4VldYMGVZRlU3ZE5xZ2o1Ym85NldtKzlZ?=
 =?utf-8?B?RDBxVmdhQXRMQWZ6TDVuRDFhRE13UFkrOWMrZEcxSGVaaGpCMFFIRVFPYnND?=
 =?utf-8?B?bmpiR2FZQWtOd0hRdHZka0ZNZmFBUEQ0WjZUS0ZiN3RHSGZQenkwR1VNQWtu?=
 =?utf-8?B?VWdNM285MGEzTTUwSnJiZ0JRaE5kMG4wRXhuVWpnTEtyYnFVaFZzVHZzNkFZ?=
 =?utf-8?B?ZERYc0w4TVd5YUVld1BLWEZxdWo1LzJaaUlZRHR3VW0xbWpiV1B5MHhPOHJS?=
 =?utf-8?B?YzR3anFuZ1lXeWxTSGhaYmVmSndqKzh6RjlnQ2VJcEdwbjJFeEx0cWN0Y08w?=
 =?utf-8?B?ai9YTmUzR2hqNG1hOGxFTVdMM2JPaUltcmlodDdzRHd0SGZYU2NmVHh0enZw?=
 =?utf-8?B?L0RnRFA2aEc3ZHJUM2l1azNsRk1sMk9XRVN6SGR5bXAwOEx0eFNnSDZmZldS?=
 =?utf-8?B?czVqY3pmUE1vMlRXU0hTcEdVNmczLzVOeTdTZ3JpL2VRVkRCVlJtTGFxcHY1?=
 =?utf-8?B?WXlJTWVNdWI0RW1jVlJtSVROVlN5THlReDM4WTIwTnM5di9ndTFVVjhsbDM3?=
 =?utf-8?B?OFdCWTlRSHZ2TnZXelRaVmRhU0hSOCtoOHRxOUh1U3BKR2c5cUVGNFNkWHVz?=
 =?utf-8?B?ZUdyNWhobC9TcUszOFM2ZTgvcThCNldXbWFTZDI3Vnh5bUdjelhucnB0OUVM?=
 =?utf-8?B?SkZOeHFKZjEvbTZMeU1SbXMvR3FITTlzQ0VpYTNqanpqM0FLdGRKMHJEQ1lB?=
 =?utf-8?B?U2duWEttYjMrTmwzT1FDWUFIZWxxL2RrK25ZVFJ6Q2g0NHowZUthM3ZGNGJs?=
 =?utf-8?B?aDQvVDJFS0IwdkR5ZzE3akxxaWZxVWpoby90QjdvcDVKTllRZUNjZnpLSlVG?=
 =?utf-8?B?b1ZXdzlXOTBGSEpSK0ZZbXZXSzBIbFU4OWRLVkw1QUZPcHBScUlybzYrVEMx?=
 =?utf-8?B?d1RndldFMFZ0YzJBNU9GYSt6OWUxM3R6TTBtV0xSMGpYODhBYkFENitKOXBO?=
 =?utf-8?B?TUE9PQ==?=
X-OriginatorOrg: wolfvision.net
X-MS-Exchange-CrossTenant-Network-Message-Id: dbde50d9-4ab0-4f8a-8760-08dc31f7ecef
X-MS-Exchange-CrossTenant-AuthSource: DU0PR08MB9155.eurprd08.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Feb 2024 09:40:08.2511
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: e94ec9da-9183-471e-83b3-51baa8eb804f
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ugH9ifatd4iBYxTgqqOVJ5pN4NNtoDbBYLlR9e4oYswyTWhrOodBuPv+G9umUwm7+oTPDbs49Xm7ZcH4xxsQMbIn2U5X/HZxduViKxWfwbI=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR08MB6763

The endpoint information such as bus width or flags are required
to set CIF (and on certain variants GRF) registers correctly. Store
the information in the private data of the driver in order to make
it accessible.

Signed-off-by: Michael Riesch <michael.riesch@wolfvision.net>
---
 drivers/media/platform/rockchip/cif/cif-common.h |  1 +
 drivers/media/platform/rockchip/cif/cif-dev.c    | 11 +++++------
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/media/platform/rockchip/cif/cif-common.h b/drivers/media/platform/rockchip/cif/cif-common.h
index 89ed9dd6f36d..34da800a0504 100644
--- a/drivers/media/platform/rockchip/cif/cif-common.h
+++ b/drivers/media/platform/rockchip/cif/cif-common.h
@@ -108,6 +108,7 @@ struct cif_device {
 	struct media_device		media_dev;
 	struct v4l2_async_notifier	notifier;
 	struct v4l2_async_connection	asd;
+	struct v4l2_fwnode_endpoint	vep;
 	struct cif_remote		remote;
 
 	struct cif_stream		stream;
diff --git a/drivers/media/platform/rockchip/cif/cif-dev.c b/drivers/media/platform/rockchip/cif/cif-dev.c
index 660e28397916..923831827be4 100644
--- a/drivers/media/platform/rockchip/cif/cif-dev.c
+++ b/drivers/media/platform/rockchip/cif/cif-dev.c
@@ -79,9 +79,7 @@ static int cif_subdev_notifier(struct cif_device *cif_dev)
 	struct v4l2_async_notifier *ntf = &cif_dev->notifier;
 	struct device *dev = cif_dev->dev;
 	struct v4l2_async_connection *asd;
-	struct v4l2_fwnode_endpoint vep = {
-		.bus_type = V4L2_MBUS_UNKNOWN,
-	};
+	struct v4l2_fwnode_endpoint *vep = &cif_dev->vep;
 	struct fwnode_handle *ep;
 	int ret;
 
@@ -92,12 +90,13 @@ static int cif_subdev_notifier(struct cif_device *cif_dev)
 	if (!ep)
 		return -ENODEV;
 
-	ret = v4l2_fwnode_endpoint_parse(ep, &vep);
+	vep->bus_type = V4L2_MBUS_UNKNOWN;
+	ret = v4l2_fwnode_endpoint_parse(ep, vep);
 	if (ret)
 		goto complete;
 
-	if (vep.bus_type != V4L2_MBUS_BT656 &&
-	    vep.bus_type != V4L2_MBUS_PARALLEL) {
+	if (vep->bus_type != V4L2_MBUS_BT656 &&
+	    vep->bus_type != V4L2_MBUS_PARALLEL) {
 		v4l2_err(&cif_dev->v4l2_dev, "unsupported bus type\n");
 		goto complete;
 	}

-- 
2.30.2