From patchwork Fri Jul 16 16:07:30 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kevin Hilman X-Patchwork-Id: 12382529 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-17.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2A54EC12002 for ; Fri, 16 Jul 2021 16:09:43 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id EE943611AB for ; Fri, 16 Jul 2021 16:09:42 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org EE943611AB Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=jYmN0qhdu4BiBQO6FfqMY3vO4siFnfeUFKwFW72y3KU=; b=h36+cmFci02Cnn jso597ECz34i9owbN0+Vz9HsAxU3fHLQHSOVOLMxjpPG8LRYiwS5pdl41UqaT1BzKTN0m/h5/lxRB 54FXR79d2hgzglZB5AddmzS0zn9+JPytod+GSnAFCqQMvMzVTHp3jEsTxVFiRxKR4+JeJZNeG4MjA t0pEtZ4M7DNmNfgvlqSBepD+GFjO3O6Y7564jfx1f8P9f/ta+xvJWf7e2JVYoDZG0bR+gau/f5IjC qY/SEmc8YB4BThS88oS8nh6hnOhCwQw9C+JmDnPrDXpujjGuEsqPE0nlNCqKaG+JVOogPo8zC0j3N ihbZ2LyobXBBR0rAACVA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m4QNF-004oFA-Df; Fri, 16 Jul 2021 16:07:37 +0000 Received: from mail-pg1-x532.google.com ([2607:f8b0:4864:20::532]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m4QNB-004oEY-B7 for linux-arm-kernel@lists.infradead.org; Fri, 16 Jul 2021 16:07:35 +0000 Received: by mail-pg1-x532.google.com with SMTP id y4so10365416pgl.10 for ; Fri, 16 Jul 2021 09:07:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=8awA/oTYaOk9R8z60joH+KUUc5uv6dZerurOLNP4+S4=; b=liXwip506zuoTFBCjOu8+ZhZ1FQDTt7H3Sbdlpq4PrpNxdqD7r3p/HPIUnAvCjDR6X ed0Tf8sIzvT3fQQgaCioAxp7uCMCpQIuBifJcMbk3nxL38cKd3xjNV6CqfR852//FKjB wGjjnMm5mift5Gqwh3K8/s+xJtC5QVgZ1UO8CL0vFw32Xe7YwSFVVe3eSesxs4SGp7IA mAPPr18BR94iPBBROBJPAT4VSKvcJy6P/wOmEkSwkiEgsVkvfy0/QpqBo6hhzOy1Zw1R eIesXVBekObGHQ85qISk8b8/gC2v4vlaDQrvBs3QiKLEQr+2tTMWTsz7DRFliZvcL4SF WUZg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=8awA/oTYaOk9R8z60joH+KUUc5uv6dZerurOLNP4+S4=; b=jATuS2W+AXHrJ0iKX7EuqxKxa406VfdkkqOwC8LWtF7WbK5pHXolPhyt05fxAZmBMZ 34S5ulY61kn9Zy1DOhXA5Kb0mrQ2cLj4F+ZHKZMAIFOtBd1eWJHMfv+U1bAc66bPRGso wV3gHtFS9wRiamTfNan6fBO/R5kdCU2Q229wnrhLmeEyAFzr33NTqkIGDiAaZEBl4uY7 VFGksXHR9J9zKjocI2QkfqRhoxbDk7TEF8Mco8sYjkDFDhPGmclIlnQB9gMyh4pMIIm7 GZftq5mr+zX81XL4eattYJ4/oWMh39dGxIb3e6eRnG5Aee2+aJaujsLezKIesEyh/bGa z5Tw== X-Gm-Message-State: AOAM5312xcyXTauvuO9rIeAXsdnhWvc2ll0FZZGq39Fb/s02rrcdS02T D4vwRPh99uvKj6OvZyqx9fjWiQ== X-Google-Smtp-Source: ABdhPJwC3/3v4i+N5JJuKFVDcWQnN6glwV6CD9Ij0poHpcB9jv4bGLK+udik2Gdt5iemkoNzI+s1dA== X-Received: by 2002:a63:a441:: with SMTP id c1mr5087596pgp.59.1626451651890; Fri, 16 Jul 2021 09:07:31 -0700 (PDT) Received: from localhost ([2601:602:9200:1465:da9d:67ff:fec6:ee6b]) by smtp.gmail.com with ESMTPSA id mu9sm10921489pjb.26.2021.07.16.09.07.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 16 Jul 2021 09:07:31 -0700 (PDT) From: Kevin Hilman To: Tony Lindgren , linux-omap@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org, Dave Gerlach Subject: [PATCH] ARM: dts: am43x-epos-evm: Reduce i2c0 bus speed for tps65218 Date: Fri, 16 Jul 2021 09:07:30 -0700 Message-Id: <20210716160730.30547-1-khilman@baylibre.com> X-Mailer: git-send-email 2.31.1 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210716_090733_475710_90959CB1 X-CRM114-Status: GOOD ( 15.60 ) 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 From: Dave Gerlach Based on the latest timing specifications for the TPS65218 from the data sheet, http://www.ti.com/lit/ds/symlink/tps65218.pdf, document SLDS206 from November 2014, we must change the i2c bus speed to better fit within the minimum high SCL time required for proper i2c transfer. When running at 400khz, measurements show that SCL spends 0.8125 uS/1.666 uS high/low which violates the requirement for minimum high period of SCL provided in datasheet Table 7.6 which is 1 uS. Switching to 100khz gives us 5 uS/5 uS high/low which both fall above the minimum given values for 100 khz, 4.0 uS/4.7 uS high/low. Without this patch occasionally a voltage set operation from the kernel will appear to have worked but the actual voltage reflected on the PMIC will not have updated, causing problems especially with cpufreq that may update to a higher OPP without actually raising the voltage on DCDC2, leading to a hang. Signed-off-by: Dave Gerlach Signed-off-by: Kevin Hilman --- arch/arm/boot/dts/am43x-epos-evm.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts index aae0af10a5b1..2aa75abf85a9 100644 --- a/arch/arm/boot/dts/am43x-epos-evm.dts +++ b/arch/arm/boot/dts/am43x-epos-evm.dts @@ -582,7 +582,7 @@ &i2c0 { status = "okay"; pinctrl-names = "default"; pinctrl-0 = <&i2c0_pins>; - clock-frequency = <400000>; + clock-frequency = <100000>; tps65218: tps65218@24 { reg = <0x24>;