From patchwork Tue May 28 23:55:42 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kuninori Morimoto X-Patchwork-Id: 102008 Received: from ny.mirrors.kernel.org ([147.75.199.223]) by linuxtv.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1sC6gl-0001TW-1Z for patchwork@linuxtv.org; Tue, 28 May 2024 23:57:07 +0000 Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 1C0D01C244AD for ; Tue, 28 May 2024 23:57:06 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5861813F442; Tue, 28 May 2024 23:55:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=renesas.com header.i=@renesas.com header.b="FOyPAPAh" X-Original-To: linux-media@vger.kernel.org Received: from JPN01-TYC-obe.outbound.protection.outlook.com (mail-tycjpn01on2041.outbound.protection.outlook.com [40.107.114.41]) (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 033DB13E8BE; Tue, 28 May 2024 23:55:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.114.41 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716940547; cv=fail; b=kUfZXGO03H25wrl9NfDL3vXIUs3r4rFva8CQ7+/rHxUSgamRYdYIM1+gjMGKlop5HCevpB7vA+u1X2kkqCMoUbhxuab7x3hVqNJbdYCQPZgku1btzHzGro4HMuySrx5pO4VSNYuLaop7KoxHssbOIm56wP2giMDx0Va7/CEyxZo= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716940547; c=relaxed/simple; bh=u2/uBfueVnq22xgrXGJY7eaKNmltqAuP2xlkEZRxez8=; h=Message-ID:From:Subject:To:In-Reply-To:References:Content-Type: Date:MIME-Version; b=WdT6vuTjIlTauZ0Qia3QyVXZD2h52p6m+qGxJuxbjgEv4ml4iKO5rMGhZj99kbXC9FoF0PewkLn0cdADFZIJt2FLSWzy88EkxUdFCIOGErG9BzRasrQzLlw2xL/ed4v+/Vpr3sIOgNPtjtFcA9OJXj9nSKIKBoi39j1QE93DvkQ= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=renesas.com; spf=pass smtp.mailfrom=renesas.com; dkim=pass (1024-bit key) header.d=renesas.com header.i=@renesas.com header.b=FOyPAPAh; arc=fail smtp.client-ip=40.107.114.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=renesas.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=renesas.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NRpnrPu4inCjci6T2Ogp3EwK66w1xlb7LA3ABT+5v4VfUQGYEAvPWANMISvHLafHupBBWE4pVkP4/OM9PyF91KiB6lGSWy0VdgqEWA3bHGCX0tTTZM3N4IaWfmQG1YkDJDyszR69rODybZn2xjXnsPNlxuQSgoWaKf7WPId9r2N3yrgnxiEWiz/CX9agZKJjhECiqbccXGd+Na0z9zJFf4EKRlzh+I2w2UxzXBG/hqsbUtpIw/2arF5bbjon9Br4dv7MU50llV0hAdbVOX9KOl/YIlIKlh//kf/t/flqxZN0xRsLonU04D30+yXIkJSgAylpB0sTEmistRaD4VWtOw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=aKyIcOtJvLvD7O2LiyJI/N2rAEc2ca7YgJ2t0jkn4VY=; b=eqWxQ3qZdSSTiHRdjZcToiBRC7L7ZhX28BISOMzcr1A+X+gTxSL3F2Uf/hcfkMHFvXmlIam/pU3xhTRq4Can4bEDnU9CStEYSlerssiDd3dWOkcPAnEheyMFvP3hmlx544c+E2OHia9wWJ1ZO8qhO8eLKVqo7L7A9dKbsV9+6WP9eQUQNn+yzvFx0tgRkOW1aBmRZQPBJOcyc14KAkAA+HpK3r+pNbQYkdc1bkoBwIavKDl3AoX9UW6oSLjn+FYPTB9Cj4hkpS50mxGlwSzMVAJb3XPVuHQdaTo4olm8hFJCTAw3AEj4CphY6ypwTDrHSo6faTACZIqsJCAiRezFuw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=renesas.com; dmarc=pass action=none header.from=renesas.com; dkim=pass header.d=renesas.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=renesas.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=aKyIcOtJvLvD7O2LiyJI/N2rAEc2ca7YgJ2t0jkn4VY=; b=FOyPAPAh9w5iXQZlm4bbePLonUTP5yMiqUQod4e4S613YT9kjpdPFuIJdCXf9dUbbiO9eT0j4NQ77P+HsVosiCcSZNpGgrZOLi6tcSncTN3HqHXTWhamLuenl6iEseD0zKGyrpByYhVVTCtvi/lXax2pRXUMTI9w8nLY3H/eMrY= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=renesas.com; Received: from TYCPR01MB10914.jpnprd01.prod.outlook.com (2603:1096:400:3a9::11) by TYCPR01MB11224.jpnprd01.prod.outlook.com (2603:1096:400:3bf::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7633.17; Tue, 28 May 2024 23:55:42 +0000 Received: from TYCPR01MB10914.jpnprd01.prod.outlook.com ([fe80::c568:1028:2fd1:6e11]) by TYCPR01MB10914.jpnprd01.prod.outlook.com ([fe80::c568:1028:2fd1:6e11%4]) with mapi id 15.20.7611.030; Tue, 28 May 2024 23:55:42 +0000 Message-ID: <87cyp54hky.wl-kuninori.morimoto.gx@renesas.com> From: Kuninori Morimoto Subject: [PATCH v2 resend 4/8] media: platform: ti: use for_each_endpoint_of_node() User-Agent: Wanderlust/2.15.9 Emacs/29.3 Mule/6.0 To: , Krzysztof Kozlowski , Alexander Shishkin , Alexandre Belloni , Claudiu Beznea , Daniel Vetter , David Airlie , Eugen Hristev , Greg Kroah-Hartman , Helge Deller , Laurent Pinchart , Maarten Lankhorst , Mauro Carvalho Chehab , Maxime Ripard , Michal Simek , Nicolas Ferre , Rob Herring , Suzuki K Poulose , Thomas Zimmermann , Tomi Valkeinen , coresight@lists.linaro.org, dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-fbdev@vger.kernel.org, linux-media@vger.kernel.org, linux-staging@lists.linux.dev In-Reply-To: <87ikyx4hm1.wl-kuninori.morimoto.gx@renesas.com> References: <87ikyx4hm1.wl-kuninori.morimoto.gx@renesas.com> Date: Tue, 28 May 2024 23:55:42 +0000 X-ClientProxiedBy: TYCP286CA0361.JPNP286.PROD.OUTLOOK.COM (2603:1096:405:79::17) To TYCPR01MB10914.jpnprd01.prod.outlook.com (2603:1096:400:3a9::11) Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: TYCPR01MB10914:EE_|TYCPR01MB11224:EE_ X-MS-Office365-Filtering-Correlation-Id: bcd31a92-56f4-456d-47d9-08dc7f71af0b X-LD-Processed: 53d82571-da19-47e4-9cb4-625a166a4a2a,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230031|52116005|1800799015|7416005|376005|366007|38350700005|921011; X-Microsoft-Antispam-Message-Info: arCV/Sfamf0olJt7SjwaysRzKtPvX8vRoXP+uViIA4Yw78kCutq/BJ16NSXIz2XSpPeDsp8ZNiakqtpHelBcuvS8U8cWVVmmUHNx0tixDdBUNnndt13eJ/ZrjyET/zIcg6OFLJr1jHxEmw/ltB9R6ilAZoMhrY2azXzVYzsMStvDyTxSQb27Q7aa9m6OT8oEJGAXfqvqWPrcIzJfDzYEG3RPFtAPmnt8gc9z98u324irICYAaHvU5LbrW/JdrlqU4PRIv43Fg3f8oGnAClOeM3Zw1t6BijwQhjroXGdk6b7fgM3NK93zPSpZTf6ByFaA15nmqRmYHeXYFYh6dRDHJpWhlfyN8JZpP7RV8VNi04Xl5L6vFnoYX7W9VtqnA3VC3/+TFbzbbSKK7GwgauZpVENLj1vV2lmokQ0tGUKri+5tiYrtuksIsJO8k/bmp32TZJFGIyNd/qSnR3onQ8tK9vc1JO9E2XmlNRsfMsjxEENSKi25/UeDLBNTvGUsRcZg9PuyS/HeRxrDxonNFDaueAYsdD6znlyel2G8To0q817u1yzykCZ3aKSeI0VTPoD4kX2eVdY4n9NpvHM0YGMFfgqWLkc7B3bw1O7pPHij4SmrK+w0t5dMmAPK+l89uKfNlMhsMFHk1bE9Rgeu9hPE2dmoTzFSV1Ml0LXWEMY3Wfm5WQmgezJEWpcvmM+ROFrkSolMqfmSwTLQbS3K5zRZSKrctDakKTLGV56svI/5lnVdkcXheATn0jgMkVubxGXg83VLI6KPSwaYGfQHyPzqxCdoVTkltTBdPPHEDefeRQaItFQyxFMWTSP+vlHjKPlJaIbVaJQ65vqd0XmiBRKuXn/Rd50jTyRolTno+U2pADQHkCBUnglitVKrPnatd6SS0Mo/SfP4ioJrelyX0q9DQQHgGOqS3MA/UqrLCZ0R0u+rAfsgJGeryBrG4J1zPxXKNfTt9Anq8mTkAxds3ELaGpIOy8i5o/evXXLN6dWJnC46QUY2P9DG17fWn1DqinaNUQ03Z+T9LFOpRn/QY5DFrRkXIg5G+g56Dlj93MXXBNAivekv6Pfap4e4ushIOjSLo+Fa2dAgL7v31+yETb2gDgVkaRlGvsN6LYjRE/OlGKJWeVFtCzUKVGdqOve3ifHdXatkcTjisqs1A6Rfd9zEIuZjL6310IHkR+AiK2PzjmQZN9F8KmeIHy7xsquWUItKTDUhMmHLPhF2Tu97ngMWU+hmI9YbNto5TdEUOvn4xSYbvp9ZgjlyDoy3WUIDmdcMFYbIJEvgQ7nPGZMLRZen5M3/JOEvWDCLb4pvsh2wKkpS21ioi8ozUoEtNqU2jkxL46fzTuiwOkXLJ6Va57ykAreaeJqbf+jJv+eY3Ymd+w0= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYCPR01MB10914.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(52116005)(1800799015)(7416005)(376005)(366007)(38350700005)(921011);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: dnMdrcfIiPiBGzwcZOAPOw/27ZFggXPa8Fi3TZyNHUwXUUdeUENvYc7GTpWg39FWmaRC1Kv/lWgQpiL0YI2Gi3ABCv11qMjNayb9U7xJs8JEKhk0JhKdqjCcPuYOTVEH3eyvbsc6YVvndwdlfXyTXYoQVZ/WzKVZPHlpHVqQETPmZMpj3uz4yMzggy1x6xb93QBAW5TyVSjJDuJsFU99ut92ANI1RIoXvpboSjpKd2H/6I1BUYAECg76Z1kx/tlaJ0dHJR2uH9Ypb/otuwFnUlHYZHXFk05nymsM1vLk6eA8QcjrrA1bwImwndwz3uSWNKzbZpHTwGZ/StnrHK/EN/qe8/C4OjMCNdftPmuAoLTg8qTh8ME3CqZPMF17fDIw5zebmF08UeNOTEIhwgJENALsHuyt48vLFe8YLoyVm0LTizT/zSYUIrf8zv3g9JRsmD49pwqc0A752ow3BT3XKE6zEOkWi3nvNdmr71sCFeIR+UPij+DvLvoSnwTm5FUOTEO91eU7afCXYIIIKd0KZIT+0HRIcfrn2GAJTmrg4MEvUKIKX0yr1bwUEDaN+me+qrRVdfwliVvADTPixQF9MwUxbc0LbSJtGaSiLnWBawLanPFjG4l4yEyaAOt1MeiBO92/cc12x+5E5lJ/kowGnxpjZdJJVcAdsMATd5mNqTvION0+yszwIFqbcGLpXUg6DjjoGjVnV2nubc+5YJ2iJCYEq8FE34QZKozh5Th1nuOOjQ04uZ9BzBayDabcYmmcyvPtDZTnXql0q7w0xWPHUWW51rPRZtcPCih7bjmvF/mb191rJBCfvpakw+awa9RLHVKhAVGaL3JOCdRjBJKkdbDAB5iXs2PjCyw/xHmUMOhMOVSMUF8xG0hRXAJsowprHt9/BOqa3Gav4n/33rnEoE3wbrOW1vPfBWHqBmYLVTSynhYy6vHzaIG524E6i355GJmhXooKDM0QRMFhAXQDM3f92etmL5skb3JJWB5DCScXJRxF03o08fJQ4gJUTOINYG2ildzm6ha5Gz4AE91Cm4TGl6O6S8GADTim+MnK+3F3vvBj+puSJ2vaxkfRVo9LkN0d0T3ZhGZo+v8Iw8rrzzXTcT7uQUgOvBpeBroU5OmISD6U3WULuR0KsPKmATrzYLXYBfLuSAMljZ2ufhHPcM3YrUDPSiSUW+n6mQocUyT7A0ZzbmupJkVVuMoDZUfePkZxZn1ORpNEeFCfPwgD5n1WmxfQhe55BMQ+ip7OSFqqQG507qYZgFYRkTG84MGnmCZKtD+3S6V+mAisLEB4u0AQFXJ2a4/zPSkwkBjRB4N0Ml81OK/jVFxH1PBnBADfclcQgK7UCEB51odHbCy+hdqw1ickCzdEdYOckPvJ+sjKHMw5YR5T5REDVwNsS+eHbkLaedLbAW2NrtHxjVB4VjE/mrDf9l4XV32geskXCQJ5BzjBwjQxFlKrsM6tmwjlCfvJJ2PS38IPCDXuwFUp0AH/BW0YpM33GrmFG06Ej0+XsQXw80BfAu/a2IuPxOcFC4GRj9GY2CPIgd3wR75mosb4A4jBYd1MjszvRtK9/6wC2I0eOL85j4CVIS3X+Evs6WdhklbPYejbcz7rtsEYD1KoNU4yzQ2yMdY5n/saS9g= X-OriginatorOrg: renesas.com X-MS-Exchange-CrossTenant-Network-Message-Id: bcd31a92-56f4-456d-47d9-08dc7f71af0b X-MS-Exchange-CrossTenant-AuthSource: TYCPR01MB10914.jpnprd01.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 May 2024 23:55:42.5664 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 53d82571-da19-47e4-9cb4-625a166a4a2a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: +Z0CyN1IR5Xf0P1Glpea6BeJcvBQNOtzwMrnZ4ymb1AQWybI3qjsFIIrGuAsb7wvYaDWWBefb2bspWCetm/26va+yQ+TBxcEZNE6yrEqEm9YwbhDMSEWZmWjfNAFioAc X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYCPR01MB11224 X-LSpam-Score: -2.6 (--) X-LSpam-Report: No, score=-2.6 required=5.0 tests=ARC_SIGNED=0.001,ARC_VALID=-0.1,BAYES_00=-1.9,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,DMARC_PASS=-0.001,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1,SPF_HELO_NONE=0.001,SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no We already have for_each_endpoint_of_node(), don't use of_graph_get_next_endpoint() directly. Replace it. Signed-off-by: Kuninori Morimoto Reviewed-by: Laurent Pinchart --- drivers/media/platform/ti/am437x/am437x-vpfe.c | 12 +++++------- drivers/media/platform/ti/davinci/vpif_capture.c | 12 ++++++------ 2 files changed, 11 insertions(+), 13 deletions(-) diff --git a/drivers/media/platform/ti/am437x/am437x-vpfe.c b/drivers/media/platform/ti/am437x/am437x-vpfe.c index 77e12457d1495..009ff68a2b43c 100644 --- a/drivers/media/platform/ti/am437x/am437x-vpfe.c +++ b/drivers/media/platform/ti/am437x/am437x-vpfe.c @@ -2287,7 +2287,7 @@ static const struct v4l2_async_notifier_operations vpfe_async_ops = { static struct vpfe_config * vpfe_get_pdata(struct vpfe_device *vpfe) { - struct device_node *endpoint = NULL; + struct device_node *endpoint; struct device *dev = vpfe->pdev; struct vpfe_subdev_info *sdinfo; struct vpfe_config *pdata; @@ -2306,14 +2306,11 @@ vpfe_get_pdata(struct vpfe_device *vpfe) if (!pdata) return NULL; - for (i = 0; ; i++) { + i = 0; + for_each_endpoint_of_node(dev->of_node, endpoint) { struct v4l2_fwnode_endpoint bus_cfg = { .bus_type = 0 }; struct device_node *rem; - endpoint = of_graph_get_next_endpoint(dev->of_node, endpoint); - if (!endpoint) - break; - sdinfo = &pdata->sub_devs[i]; sdinfo->grp_id = 0; @@ -2371,9 +2368,10 @@ vpfe_get_pdata(struct vpfe_device *vpfe) of_node_put(rem); if (IS_ERR(pdata->asd[i])) goto cleanup; + + i++; } - of_node_put(endpoint); return pdata; cleanup: diff --git a/drivers/media/platform/ti/davinci/vpif_capture.c b/drivers/media/platform/ti/davinci/vpif_capture.c index c28794b6677b7..078ae11cd0787 100644 --- a/drivers/media/platform/ti/davinci/vpif_capture.c +++ b/drivers/media/platform/ti/davinci/vpif_capture.c @@ -1517,16 +1517,12 @@ vpif_capture_get_pdata(struct platform_device *pdev, if (!pdata->subdev_info) return NULL; - for (i = 0; i < VPIF_CAPTURE_NUM_CHANNELS; i++) { + i = 0; + for_each_endpoint_of_node(pdev->dev.of_node, endpoint) { struct v4l2_fwnode_endpoint bus_cfg = { .bus_type = 0 }; unsigned int flags; int err; - endpoint = of_graph_get_next_endpoint(pdev->dev.of_node, - endpoint); - if (!endpoint) - break; - rem = of_graph_get_remote_port_parent(endpoint); if (!rem) { dev_dbg(&pdev->dev, "Remote device at %pOF not found\n", @@ -1577,6 +1573,10 @@ vpif_capture_get_pdata(struct platform_device *pdev, goto err_cleanup; of_node_put(rem); + + i++; + if (i >= VPIF_CAPTURE_NUM_CHANNELS) + break; } done: