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 886B7CCD1A5 for ; Wed, 22 Oct 2025 01:49:39 +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:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=hWJ+CXv+tvjRm8Xst5+IM905saN+WSrHjY0k/bvxfHM=; b=Ap61sRVeq2ltPN rdH6u2s7sNKqpGCkaYwswbQjdihLp6QB+px+1MOtSHycM6QE6zRTSvG1yGHR06YPkfBQii+QFPzEV YLIVToFFx+2JapBsT2ZM0NV2hrMOGGYFAHHSRKvCpBsjJxT7JqalcLA80kMVPB6s1ud9nlA0EbgKZ Cj6wOOxyguWu6eKOaYsgAb4WCq/2UAb7igqudo+vYIkipTtnfuDDE1IRMXauhPvW/rYH0tNDFzzO7 xJGxHELz+L6aM+1E9n6gXAL8e6Gi/ezmmUkBOeMNxvcc1GuTqigYInJv0g3t7sguIg8sl4ZfOmmUY vJ36YQlKRi+QLqywHoug==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vBNyQ-00000001B0R-1v2T; Wed, 22 Oct 2025 01:49:10 +0000 Received: from smtpbgau1.qq.com ([54.206.16.166]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vBNyN-00000001Azl-0VBL for linux-riscv@lists.infradead.org; Wed, 22 Oct 2025 01:49:09 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.spacemit.com; s=mxsw2412; t=1761097692; bh=X7vhDG495g1PdZKo8SLtLFS6XGZDbpBLt2YVQRk/xzU=; h=Date:From:To:Subject:Message-ID:MIME-Version; b=V0JH4R/40gAcKh+inDPGZmYd64cPK9KggsL9RFO83woRgMyGLnX4Yb6W8qsj9Byco mKRtA3Vg9+lu/95xHmjzEzGCnxyaA/75eFJO9daK33W2PA3PP0/Bs7lrsPmuwrEtVu kl65k5yOZ+0xMf5U0fmphiiT7m8hCIQnLCzAM3eE= X-QQ-mid: esmtpsz11t1761097685t6631861a X-QQ-Originating-IP: XeCO6qz22dmz9Hct7O9nhli4miv2nRud+MoWQJbKTto= Received: from = ( [14.123.255.147]) by bizesmtp.qq.com (ESMTP) with id ; Wed, 22 Oct 2025 09:48:04 +0800 (CST) X-QQ-SSF: 0000000000000000000000000000000 X-QQ-GoodBg: 0 X-BIZMAIL-ID: 87910848643870731 EX-QQ-RecipientCnt: 7 Date: Wed, 22 Oct 2025 09:48:04 +0800 From: Troy Mitchell To: Andi Shyti , Troy Mitchell Cc: Yixun Lan , linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, spacemit@lists.linux.dev Subject: Re: [PATCH v4] i2c: spacemit: configure ILCR for accurate SCL frequency Message-ID: <748544ABF6A72D31+aPg31PBvHDkBALoa@kernel.org> References: <20251017-k1-i2c-ilcr-v4-1-eed4903ecdb9@linux.spacemit.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-QQ-SENDSIZE: 520 Feedback-ID: esmtpsz:linux.spacemit.com:qybglogicsvrgz:qybglogicsvrgz3a-0 X-QQ-XMAILINFO: NDtUtlvFer7ve2K//w09cAR88dsVH4wGZELugfbnvP/F/FT2vXAdwHKf Rr5Z8b4KfgRc9OHxuHdwnZwfKVfU/uGmbdfac+DQhysUbqMWj3/e0dkCtvUJdCbIitBszoV dVQar2Qniz5Dbo3ckPTZSumd3+/Dr+rJEueQBXTMgkD2Q2mKdKevG5C8UEvUQhjKwFRzoTy gqzJ+BDAkFcmCp+1Y3KJoKKZ6XafpCdcKmvvN9Decj5WhxD6BmnreKn+/I9+/TDzGXji4dM /tLvcSvv/oBmB6uhdCEjMQhahabXSWnaHJold3SMU9TXbirVR6b1r1xUgriUIQjIcVqi11o vKT0RTF7kkE3bBuykyw0TKSSk/+P0aOcztZth6UgjOpC36VC/uqXR3zshSWbdsL1HM2nHpO rlb5T+40gegKMISEs6rw5gjGs6gUTIx+RRc10ztgXALYFCjxGlGt8k7a+aiISYDSwQITrs/ lvvPNH/qP89VsGSiZnT+/RCIW+0qM++UgN+0+bs+hXNT73fY8dYCXOueB6jaIod7Ln9z2LO fbofD1zt938vSvu8vvUEGK1iioWGco83rhR7e+GWisgcXhod+p1UtjHYXcw/hExzzWnJhy+ /fIV/JsEak9s4mN31zn3snE6sVonv79B2VAtevz1/qrIBgfxxhBvrO4h49oCXBgZl2cCScc RO96nn/Poujn2hT6Uro1wcjNl6OW6nseguJ7/QnVfeXGmYZGI4aeybayFsLXLb2aoCrlvxO BmIZ3PvXAEhRe9iesn93SQZD8qm/aMBAFQ7FxybsAIxJ7ufhaax3aRioQjEOLOYdHu/7oXp KinojmAuy7kfZcMtWzSGc72AJSb29J06hkR3UnV1xFc/mcEJmUDK0vr4b7hnka2ZtQHdsqS YqX7bbTgcIb2MHD18fELke9G9rOccqSF1laE/l1KFlD7uZ65vyPCsvrKdwIZpKqiBwyJXNU c6Q3bCNCv66Vn1ZkUlgQtQQQfS9E77qY61+dIYJ1lyD9UUF6/7fCQ35h+Y2ft33/4FImR5i NH5852JbblGtvMUhInuJoV1giZRETcyKv4HJXf+ZdAPnw/rAmQ X-QQ-XMRINFO: MSVp+SPm3vtS1Vd6Y4Mggwc= X-QQ-RECHKSPAM: 0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251021_184908_183615_7B70E5C3 X-CRM114-Status: GOOD ( 21.36 ) X-BeenThere: linux-riscv@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-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org Hi Andi, On Tue, Oct 21, 2025 at 06:03:34PM +0200, Andi Shyti wrote: > Hi Troy, > > On Mon, Oct 20, 2025 at 11:28:45AM +0200, Andi Shyti wrote: > > On Fri, Oct 17, 2025 at 03:27:39PM +0800, Troy Mitchell wrote: > > > The SpacemiT I2C controller's SCL (Serial Clock Line) frequency for > > > master mode operations is determined by the ILCR (I2C Load Count Register). > > > Previously, the driver relied on the hardware's reset default > > > values for this register. > > > > > > The hardware's default ILCR values (SLV=0x156, FLV=0x5d) yield SCL > > > frequencies lower than intended. For example, with the default > > > 31.5 MHz input clock, these default settings result in an SCL > > > frequency of approximately 93 kHz (standard mode) when targeting 100 kHz, > > > and approximately 338 kHz (fast mode) when targeting 400 kHz. > > > These frequencies are below the 100 kHz/400 kHz nominal speeds. > > > > > > This patch integrates the SCL frequency management into > > > the Common Clock Framework (CCF). Specifically, the ILCR register, > > > which acts as a frequency divider for the SCL clock, is now registered > > > as a managed clock (scl_clk) within the CCF. > > > > > > This patch also cleans up unnecessary whitespace > > > in the included header files. > > > > > > Signed-off-by: Troy Mitchell > > > > merged to i2c/i2c-host. > > I'm sorry, because of the report from LKP(*) I reverted your > patch. I2C_K1 is selected by MFD_SPACEMIT_P1, so that we get the > following warning: > > WARNING: unmet direct dependencies detected for I2C_K1 > > and compile I2C_K1 without COMMON_CLK. Please, fix it and > resubmit the patch. Yes, I noticed that too. The issue is introduced by PMIC driver(P1) instead of I2C. Should I resubmit this patch right after sending the fix, or wait until the fix gets merged first? - Troy > > Thanks, > Andi > > (*) https://lore.kernel.org/oe-kbuild-all/202510202150.2qXd8e7Y-lkp@intel.com/ > _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv