From nobody Sun May 24 21:39:23 2026 Received: from DB3PR0202CU003.outbound.protection.outlook.com (mail-northeuropeazon11010068.outbound.protection.outlook.com [52.101.84.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6A327346FA7; Thu, 21 May 2026 09:07:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.84.68 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779354460; cv=fail; b=OYp9dYvgjJDQAZDkKvVcqnpO6fZKfb3SsrFlsPRFXz44HwzP5UCGsdVfS6EUCLlMm0gUd1HYbkbT6oZq6t+N67gs/NFm0AclcG87A7+mI9a5QBEb4LF5QBaxWA095d7y6IYfIb7jP+B5arToGCGdQ++H3VXkIGAf/3oL6eZ7gDg= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779354460; c=relaxed/simple; bh=EkGGKSNyw7AnehXSs5Xl89VuIdDd9orbG87qyje5T5o=; h=From:Date:Subject:Content-Type:Message-Id:References:In-Reply-To: To:Cc:MIME-Version; b=IByJijNMzdX1BF+f22b+LASmxTH2RLitaZ2ezrT9J5gKFSp3B7riWZrIsxmavDX3wn+qnMfAY1bV1jTNaNmvYNKHVDQVV0UbfzaK2Ynxpb1canNwUDuZfaj2Z7O2xNDoh+yNLg2H8hBF3+wwZn5LmrUnF7LXl4C2qlnwudqvY7g= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com; spf=pass smtp.mailfrom=oss.nxp.com; dkim=pass (2048-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b=ByhuevAG; arc=fail smtp.client-ip=52.101.84.68 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b="ByhuevAG" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=oc4kgkkipaGs2CmNnmFgGqvO1xnntchDKfhiiWQkxnod1WKe5ancg6JlXQH/g1tZA9/gvvtREkY7ILSNiGZwBrRTw2CJgXs1X63Bhel5pVIVjjRWLxDSnrpGsCS9DzDXcZc0s8ea7ePBoBRBExj9qk6oaV9iqgnlnhG5T8bwsxMvg0N25OHN6K55geR/xKp7M0vhpxeMfHfaw5y+ouPCDBxd0pIuMz0a4zsnxnjR75qh/EzCFKPny3LpO9FDUu7tGZPifvnPDrA9QM7IPMWDs/Ro7Boq7UKw/s/WURRnMoOmxxTNR2i+JbuOatyK4o1qyFSwJIFRIk1BsQ6gCp4U9A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=wE0+e4pHjJ4pVj5BWvxUiksQFKOCayHXihzU8rfTBIA=; b=UyoxPdxmXBwL1uNK+E8ugN/T3nLk9TPaU0uM+sDxgrp+XGqz3Iag8I61OGAaLpTd5f6XDOfI1fsnuU3PKO/A57eidHEFbn7VVlVazkpPl6Iqc2OPTZRKw+u7FxurqIpg641tCD48hN684OBV/9P/YzFu7IA9wKb+fsJft9SfAeVAcBbfAbMCF2C97TQ6HyYWI8XgLP47u7sIg+KNbcf+n0IJUHCmHTcPXGHOqHU2nMAX+5XibpGwxcPfY5y8knHKpIgC4hfYJsnz7EQ2Y8dX4Z/cdmDLo69/53r4IyidlghFEBqMpHq91EM96uWQR3ORai6DHwatDI785Z/dz4hovA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oss.nxp.com; dmarc=pass action=none header.from=oss.nxp.com; dkim=pass header.d=oss.nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NXP1.onmicrosoft.com; s=selector1-NXP1-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=wE0+e4pHjJ4pVj5BWvxUiksQFKOCayHXihzU8rfTBIA=; b=ByhuevAG0mPMwCrDQkQ9mbWsM/X3Yyy4IzYpyeHASJa28VcxvNmILvEi4xPQfoN7VQnfq1vWkdyAoLQWxqj1J2tojl3vxomY4+ZaYLch8zRLq3jLHcmNSwcIEFJFVHVl+5VXbohT3GboMWchdsVBPgoSinyIvjYg2/HV3id30RKkn65NTKtApMG/wNaLrUKV8qGxOEF7qcjoA/wUmaDUr/NfxoSGtT3ReFTa6M7SVx8WXRbA+pJLe4EPojmoOzpR1g50oFNzg7rjNzvmB+8oGK7gn9gleOvUCTRZoQUkiZmCJb7IH5hS1MY1LZIh4G+yrgMZXwpOILO+Fld+hnIs8g== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oss.nxp.com; Received: from AS8PR04MB9080.eurprd04.prod.outlook.com (2603:10a6:20b:447::16) by AS5PR04MB11467.eurprd04.prod.outlook.com (2603:10a6:20b:6c3::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.48.17; Thu, 21 May 2026 09:07:33 +0000 Received: from AS8PR04MB9080.eurprd04.prod.outlook.com ([fe80::92c2:2e03:bf99:68eb]) by AS8PR04MB9080.eurprd04.prod.outlook.com ([fe80::92c2:2e03:bf99:68eb%6]) with mapi id 15.21.0048.016; Thu, 21 May 2026 09:07:33 +0000 From: Guoniu Zhou Date: Thu, 21 May 2026 17:10:04 +0800 Subject: [PATCH v5 1/2] media: imx8-isi: crossbar: Add get_frame_desc operation Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260521-isi_vc-v5-1-a38eb4fcd58e@oss.nxp.com> References: <20260521-isi_vc-v5-0-a38eb4fcd58e@oss.nxp.com> In-Reply-To: <20260521-isi_vc-v5-0-a38eb4fcd58e@oss.nxp.com> To: Laurent Pinchart , Mauro Carvalho Chehab , Frank Li , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam Cc: Aisheng Dong , linux-media@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Guoniu Zhou X-Mailer: b4 0.14.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1779354625; l=1554; i=guoniu.zhou@oss.nxp.com; s=20250815; h=from:subject:message-id; bh=XQauEoHr2n0XOpfIAH225oPyqKAumXox6VGii8oO6Qs=; b=ZRbYrgCzv829ATDpUY1KvclTxgGhv2RXapEtc9YmIDNTFuuV/sZdTsfv/6QFoy4tta8PUIO9O rti+F/xlE+rC6KMQv6+0rLlzDxQqj52xu3K6hsNZV+UqfyB+42WBglD X-Developer-Key: i=guoniu.zhou@oss.nxp.com; a=ed25519; pk=MM+/XICg5S78/gs+f9wtGP6yIvkyjTdZwfaxXeu5rlo= X-ClientProxiedBy: SI1PR02CA0032.apcprd02.prod.outlook.com (2603:1096:4:1f6::12) To AS8PR04MB9080.eurprd04.prod.outlook.com (2603:10a6:20b:447::16) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS8PR04MB9080:EE_|AS5PR04MB11467:EE_ X-MS-Office365-Filtering-Correlation-Id: 23f70f75-bbca-4584-3499-08deb718649e X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True X-LD-Processed: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|376014|52116014|1800799024|19092799006|18002099003|22082099003|56012099003|38350700014|11063799006|6133799003; X-Microsoft-Antispam-Message-Info: g1vtot14gyEPAOi8TV8cA8juG5ZE9k701dELD8S8QWJNmSbqSrkfw11QI5bTcznz7EGZ6UZw9pJvn+3ZINOafH1KvdeyvB0hwcNmckN6r3E7Cuu6aJfVvIelUG+FBItBsNCJmtnM88U1uyc2pW1OgktfDZSg/EaySH9VZ/zeTTTbdEHeMOxfHHMYIZjBaxvuTmuLCegVeIbZEguCbQBcSTwhuEWTwhfJZHVJwnAJDrCF59RhAAClu2gmHVIM5/MLan+T2r/O6D/jC9AGnaaN5O4Tw/wbvEzU3S/uV0owBJmgD29dwwdN4t/y8id4VIEFc6niXcAKw+AYHN6U5zT7VJ2LnkgxGVIiClZt2pROTSPHzYGxj6uRoIkk1xx3jtDte3n4c/qvHmbl3PpnVHxOudN3nmIydLsmQGxhYMsvk/gvn2y9dDLOeA3mF2cqzjh16kamzGUxuQ400wvcRfW8ScONL/VI8hs8hrq/BdcKWQNxk49plv/Nb7qHtn/VKPTGdbM5VWUIQCl924Uscfp57LAx7ZOvSQYMqTqvev3tbFRJ2uixoI+VOSqiZwfr6d2ygk3VmBkp/bDND6khkECwVfHCd57ICoTs/OGC0weuFPHF87T7+8xmoWS1+Jdj4VOWtzkOf951pGuBiiXvTud2hHiEBbxsIzP1eOSPvvFVGUYaGM1gFXZejZNroAd8MDDrzhA/AcJXVMZolMknslpnK/V2ahUEQcX0QtrQBadFbq8XeR6fSEL75DO/gCe+zadR X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AS8PR04MB9080.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(376014)(52116014)(1800799024)(19092799006)(18002099003)(22082099003)(56012099003)(38350700014)(11063799006)(6133799003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?VEFOS3FyRTZucTkwVUhvcXpGK0JHV1pRemVMRnVnajc4d1BqNG0yQmk0c056?= =?utf-8?B?VE9meGxWZjNaRVh6aVBxYnJxd0dzQlFTVEEvWTdVanR2V2RSeFJWaGVtVktH?= =?utf-8?B?TUZZMzZVRkVFa0kxSHpjRTNlSUtQREUvUFl6SnI0cTR5V0puZWhuYXJvME5k?= =?utf-8?B?Q0FIMFk4dWVERUFZQ2F0VHU4Sk1sYlI2cHRscHlKalptVCtZVXV1bUFFaUJy?= =?utf-8?B?WHJEN000VkF5QnY1QUsyUHNVV2NDTHBsMWcwVHBTVkdGb0owNkVjYkZwdkov?= =?utf-8?B?Y1lxQ1ozNEhVK0dDRzBnWHF3RUVYOXd2MndFbGJzTjkyZ3JQUk5VRitUMTBE?= =?utf-8?B?UlBHaU9LdVVRTHJ0SHBxby8xekNhclBHZytQNWI4MDNyM3V4RkJSZnJpOEVz?= =?utf-8?B?UUtidk4rbWhMeklLcWdTdWZwajYwZ1BWV1pBVkZ6T01SajNUUm5IWWRPRlEr?= =?utf-8?B?YkNsQUJxbkNIVzkrQWNVUHdneHpPL3ZaM3FyOG5RYUlFWHB3YUZoWCsrbXNT?= =?utf-8?B?bVdzMkJQRHBUZkRSYzA1MFU1dTRjM3luRXlEWjBYaEp2M3hqRnRBTi9tckdZ?= =?utf-8?B?VUVBdmlsMVh4YjdyT2dwcFJBZ003VUxGRzNwVVgxejArK3pHRGZieHFNc3lB?= =?utf-8?B?WHBtMVhoZXBhVjNJUzNnQmlxRGZocVZCUWswZFBnZ0M0TVB6RkNxRHFXb0VG?= =?utf-8?B?ZEJndUdHZmhuZzZGdGdtTThLWklnOExQSGRxM1N6OThzZi9BbGhMUkFSaUxC?= =?utf-8?B?cnB6Y3JCNVdCODdBT2svQ1IvQXNRUUZVSzJyOWtiTEhrN2dydFdSWDJMeXd5?= =?utf-8?B?bEJkZmkzbnhuQTdPVzErMGNRVmdNMUJ3elo3a2VXb2MyOENOME45cXlWRzJP?= =?utf-8?B?c3IveTg4c3pQVkF6OTk2dWhBTkZxL2dGSnJwc1IwR05pSXN0V2dHVUZVZWd6?= =?utf-8?B?bUwyWDZ6ZTVZZGsvNXZLclNEb0ZqZnBvZG9pZGxBcHlrNTkyVG5DWVg0Vmpw?= =?utf-8?B?MmppRkIrUW5Qb2Jua3F6cjN4L2tXdVFrUG9TM09LMWpDZVl4eGJpRXR1WkVq?= =?utf-8?B?VDBlT0VHcGRIMEtxcVptME9Db3pZeUhINlRMR2JaSG51a1Q4VjFIOXVQSjQz?= =?utf-8?B?OGtGYlNWRHJMcEJSSUI4ZFpLUUxNNFcyL2pKYit5MmJQa3VMUTh1d0NuQUNO?= =?utf-8?B?Rld5S1VZMjVnNll3ZE5lZVhrMW53SC9uVFM3ZVdaSVNjNDZrOFpJNDZRUlN6?= =?utf-8?B?a0NDRnVGZXFLMnZsQm8vWUEzNEhmSzhoVVZBQWtPQ1N6RlBLblMvZGhrTEtY?= =?utf-8?B?aWg2aTYrTEUvZHBxN0JRWFVKSFhlNzlJMjhWUkwxbldvQTRCV1FyVjk4ZXNP?= =?utf-8?B?aitzSGoxcUxPekxzSnZBUU8xaHFiMkdsSisrK2lDeWJ3bXJtZUFCaDNiRFBh?= =?utf-8?B?eHh0SmFMNCtiYTVqYTlhTjVadnR5ZS9xZWcrNnhpS09Tckwxb3RQUjJlSG5Z?= =?utf-8?B?OFl6SVlMaW50c0V1ZmJmajY5cHlXY2dZS3ZnNk9hOXZhbnh4eGNaNStVQ0x4?= =?utf-8?B?T1IyWTZqK2x1NGJVbGxnWmNGY0VMS05BMGJBWXM5VmNwUDIwNVF5UGNDZEpz?= =?utf-8?B?dUtOUUZKdjcyRjJ2QTNkK3NWWTNjY3Vyd0NGN0NDSVQ5eWora0NCaGhSS3pX?= =?utf-8?B?ckp0UFBJb3VyK3I1d2dCUG9DWjFmaDlib1J3UGU4UW1JTVRxSGVnUmRyc29F?= =?utf-8?B?L1d0MUZQVzk2MGJwaEhtdTVmT0hkTHB4VjlLZEw0cDN2eDdkZ0lHdW5UNGZ5?= =?utf-8?B?R3RlcVZ0cUxBRWN6QVZ5bGVUZUpsdXQ2K1RNaWwvZ1JrbTlkZmNPb09DN3lI?= =?utf-8?B?eVhmZmNDbVF2ZGRNbkdLTGM2cXhCZ2ZLVmh2TEJDUjRxcXZadnBqeFNJZDcx?= =?utf-8?B?UmlmcUh5WmNJdk1SZzJMSlcxTUlZdkQrNEZqcitDeCtmQXIzSUJTMmRSTFRW?= =?utf-8?B?RXIwUTBkS09hcm9vc2JLT3hmRHZEcmFOKzFudHBqU3ZXNjVjQVkvS3VoL3VG?= =?utf-8?B?TGlWK0JMYk1mY3VtY0ZjazB5cXVhMldpVjVCS1BETStMRzA3Vmx5YWNRbk1F?= =?utf-8?B?SjZiV1lXSEF2VG4wUm1sYlE1K2laWnpvMUkrUnNwYVgzUU1aSmIvQ1Z5dGdX?= =?utf-8?B?U25GQndkaXdxdHhWeDNmanIyUTEydVRHcUdnU3J1MDRFdUZMMlRyVWpEa0l3?= =?utf-8?B?eHJhT1hWQTB0bDdUdFFiQndwYVZxVy9GUzZSdW9NSmo1N2M4dkJxQkZqemdz?= =?utf-8?B?clBzV0ZOZlBGZmVGK3RoUU9YTFp3am1ZNlVXL2pWcnhpdzNNaFpvQT09?= X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 23f70f75-bbca-4584-3499-08deb718649e X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB9080.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 May 2026 09:07:33.1172 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: GT8Yvyf3nlsYi0AjK88yLFdsQS+i5VhqnMZ4yM5fwj+2qIMmKiUWkb/jHEJSkHbp5lTqqXBWGnAU/LFYlecOiQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS5PR04MB11467 From: "Guoniu.zhou" Implement the get_frame_desc pad operation for the crossbar subdev using the v4l2_subdev_get_frame_desc_passthrough() helper. This allows the crossbar to properly propagate frame descriptors from its sink pads to its source pads, which is necessary for proper stream configuration in multiplexed streams scenarios. Signed-off-by: Guoniu.zhou --- Changes in v5: - Use v4l2_subdev_get_frame_desc_passthrough helper - Rewrote commit message Changes in v4: - Use %d instead of %u for ret variable in error messages - Fix potential -ENOIOCTLCMD leak by resetting ret to 0 on continue Changes in v3: - New patch added based on feedback from Laurent Pinchart --- drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c b/driv= ers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c index 605a45124103..0b593aed618b 100644 --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c @@ -404,6 +404,7 @@ static const struct v4l2_subdev_pad_ops mxc_isi_crossba= r_subdev_pad_ops =3D { .enum_mbus_code =3D mxc_isi_crossbar_enum_mbus_code, .get_fmt =3D v4l2_subdev_get_fmt, .set_fmt =3D mxc_isi_crossbar_set_fmt, + .get_frame_desc =3D v4l2_subdev_get_frame_desc_passthrough, .set_routing =3D mxc_isi_crossbar_set_routing, .enable_streams =3D mxc_isi_crossbar_enable_streams, .disable_streams =3D mxc_isi_crossbar_disable_streams, --=20 2.34.1 From nobody Sun May 24 21:39:23 2026 Received: from PA4PR04CU001.outbound.protection.outlook.com (mail-francecentralazon11013054.outbound.protection.outlook.com [40.107.162.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0348E374758; Thu, 21 May 2026 09:07:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.162.54 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779354463; cv=fail; b=bEf8qrvVcw/Bhu5SIa/LTD6JXVTyN5yFHZ2i9e1VfPBgFgDfXMBUjuHkR2gqLctUVNbc2EPceS+sT22r5d4ot5j/2ZujbVoFK5nzFDU+FVagMSS/zwy/neVy6sffU7g2N2CBNwMyDzqq9bbdc8VKXiB/ZDj6EalVtpAlqLUKGKg= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779354463; c=relaxed/simple; bh=AwcrFzcWudNZburQvtWYCWJyRTyvAezqremiN31u2+E=; h=From:Date:Subject:Content-Type:Message-Id:References:In-Reply-To: To:Cc:MIME-Version; b=lmzeoVwQ9b3zVbpjA5Z5GisNpeI5XyD+/VgM6SPh12Le07D+U7v25T6FZNLzH/oq0Mze0J3mPJCgnqBDcOBnDutyY1oIO/IkjuhSe2nvSzM8OOF2YmVbIch4cnPwab8Ry3FpcwPM9tJdqAdqVHvlSI3Z2sPPCfa0n4UZV4j6SEo= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com; spf=pass smtp.mailfrom=oss.nxp.com; dkim=pass (2048-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b=AtUu322U; arc=fail smtp.client-ip=40.107.162.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b="AtUu322U" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=jiZnaRe4hKlUSu0mUNhVMaHC2LHBPAuuPQMgpY1g+0L0QVrOuCaXf6wXd7QQ52uxf0C2gXFPHdX0I2H1uXRs/O2aIeZ2gFRfWd6IL1swRzUP5mDJksvwwUU3e3DXEP2wXrE0ZWFTsajS6PIBPM+hYkf5w2U5sejhfZuDw8lXtL7Q0jKH/PlSiZyPLIsYOhu1pVoNGxCtrhb6YmHwq5UbRoSkChHYZLIaqW1elMY7NhZKmlzcSzsSmVFefgO9vP9iW9lMSjNxPSWmVc2H0qSvb+WndG1l2FjG4R5l6PwqW/j+MAJras2xlVvGuqLcFC0Ev76uU04tFDJ3BzdGJMfWqw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=MeeAPEbPZ0YkC+eMb+Mwxfw43DMsqEAYdMcvZN7lSb8=; b=nYmqnYPGp+Wn/VGwTT6T4rrg87ixOQlEq7SNF5BirDcvXZ5iM2b/19Igwn2XV5X8gzLpjjgfPbSbCl29MKzTm+n/SlkyQi6ToilCnMi4NITN5Rrt1RT9mkHpHJIOWQ6AXxSLBtdkCaU4Rl5wOK1Kng0wYvk7WXNcPBWH43LhRAC0QoZxLRrV/IRNATqLJZOVpm2EYgdhl2ThHfGDCcWvcijkifk/g0Ys4+54IJWiID2JRRWZHmDzDMFkd88SW0SU6Nl7ocA0I0+7u+CpGGu5LdPstQfU+BYayh/050TlvFH6MVNNEHlPaUCeZ01Rk3l6l3Xs7Kg1Jt8jmBRarFankA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oss.nxp.com; dmarc=pass action=none header.from=oss.nxp.com; dkim=pass header.d=oss.nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NXP1.onmicrosoft.com; s=selector1-NXP1-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=MeeAPEbPZ0YkC+eMb+Mwxfw43DMsqEAYdMcvZN7lSb8=; b=AtUu322UUKLGZQktqVtQ82tnm7odlVh6VbOjFKXR55opVqBD2KBU4D0OdYhWFrVdCz3VPY2Qh1mw93vezi07UhDKaXzluAAfS2Xxe4pTCO3BTp/O22m7pWOaxPY3FB7Rqv1wSexe/LvOpYZgTqKe3es4jkgc/FQlpGWhRwD1XtyLuP3+mgzf9q7Se3uwNML2yiT/MEpxTCdbt3JHKRiMhDs6aGEmh0uycCryVVlkKIZrVmWRZUsf8BetVcKGjABkPUVl7fEhRNwuuYitFv+RuQZYiOa6Nzm82KnDLhN5wd39Z6EnKpYBdSUCuumsjctjbtpf6kydpo4rdJPjzY+c5A== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oss.nxp.com; Received: from AS8PR04MB9080.eurprd04.prod.outlook.com (2603:10a6:20b:447::16) by AS5PR04MB11467.eurprd04.prod.outlook.com (2603:10a6:20b:6c3::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.48.17; Thu, 21 May 2026 09:07:39 +0000 Received: from AS8PR04MB9080.eurprd04.prod.outlook.com ([fe80::92c2:2e03:bf99:68eb]) by AS8PR04MB9080.eurprd04.prod.outlook.com ([fe80::92c2:2e03:bf99:68eb%6]) with mapi id 15.21.0048.016; Thu, 21 May 2026 09:07:38 +0000 From: Guoniu Zhou Date: Thu, 21 May 2026 17:10:05 +0800 Subject: [PATCH v5 2/2] media: nxp: imx8-isi: Add virtual channel support Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260521-isi_vc-v5-2-a38eb4fcd58e@oss.nxp.com> References: <20260521-isi_vc-v5-0-a38eb4fcd58e@oss.nxp.com> In-Reply-To: <20260521-isi_vc-v5-0-a38eb4fcd58e@oss.nxp.com> To: Laurent Pinchart , Mauro Carvalho Chehab , Frank Li , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam Cc: Aisheng Dong , linux-media@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Guoniu Zhou X-Mailer: b4 0.14.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1779354625; l=10399; i=guoniu.zhou@oss.nxp.com; s=20250815; h=from:subject:message-id; bh=zzRGyQyt6Dy94J3DrdiiKwNjjqK+wngDnAO6UylWkeo=; b=u/xoKezxlg8LEKGR1SgzzGlKS2+8wAWy4BZxmd4oxk5hWH+gI+quYCzXqTJsI7u2lvf+gMOrl 5WGHgYEHGwiChwr5vkU4enkVZsW23kcIkLMc0EKjD4G/XGODnZ+NSLt X-Developer-Key: i=guoniu.zhou@oss.nxp.com; a=ed25519; pk=MM+/XICg5S78/gs+f9wtGP6yIvkyjTdZwfaxXeu5rlo= X-ClientProxiedBy: SI1PR02CA0032.apcprd02.prod.outlook.com (2603:1096:4:1f6::12) To AS8PR04MB9080.eurprd04.prod.outlook.com (2603:10a6:20b:447::16) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS8PR04MB9080:EE_|AS5PR04MB11467:EE_ X-MS-Office365-Filtering-Correlation-Id: 4df8530e-bd19-4b2a-500f-08deb718678e X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True X-LD-Processed: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|376014|52116014|1800799024|19092799006|18002099003|22082099003|56012099003|38350700014|11063799006|6133799003; X-Microsoft-Antispam-Message-Info: UkY+O9Ib58FaLY/DUNI0mmEUDF3dvl7mcgsnaXpMwkNHRwzkUgwlnIgb9AJ0RUfFWoxYTLGWPR5b1Khg19WBiWELMXi4ZE7taDFgWpZJWfR0lEBm5tNVnbjv26CXkmnZ9ugsEVsHEbcY2aE4QmD3aJFl9UQ51zQPHHlphvu0TA+zz04eCEb/k2DmHKRsizE1yRBfoJmWwEyjnxnSCkx8PVovBqmOgatV2dr0HHS8ZNGuXlruRjmd4rTSqc6haygj3cYfybcje0+gToyX0Ix7sgCzDeoJ5xqfbOzDJe7Xpu441GjYkAdm6Rl8neHoqmAS+LxJ0hpXTFx4VSl7Zhgyr9lzMNArE8xa5hhVJO806cRBw3vRGcrsQpLC1RKCojqMSj3NWwTrVQSSXzEWfHhk1mhTYkEU/baiWrjWhxPt/E7IND9h0w2s3LFYwnj9UImGuEfMlgqGdbMlwKGExebWh059ZvtcqrpYSNejtSPBiDAG+Ig45h0Xqzh5WA4A/+ugWBUYYm1OHyPTSMvGPhVUbwNbEyPvBmlrfLqOZea0bGfriY65K9t0usDGM8uXSr0BTCNFI6JfeSAh49bUHs4SMDNUNvfpRS880SEn1B4ZaZZAf2iwbc3WIO3QCWV4ZyHyo4waEv0qovrZ4YF8F+DXE/u4RH6fxU34ihKbvsngj880ScYOsb6CjqcwpFi9MVAkevnxKahVRYbSyaAefytfuIKTpIZ2KPhJTYtFsBwTaFD/tWhU4dXFjloEXgnkWBNe X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AS8PR04MB9080.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(376014)(52116014)(1800799024)(19092799006)(18002099003)(22082099003)(56012099003)(38350700014)(11063799006)(6133799003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MTBYdWtwV0ZrU09nc1FSVXpWZzZ1enVmZ2NsZThHSmRhQ3p4UXUxOFZRMVk4?= =?utf-8?B?Vis2TkM5VGZ5OFBoSldIaHByZHR0bDl6YU8xRGR2SVBZQzJvcHpQVnliYXdM?= =?utf-8?B?bkgyNDdnOGRlNi9qcEtDdzg1YTcyNTRkWDdsR25jd2VtbXh5ZUZDSGRuaHNC?= =?utf-8?B?WjcwOFl0ajl1K3hBSmtjbUJRa3BVMmlpR3c5YVRuckl4T2pLT2ROKzNralRV?= =?utf-8?B?aWZWV0lyQWd0cUVaa1hhTExOS3JpeWhMNVk3RktSZkczZyt6azFPdFBKTzdL?= =?utf-8?B?aU5SVG9sbTFINXFkMnpHWGhyNko1S3BxUUc1Ymo0TUF2Yjl1UjNKVms4OGNo?= =?utf-8?B?VURScDZXcXc0ejZGaFhTS0pPdVJaUFI3dlJPREZycVUxSlBVMW9sUldhTUlY?= =?utf-8?B?ZW1YQldZVkhySnhMaVhIMkdXaDlhQUN0Q3lCOWlLMEpzWFd5Vml5WVVXOEZD?= =?utf-8?B?dHVQVHZlRFhMNFoxOHlRTjhZY3poV1N0ZW02MTJRMGc2NjNTWEpmZGNuLzlY?= =?utf-8?B?c2tDeTIyc0Fpa1dML202Tm43cWdLRnhHMWZjSndtYW9zeFkvTHNMZmRYbFFo?= =?utf-8?B?YkU3NkI5Q3daMlk2OUI3dGwyVFpsTzJjQnF2c1dpOCtSQmpkRHZpZXRXYXlQ?= =?utf-8?B?ck13SWV6bzNvZEl5YXJ3RysvbjlIL2c0UGxVZ2ViNlFpUVA2ZmJST3ZKYS84?= =?utf-8?B?TlA1WFNJUXRPZC9zSnVuMFc1VVU1VGFUQVdrcmFNSmZoaUVyR2RocmpINUZ3?= =?utf-8?B?TUdqdWsrS0RTRHFhVm9CSHU5RWhLZUZ3TUx3aHkrN3lBZG1OV1VQNDdGalZo?= =?utf-8?B?bFNDbmVJVVRzZFVIcmZDU2FyMmFCUVlzeHE3YS9ibGxLUUxESURzR0I4NGtL?= =?utf-8?B?S2pqZUNNbVdUWC9XOHhlem44UGZMTUJCSk1OREh3VThJcGRUT2VKQjVFaWw2?= =?utf-8?B?QnN3NkR3dWh6SDYyUHhtTWxGQkVsUmkzOExJeTdUWGJxcUlOUWx3K2xxakN4?= =?utf-8?B?ZXh6azRHeGQ5MWRFRUZhVzY0SG9tNktXc2FBRngzSDRJQ3ZRNnJXSzlsKzVh?= =?utf-8?B?YVNsUms4TGtsSHZHek8rdnRsOU85NUpJMjI4dUFmY0tPandFekYyNVUzbmk0?= =?utf-8?B?VUZRNHBtMmEySEVGRC9oYzlaTkttVHlJRk5haGVzaGlwaGRaN0F5TDkzR1I5?= =?utf-8?B?UVMvSXQ0K01ycGFYcWNVYlhURG9vRTF4RGtrdDdFUEMzQUFuNVN0STRNZVlX?= =?utf-8?B?MkpPWm96eW9GZ1Bzc2tjeWxKaHJvd3RyRTFydmFQMzNZK256UnRLZUxySTlj?= =?utf-8?B?V2RqSmlUcWdkTHU5cGZnT3AzenVQVFpndWJQZFQ2UmtWcEIxNjI4UTY4d3N3?= =?utf-8?B?am5MaE9pVzl6eHhYMTJ4bWYvQmlMNkozbW92a1VFMFRUcG9xdkU0R3JyR2Q2?= =?utf-8?B?MXplaDBPelYwWDNTdENHeC92cGxwaXcxT2V5c2VCZGZkVXFnR0pCSGtTd0hG?= =?utf-8?B?cy8vVnFXUHFTaVBvTmxwQ3pySW1CUlE1MXNvQkduRW5tMUdSYXIyeHN1eU5v?= =?utf-8?B?SUxwd3AwWDB5OW5oeTExOThWWnBmVVFyczNuQUthMDFGNTU0ZTlhNDE3R0tL?= =?utf-8?B?NkpSK3NkOHRSR0VmUm5BUUdydThTWmpITDBteW05UUhMcWVyQjBXb1hlSnlH?= =?utf-8?B?ckkremRKci9nV005cXRQNUJ0N25JREU5SFJGRXNMeWk3b3AxVU9pc3VNMlgz?= =?utf-8?B?ckpHYkQ1Z1Ftdm1ybHlrcXdESDFKY0RYemFhUWllbWg2a2ZnN2JUMkVzZTdr?= =?utf-8?B?bUFseW0vaGRHaTlueEFiSTlGZ0N2WUh3K2docHovK2psOHAzSFhmdFprYW42?= =?utf-8?B?bnQwVlVlQk00TUd6OVpoTXhoOHlGZE1peW9CQTVxcDZPSmNLUWw3cmZJWTRy?= =?utf-8?B?YStNNTUxQWJSUUsxWmRPRjd1eXJUK3cwaHdMbUNueXBaNHE5U1VEREl4a2Uz?= =?utf-8?B?TDZkMUpHNnMxZXVwcE9NdWtmRURIVnFpWnM3amJCZFRybDlGTC9OQk8ySUw1?= =?utf-8?B?M2kzdUsxaUpkRVVoTllGY2gyUEg0V0xJdE8ycDZvclFVYWxlcE8vSUZ6QjZm?= =?utf-8?B?MS9uTmZrWGFqUEp5VG9BYVZKZjZrbERwQitMQldtZkhaWFp5YUhsZkUyRklM?= =?utf-8?B?ei9lTXY3bUI5Mkx3YlVVR1hiWGZib1RMYjJEeFBObXc4RTExY2FrcVpGclVu?= =?utf-8?B?UVUreE5VLzcxc2o2SzNmdEVZdit3bC9yRGd4WU81R1k3MUV4SDhGd2pGdy9m?= =?utf-8?B?K1lXb3N3dDgxMVQyMERsL3k0UmVielB0L0JPdE1LQUlqSGYyWUlzdz09?= X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4df8530e-bd19-4b2a-500f-08deb718678e X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB9080.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 May 2026 09:07:38.2109 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: kqkV7pTkpxjnh3MIXJPfbqIMrYQ1bZI1RGznqfKD3XW0NgeKVRZVKroN4MFQ5OdJmlxGchQ+XQrBPAEXlZpjSg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS5PR04MB11467 From: Guoniu Zhou The ISI supports different numbers of virtual channels depending on the platform. i.MX95 supports 8 virtual channels, and i.MX8QXP/QM support 4 virtual channels. They are used in multiple camera use cases, such as surround view. Other platforms (such as i.MX8/MN/MP/ULP/91/93) don't support virtual channels, and the VC_ID bits are marked as read-only. Reviewed-by: Frank Li Signed-off-by: Guoniu Zhou --- Changes in v5: - Return -EPIPE instead of -EINVAL for stream configuration errors - Clear VC_ID_1 after generic mask to follow generic-then-conditional order - Pass vc as function parameter instead of storing in pipe structure. - Drop get_frame_desc fallback as crossbar now implements the operation - Remove redundant num_entries check in mxc_isi_get_vc(). - Set vc to 0 for M2M as it doesn't support virtual channels. Changes in v4: - Fix VC boundary check: use num_vc (virtual channels count) instead of num_channels (ISI pipelines count) - Set VC to 0 when frame descriptor has no entries - Move platform-specific comments to block style to fix line length warnings Changes in v3: - Add num_vc field to platform data to indicate VC support - Clear VC_ID_1 bit after reading CHNL_CTRL for proper VC switching - Set VC_ID_1 only on platforms with num_vc > 4 - Improve mxc_isi_get_vc() error handling - Add back CHNL_CTRL_BLANK_PXL and document platform-specific register fiel= ds --- .../media/platform/nxp/imx8-isi/imx8-isi-core.c | 3 ++ .../media/platform/nxp/imx8-isi/imx8-isi-core.h | 2 + drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c | 17 +++++++- drivers/media/platform/nxp/imx8-isi/imx8-isi-m2m.c | 2 +- .../media/platform/nxp/imx8-isi/imx8-isi-pipe.c | 50 ++++++++++++++++++= +++- .../media/platform/nxp/imx8-isi/imx8-isi-regs.h | 12 ++++-- 6 files changed, 79 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.c b/drivers/= media/platform/nxp/imx8-isi/imx8-isi-core.c index 4bf8570e1b9e..837ac7046cf2 100644 --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.c +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.c @@ -318,6 +318,7 @@ static const struct mxc_isi_plat_data mxc_imx95_data = =3D { .model =3D MXC_ISI_IMX95, .num_ports =3D 4, .num_channels =3D 8, + .num_vc =3D 8, .reg_offset =3D 0x10000, .ier_reg =3D &mxc_imx8_isi_ier_v2, .set_thd =3D &mxc_imx8_isi_thd_v1, @@ -329,6 +330,7 @@ static const struct mxc_isi_plat_data mxc_imx8qm_data = =3D { .model =3D MXC_ISI_IMX8QM, .num_ports =3D 5, .num_channels =3D 8, + .num_vc =3D 4, .reg_offset =3D 0x10000, .ier_reg =3D &mxc_imx8_isi_ier_qm, .set_thd =3D &mxc_imx8_isi_thd_v1, @@ -340,6 +342,7 @@ static const struct mxc_isi_plat_data mxc_imx8qxp_data = =3D { .model =3D MXC_ISI_IMX8QXP, .num_ports =3D 5, .num_channels =3D 6, + .num_vc =3D 4, .reg_offset =3D 0x10000, .ier_reg =3D &mxc_imx8_isi_ier_v2, .set_thd =3D &mxc_imx8_isi_thd_v1, diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.h b/drivers/= media/platform/nxp/imx8-isi/imx8-isi-core.h index 14d63ec36416..2957119c81f2 100644 --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.h +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.h @@ -169,6 +169,7 @@ struct mxc_isi_plat_data { enum model model; unsigned int num_ports; unsigned int num_channels; + unsigned int num_vc; /* Number of VCs, 0 =3D no VC support */ unsigned int reg_offset; const struct mxc_isi_ier_reg *ier_reg; const struct mxc_isi_set_thd *set_thd; @@ -377,6 +378,7 @@ void mxc_isi_channel_unchain(struct mxc_isi_pipe *pipe); =20 void mxc_isi_channel_config(struct mxc_isi_pipe *pipe, enum mxc_isi_input_id input, + unsigned int vc, const struct v4l2_area *in_size, const struct v4l2_area *scale, const struct v4l2_rect *crop, diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c b/drivers/me= dia/platform/nxp/imx8-isi/imx8-isi-hw.c index 0187d4ab97e8..a98d7bec731d 100644 --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c @@ -301,6 +301,7 @@ static void mxc_isi_channel_set_panic_threshold(struct = mxc_isi_pipe *pipe) =20 static void mxc_isi_channel_set_control(struct mxc_isi_pipe *pipe, enum mxc_isi_input_id input, + unsigned int vc, bool bypass) { u32 val; @@ -312,6 +313,10 @@ static void mxc_isi_channel_set_control(struct mxc_isi= _pipe *pipe, CHNL_CTRL_SRC_TYPE_MASK | CHNL_CTRL_MIPI_VC_ID_MASK | CHNL_CTRL_SRC_INPUT_MASK); =20 + /* Clear the VC_ID_1 bit on platforms supporting more than 4 VCs. */ + if (pipe->isi->pdata->num_vc > 4) + val &=3D ~CHNL_CTRL_VC_ID_1_MASK; + /* * If no scaling or color space conversion is needed, bypass the * channel. @@ -338,7 +343,14 @@ static void mxc_isi_channel_set_control(struct mxc_isi= _pipe *pipe, } else { val |=3D CHNL_CTRL_SRC_TYPE(CHNL_CTRL_SRC_TYPE_DEVICE); val |=3D CHNL_CTRL_SRC_INPUT(input); - val |=3D CHNL_CTRL_MIPI_VC_ID(0); /* FIXME: For CSI-2 only */ + val |=3D CHNL_CTRL_MIPI_VC_ID(vc); /* FIXME: For CSI-2 only */ + + /* + * On platforms with more than 4 VCs (i.MX95), the VC ID is + * split across VC_ID_0 (bits 7:6) and VC_ID_1 (bit 16). + */ + if (pipe->isi->pdata->num_vc > 4) + val |=3D CHNL_CTRL_VC_ID_1(vc >> 2); } =20 mxc_isi_write(pipe, CHNL_CTRL, val); @@ -348,6 +360,7 @@ static void mxc_isi_channel_set_control(struct mxc_isi_= pipe *pipe, =20 void mxc_isi_channel_config(struct mxc_isi_pipe *pipe, enum mxc_isi_input_id input, + unsigned int vc, const struct v4l2_area *in_size, const struct v4l2_area *scale, const struct v4l2_rect *crop, @@ -374,7 +387,7 @@ void mxc_isi_channel_config(struct mxc_isi_pipe *pipe, mxc_isi_channel_set_panic_threshold(pipe); =20 /* Channel control */ - mxc_isi_channel_set_control(pipe, input, csc_bypass && scaler_bypass); + mxc_isi_channel_set_control(pipe, input, vc, csc_bypass && scaler_bypass); } =20 void mxc_isi_channel_set_input_format(struct mxc_isi_pipe *pipe, diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-m2m.c b/drivers/m= edia/platform/nxp/imx8-isi/imx8-isi-m2m.c index a39ad7a1ab18..291907ef44cb 100644 --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-m2m.c +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-m2m.c @@ -144,7 +144,7 @@ static void mxc_isi_m2m_device_run(void *priv) .height =3D ctx->queues.cap.format.height, }; =20 - mxc_isi_channel_config(m2m->pipe, MXC_ISI_INPUT_MEM, + mxc_isi_channel_config(m2m->pipe, MXC_ISI_INPUT_MEM, 0, &in_size, &scale, &crop, ctx->queues.out.info->encoding, ctx->queues.cap.info->encoding); diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-pipe.c b/drivers/= media/platform/nxp/imx8-isi/imx8-isi-pipe.c index a41c51dd9ce0..03e0115b5b5a 100644 --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-pipe.c +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-pipe.c @@ -232,6 +232,47 @@ static inline struct mxc_isi_pipe *to_isi_pipe(struct = v4l2_subdev *sd) return container_of(sd, struct mxc_isi_pipe, sd); } =20 +static int mxc_isi_get_vc(struct mxc_isi_pipe *pipe) +{ + struct mxc_isi_crossbar *xbar =3D &pipe->isi->crossbar; + struct device *dev =3D pipe->isi->dev; + struct v4l2_mbus_frame_desc fd =3D { }; + unsigned int source_pad =3D xbar->num_sinks + pipe->id; + unsigned int max_vc; + unsigned int i; + int ret; + + ret =3D v4l2_subdev_call(&xbar->sd, pad, get_frame_desc, + source_pad, &fd); + if (ret < 0) { + dev_err(dev, "Failed to get source frame desc from pad %u\n", + source_pad); + return ret; + } + + /* Find stream 0 in the frame descriptor */ + for (i =3D 0; i < fd.num_entries; i++) { + if (fd.entry[i].stream =3D=3D 0) + break; + } + + if (i =3D=3D fd.num_entries) { + dev_err(dev, "Failed to find stream from source frame desc\n"); + return -EPIPE; + } + + max_vc =3D pipe->isi->pdata->num_vc ? : 1; + + /* Check virtual channel range */ + if (fd.entry[i].bus.csi2.vc >=3D max_vc) { + dev_err(dev, "Virtual channel %u exceeds maximum %u\n", + fd.entry[i].bus.csi2.vc, max_vc - 1); + return -EPIPE; + } + + return fd.entry[i].bus.csi2.vc; +} + int mxc_isi_pipe_enable(struct mxc_isi_pipe *pipe) { struct mxc_isi_crossbar *xbar =3D &pipe->isi->crossbar; @@ -244,6 +285,7 @@ int mxc_isi_pipe_enable(struct mxc_isi_pipe *pipe) struct v4l2_subdev *sd =3D &pipe->sd; struct v4l2_area in_size, scale; struct v4l2_rect crop; + unsigned int vc; u32 input; int ret; =20 @@ -280,8 +322,14 @@ int mxc_isi_pipe_enable(struct mxc_isi_pipe *pipe) =20 v4l2_subdev_unlock_state(state); =20 + ret =3D mxc_isi_get_vc(pipe); + if (ret < 0) + return ret; + + vc =3D ret; + /* Configure the ISI channel. */ - mxc_isi_channel_config(pipe, input, &in_size, &scale, &crop, + mxc_isi_channel_config(pipe, input, vc, &in_size, &scale, &crop, sink_info->encoding, src_info->encoding); =20 mxc_isi_channel_enable(pipe); diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-regs.h b/drivers/= media/platform/nxp/imx8-isi/imx8-isi-regs.h index 1b65eccdf0da..e795f4daf3ff 100644 --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-regs.h +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-regs.h @@ -6,6 +6,7 @@ #ifndef __IMX8_ISI_REGS_H__ #define __IMX8_ISI_REGS_H__ =20 +#include #include =20 /* ISI Registers Define */ @@ -19,9 +20,14 @@ #define CHNL_CTRL_CHAIN_BUF_NO_CHAIN 0 #define CHNL_CTRL_CHAIN_BUF_2_CHAIN 1 #define CHNL_CTRL_SW_RST BIT(24) -#define CHNL_CTRL_BLANK_PXL(n) ((n) << 16) -#define CHNL_CTRL_BLANK_PXL_MASK GENMASK(23, 16) -#define CHNL_CTRL_MIPI_VC_ID(n) ((n) << 6) +/* + * CHNL_CTRL_BLANK_PXL: i.MX8{QM,QXP} only + * CHNL_CTRL_VC_ID_1, CHNL_CTRL_VC_ID_1_MASK: i.MX95 only + */ +#define CHNL_CTRL_BLANK_PXL(n) FIELD_PREP(GENMASK(23, 16), (n)) +#define CHNL_CTRL_VC_ID_1(n) FIELD_PREP(BIT(16), (n)) +#define CHNL_CTRL_VC_ID_1_MASK BIT(16) +#define CHNL_CTRL_MIPI_VC_ID(n) FIELD_PREP(GENMASK(7, 6), (n)) #define CHNL_CTRL_MIPI_VC_ID_MASK GENMASK(7, 6) #define CHNL_CTRL_SRC_TYPE(n) ((n) << 4) #define CHNL_CTRL_SRC_TYPE_MASK BIT(4) --=20 2.34.1