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 9B6B0C41535 for ; Tue, 19 Dec 2023 16:56:53 +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-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-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Oezvvi8VrH9W0j1QMFUnSbDJWDZhMEv82dBTDvY7NZM=; b=fEdceDSXpBaWqp0I1XuA9X3oTZ eaz7tZTazKAHhB4DlMuGaM0JGVkxo0+FrUWKaJ2C7iFY4MeVgrwanWXx3jBYPjJKSWTRw9WbzKQ3X DWmQcVOwUsksMLX+A/ASanpMeJ3w99Di/9qBjtVr8RavZTSV63Yjg4g7ZqnyVkmHV2qyxmaf4aOd6 nd7muNwq2SN9vyFvZ3nWxQNAEfraVcAkr2n8MlAhuO/aMq04f0haqzaEaYkT6JbLjhQqXMwTVTfZ3 Ivv4TDtR0auuU9MSrO+Y6adYSj0RjfDK37U6xxuAE/trmF8wN6DoC/KIKMPUaSnwyd1afPZJ7WdVG MSltXYUw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rFdOL-00EmyV-0n; Tue, 19 Dec 2023 16:56:25 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rFdOI-00Emy5-2u for linux-arm-kernel@lists.infradead.org; Tue, 19 Dec 2023 16:56:24 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 5837161384; Tue, 19 Dec 2023 16:56:22 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6A844C433C7; Tue, 19 Dec 2023 16:56:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1703004982; bh=SnNbmDtLT6+P7umkLGp9NLZ5G92Et4onaiBrbSI3QD0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=q2NXajxRRGfNKgOLAAfz1w39X8+DhtJYo/8mOfrDLM/9sgTL03YyZfJzkjgwyi0GA vmToktwSwBKwa2/+uHQJ64+DNw1olIs53XeY7O1x9jLB4RP5VOtboG4T1jc3NISeIb l/StNTxLt1g20JbyiKv2Agch67Fd8sC6FyuiPXVhuHuGjyFmG7SPhfJjs2BAdkniTm LF1xmgOHSy49uOjzUvYknMpO7P/AlY264W86aKltQ0kU/2bRMhk11wGMAw1k79iBfZ Hz2y5y0dDoYS1g4InONV/+gqQb1tpvuM3JAa76Mt/tmO441OGRIm/XZVDhYEHYHSva OC0vnr/fEJ52w== Date: Tue, 19 Dec 2023 17:56:15 +0100 From: Wolfram Sang To: Alexander Stein Cc: Andi Shyti , Dong Aisheng , Shawn Guo , Sascha Hauer , Fabio Estevam , Alexander Sverdlin , Pengutronix Kernel Team , NXP Linux Team , linux-i2c@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v7 1/1] i2c: lpi2c: use clk notifier for rate changes Message-ID: Mail-Followup-To: Wolfram Sang , Alexander Stein , Andi Shyti , Dong Aisheng , Shawn Guo , Sascha Hauer , Fabio Estevam , Alexander Sverdlin , Pengutronix Kernel Team , NXP Linux Team , linux-i2c@vger.kernel.org, linux-arm-kernel@lists.infradead.org References: <20231107141201.623482-1-alexander.stein@ew.tq-group.com> <20231109091046.4hrvxr7g5imfrykq@zenone.zhora.eu> <20231110122720.cyxtnpj5k6bip3ok@zenone.zhora.eu> <2912069.e9J7NaK4W3@steina-w> MIME-Version: 1.0 In-Reply-To: <2912069.e9J7NaK4W3@steina-w> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231219_085623_019936_9B38B6BB X-CRM114-Status: GOOD ( 14.41 ) 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: multipart/mixed; boundary="===============0840330017176173109==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============0840330017176173109== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="kHQ9lrHlsJlBeH8u" Content-Disposition: inline --kHQ9lrHlsJlBeH8u Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, > > ... improve the commit message, reporting the real deadlock case > > instead of a lockdep warning and... >=20 > I've improved the commit message about an actual deadlock. That means a v8 is in your queue? > > ... find a more appropriate error number, I will ack this patch. >=20 > Thinking about this again, I think EINVAL is an appropriate error code. > The parent clock frequency is also an input for the i2c transfer. So if, = for=20 > whatever reason, that clock frequency is 0, it is an invalid value (argum= ent). > I've checked other drivers what they do if that clock is 0. Unfortunately= most=20 > don't consider this case at all. But some do, so e.g. i2c_lpc2k_probe() o= r=20 > dc_i2c_init_hw() both return EINVAL if the clk or a calculated divider is= 0. IMHO, the return value doesn't matter that much as we have a descriptive text accompanying it anyhow. Happy hacking, Wolfram --kHQ9lrHlsJlBeH8u Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmWByysACgkQFA3kzBSg KbaYxg/+LA9ZHucDgFYv8gSb8Hy4hMfyzQgtPSCwkI8Goqa2nXbBE4EnYL6NrHs3 glg9RUu0AJYQ/RFZbGEj/Y5ZdY9xNMGT2uJ+DOrtwn8M1nHqUHlWacghEFvR1mtW aQ/njOWc+H8Z0Sl5wI1FXLk6U0jsQgh3slk11DINwHgvaVvm3tc9nKCZnXl/9FIu xAI98VwF71jXLLT6aITWPFVwJu3LV74BUaK3LYsax071g1uZIhJRCebNBwoHjx+7 Abysd+RHOSn/knlkEk1lTtPRq67cefmePyrf28lgMfn/UynLjfklvV/Rn4oNIk1T evKkZT6wMa/KCowJFSqKzdxc2/iQV9EsDob+c6lPZ1tsDyDX+cP4nY9/qf1AcQfP JkuKOoOmA3lnXHuHN6s6r/uowv9vJZiiw9l5Y3KJ+kC57aa1Wa7zRVWTYB4Hbqj3 Tmd2ing3xGzV3/jbOFsaNltjK6pWro76bI720CPPDBwp44p7W3+cyuaXxWousTDj uUluTKqjRGOu9SEECG7ir4y+N5o8/vimwS6UJlfbTdFWMMGKOPIUDAt+j4+o1RMH ibrX2S1nUw8Aps/TfTLp0ZSKjKfwh6ubqN3SanESnNVGKkLI17YGwqSQIMN1lwEe evryrutbmlS/MmYWjWW8hGK2GV4u8Vt1hX0DxIrq6P9dZSvlXJ0= =c1MU -----END PGP SIGNATURE----- --kHQ9lrHlsJlBeH8u-- --===============0840330017176173109== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============0840330017176173109==--