From mboxrd@z Thu Jan 1 00:00:00 1970 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 0BC13C433F5 for ; Sat, 5 Mar 2022 05:23:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231251AbiCEFYo (ORCPT ); Sat, 5 Mar 2022 00:24:44 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42330 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231276AbiCEFYl (ORCPT ); Sat, 5 Mar 2022 00:24:41 -0500 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2074.outbound.protection.outlook.com [40.107.237.74]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E33FA1965C1; Fri, 4 Mar 2022 21:23:50 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FH20IXb/zSitqlUDKnsLFRks7//jU7eyQPPAw1W3LBNrPsFYq/vdjCoSlY1zU8poN8MDnqnzksfgB0U7X0oxb9KAE56pb9/fxoN6q8GeXZ+GPiLkWPrEc4co0wS262NBAc7cyPZBj2/t/w4yc5k1mQys/hSIvvOUe9Xg9ymbIpH7ePnt+CqSqzAg6z8I8MnUAk1Cq6rw9WoGYdTgfM4bwIVDrjxA+aaP20lXojXmFElQCW85tjbui5mzxNJ8BKjsH3CjT8RrNvLz9/Ad50nj1OwK0MEhKEe1EW24NRRZRtlaQ6QUbu7uDhMfdTtQX+Fi9jzBEu+08U6LSnRMVFcz+w== 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=qdmgBRgPiavesm0saAo05tNSNGEzTR1CbgfRdTnIAxc=; b=FRKlAjMlE0K15dvmgYatdQjAfG/YmXbYuO/gcmTyUNOYkRNWDDWNcmSF9KeqTA/dfS1G4iXCiNoz8L35wd69LPPZvt0vzeVv2Z6ElBO95Gp52gcX9CzmpkMHAdhodmLKuS5rBz+h3ldHFA9h1dNqOnYzj5PfnJaAn2CG3XJpy/cET9j3FJ9z3p2U0cu6W0beXULJ7aj2zzcv93siij9cGwsfLo7mTxw9Y8PsrMxw9TV5YhdRMVnFPhCsixNJRZ5YA9AB6GtCVH2zpfAL6cYMHJgXqw3Ytb0J/SNXlpX+V5u/GLQvypMxtABArK4vMr4AJixFc546CGKYx5YqMP0whg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 149.199.62.198) smtp.rcpttodomain=infradead.org smtp.mailfrom=xilinx.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=xilinx.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xilinx.onmicrosoft.com; s=selector2-xilinx-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qdmgBRgPiavesm0saAo05tNSNGEzTR1CbgfRdTnIAxc=; b=Acwv36unsXQUpRPfGRFoc5c5+CokFICwBAgBayKD0WssRPon6V7fc301ONbNXa2yBdcIkVwTvioCwJJi4O5dPL7Giq8HTlXWVYN2MDgOlyVt2rB0QQchhrqVGmVvhw538LTfVc5jVh0rSOHQVDpeY9t2dQ/F8EohQJ9T/jLdHhQ= Received: from SA9PR11CA0006.namprd11.prod.outlook.com (2603:10b6:806:6e::11) by PH0PR02MB8730.namprd02.prod.outlook.com (2603:10b6:510:d9::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5038.15; Sat, 5 Mar 2022 05:23:47 +0000 Received: from SN1NAM02FT0043.eop-nam02.prod.protection.outlook.com (2603:10b6:806:6e:cafe::40) by SA9PR11CA0006.outlook.office365.com (2603:10b6:806:6e::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5038.14 via Frontend Transport; Sat, 5 Mar 2022 05:23:47 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 149.199.62.198) smtp.mailfrom=xilinx.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=xilinx.com; Received-SPF: Pass (protection.outlook.com: domain of xilinx.com designates 149.199.62.198 as permitted sender) receiver=protection.outlook.com; client-ip=149.199.62.198; helo=xsj-pvapexch02.xlnx.xilinx.com; Received: from xsj-pvapexch02.xlnx.xilinx.com (149.199.62.198) by SN1NAM02FT0043.mail.protection.outlook.com (10.97.5.57) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.5038.14 via Frontend Transport; Sat, 5 Mar 2022 05:23:47 +0000 Received: from xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) by xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.14; Fri, 4 Mar 2022 21:23:46 -0800 Received: from smtp.xilinx.com (172.19.127.95) by xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) with Microsoft SMTP Server id 15.1.2176.14 via Frontend Transport; Fri, 4 Mar 2022 21:23:46 -0800 Envelope-to: dwmw2@infradead.org, yilun.xu@intel.com, mdf@kernel.org, robh@kernel.org, trix@redhat.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org Received: from [172.19.72.93] (port=44432 helo=xsj-xw9400.xilinx.com) by smtp.xilinx.com with esmtp (Exim 4.90) (envelope-from ) id 1nQMtO-000BZ8-Lt; Fri, 04 Mar 2022 21:23:46 -0800 Received: by xsj-xw9400.xilinx.com (Postfix, from userid 21952) id 39BAE600196; Fri, 4 Mar 2022 21:23:10 -0800 (PST) From: Lizhi Hou To: , , CC: Lizhi Hou , , , , , , , , , , , Max Zhen Subject: [PATCH V1 RESEND 4/4] of: enhance overlay applying interface to specific target base node Date: Fri, 4 Mar 2022 21:23:04 -0800 Message-ID: <20220305052304.726050-5-lizhi.hou@xilinx.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220305052304.726050-1-lizhi.hou@xilinx.com> References: <20220305052304.726050-1-lizhi.hou@xilinx.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: f8381eb2-dc22-4284-8d3e-08d9fe6852fa X-MS-TrafficTypeDiagnostic: PH0PR02MB8730:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 6hmkloC6VHpLZ7F6CCoi3iozinOWw66Rsthi+kfo6gVro2Nmhw170UOXVnGrPv42WNUNOJWo45uH7BZ7ku6yZ3loNWpGEwxysVPIaKWTVGwCKRL/1uRYGUXZiZp2n9lZbu45SvJrUMwVsT7yzJvRFxeZ6Q/DREFzU7qxWMT6mp9sISoIFAmGvx2gNxThjyv3naa1pBQ+NpPmkmUUxvHxwMI4GJa2zCJD+kxxNI0kiKVzdUebm4ugRuMM+5Dr+RMNsCiPpa34fZmyJHvM7RMaqaQ82ihgncIAlZVvf77zaJ87W51PWRY1Wu4yp55xnD1zjse8L/hFA6eM3ekOVrI7D3zvCOLjp5g48LNRhFioE7IEX4g1XYCmc1KUki7f7tyRMnDk6OW9KJVQTMcThYeVfFEOmziOj2l7yw6Zkh9J1qYCnXK+nMGJZG4naBt7HXfrMY7C25cvG0ILpcNBPu76v/We3aS50LEGYzxK5AChG3mpPzov9waBZf4APSfERmqLGaxGev8GNCMLozwtm1yTfH4gORWlStNBi+QWMtnDx6APWYtXx75QnOzVFvR3dGFUrPSGuHK3Q+RY3Uu+UaEBZKBVP66KubV7aAjJwbjxyVoMMvPufnCwtDTniePGc7HBirvkXplhbpCNuix45sZKww== X-Forefront-Antispam-Report: CIP:149.199.62.198;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:xsj-pvapexch02.xlnx.xilinx.com;PTR:unknown-62-198.xilinx.com;CAT:NONE;SFS:(13230001)(4636009)(46966006)(36840700001)(336012)(426003)(83380400001)(5660300002)(2906002)(42186006)(7636003)(36756003)(26005)(44832011)(6266002)(356005)(1076003)(8936002)(186003)(107886003)(2616005)(8676002)(508600001)(70206006)(70586007)(110136005)(54906003)(82310400004)(36860700001)(47076005)(4326008)(316002)(6666004);DIR:OUT;SFP:1101; X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Mar 2022 05:23:47.1060 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: f8381eb2-dc22-4284-8d3e-08d9fe6852fa X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.62.198];Helo=[xsj-pvapexch02.xlnx.xilinx.com] X-MS-Exchange-CrossTenant-AuthSource: SN1NAM02FT0043.eop-nam02.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR02MB8730 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The self discovered device, e.g. PCIe device, may carry a device tree overlay to describe its downstream devices. In this case, the device node is created dynamically by device driver. And in the overlay it does not specify a fixed target path. Instead, a relative path to the device node is specified. Thus, a new parameter is added to overlay applying interface. This parameter is the pointer of target base node. Then the entire overlay target path is target base node path plus the relative path specified in the device tree overlay. Signed-off-by: Sonal Santan Signed-off-by: Max Zhen Signed-off-by: Lizhi Hou --- drivers/fpga/xrt/mgmt/Makefile | 1 + drivers/fpga/xrt/mgmt/dt-test.dts | 15 ++++++++++ drivers/fpga/xrt/mgmt/dt-test.h | 15 ++++++++++ drivers/fpga/xrt/mgmt/xmgmt-drv.c | 41 +++++++++++++++++++++++++++- drivers/gpu/drm/rcar-du/rcar_du_of.c | 2 +- drivers/of/overlay.c | 37 +++++++++++++++++-------- drivers/of/unittest.c | 2 +- include/linux/of.h | 2 +- 8 files changed, 100 insertions(+), 15 deletions(-) create mode 100644 drivers/fpga/xrt/mgmt/dt-test.dts create mode 100644 drivers/fpga/xrt/mgmt/dt-test.h diff --git a/drivers/fpga/xrt/mgmt/Makefile b/drivers/fpga/xrt/mgmt/Makefile index b893c7293d70..a87ab1f6b403 100644 --- a/drivers/fpga/xrt/mgmt/Makefile +++ b/drivers/fpga/xrt/mgmt/Makefile @@ -9,4 +9,5 @@ obj-$(CONFIG_FPGA_XRT_XMGMT) += xrt-mgmt.o xrt-mgmt-objs := \ + dt-test.dtb.o \ xmgmt-drv.o diff --git a/drivers/fpga/xrt/mgmt/dt-test.dts b/drivers/fpga/xrt/mgmt/dt-test.dts new file mode 100644 index 000000000000..3bc3d0c95180 --- /dev/null +++ b/drivers/fpga/xrt/mgmt/dt-test.dts @@ -0,0 +1,15 @@ +// SPDX-License-Identifier: GPL-2.0 +/dts-v1/; + +/ { + fragment@0 { + target-path=""; + __overlay__ { + pr_isolate_ulp@0,41000 { + compatible = "xlnx,alveo-pr-isolation"; + reg = <0x0 0x41000 0x0 0x1000>; + }; + }; + }; +}; + diff --git a/drivers/fpga/xrt/mgmt/dt-test.h b/drivers/fpga/xrt/mgmt/dt-test.h new file mode 100644 index 000000000000..dee0de570c8a --- /dev/null +++ b/drivers/fpga/xrt/mgmt/dt-test.h @@ -0,0 +1,15 @@ +/* SPDX-License-Identifier: GPL-2.0 */ +/* + * Copyright (C) 2020-2022 Xilinx, Inc. + * + * Authors: + * Lizhi Hou + */ + +#ifndef _DT_TEST_H_ +#define _DT_TEST_H_ + +extern u8 __dtb_dt_test_begin[]; +extern u8 __dtb_dt_test_end[]; + +#endif /* _DT_TEST_H_ */ diff --git a/drivers/fpga/xrt/mgmt/xmgmt-drv.c b/drivers/fpga/xrt/mgmt/xmgmt-drv.c index 60742a478a43..54078ffe37fe 100644 --- a/drivers/fpga/xrt/mgmt/xmgmt-drv.c +++ b/drivers/fpga/xrt/mgmt/xmgmt-drv.c @@ -14,8 +14,11 @@ #include #include #include +#include #include +#include "dt-test.h" + #define XMGMT_MODULE_NAME "xrt-mgmt" /* PCI Device IDs */ @@ -25,17 +28,53 @@ static const struct pci_device_id xmgmt_pci_ids[] = { { 0, } }; +struct xmgmt { + int ovcs_id; +}; + static int xmgmt_probe(struct pci_dev *pdev, const struct pci_device_id *id) { - devm_of_pci_create_bus_endpoint(pdev); + struct device_node *dn; + struct xmgmt *xm; + int ret; + + xm = devm_kzalloc(&pdev->dev, sizeof(*xm), GFP_KERNEL); + if (!xm) + return -ENOMEM; + pci_set_drvdata(pdev, xm); + + ret = devm_of_pci_create_bus_endpoint(pdev); + if (ret) + return ret; + + dn = of_pci_find_bus_endpoint(pdev); + if (!dn) { + dev_err(&pdev->dev, "does not find bus endpoint"); + return -EINVAL; + } + + ret = of_overlay_fdt_apply(__dtb_dt_test_begin, + (u32)(__dtb_dt_test_end - __dtb_dt_test_begin), + &xm->ovcs_id, dn); + of_node_put(dn); + if (ret) + return ret; return 0; } +static void xmgmt_remove(struct pci_dev *pdev) +{ + struct xmgmt *xm = pci_get_drvdata(pdev); + + of_overlay_remove(&xm->ovcs_id); +} + static struct pci_driver xmgmt_driver = { .name = XMGMT_MODULE_NAME, .id_table = xmgmt_pci_ids, .probe = xmgmt_probe, + .remove = xmgmt_remove, }; static int __init xmgmt_init(void) diff --git a/drivers/gpu/drm/rcar-du/rcar_du_of.c b/drivers/gpu/drm/rcar-du/rcar_du_of.c index afef69669bb4..8799ed235f83 100644 --- a/drivers/gpu/drm/rcar-du/rcar_du_of.c +++ b/drivers/gpu/drm/rcar-du/rcar_du_of.c @@ -60,7 +60,7 @@ static int __init rcar_du_of_apply_overlay(const struct rcar_du_of_overlay *dtbs ovcs_id = 0; return of_overlay_fdt_apply(dtb->begin, dtb->end - dtb->begin, - &ovcs_id); + &ovcs_id, NULL); } static int __init rcar_du_of_add_property(struct of_changeset *ocs, diff --git a/drivers/of/overlay.c b/drivers/of/overlay.c index d80160cf34bb..4d192dff5055 100644 --- a/drivers/of/overlay.c +++ b/drivers/of/overlay.c @@ -686,9 +686,11 @@ static int build_changeset(struct overlay_changeset *ovcs) * 1) "target" property containing the phandle of the target * 2) "target-path" property containing the path of the target */ -static struct device_node *find_target(struct device_node *info_node) +static struct device_node *find_target(struct device_node *info_node, + struct device_node *target_base) { struct device_node *node; + char *target_path; const char *path; u32 val; int ret; @@ -704,10 +706,23 @@ static struct device_node *find_target(struct device_node *info_node) ret = of_property_read_string(info_node, "target-path", &path); if (!ret) { - node = of_find_node_by_path(path); - if (!node) - pr_err("find target, node: %pOF, path '%s' not found\n", - info_node, path); + if (target_base) { + target_path = kasprintf(GFP_KERNEL, "%pOF%s", target_base, path); + if (!target_path) + return NULL; + node = of_find_node_by_path(target_path); + if (!node) { + pr_err("find target, node: %pOF, path '%s' not found\n", + info_node, target_path); + } + kfree(target_path); + } else { + node = of_find_node_by_path(path); + if (!node) { + pr_err("find target, node: %pOF, path '%s' not found\n", + info_node, path); + } + } return node; } @@ -730,7 +745,7 @@ static struct device_node *find_target(struct device_node *info_node) * detected in @tree, or -ENOSPC if idr_alloc() error. */ static int init_overlay_changeset(struct overlay_changeset *ovcs, - const void *fdt, struct device_node *tree) + const void *fdt, struct device_node *tree, struct device_node *target_base) { struct device_node *node, *overlay_node; struct fragment *fragment; @@ -792,7 +807,7 @@ static int init_overlay_changeset(struct overlay_changeset *ovcs, fragment = &fragments[cnt]; fragment->overlay = overlay_node; - fragment->target = find_target(node); + fragment->target = find_target(node, target_base); if (!fragment->target) { of_node_put(fragment->overlay); ret = -EINVAL; @@ -914,7 +929,7 @@ static void free_overlay_changeset(struct overlay_changeset *ovcs) */ static int of_overlay_apply(const void *fdt, struct device_node *tree, - int *ovcs_id) + int *ovcs_id, struct device_node *base) { struct overlay_changeset *ovcs; int ret = 0, ret_revert, ret_tmp; @@ -947,7 +962,7 @@ static int of_overlay_apply(const void *fdt, struct device_node *tree, if (ret) goto err_free_tree; - ret = init_overlay_changeset(ovcs, fdt, tree); + ret = init_overlay_changeset(ovcs, fdt, tree, base); if (ret) goto err_free_tree; @@ -1015,7 +1030,7 @@ static int of_overlay_apply(const void *fdt, struct device_node *tree, } int of_overlay_fdt_apply(const void *overlay_fdt, u32 overlay_fdt_size, - int *ovcs_id) + int *ovcs_id, struct device_node *base) { void *new_fdt; void *new_fdt_align; @@ -1053,7 +1068,7 @@ int of_overlay_fdt_apply(const void *overlay_fdt, u32 overlay_fdt_size, goto out_free_new_fdt; } - ret = of_overlay_apply(new_fdt, overlay_root, ovcs_id); + ret = of_overlay_apply(new_fdt, overlay_root, ovcs_id, base); if (ret < 0) { /* * new_fdt and overlay_root now belong to the overlay diff --git a/drivers/of/unittest.c b/drivers/of/unittest.c index 70992103c07d..471bf6f2ec9a 100644 --- a/drivers/of/unittest.c +++ b/drivers/of/unittest.c @@ -3010,7 +3010,7 @@ static int __init overlay_data_apply(const char *overlay_name, int *ovcs_id) if (!size) pr_err("no overlay data for %s\n", overlay_name); - ret = of_overlay_fdt_apply(info->dtb_begin, size, &info->ovcs_id); + ret = of_overlay_fdt_apply(info->dtb_begin, size, &info->ovcs_id, NULL); if (ovcs_id) *ovcs_id = info->ovcs_id; if (ret < 0) diff --git a/include/linux/of.h b/include/linux/of.h index 2dc77430a91a..7c5ecafb98c8 100644 --- a/include/linux/of.h +++ b/include/linux/of.h @@ -1567,7 +1567,7 @@ struct of_overlay_notify_data { #ifdef CONFIG_OF_OVERLAY int of_overlay_fdt_apply(const void *overlay_fdt, u32 overlay_fdt_size, - int *ovcs_id); + int *ovcs_id, struct device_node *target_base); int of_overlay_remove(int *ovcs_id); int of_overlay_remove_all(void); -- 2.27.0