[{"id":3188877,"web_url":"http://patchwork.ozlabs.org/comment/3188877/","msgid":"<20230927-proofread-spoken-79f13b074a5f@spud>","list_archive_url":null,"date":"2023-09-27T15:05:28","subject":"Re: [PATCH 1/3] dt-bindings: i2c: mv64xxx: update bindings for\n unstuck register","submitter":{"id":84372,"url":"http://patchwork.ozlabs.org/api/people/84372/","name":"Conor Dooley","email":"conor@kernel.org"},"content":"On Wed, Sep 27, 2023 at 12:47:59PM +1300, Chris Packham wrote:\n> Some newer Marvell SoCs support an \"unstuck\" function for I2C bus\n> recovery. This is an alternative to the generic GPIO based recovery that\n> the older SoCs use. The unstuck register falls outside of the usual\n> address block for the I2C controller so requires an additional cell in\n> the register property. This is optional and does not need to be\n> supplied.\n> \n> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>\n\nAcked-by: Conor Dooley <conor.dooley@microchip.com>\n\nThanks,\nConor.\n> ---\n>  .../devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml         | 5 ++++-\n>  1 file changed, 4 insertions(+), 1 deletion(-)\n> \n> diff --git a/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml b/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml\n> index 984fc1ed3ec6..461d1c9ee3f7 100644\n> --- a/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml\n> +++ b/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml\n> @@ -45,7 +45,10 @@ properties:\n>        auto-detects this and sets it appropriately.\n>  \n>    reg:\n> -    maxItems: 1\n> +    minItems: 1\n> +    items:\n> +      - description: I2C controller registers\n> +      - description: I2C unstuck register\n>  \n>    interrupts:\n>      maxItems: 1\n> -- \n> 2.42.0\n> \n>","headers":{"Return-Path":"\n <devicetree+bounces-3906-incoming-dt=patchwork.ozlabs.org@vger.kernel.org>","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=jbH0OCrR;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3906-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4Rwg0K0kByz1ynX\n\tfor <incoming-dt@patchwork.ozlabs.org>;\n Thu, 28 Sep 2023 01:05:37 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id 5F9112819A6\n\tfor <incoming-dt@patchwork.ozlabs.org>; Wed, 27 Sep 2023 15:05:35 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 93B0936AFE;\n\tWed, 27 Sep 2023 15:05:33 +0000 (UTC)","from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org\n [10.30.226.201])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 69BA727EF9\n\tfor <devicetree@vger.kernel.org>; Wed, 27 Sep 2023 15:05:33 +0000 (UTC)","by smtp.kernel.org (Postfix) with ESMTPSA id CA17CC433CB;\n\tWed, 27 Sep 2023 15:05:30 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1695827132;\n\tbh=3DNgn9Pn+10uv+Th5OTDqc8kfv8Y/NmLhg9ZrOr/5ow=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=jbH0OCrR4ic5XiA0klM7mHBZ3BFHbZRhRWZzNqd0TwKJMaoo/m75k4goJX2XRCqQb\n\t iI7C1whDKeD9Nj2hR3gIGJJ45yxPweCXkUdozQDnEF51QTuyoQ8ScQ5UQG0Q/fV0ij\n\t 7ek6S/w+zSBjFMwIO+Vxbsd4FF/3YhIH2UxQ9Omitvq2zYs2XWA8PeT6eUh3RO76wD\n\t RM/ow2nxNdcPTZSEeLwjahN9cLaT/ALxzFwvhMdIWiWG+0hwpxQbMvrUzNljic0h4m\n\t 0SQ47iy4m+xYTwG/b6IFJfA0JAzb2QZ5tZFJxzJHz2NMKgO+sxnaLNbDrzAhyoolNT\n\t kRcTZn7y0qFHA==","Date":"Wed, 27 Sep 2023 16:05:28 +0100","From":"Conor Dooley <conor@kernel.org>","To":"Chris Packham <chris.packham@alliedtelesis.co.nz>","Cc":"gregory.clement@bootlin.com, andi.shyti@kernel.org, robh+dt@kernel.org,\n\tkrzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,\n\tpierre.gondois@arm.com, linux-i2c@vger.kernel.org,\n\tdevicetree@vger.kernel.org, linux-kernel@vger.kernel.org","Subject":"Re: [PATCH 1/3] dt-bindings: i2c: mv64xxx: update bindings for\n unstuck register","Message-ID":"<20230927-proofread-spoken-79f13b074a5f@spud>","References":"<20230926234801.4078042-1-chris.packham@alliedtelesis.co.nz>\n <20230926234801.4078042-2-chris.packham@alliedtelesis.co.nz>","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"<devicetree.vger.kernel.org>","List-Subscribe":"<mailto:devicetree+subscribe@vger.kernel.org>","List-Unsubscribe":"<mailto:devicetree+unsubscribe@vger.kernel.org>","MIME-Version":"1.0","Content-Type":"multipart/signed; micalg=pgp-sha256;\n\tprotocol=\"application/pgp-signature\"; boundary=\"yq9mmXrPR6AquKUP\"","Content-Disposition":"inline","In-Reply-To":"<20230926234801.4078042-2-chris.packham@alliedtelesis.co.nz>"}}]