Received: by 2002:a05:6a10:9848:0:0:0:0 with SMTP id x8csp374721pxf; Wed, 7 Apr 2021 01:26:52 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwdHq1fgBkm7/5ShWjSQRJh1WU8uUmogjWHvu8T0fyRjwuC30C41nhb2WZ9NK8dpKWbd4N7 X-Received: by 2002:a05:6e02:1b01:: with SMTP id i1mr1906788ilv.253.1617784012540; Wed, 07 Apr 2021 01:26:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1617784012; cv=none; d=google.com; s=arc-20160816; b=jQCzegeBDA48wCO3m0bGNuTSKeHYhgmy9yUwmVX1BmpjGsXlLdV8rqKWKRjIbpnzs+ WFXe/mZGiHHpYeq+PzB0AJahn87VOLHI6srCMoacuMM+ckXnHRNGDBxx9tszZovmp8F9 aT4D+kEYijrA7joPpEGnzBZm4t3IOwyrBlK61OfTg77i7B3Ynbw1WXJ5+rlkJfyCbFvq w7fkFow8zcljhk+0ycub7jHSPrulvYdnXcuUE0lVVAFbRGhdZkD1uxGG5nUQKP7q56QQ 2B72RNPtaYqLOEYmDoujV9+aHsfcfaDiSwLVlRfPaKAH8mGP7jGoDqR/7axta6gfI0p4 Ww1A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:mail-followup-to:message-id:subject:cc:to:from:date :dkim-signature; bh=I/LJ3cUrwS9aB0C2I9++lGDwdqSdmaXTkVNzxYpVMVs=; b=W+hxXVSkCjwopumbUFX6GAwbEVLlLEGZ3MQMDEqp3rFctTqSOsw9mXLp42aui2NHML srinc40z/TW8p0e4xYxSALeXB8IfgIKr0ZGvWam/5J2SnzAVgxu9ujptPBCfx9i2PFUc IzvwaxdM65H6zfH+ER8Rd9Li9a/gq4rJmaV0mR2LRNzTxypgxH2WjWfBF6+aCGdEO2w1 DCbY0z+Obxl9mpmRr8do/E/ivkGb9BEzHVK13Lo9CmQ/bhcWObJRg30DH5FRNku524g3 dkQjpunlIa0uLjHhDN+2RVJlsOU8Exihf+4mc/PRbZ2dbZgWwVU4MhhpKsvFbp+3kCVX +RFw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=IRODlppE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id z17si20099874jat.122.2021.04.07.01.26.40; Wed, 07 Apr 2021 01:26:52 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=IRODlppE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S245129AbhDFUDD (ORCPT + 99 others); Tue, 6 Apr 2021 16:03:03 -0400 Received: from mail.kernel.org ([198.145.29.99]:32874 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234411AbhDFUDC (ORCPT ); Tue, 6 Apr 2021 16:03:02 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id BE8D46136A; Tue, 6 Apr 2021 20:02:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1617739373; bh=P889avgBFB0bi2ojrpin8Uk1g11A/55qILssYLXrdyg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=IRODlppE4u/hQX7UdelwAg6h8sw6WEQUc1R0QNj2frM7lFX0mSvgr2H5RChOeFcyx bLPu6AYe4TMHteN6XNRFn1oPcIC0hs1wru3W7DHLGHY95SVqve7EcF3TGQopxWDMnt agT93iaU+BIH8VNvb9Tn/BswbxoddGgVxMEh3Bl+dYRzw270mCgctFZQK9KY/0KpoX TIfqD6i2jKVQcZeTP4gtJIb2/GVjDAG2CyPTRuX3SL7bQcyLsMYjT+WHsxSAJzK6av aSpNZm6rRKMNzQqkOxDh9CmM6LmwD8NME6I/VGsYNPQlSTDdBPBwxxeLRc+2ZKemYO l2OksajcNsp9w== Date: Tue, 6 Apr 2021 22:02:48 +0200 From: Wolfram Sang To: Yicong Yang Cc: andriy.shevchenko@linux.intel.com, linux-i2c@vger.kernel.org, Sergey.Semin@baikalelectronics.ru, linux-kernel@vger.kernel.org, digetx@gmail.com, treding@nvidia.com, jarkko.nikula@linux.intel.com, rmk+kernel@armlinux.org.uk, song.bao.hua@hisilicon.com, john.garry@huawei.com, mika.westerberg@linux.intel.com, prime.zeng@huawei.com, linuxarm@huawei.com Subject: Re: [PATCH v6 3/5] i2c: add support for HiSilicon I2C controller Message-ID: <20210406200248.GH3122@kunai> Mail-Followup-To: Wolfram Sang , Yicong Yang , andriy.shevchenko@linux.intel.com, linux-i2c@vger.kernel.org, Sergey.Semin@baikalelectronics.ru, linux-kernel@vger.kernel.org, digetx@gmail.com, treding@nvidia.com, jarkko.nikula@linux.intel.com, rmk+kernel@armlinux.org.uk, song.bao.hua@hisilicon.com, john.garry@huawei.com, mika.westerberg@linux.intel.com, prime.zeng@huawei.com, linuxarm@huawei.com References: <1617197790-30627-1-git-send-email-yangyicong@hisilicon.com> <1617197790-30627-4-git-send-email-yangyicong@hisilicon.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="y9PDtDHaFrXNoMPU" Content-Disposition: inline In-Reply-To: <1617197790-30627-4-git-send-email-yangyicong@hisilicon.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --y9PDtDHaFrXNoMPU Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Only super minor stuff. Thanks to all the contributors and reviewers! > +#define HZ_PER_KHZ 1000 KHZ_PER_HZ? > + ret = devm_i2c_add_adapter(dev, adapter); > + if (ret) { > + dev_err(dev, "failed to add i2c adapter, ret = %d\n", ret); No need to print that. The core prints messages on failures. --y9PDtDHaFrXNoMPU Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmBsvmgACgkQFA3kzBSg KbZ0Nw//boFzeFDX9pJMYnU9ZpW5WHJO7lXahbT9MjrWEw9/pILqtOQvdbYuU5jk U+NWcQ3iyzI+9P0OqfZIub5MnqgMBbE/uEAoJt3+btv/OEw6sf5zux7wa0nEQAP+ JdXjaMNGKm7wP7LJv5GgStsNoDzHNrtyH18BfDYL8hHVJq5cvmFwI4JyK4Wj3tVJ MnHK0o+7HqxyDzgUZY6FU+/oJMCYW5WIE7JI7Gqa0EKcsc3HUfgZG3EvA0tnrv5K dG6552xRFDHNLGwdNsSnOEedRPIar0fjHvbHQwi6rE75r/FXO/8qzXITLMXU6q9X 8cY/U4vNxTl4GlMUnIZRioeA3MWTspAl/DWeeYaELc+uLdhEFzu9h2Rg6oDKV5St YGKblb5r8JCO+/59aSWqjiGQPzDETlhrXQe1zCUBHq7KEdH41sLGDsGEG4dug4Yp bz751h0LbzYhGL8Rzuu3aF+JbCNQQLC/Z7yjKTG+GzwGP2wnznGQTpejDwjEYzmE Qma/SnBZqMR1ulEor47mslRdBuS1/v1vljTGqv+n4hWrsuSmTwsCqXjsr8nDcgpQ 26puJE1padPiaba9XqpSIZUEpZMMIXRkjIbmKEtUQVU+8ddT2aaCJocY3yol2XMX FMCdTee++Ekcoaw1xG08zxwAlJeMtHa/I0+UhGVyZirqQ9WwV2k= =QjPO -----END PGP SIGNATURE----- --y9PDtDHaFrXNoMPU--