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 E84C2C0015E for ; Thu, 3 Aug 2023 06:35:16 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:CC:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=DrgxVKQf5QjSmq9vtSC9w4McWZ04u9uqwWXE1Rsk3ak=; b=bpHUTQFDlyDuPZ 1Dvi1TBllpDYrPW6P3MtmGzhb0/UC1hANgMtjH9nMHdYTplB1n0hsdHzmOB5d0yC5EvzLGCvI2719 //cRvlGT0LARaOagP/xBgcrZdPsW5LPUKaQeAVVe9ToUtLivnZkMKtk/CflWs8DTOqYqAv7C6kDDP pYSBGB7exQEiy2GYFExt5Wf8tfaJRQVzKh65WCJab0G4HGPzrd1MDX9ZLJKkBE8I0TYv8LHIrLKJ4 IxKnXty8BX3Ji/yAKIv3hZ+jmMzkTDhkp7YjxhEPobioD3d1CFjh1gDG8rXdTeybcVXmvyR8G9WBi 6dyx8LJct1T5TrT8IbAw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qRRv8-006mX4-08; Thu, 03 Aug 2023 06:34:50 +0000 Received: from lelv0142.ext.ti.com ([198.47.23.249]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qRRv4-006mWT-25 for linux-arm-kernel@lists.infradead.org; Thu, 03 Aug 2023 06:34:48 +0000 Received: from fllv0034.itg.ti.com ([10.64.40.246]) by lelv0142.ext.ti.com (8.15.2/8.15.2) with ESMTP id 3736YY05084557; Thu, 3 Aug 2023 01:34:34 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1691044474; bh=l/dJL8sYApap4cmNcA7wUrOOxRli0l2lUdTGCW+hBTI=; h=Date:Subject:To:CC:References:From:In-Reply-To; b=sOgsooCQodMlFzxrchLC7ojEei8pZwJ0w536P2/Bvd7/YAlvqiJ401BEBY1/coPJ4 bdhV8BqAPUrbJZROtkCrwTdZSTSKArIIbbmd9DibtkP7I0laH9+ReOT5Wq+3xAqoNX uV548Na94kRKVC3393CUD9lj3YNQeOIpgnrKAcR8= Received: from DFLE106.ent.ti.com (dfle106.ent.ti.com [10.64.6.27]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 3736YYSo127960 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Thu, 3 Aug 2023 01:34:34 -0500 Received: from DFLE104.ent.ti.com (10.64.6.25) by DFLE106.ent.ti.com (10.64.6.27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Thu, 3 Aug 2023 01:34:34 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DFLE104.ent.ti.com (10.64.6.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Thu, 3 Aug 2023 01:34:33 -0500 Received: from [10.249.128.142] (ileaxei01-snat.itg.ti.com [10.180.69.5]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 3736YRsM047295; Thu, 3 Aug 2023 01:34:28 -0500 Message-ID: Date: Thu, 3 Aug 2023 12:04:27 +0530 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.14.0 Subject: Re: [PATCH v3 1/8] arm64: dts: ti: k3-am62x-sk-common: Update main-i2c1 frequency Content-Language: en-US To: Nishanth Menon CC: Vignesh Raghavendra , Tero Kristo , Rob Herring , Krzysztof Kozlowski , Catalin Marinas , Conor Dooley , Devicetree List , Linux Kernel List , Linux ARM Kernel List , Tomi Valkeinen , Rahul T R , Devarsh Thakkar , Jai Luthra , Jayesh Choudhary References: <20230728173438.12995-1-a-bhatia1@ti.com> <20230728173438.12995-2-a-bhatia1@ti.com> <20230730163052.w5y5a25xgpxzuhzs@serving> From: Aradhya Bhatia In-Reply-To: <20230730163052.w5y5a25xgpxzuhzs@serving> X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230802_233446_791004_6DB0510D X-CRM114-Status: GOOD ( 25.10 ) 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: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Nishanth, On 30-Jul-23 22:00, Nishanth Menon wrote: > On 23:04-20230728, Aradhya Bhatia wrote: >> The Display Data Channel (DDC) transactions between an HDMI transmitter >> (SIL9022A in this case) and an HDMI monitor, occur at a maximum of >> 100KHz. That's the maximum supported frequency within DDC standards. >> >> While the SIL9022A can transact with the core at 400KHz, it needs to >> drop the frequency to 100KHz when communicating with the monitor, >> otherwise, the i2c controller times out and shows warning like this. >> >> [ 985.773431] omap_i2c 20010000.i2c: controller timed out >> >> That feature, however, has not been enabled in the SIL9022 driver. >> >> Since, dropping the frequency doesn't affect any other devices on the >> bus, drop the main-i2c1 frequency from 400KHz to 100KHz. >> >> Signed-off-by: Aradhya Bhatia >> --- >> arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi >> index 34c8ffc553ec..540ed8a0d7fb 100644 >> --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi >> +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi >> @@ -300,7 +300,7 @@ &main_i2c1 { >> status = "okay"; >> pinctrl-names = "default"; >> pinctrl-0 = <&main_i2c1_pins_default>; >> - clock-frequency = <400000>; >> + clock-frequency = <100000>; >> >> tlv320aic3106: audio-codec@1b { >> #sound-dai-cells = <0>; >> -- >> 2.40.1 >> > > I know we had issues with level shifters on the dev platform previously, > but do we need this patch anymore with the production boards? > > if so, use Fixes and I can pick for next. > This doesn't have anything to do with the level shifter issue we have had previously. The DDC communication between the monitor and the HDMI bridge requires that the bus frequency be 100KHz. Some monitors still end up supporting higher frequencies on higher quality HDMI cables, but to support a larger variety, we need to ensure that it doesn't exceed 100KHz. Hence, this is just as much applicable on the production boards. Since the capability of the SIL9022 to drop down the transaction frequency to 100KHz has not been enabled in the driver, I am changing the bus frequency for the whole i2c-1 bus. And this patch is not required if display is not being used, so I don't think we need to back-port to older versions via "Fixes" tag... unless I am missing a deeper point you were trying to make? If so, let me know, and I will post a new version with the said tag. Regards Aradhya _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel