[{"id":1776843,"web_url":"http://patchwork.ozlabs.org/comment/1776843/","msgid":"<87wp4jgssp.fsf@free-electrons.com>","list_archive_url":null,"date":"2017-09-28T07:56:38","subject":"Re: [PATCH 2/2] dt-bindings: i2c: Update documentation for\n\t\"mv78230-a0-i2c\"","submitter":{"id":15771,"url":"http://patchwork.ozlabs.org/api/people/15771/","name":"Gregory CLEMENT","email":"gregory.clement@free-electrons.com"},"content":"Hi Kalyan,\n \n On jeu., sept. 28 2017, Kalyan Kinthada wrote:\n\n> This commit modifies the documentation for\n> \"marvell,mv78230-a0-i2c\" compatible string.\n>\n> The \"marvell,mv78230-a0-i2c\" compatible string enables the workaround\n> for an i2c repeated start timing violation, but unlike\n> \"marvell,mv78230-i2c\" it disables the i2c offload support. This is\n> applicable to a number of SoCs such as early revisions of the Armada XP\n> and the Armada 38x.\n>\n> Deleted the special note for \"marvell,mv78230-a0-i2c\" as it can\n> be used by other SoCs as well.\n>\n> Signed-off-by: Kalyan Kinthada \n> ---\n> Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt | 4 ----\n> 1 file changed, 4 deletions(-)\n>\n> diff --git a/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt b/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt\n> index 5c30026921ae..37c00ad1c0bb 100644\n> --- a/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt\n> +++ b/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt\n> @@ -10,10 +10,6 @@ Required properties :\n> - \"marvell,mv64xxx-i2c\"\n> - \"marvell,mv78230-i2c\"\n> - \"marvell,mv78230-a0-i2c\"\n> - * Note: Only use \"marvell,mv78230-a0-i2c\" for a\n> - very rare, initial version of the SoC which\n> - had broken offload support. Linux\n> - auto-detects this and sets it appropriately.\n\nI think here we should pour more details like what you wrote in the\ncommit log. Maybe something like:\n\n - \"marvell,mv78230-a0-i2c\": for controller which\n do not support offload but still have the i2c\n repeated start timing violation issue, such as\n the ones found in the Armada XP A0 and the\n Armada 38x SoCs.\n\n\nAnd I still have to check it for Armada 375 and Armada 39x, but of\ncourse it is not preventing merging your patches.\n\nGregory\n\n\n\n\n> - interrupts : The interrupt number\n> \n> Optional properties :\n> -- \n> 2.14.1\n>","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"sgklafCN\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2n8j3htxz9s9Y\n\tfor ;\n\tThu, 28 Sep 2017 17:57:09 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxTgt-0003KZ-IG; Thu, 28 Sep 2017 07:57:03 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxTgp-0002s9-P1 for linux-arm-kernel@lists.infradead.org;\n\tThu, 28 Sep 2017 07:57:02 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid A966620928; Thu, 28 Sep 2017 09:56:37 +0200 (CEST)","from localhost (242.171.71.37.rev.sfr.net [37.71.171.242])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id 7DA9D20926;\n\tThu, 28 Sep 2017 09:56:37 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:In-Reply-To:\n\tDate:References:Subject:To:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=4iiFiQI4NfGgItX/0QbwjnMhzoiEgvefNGZJF35Ud/o=;\n\tb=sgklafCNR9BPNG\n\tLTQEMzdsNDt5zBtJCAojNdng0thZsTTz0JKrKzbUBIJHmMdNUta2sxGG2zReaAcQeV1opLcS6AdR2\n\tnpHuRwJcq5zljMUeVaNguzToj14dDo13TdV1HKQeU4gZAL5PvHnErnvC1WZO0PWzj853J/IpVuRMZ\n\t478X4kUeKKrFgqj6+jx2qFdGry009Yw4r5/5BZtRJB599yzj2SqBgueoGBm/fDnMRBmdtQfSxFECu\n\tLyIVbORhpHoRNuA2xl8JOLFj7/uayU57m8xhPHehx6WjR2bY6L20ef3F9tkFahoFtYzw0XsEDmqm4\n\tEoF+wapk60Hrz0j1TUfA==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT,\n\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","From":"Gregory CLEMENT ","To":"Kalyan Kinthada ","Subject":"Re: [PATCH 2/2] dt-bindings: i2c: Update documentation for\n\t\"mv78230-a0-i2c\"","References":"<20170928040407.8933-1-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20170928040407.8933-3-kalyan.kinthada@alliedtelesis.co.nz>","Date":"Thu, 28 Sep 2017 09:56:38 +0200","In-Reply-To":"<20170928040407.8933-3-kalyan.kinthada@alliedtelesis.co.nz>\n\t(Kalyan Kinthada's message of \"Thu, 28 Sep 2017 17:04:07 +1300\")","Message-ID":"<87wp4jgssp.fsf@free-electrons.com>","User-Agent":"Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170928_005659_968429_91E206ED ","X-CRM114-Status":"GOOD ( 15.75 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, andrew@lunn.ch, jason@lakedaemon.net,\n\twsa@the-dreams.de, linux@armlinux.org.uk, linux-kernel@vger.kernel.org,\n\tdevicetree@vger.kernel.org, robh+dt@kernel.org,\n\tlinux-i2c@vger.kernel.org, \n\tChris Packham ,\n\tlinux-arm-kernel@lists.infradead.org, sebastian.hesselbarth@gmail.com","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]