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 smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 D0725C433F5 for ; Tue, 12 Apr 2022 09:05:22 +0000 (UTC) Received: by smtp.kernel.org (Postfix) id 9AC0CC385A5; Tue, 12 Apr 2022 09:05:22 +0000 (UTC) Received: from mail-ed1-f48.google.com (mail-ed1-f48.google.com [209.85.208.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.kernel.org (Postfix) with ESMTPS id D49D9C385AB; Tue, 12 Apr 2022 09:05:21 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 smtp.kernel.org D49D9C385AB Authentication-Results: smtp.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.kernel.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-ed1-f48.google.com with SMTP id d10so21616014edj.0; Tue, 12 Apr 2022 02:05:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=JvUzj4eJQSW6VlbLUjMSITUiq97FPtxDr7ncwYmoRZw=; b=PQN4pGhpOlIZoayWPrUBzGowazBqMXrd4jRr1snIyitef4NjsLhheTAe0qQ4xxS53b 3Bt6MF+gTSAJvT6Dr3An/eqtqMwA/5zAkSdx9c7loYNVNFh9WM+LFbECAWMLVRdG5mYk EHaD8+YmvSfCswWysoOwVBSMt1FW0TYI80ssVuaKlsX40TIA1HRDORT8I2xQLJg1MiMs Yx/pt7YehaJjEiMKaTBxBkXZNXuWm38BnB890ruRydAj6g6KeFCaBmrdhEULfmwVXtvW JuXDdKRrxlc7wpyYmrYqigcamJk84SaFG6J8AQDlYbDZrY894fzLN+jF/gyZI2J22cJA G+3g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=JvUzj4eJQSW6VlbLUjMSITUiq97FPtxDr7ncwYmoRZw=; b=XZOOMtwzV9YhBG/Ch+AkrLBDb4tjzI486jr7e1d7TksDQhobFNzGwH9GRCHzK+hyC2 0NjFQJAmfkWD17AYvmZ8sElDy7ho9IdJM3qoSRrwN68WyMcqGqgUW5m4h+ypjqm3Wz0W bUmg/buPsiyEVmVa2x2RnkUeD9OKV+5cWlq9wLH86PM/mjFVOkFzQMHa7zeZ7CLQXFGF 67eiJRC+VRwOEqyZug7MSljuC91aUwRHKyuaXf1cDP5GPulTWYFlj85bMiluTKU6RUkv mn2bNVvbmRhvzT5FBURZ1sNJBVZYb6b5n1Dln9H4BUPCZcoGs8LatauIRf0mImTxXb9x pOZQ== X-Gm-Message-State: AOAM533SvqYREL0Q3cNtM0rb3EOuZq3cxp+VIQSOJ94lBs/YDE4+p0Hv zdcBhZjVCRBVIzd5xZ4WQ6NFpFEKifc= X-Google-Smtp-Source: ABdhPJy7UIlkAw90AvWa8CNYC9UtMZpU5M8mz2CPjgf62C2T/CEb2v4+GAYAJGpj0vByS1chiMtK+A== X-Received: by 2002:a05:6402:27d1:b0:419:1b02:4a04 with SMTP id c17-20020a05640227d100b004191b024a04mr37474442ede.218.1649754319553; Tue, 12 Apr 2022 02:05:19 -0700 (PDT) Received: from orome (pd9e518f7.dip0.t-ipconnect.de. [217.229.24.247]) by smtp.gmail.com with ESMTPSA id d26-20020a1709064c5a00b006e89debe3f4sm1595260ejw.9.2022.04.12.02.05.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Apr 2022 02:05:18 -0700 (PDT) Date: Tue, 12 Apr 2022 11:05:15 +0200 From: Thierry Reding To: Rob Herring , Arnd Bergmann List-Id: Cc: Krzysztof Kozlowski , Jonathan Hunter , Bjorn Andersson , Avi Fishman , Tomer Maimon , Tali Perry , Patrick Venture , Nancy Yuen , Benjamin Fair , Maxime Coquelin , Alexandre Torgue , linux-stm32@st-md-mailman.stormreply.com, soc@kernel.org, devicetree@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RESEND PATCH] arm64: dts: tegra: Fix boolean properties with values Message-ID: References: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="NBsBIpgqCfiMExLJ" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/2.2.1 (c8109e14) (2022-02-19) --NBsBIpgqCfiMExLJ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Apr 06, 2022 at 02:17:30PM -0500, Rob Herring wrote: > Boolean properties in DT are present or not present and don't take a valu= e. > A property such as 'foo =3D <0>;' evaluated to true. IOW, the value doesn= 't > matter. >=20 > It may have been intended that 0 values are false, but there is no change > in behavior with this patch. >=20 > Signed-off-by: Rob Herring > --- > Can someone apply this for 5.18. >=20 > arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi | 8 ++++---- > .../boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts | 8 ++++---- > arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi | 6 +++--- > arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi | 6 +++--- > arch/arm64/boot/dts/nvidia/tegra210-p2180.dtsi | 6 +++--- > arch/arm64/boot/dts/nvidia/tegra210-p2894.dtsi | 8 ++++---- > arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts | 8 ++++---- > arch/arm64/boot/dts/nvidia/tegra210-smaug.dts | 4 ++-- > 8 files changed, 27 insertions(+), 27 deletions(-) This causes multiple regressions on Tegra boards. The reason for this is that these properties are not in fact boolean, despite what the DT bindings say. If you look at the code that handles these, you'll notice that they are single-cell properties, typically with <0> and <1> values. What may have led to the conclusion that these are boolean is that there is also a special case where these can be left out, but the meaning of that is not the "false" (<0>) value. Instead, leaving these out means that the values should be left at whatever is currently in the register. See pinconf_generic_parse_dt_config() and parse_dt_cfg() specifically in drivers/pinctrl/pinconf-generic.c. Arnd, can you please revert this so that these boards can be unbroken? It's a bit unfortunate because there seem to be other platforms that also employ these in the boolean form that Rob mentioned, but I think it is those that probably need fixing instead. Not sure what the intentions were for those. Adding Bjorn for MSM, the Nuvoton and STM32 folks. Thierry --NBsBIpgqCfiMExLJ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmJVQMgACgkQ3SOs138+ s6FAcQ//f9+PoHjpquYwfy+qc69MSz549E+MT1SBelK6GGDpNc7CWrtmujpeiTP/ bgA1e0TzCagra2528fAUyOtyAaVKPHS8BVt6OxptgyEgvwP2fygn83+2BIQsVar1 Ls092mvWt1GeHYri19DRt3L43JH1RlcYxaPsUbgWU2tucetEtEDndOVUki87oclX se/tsVqGRA5B1FMFHPiVgJyTaFzJtiwkCZnH31BN3DRXSsOnkJr33fV5bQM4Ylg5 vxWmwURr1T7+vTgUSCxi2RNzYGAuveI/IGFs1QD/jddo8MEHqF/EHmGmPWHF5fta iP3T6u7bSb0y433imp3GDtLFch/cJ4BZ9sIWfLqPXP06sVVMfbLq2joo0p26XEZ1 q7hxFhNrpWBpmEna+RtNQWLCbcW7BBI+v0f4VYml+xNVpYehqUfHwrOqLdURucPO WbriU9+nODw81tLTJbSr2EGGvYi4uiGBZw302ZuApk2pOhF+Az9WnlxUHiM3Q64D WFubEG7Ezq1lEFO4hD4zZs9bMhSycTi6LMIJRTz7HHH/qPubJ8DWRi/Npn7WjN5S uzQyjmD9ei7E/jLb28Sa0TCk4zAKiZiSVWZF/ONXhJ4aGzkgLhebFgMlGOihj4Gj 4J3cLi4fOm2o4qU7axaSMb8yd6hAD93kKRIylehK0Lfdmt9pM4I= =kKPk -----END PGP SIGNATURE----- --NBsBIpgqCfiMExLJ--