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 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 5E52CC4332F for ; Tue, 12 Dec 2023 17:15:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=K+v0b4OG+ED8scGNjywt9t8Tm0WvJ8lq2SPgsGR3hAk=; b=BMnOEVAHnF7VOPfmkTO09ZJhIP DoF+/ACYpAGJ9x+i+dT65HYudydMxllrMwspY8EzKJPpvGMJGRX8BUN5kYlHrn86AH41gkQntXHd+ YUaJAR8QtVxKfRwrhbV/WHg5V9AVq4Ih2rRJg/CaRTfXy2TKy1J8NB323A61CmYhN8aK9zisXFc6R BlsemenasBK7GFl2V6wOk3EYDzDwSfbHyZhIZ0qRvTEsUqSd3HmOfRTP+3yaLN+it5XXlXciKXEUk xpCH4K1myYmcZA6zXOliiVAKSSZszQMCLPJG13WfPAeV17CRGs0i0hwnX7+7GjiW5BbiAox+Qs66l 7ayI2cPQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rD6LB-00CLTT-0k; Tue, 12 Dec 2023 17:14:41 +0000 Received: from ams.source.kernel.org ([145.40.68.75]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rD6L6-00CLRe-2N for linux-arm-kernel@lists.infradead.org; Tue, 12 Dec 2023 17:14:38 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by ams.source.kernel.org (Postfix) with ESMTP id 13DE2B817C8; Tue, 12 Dec 2023 17:14:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DCE9EC433C7; Tue, 12 Dec 2023 17:14:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702401274; bh=Zcj0fXIYj05PfXAXJRRaba5K0+vOckPktiL630dMYgs=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=YaQxsfHFiB42qftcR6JIjgF6HOa9mFE3f99F0/2J8URfdFND6pZ217M2SLq7InFZN oEFXu+Lf/wQ7YSvLx1w5Q9jzNBY/Qc1WVKretbOmlcoLt/Swv1ld9vz/dalKNU5pLm 9wIhqZnctSuPaMCfLkR+9uHaZDlOXJG9EvVhYP2OSGpSfmb47zwx4z2ebcCcSucmB9 48ntdXs609bTUl4lCqHP3vA13xXAsU2dALDh2vvOolorLHqJNKS+P1hcp9UVsSeuO9 K+8KE1iKh3+/3phhUlltwDyg+gKlHq8nTyD7OdMOGm2gHZ8Wiviu8N6UG2cwp1JY9S EwD7Iz17sNZyw== Date: Tue, 12 Dec 2023 17:14:26 +0000 From: Conor Dooley To: Ninad Palsule Cc: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, joel@jms.id.au, andrew@codeconstruct.com.au, peterhuewe@gmx.de, jarkko@kernel.org, jgg@ziepe.ca, keescook@chromium.org, tony.luck@intel.com, gpiccoli@igalia.com, johannes.holland@infineon.com, linux@roeck-us.net, broonie@kernel.org, patrick.rudolph@9elements.com, vincent@vtremblay.dev, peteryin.openbmc@gmail.com, lakshmiy@us.ibm.com, bhelgaas@google.com, naresh.solanki@9elements.com, alexander.stein@ew.tq-group.com, festevam@denx.de, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org, linux-integrity@vger.kernel.org, linux-hardening@vger.kernel.org, geissonator@yahoo.com Subject: Re: [PATCH v1 2/8] dt-bindings: tpm: Add schema for TIS I2C devices Message-ID: <20231212-amusement-elevation-28e42bcccc35@spud> References: <20231212164004.1683589-1-ninad@linux.ibm.com> <20231212164004.1683589-3-ninad@linux.ibm.com> MIME-Version: 1.0 In-Reply-To: <20231212164004.1683589-3-ninad@linux.ibm.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231212_091437_071310_758FDCD8 X-CRM114-Status: GOOD ( 19.98 ) 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: , Content-Type: multipart/mixed; boundary="===============1785894248891323024==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============1785894248891323024== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="2PG9iIB6csIKhjsS" Content-Disposition: inline --2PG9iIB6csIKhjsS Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hey, On Tue, Dec 12, 2023 at 10:39:58AM -0600, Ninad Palsule wrote: > From: Johannes Holland >=20 > Add a dt schema to support device tree bindings "Add bindings for..." > for the generic I2C > physical layer. Refer to the TCG PC Client Platform TPM Profile (PTP) > Specification for TPM 2.0 v1.04 Revision 14. >=20 > This includes descriptions for the Nuvoton and Infineon devices. >=20 > OpenBMC-Staging-Count: 3 I have no idea what this is, but it needs to be removed from the patch. > Signed-off-by: Johannes Holland > Signed-off-by: Joel Stanley > Signed-off-by: Ninad Palsule > --- > .../bindings/security/tpm/tpm-tis-i2c.yaml | 50 +++++++++++++++++++ > 1 file changed, 50 insertions(+) > create mode 100644 Documentation/devicetree/bindings/security/tpm/tpm-ti= s-i2c.yaml >=20 > diff --git a/Documentation/devicetree/bindings/security/tpm/tpm-tis-i2c.y= aml b/Documentation/devicetree/bindings/security/tpm/tpm-tis-i2c.yaml > new file mode 100644 > index 000000000000..de1e34065748 > --- /dev/null > +++ b/Documentation/devicetree/bindings/security/tpm/tpm-tis-i2c.yaml > @@ -0,0 +1,50 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/security/tpm/tpm-tis-i2c.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: I2C PTP based TPM Devices > + > +maintainers: > + - Johannes Holland > + > +description: > + Device Tree Bindings for I2C based Trusted Platform Module (TPM). s/Device Tree Bindings for //. Doesn't dt_binding_check now complain if you have this in a title or description? > +properties: > + $nodename: > + pattern: "^tpm(@[0-9a-f]+)?$" > + > + compatible: > + oneOf: > + - description: Infineon's Trusted Platform Module (TPM) (SLB9673). > + items: > + - const: infineon,slb9673 > + - const: tcg,tpm-tis-i2c > + - description: Nuvoton's Trusted Platform Module (TPM) (NPCT75x). > + items: > + - const: nuvoton,npct75x > + - const: tcg,tpm-tis-i2c > + - const: tcg,tpm-tis-i2c IMO this should be removed and this fallback should only be used in combination with device specific compatibles, like you have here for the infineon and nuvoton devices. Cheers, Conor. > + reg: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + i2c { > + #address-cells =3D <1>; > + #size-cells =3D <0>; > + > + tpm@2e { > + compatible =3D "nuvoton,npct75x", "tcg,tpm-tis-i2c"; > + reg =3D <0x2e>; > + }; > + }; > +... > --=20 > 2.39.2 >=20 --2PG9iIB6csIKhjsS Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZXiU8gAKCRB4tDGHoIJi 0rfJAP4nZ5EnLrc7JoF/x1QncshpSRifSSj5EXGcysTDO4Sr+wEAwPGS175s6OHs QOXCdCmeYeUbjJsKyPpDYvec+54oKgg= =kpn3 -----END PGP SIGNATURE----- --2PG9iIB6csIKhjsS-- --===============1785894248891323024== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============1785894248891323024==--