From patchwork Tue Nov 12 06:41:32 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Josua Mayer X-Patchwork-Id: 13871758 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E69F0D41D5F for ; Tue, 12 Nov 2024 06:47:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version:Cc:To: In-Reply-To:References:Message-Id:Content-Transfer-Encoding:Content-Type: Subject:Date:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=DBP1lqm4a8r+XPGPyakaL2LkPPDwdDkPI1a+IG7CK6w=; b=Vk+66vV3VrlD2+6sP3oBz8JKtQ AByrAh4s1e5Izx4hQKkaQhz74S8d43YvADoiSqaiQJ85sdHakrGphgcpyY5ool8LpF8Oc+eh6Nhgz EEcLZDOhL9Jxv4du4lA3Fg2wCgde/XFtDbHRWNxaPsI+aClZRgfdaOQGSKM+912V9rSQTgkpEm+fT HYwzeUrz5fc8dMiuVkK+aG1GnrQuc+cN5eikYPNYqOJuoSSrxZH7OU5n6CgT4O18Oi4hXLUoJtLyK rMqLkJM4MK7GSrwx2GQCQ9tmzDTg8FecMuP7PJlz4Rve9S2mRE79X1GO/+EDPRVwTUVQh3M+0r3IK u6uUHKQA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tAkgG-00000002Moa-4BeX; Tue, 12 Nov 2024 06:47:17 +0000 Received: from mail-vi1eur05on2071d.outbound.protection.outlook.com ([2a01:111:f403:2613::71d] helo=EUR05-VI1-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tAkbE-00000002M29-2NuJ for linux-arm-kernel@lists.infradead.org; Tue, 12 Nov 2024 06:42:05 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=NeDKtHg33/2L5Xe4cSB/mcXz2NoWordWhM46Jv5UK/2cnDrTMgysPMrMU8fW/Vt66WmNg7kbvYwa47Ad2gdUQtE3mtLtjL4AIFXo8ldQCeByDhegTOH0aMJQ45OR0C3rTWO23zU4ElFVmRfFCBJ7o1D1t1Zcy8BRjDWzpBXXf3r8vc189wfXCdSn/OR9k1FqGM3ZzZA03bVSbK6lzhwSkWh7uyRsFZQws6Q3+ZfBi593QS6zMgHuyUxnzBzVVhNg6uaREhmmUW7OCRlRmrPG5+l/ndhIyYK7w6vbFEKTekk164lrgBiEOwv+1C/FGC7mLfhLF0daQmdt1k2YrDQWdQ== 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=DBP1lqm4a8r+XPGPyakaL2LkPPDwdDkPI1a+IG7CK6w=; b=SekHJfsBPsz9cueSaShq+yqlCJOhRxaIB1PG+/r/4Q8wN/1s4+Eq0HrbnuqZFAy0dm4vvxHduJMO4SRfONc6PKnkmOq2IalQYhz3p6hOaTXjvz+01rBj+Udf2Rg3cW6sdyf1nE71JBwX/gvQyyym920wH/srHO4LMq5AIektXFOs+ZVj8EIueoDbqIbeYkbHGVAMG4iluiMLQpdS++wlpB79KViQfdcQ1Qd425Rd8um7ZOeE8v5UI7U0t9Aq+ZqUwZd03i3MXQ2/7N0bNWpbI2cnvGVVaZd6Ssx8aObaVdYIuNRGCvaIl+sremxVPyWekeizXNjwhzhuJ5DzeQ+6GQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=solid-run.com; dmarc=pass action=none header.from=solid-run.com; dkim=pass header.d=solid-run.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=solidrn.onmicrosoft.com; s=selector1-solidrn-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=DBP1lqm4a8r+XPGPyakaL2LkPPDwdDkPI1a+IG7CK6w=; b=XZXcaUmKotSyIxgweOHbOmrfcTfJhEJk15LetMl3VJ/V0P9e1ZsTPOOaiY2RE7lB+NC9u3KOd4Wg1V8cvX8wGSDyor7+ae5nE6U5xVNs9zrTBS5+PTtAoo0+qcjbdomFwrRq4KsDcrvXMHC6sxy8li4eF4ZAkfCjgLYbzwyZT60= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=solid-run.com; Received: from AM9PR04MB7586.eurprd04.prod.outlook.com (2603:10a6:20b:2d5::17) by DU4PR04MB10959.eurprd04.prod.outlook.com (2603:10a6:10:586::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8137.28; Tue, 12 Nov 2024 06:41:54 +0000 Received: from AM9PR04MB7586.eurprd04.prod.outlook.com ([fe80::c04e:8a97:516c:5529]) by AM9PR04MB7586.eurprd04.prod.outlook.com ([fe80::c04e:8a97:516c:5529%4]) with mapi id 15.20.8137.027; Tue, 12 Nov 2024 06:41:54 +0000 From: Josua Mayer Date: Tue, 12 Nov 2024 08:41:32 +0200 Subject: [PATCH 1/2] of: add support for value "false" to of_property_read_bool Message-Id: <20241112-am64-overlay-bool-v1-1-b9d1faff444e@solid-run.com> References: <20241112-am64-overlay-bool-v1-0-b9d1faff444e@solid-run.com> In-Reply-To: <20241112-am64-overlay-bool-v1-0-b9d1faff444e@solid-run.com> To: Rob Herring , Saravana Kannan , Nishanth Menon , Vignesh Raghavendra , Tero Kristo , Krzysztof Kozlowski , Conor Dooley Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Geert Uytterhoeven , Josua Mayer , Jon Nettleton , Yazan Shhady , rabeeh@solid-run.com X-Mailer: b4 0.14.2 X-ClientProxiedBy: PA7P264CA0446.FRAP264.PROD.OUTLOOK.COM (2603:10a6:102:398::29) To AM9PR04MB7586.eurprd04.prod.outlook.com (2603:10a6:20b:2d5::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM9PR04MB7586:EE_|DU4PR04MB10959:EE_ X-MS-Office365-Filtering-Correlation-Id: 423749e5-5507-4074-405c-08dd02e51900 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|366016|1800799024|52116014|38350700014; X-Microsoft-Antispam-Message-Info: =?utf-8?q?eFwCBxs/qPJqhfHQZJsP5EGfdrmrk9b?= =?utf-8?q?EDszRN7cOKcBhRxFIbc1gi3GaAIfwTVtCDKTqDLiukfXXHDaWGh477jk4B4+K8NBE?= =?utf-8?q?entdya6xmE7LYg8PDnQr+ukqxEb/uxRhuhnshjG46Oa0ANeXuh4qZr+Q+yjyckvZ8?= =?utf-8?q?r8nRKRFUqZLAXErZwJ4VZuSt58suZNAx8q4fVLa19bY7RTT1x3KOvwOA+rmDDCB57?= =?utf-8?q?+QiZv5WKe0o8cFIJkIbjHxsmT0atq39R20LqVPMAnVstcS5HXbVth3BQnSXQ+xWvt?= =?utf-8?q?XA+M+hea22BcrSNLA2kQOO1/ZwTc7f9ILO/VGdEQj6Rw1wj6GCZ7FHnQRaHI3te0q?= =?utf-8?q?zgeT+w1DTTVOjXrLMOEDdrcnYwxyeu1JN5G8FHXxtUpjo3QczWRPBOqGM58p0fJyg?= =?utf-8?q?akvCFlc0SYRHaRPzl4G2eJqamwBv1zqygznWQ8tps6it3yaWTbZxltgxSYffT0AgR?= =?utf-8?q?0xU+HODeNkB0AKVt7UxyIDQhgjupMk1C8OGc4LV3F1qsTRYNxeCS9ZZMnfXEWi2dS?= =?utf-8?q?/Am7NpVFL/Ebna9HPybpWa4KrH4DEcRczM4DwbCDr6fq3rf/3MjOZ3AqUXOcG2YQ1?= =?utf-8?q?0tGdPnpDildb+qqFivWOlWOuz9BS8gi+8yEu7hS/1ongKM/LkRpz+T2Zd+i7XMd72?= =?utf-8?q?qOQSiBgo4UGEP4M8IKf2uCw0mKoqKWNDUyqr3YNONn9dDqM7MXeLzhR4zrQWv9ww7?= =?utf-8?q?QmH5dgvNawt3wKMmDuMa2zyZX7rUapKpNmvooXRP5XDgXb+/PJ/1trN6Oyakhg1uB?= =?utf-8?q?Uq65nIiy5TDtL+T7lHwGl1Apl8M+Tu+3DqP6SW1Mv7O6LsAYYPQWhPSUKTS5TtsxD?= =?utf-8?q?TNnplGab4v0S8QbAcIWK5Zt/5xxYbrI4aBkWYz+VfLydfokCiB+ON8tlIGX3HdCav?= =?utf-8?q?4an/aEZJuMTXPIublZFDpwk7EMRizRCsvU6Zut/a67xNckxuFebe6rrM1ixVygzhB?= =?utf-8?q?GOfspgOw9Daxj8NXsC80dkOsFK+mywZl9EMUIi3wWwcBi3arGuarsZrHnMZJ/GEH2?= =?utf-8?q?PctaVX4V60aJBxvKhGcCpFf5n8Sqaf2jpzPMbI/h+uy9evoOEcK/zQvMUIyu5ms+P?= =?utf-8?q?/WrSTrIoKUjQQiESEtPazcSqC8ajs+N7r6kryQLAf/2g2f09ZUKVo5oGBn1e325E7?= =?utf-8?q?3W6s44WBwHojLJeEmtOfwZV0Us9c3ZFGITTz5hOAOx6wbW4gWC2ILunkPFvcrffVB?= =?utf-8?q?e8urszNWOjHEedvW0Ir+pFgpE6w2tX1spgT13iFLnuGYsTayrzY4JNpAyRyTpIHo7?= =?utf-8?q?9Teuyx9EFzpSN70ogK3xuB1tOJtc1s1YHdN+ygvNs3CABfqeYFHFWbOA=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM9PR04MB7586.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(7416014)(366016)(1800799024)(52116014)(38350700014);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?k62pTEqVQ5qkcvFOJWmd2ayv7XCq?= =?utf-8?q?sGkSYg6duXWpTdB8mCak5+vw4CdpAKw2XntJj6G5maKgY/ogEoVTWosywP7v0eyiG?= =?utf-8?q?FF4ZPWAwuLFLZWd4FZLeZRCwezSh0hUnGDV77sqS9pZR9TJA1NkiQP/fFSH3Grf3q?= =?utf-8?q?ssEc3COfhNtblFRhzZgv37Hr6iLa6DrbroRIMJJQLWCwGZiLHCRlVdnrIOfMcBJyw?= =?utf-8?q?l3DXAoK2edKB6lP15rnKQ5cxhACesu3hYyoaHtnEDyk8y27WdLOTIBrCq6tGp8V2N?= =?utf-8?q?sRWuSu9bCB93ux7Tw5SF8m899IQAZjYBBmG2+ZKlrpQuGJ+x5UNZX2D+U8XZDs/aV?= =?utf-8?q?JRfPUgv0PSNZJ4zaUUWX5j3f3+XCi4DTJgfvxVGrIBsgrhDKUjBs8omd3wGjR/S77?= =?utf-8?q?VC/IlxRt5WHv5kJSlhqAqiMxTEceGY3Jtz2zkVtBw4Sml/Waqx9mFdlZ7caUAb2Sp?= =?utf-8?q?+PV0TlrpqdDOJb2hBbtiKtL1sgkL5hxudc5Ch8GVohFcGcMDdJzA1rP/gU9kOoDiL?= =?utf-8?q?Tc0hMgmTUt0nLi6wRrqeLbrh9DSQC+4nNQyur8wdc0/D5ZLYSIr1qrgoymsWOYpU2?= =?utf-8?q?5OyZDmzU4XPCp0JIlSS0xolRmOnu3RrvDh6nkt1slybWbj5iIvhD8jn2bmmtMfO4N?= =?utf-8?q?UgDAVv0KcwZ7Lo2bLfkU85pdN1sVvNINnCV5D3wzG+nnF+fnG/rGRoazKx1uPGyVc?= =?utf-8?q?3T1eBhgwF11JQOcPLxYPRbB226ztiQKUXfDhlKKuh7vQLVWiyBkSwkota2MZpR4XK?= =?utf-8?q?W+kKNyP7k5G3R5kxcEMOOvHfELw0Egxp9yel9I4JrTOsIaHgeL6TyDeSagezT7Gm7?= =?utf-8?q?7ag1FrxhM8+gZBsypNhRg3YJRHTIpfUaqqgZoafVstY+QdkkdExYZgXxIUpSF2e5K?= =?utf-8?q?2v1bFEoMTl8/PTPVSy8hqt53M7ByFAtfVZDmkRALhSCh+zTMWtllsFEAft/qxBzjV?= =?utf-8?q?A5XYiKH8/zYQxXVg9mZj+YfyXQ/UMOkskDYTKzgkfDCGSTNOLH6PIl3XRImGL6wIu?= =?utf-8?q?RFkGCu6f6M0zjZS0pLd+WOWPM0GJyD/vjRgiGgv3AvngHDoPkF39oqbSS4VFH98Vy?= =?utf-8?q?gjnDtO5HKqXX2vHGxOWuxdLfloNzSEr9h68oKtB9/jAqlXJDQOdy5Ab2gbxxzyrew?= =?utf-8?q?/ejffPG4Ukg7Bebf8ROx1RSd92K0/9BxFvm7mrWQVY7w72g1WbozaP/n8R6gx+yi0?= =?utf-8?q?8DLNwpQB2RAhw8u12M2q2J9i0Q3tfOa1D/uKdUAoPM2WXrBRFaMuH2mhqq9vBrAL2?= =?utf-8?q?vOs4ABCiBRP8KQHmy8hop/kWeJKDP9dZVYVp9dBvLNuuEqn4HrmkLNkisZaUubFvq?= =?utf-8?q?2a+2yAW7Hk0zCUPSvhWD0htGQa9fkj7BLRhqiMBHk5mtQJabosh7kspwp/IUyxPS6?= =?utf-8?q?JaGSI+nC+wYbyUMvaYf1WZiHukfGAa28AWsYHRKsMqY7KofuT0LOl5Ahfu9MX+ZBJ?= =?utf-8?q?Cmd0LXlsEu7QqarHxAMFRg/jL8ZIhw7/qEQD/Flx2HLrU4q6vtstCDUSp0Xtt/exW?= =?utf-8?q?/15hOmkxwBZu?= X-OriginatorOrg: solid-run.com X-MS-Exchange-CrossTenant-Network-Message-Id: 423749e5-5507-4074-405c-08dd02e51900 X-MS-Exchange-CrossTenant-AuthSource: AM9PR04MB7586.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Nov 2024 06:41:54.8175 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a4a8aaf3-fd27-4e27-add2-604707ce5b82 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: tYGuxNo9pV0+45AfaluW49HnFKjGY5Ze1SKxkSWJhP9HDZbjtABuyrQtDqf/n3VQsVEl42/6ciFn8bSktFrqKA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU4PR04MB10959 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241111_224204_649237_9EAEDDEB X-CRM114-Status: GOOD ( 16.11 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Boolean type properties are usually considered true if present and false when they do not exist. This works well for many in-tree board dts and existing drivers. When users need to overrride boolean values from included dts includes, /delete-property/ is recommend. This however does not work in overlays (addons). Several places use string "true" ([1], [2], [3]) and one uses string "false" ([1]). This suggests that at some point the value of a type string property was to be taken into account during evaluation. Change of_property_read_bool to only return true if a property is both present, and not equal "false". Existing usage in drivers/of and include/linux/of.h are updated accordingly. Other places should be reviewed as needed wrt. changed semantics. [1] Documentation/devicetree/bindings/sound/rt5651.txt [2] Documentation/devicetree/bindings/sound/pcm3060.txt [3] arch/arm/boot/dts/ti/omap/am335x-baltos.dtsi Signed-off-by: Josua Mayer --- drivers/of/property.c | 2 +- include/linux/of.h | 13 ++++++++----- 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/drivers/of/property.c b/drivers/of/property.c index 11b922fde7af167cc69f6dd89826219653cd1ee8..a51132b883fb3fa0b1a120bacb298abf72f67c5b 100644 --- a/drivers/of/property.c +++ b/drivers/of/property.c @@ -893,7 +893,7 @@ of_fwnode_device_get_dma_attr(const struct fwnode_handle *fwnode) static bool of_fwnode_property_present(const struct fwnode_handle *fwnode, const char *propname) { - return of_property_read_bool(to_of_node(fwnode), propname); + return of_property_present(to_of_node(fwnode), propname); } static int of_fwnode_property_read_int_array(const struct fwnode_handle *fwnode, diff --git a/include/linux/of.h b/include/linux/of.h index 85b60ac9eec50bb202aa774cab273e1d947e394d..ea65c7d1176194d6ce79432782361f2cdab84f8b 100644 --- a/include/linux/of.h +++ b/include/linux/of.h @@ -1247,14 +1247,15 @@ static inline int of_property_read_string_index(const struct device_node *np, * Search for a boolean property in a device node. Usage on non-boolean * property types is deprecated. * - * Return: true if the property exists false otherwise. + * Return: true if the property exists and value is not "false", + * false otherwise. */ static inline bool of_property_read_bool(const struct device_node *np, const char *propname) { - struct property *prop = of_find_property(np, propname, NULL); + int ret = of_property_match_string(np, propname, "false"); - return prop ? true : false; + return ret == -ENODATA ? true : false; } /** @@ -1268,7 +1269,9 @@ static inline bool of_property_read_bool(const struct device_node *np, */ static inline bool of_property_present(const struct device_node *np, const char *propname) { - return of_property_read_bool(np, propname); + struct property *prop = of_find_property(np, propname, NULL); + + return prop ? true : false; } /** @@ -1679,7 +1682,7 @@ static inline int of_reconfig_get_state_change(unsigned long action, * of_device_is_system_power_controller - Tells if system-power-controller is found for device_node * @np: Pointer to the given device_node * - * Return: true if present false otherwise + * Return: true if enabled, false otherwise */ static inline bool of_device_is_system_power_controller(const struct device_node *np) { From patchwork Tue Nov 12 06:41:33 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Josua Mayer X-Patchwork-Id: 13871759 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D079ED41D5F for ; Tue, 12 Nov 2024 06:49:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version:Cc:To: In-Reply-To:References:Message-Id:Content-Transfer-Encoding:Content-Type: Subject:Date:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=6T4IZs1Gv41w/SNtX56/TchU7qWinfqOFYQSgWYfxnM=; b=mcsyXJ4X5apWbj1M+kXPeusJpm Z8ldWSqg9eIpNrNB4UNVcIdVZygM8Hzva7u/757tNDmnQjzbVDinxUsO51EX5bPOzSV+u9V2i2otK 7CfR3bubPj/BZm6lOL2dRQo6HznnfBfBz+0P6fHj5OCLI7DJ8HylCqPcJnbHDRl63uvQ4ZwHCe2oT CPpbOfYSQkoqFYbkc+id/uoT2eTkAF+nFbwN+++XVo9y++ZKd0QHljRI3L+rv/ztuJMlXBlhIjUnC TBom55bzTSBF+BBeEDneVfyxRag4+FMPI+Ih1usKjbsU6ZrvnOHuuKf9VSXqs7OI/juHxUzK0/01c linwrR+Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tAki0-00000002N1I-1yY1; Tue, 12 Nov 2024 06:49:04 +0000 Received: from mail-vi1eur05on2071d.outbound.protection.outlook.com ([2a01:111:f403:2613::71d] helo=EUR05-VI1-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tAkbG-00000002M29-1ZAe for linux-arm-kernel@lists.infradead.org; Tue, 12 Nov 2024 06:42:07 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=r4gSb7HGQo6o+0FQvP49In6AHc/Sp/y/kCDXaufaH+QFJzfIOn5XsGiDWeRlFjwD+mJJ6xkDWktfn8OWUnYR9S7aGT8gQUknDEYv4FremrnO2v+YeA80AFuUiPzxgH0I5rQJCqKfEHNJoFaenGGVFqmzIi7keIljlX1yRTvlmptXZ7n9RDbGUUctQwiiTWeB/Ar0tueEO0DY03BhQpxGvGvMndiatemohcxXSXrCDd3bgVt35fB+qZcpPm/BuRFmlqkaYuXsQNq24dG8E+MzwY5ffh3QV2LyrMZuzk3yUzULWL/l6tcuh9JzHi9VWcbUXAeh9M0DaLJF8JDcUOHC1g== 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=6T4IZs1Gv41w/SNtX56/TchU7qWinfqOFYQSgWYfxnM=; b=w4u32oPED5taCPVewH6FSzitDNL02uqyz1iQn7XuvxITWFZp7EDJL8wWUelYVxpddc38995U5q+twoR6q19RwS+HczBjAxsT5isnYGjS0ug/AjEVHzIEO5TPJ/Hp/sZ6h9eeN37ypg8EFGdhnCSLdKExM4znN7itaAiaNuDiB0bXzA5+oUJicnLUjrdky0WBDGTbqjlmivGMt8gNekDYGNdVFoFKsi8tLH8YDp+95YF/n755AgMTEmcs8m7I5wsOtjzxxj9DP4JATYWDcgVyppgjZ2ASz3M+EhQVXBK1lygR+akJWRYW9QgTRIMGEtp4tf5TeqCAGjsoMwmHwZhJSQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=solid-run.com; dmarc=pass action=none header.from=solid-run.com; dkim=pass header.d=solid-run.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=solidrn.onmicrosoft.com; s=selector1-solidrn-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=6T4IZs1Gv41w/SNtX56/TchU7qWinfqOFYQSgWYfxnM=; b=XqYZXt2tfm95arVJ0YsnHejeZw8Cque0cSk05SkFzXrkjQyamhyGFHLj89pfC99+x7y4L4MplFVX+D5s0NcHnRIS22bKjRBYhId9XdHjG43E2OC74U+pZcoW5NtiQNi7rEKIpW/jClK7SkWY11zRDB82zfuWST9TaJjZeCjZ7DE= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=solid-run.com; Received: from AM9PR04MB7586.eurprd04.prod.outlook.com (2603:10a6:20b:2d5::17) by DU4PR04MB10959.eurprd04.prod.outlook.com (2603:10a6:10:586::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8137.28; Tue, 12 Nov 2024 06:41:56 +0000 Received: from AM9PR04MB7586.eurprd04.prod.outlook.com ([fe80::c04e:8a97:516c:5529]) by AM9PR04MB7586.eurprd04.prod.outlook.com ([fe80::c04e:8a97:516c:5529%4]) with mapi id 15.20.8137.027; Tue, 12 Nov 2024 06:41:56 +0000 From: Josua Mayer Date: Tue, 12 Nov 2024 08:41:33 +0200 Subject: [PATCH 2/2] arm64: dts: ti: k3-am642-hummingboard-t-usb3: fix overlay boolean value Message-Id: <20241112-am64-overlay-bool-v1-2-b9d1faff444e@solid-run.com> References: <20241112-am64-overlay-bool-v1-0-b9d1faff444e@solid-run.com> In-Reply-To: <20241112-am64-overlay-bool-v1-0-b9d1faff444e@solid-run.com> To: Rob Herring , Saravana Kannan , Nishanth Menon , Vignesh Raghavendra , Tero Kristo , Krzysztof Kozlowski , Conor Dooley Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Geert Uytterhoeven , Josua Mayer , Jon Nettleton , Yazan Shhady , rabeeh@solid-run.com X-Mailer: b4 0.14.2 X-ClientProxiedBy: PA7P264CA0446.FRAP264.PROD.OUTLOOK.COM (2603:10a6:102:398::29) To AM9PR04MB7586.eurprd04.prod.outlook.com (2603:10a6:20b:2d5::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM9PR04MB7586:EE_|DU4PR04MB10959:EE_ X-MS-Office365-Filtering-Correlation-Id: 644c137a-1640-4421-47ee-08dd02e519d5 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|366016|1800799024|52116014|38350700014; X-Microsoft-Antispam-Message-Info: =?utf-8?q?g7Ny28SEPdyXxFdPsEvn7LjDdFMHf2x?= =?utf-8?q?UXqk7WuyjtbNMTKm+FSdMsyhnGanZx6cK9F5PWmKNrmaSfo5AxvsR7O0zFJYWKEUv?= =?utf-8?q?+VGf40kIAGvsx2HKVK2gIOEOobHuEOrls+HkcFSEe/U1Ln4n+F68Ny/LU9ggW3pw1?= =?utf-8?q?VgABxkmKGijlcOG6pfDfmsDDaPpBliP/Mxd2Vb+ODaNpydpMvUj/c3c61/iO2V2Ob?= =?utf-8?q?oUm6SSChjp5gGjpmQzs7eMFDsexSTJzDbhNoZGWOHHFrtQRNlsWxkuHtZeCk5RirF?= =?utf-8?q?RBwPwCD5WhmJ321HxSYWo3tXZzgPW9XdSUC7965sx9j/LS6Yi4xkYvG6EHZmn/5dt?= =?utf-8?q?+PkUYUK5ea0UANeUSi2sZ+LK5iq422nGw1gxL6w1PwXS0f7Zld78FjKVf4iAEFoWh?= =?utf-8?q?szZkb9JrtHzrG1Aio+6Deo9hFzKsID8kB02tX26jYfjAGPK1iIWnLTTTvdISTIh96?= =?utf-8?q?CFcyFiSlC8X06t6zLBHXfnD2veP3Rdr9OjaPaHvHmYIbfol6HTWuuJ+MNLKcVIESl?= =?utf-8?q?xs+yYdJV3XohxVhs9LVR1RppHftW9O+9gUOcRnPXhgNo4mnr7XmaaWBo+dXb4Y2K5?= =?utf-8?q?zznM2dqaJUjCWIuELePt/bYLrNowLrVNNV2Pjjt5KlFb4N311477enISvJyjdwXlC?= =?utf-8?q?+5MJHSoZ+g0xhu/MkTmoXG3KX2yEKfo7Elln7E50hgosIlksBSFbo4MUNVhvXWHj7?= =?utf-8?q?4D10+cWxC2ngOa50wKPgm2rS0NP5wDpUDPy5xMqY+WyJY+GACFbEtKWJVwtPc1Uzw?= =?utf-8?q?1XpWyeDNd/AJrGlQSEoRRxly0poCr5QXm/QcZWTMTZMY0+8KWWc48tA8OvCnFEppB?= =?utf-8?q?iRSx2JS4TI08AD95AWfloudwksTi7xoIjgj4FHDRbIFnzCCQD/IJAIdOlJt/Pfl6/?= =?utf-8?q?IN+XVPq16OBzD8TPtSGy+tsv9C3ENkHeSiioHIN+yvAzTUZNob4xy7D5mVsWGPRO+?= =?utf-8?q?9UzTrb4cgW448Qa+5p9EvHKmP1nqOT0g5S8AcPTkgaI4PTvy3cCY1fxuKnqYPygFS?= =?utf-8?q?19lVEck+B5ZjUFOeTsGS+z7BQC14BKPd3ffDwj5Jok9Wbo+3ppeY23elfhlZVDz8X?= =?utf-8?q?PNv8JrDu3nRTow0AR0QWCSqnzW9RLw4vhd9NjHSnQpCHouuE+34RMVhl2EYy3g0qs?= =?utf-8?q?ZYGPpNDeatoHHwSjx2wsVamb1t6AYHXnCiUx2OBK9Mu1Ywi4a1vJ/TT5KBxxdTPhF?= =?utf-8?q?raSo1uBZyMQLSgeNBuE1Rcrg5MwbHEUsGQTpQkT3v5I4JjrvZSsbtTPs9hkfLdpyT?= =?utf-8?q?d23WtkkzlycjGmaTj+dPg9TOzGQw1OSYFxMGpvTxJ5xfP/LopBeFXZHMOA/gCMFK2?= =?utf-8?q?GzqUhGQWAlwB?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM9PR04MB7586.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(7416014)(366016)(1800799024)(52116014)(38350700014);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?bZjSNhM1U/r2DY7ly3Zx5oiXIn4k?= =?utf-8?q?NtkjRQNrXyvXCIVsgurHcQNIorccVb8la5ukCntD1mVN6vnDuxR6mRSri+jEmXtVR?= =?utf-8?q?9r8pIzK5+rCkXfSOwmayo2R3eHP4kjCqryV5XmCmvOuhOed6s0CZ7yCTM2BCh964+?= =?utf-8?q?uLbXe0WVvk7cRj161Uqn3v1BIy96pfPvmCF64LUCHU3kGPvvPp4FLbprELSB6tHbS?= =?utf-8?q?aEiYvCa3vaEAEjj6cu15ouB8BPC2qzsaLiBDAaboS1+nltttcq103DS75U2+tHZnp?= =?utf-8?q?I+zbhd3yrNBW4QIOWlPffBtnszD0lon+yk+jw8+J207A+RNZGHTnLGyod2naytJ3M?= =?utf-8?q?B59EctLHZw77uV56NRVblOxkeBK8J7mPATZpUdP6c9qIf2XDZX7qVpJdjKGo2GgZl?= =?utf-8?q?b5tGKpahcHI6TN3Cr6hO/99MQvNzVXhtVqBEzsKM+Im6/hWma9DsYu0lPwkZCLg6Q?= =?utf-8?q?LTfXKyWdZNlbfMdKkn7bLdiJhxFacnwdcsZ8oBBeySSS3BKDJB61aa3hXt1hjK88z?= =?utf-8?q?KJNpGwfxdFVS3hbZSo1Z+y0G62WC1IwCMvixDnFT6s+GxOayoPblZR+GG9hWkzS7H?= =?utf-8?q?BwgVdvL/z52wo1sSrfC1HemWuYfva7T+GfNRCTgHzJjwno4qWjVQSZKIN+rPL+tcU?= =?utf-8?q?/IGHJ8Qb3TZOrFjReA475UaYOmix8gA4IondBSkixjo9lGh88imMIPA0Hk8TUAMlx?= =?utf-8?q?UwzZeygDKXv8IgxoD1PaAgb9LhAr2KFEUGQHfWz2DFgnlQ5QPxJBDmKSK7Y6NNToC?= =?utf-8?q?kRpELFIgxEH2OC+0xB6mf6dGXJmq3y4iXK90Or3U5gc+QLE9sSuwUgxuPQod4fwMo?= =?utf-8?q?Wlv3gcbv7FuEaDvK/srQWchT257KwgyoVAFfIGIOm7QfhmTwU4DIEef0ITi2Q3nxc?= =?utf-8?q?fBWNEWveMy0ZrV433VPKrrQGoguYvZlbo1IRq0J3GDFwNJ0L7kU39IYqjcZrfdID0?= =?utf-8?q?18KacJX2m10il33DJsLKNFAy2z0GZydc+7QSaeJySA57otCLTOYizfbne/u8XdrZv?= =?utf-8?q?FsnAEBra85MVpzafi4Na1QqBDteV/aay+QlLI1vmF/oAGnsiFa4yfofRvVRTw0WSb?= =?utf-8?q?ugYm5cACazVYFZMbBW4jQ8ySpKHu7TylpMFIvObHOUJJEMY5dT6ctEKCoh98de0sH?= =?utf-8?q?0Esz30irpLvhPaguYulzyW3HtL52cNzBF3ulEogl4pZLKGAW/XDrXvr6A/m4AgOBZ?= =?utf-8?q?xHff5eLDbS5g2z1u6VYlujfr9MF6xf8N71/OWDb2ojNU/3blkLQQDHN9j2JCPOXNz?= =?utf-8?q?KH2zeeQqRiuk7g+2LKO2WtHzWa6Rh8LLdE9SKFBhgg9ZQBj+pNeATSWd/6TC66O4f?= =?utf-8?q?noVrZ6AJDOcgmHfxNezaT+stqVCiFDhfANmqxMUf0bnxmXmahUL/GnnPNr7YgxJ7s?= =?utf-8?q?iJrkWZXAu6iABSZVW5XDTOZPMjTYtlwvb2p2Bax7dIFg2pAniLZFya/Q/yzdpskoF?= =?utf-8?q?EcO5Br3kLb2cdpNv8mvJ6lJi23y5z/oQw5JkaomCFBQVg8zmsvT4QjqtZvuI2KpJg?= =?utf-8?q?ohrr8c6tdzl2?= X-OriginatorOrg: solid-run.com X-MS-Exchange-CrossTenant-Network-Message-Id: 644c137a-1640-4421-47ee-08dd02e519d5 X-MS-Exchange-CrossTenant-AuthSource: AM9PR04MB7586.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Nov 2024 06:41:56.2109 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a4a8aaf3-fd27-4e27-add2-604707ce5b82 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: NPr3qBfzFqX9EQecaPpEdygCg+ox2H8XkSmQ2yxG/IzwylFCLPLV5G92ulxwg4lg0Icq6ftS5Ln+cpPXLC6aXw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU4PR04MB10959 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241111_224206_452729_1E6BAB65 X-CRM114-Status: GOOD ( 11.07 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org The USB-3.1 overlay uses /delete-property/ to unset a boolean property on the usb controller limiting it to USB-2.0 by default. Overlays can not delete a property from the base dtb. Change the overlay to instead assign value "false" on the ti,usb2-only boolean property, which is of_property_read_bool now evaluates as false. Reported-by: Geert Uytterhoeven Closes: https://lore.kernel.org/linux-devicetree/CAMuHMdXTgpTnJ9U7egC2XjFXXNZ5uiY1O+WxNd6LPJW5Rs5KTw@mail.gmail.com Fixes: bbef42084cc1 ("arm64: dts: ti: hummingboard-t: add overlays for m.2 pci-e and usb-3") Signed-off-by: Josua Mayer --- arch/arm64/boot/dts/ti/k3-am642-hummingboard-t-usb3.dtso | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/ti/k3-am642-hummingboard-t-usb3.dtso b/arch/arm64/boot/dts/ti/k3-am642-hummingboard-t-usb3.dtso index ffcc3bd3c7bc5d47ce9926a95a13af3f61182a2b..3ed709ff11fea1ba07e180d442ce6fda6acaa2a3 100644 --- a/arch/arm64/boot/dts/ti/k3-am642-hummingboard-t-usb3.dtso +++ b/arch/arm64/boot/dts/ti/k3-am642-hummingboard-t-usb3.dtso @@ -34,7 +34,7 @@ &serdes_mux { }; &usbss0 { - /delete-property/ ti,usb2-only; + ti,usb2-only = "false"; }; &usb0 {