Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Date:   Mon, 23 Oct 2023 17:12:40 +0200
From:   Wolfram Sang <wsa@kernel.org>
To:     Camel Guo <camel.guo@axis.com>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Alim Akhtar <alim.akhtar@samsung.com>,
        linux-i2c@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
        kernel@axis.com
Subject: Re: [PATCH] i2c: exynos5: Calculate t_scl_l, t_scl_h according to
 i2c spec
Message-ID: <ZTaNaH8qo3Ho+yvL@ninjato>
Mail-Followup-To: Wolfram Sang <wsa@kernel.org>,
        Camel Guo <camel.guo@axis.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Alim Akhtar <alim.akhtar@samsung.com>, linux-i2c@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
        kernel@axis.com
References: <20220912085943.1098651-1-camel.guo@axis.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="NHBuWLbeQWJ0REJ9"
Content-Disposition: inline
In-Reply-To: <20220912085943.1098651-1-camel.guo@axis.com>
Precedence: bulk


--NHBuWLbeQWJ0REJ9
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Sep 12, 2022 at 10:59:43AM +0200, Camel Guo wrote:
> Previously the duty cycle was divided equally into h_scl_l, t_scl_h.
> This makes the low period of the SCL clock in Fast Mode is only 1.25us
> which is way lower than the minimal value (1.3) specified in i2c
> specification. In order to make sure t_scl_l, t_scl_h always fullfill
> i2c specification, this commit calculates t_scl_l using this formula:
>=20
> t_scl_l =3D clk_cycle *
>     ((t_low_min + (scl_clock - t_low_min - t_high_min) / 2) / scl_clock)
>=20
> where:
> t_low_min is the minimal value of low period of the SCL clock in us;
> t_high_min is the minimal value of high period of the SCL clock in us;
> scl_clock is converted from SCL clock frequency into us.
>=20
> Signed-off-by: Camel Guo <camel.guo@axis.com>

Applied to for-next, thanks!


--NHBuWLbeQWJ0REJ9
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmU2jWgACgkQFA3kzBSg
KbaidQ/+Kzl5eb4XcCOXgAi1kIsBaqIK0qMCqygYCEr0/uQi7gVIcTemI0AM86uu
M1M6UwxYcVGms/5kUHiEsCBjHgDVkistUr49W1Dur4zVvHj/aPwDoDgyXwpCGL08
0H7L/aJilgj86wGiDASH8/ZLaYrkV8HKWae+ocXVJDLTOrgWlVyjLnklZpK2e0WL
OCuKjD758ebaTNITimU0BaMZRS08NBXsxlunQN0IaIYdrqa48mSyHb7/EVhcKnp/
AxjWzxmJxtDakVKPCFNHjLFoeUp2NplMZ/pORRR6k6n/rfa9FWTFFxnuO7TTEDv6
lhnKqszlYAjFxyToF5prTQMfzKPqJcCVvEnskZjMdIB2/kZyvsW2v3clfx4R0RHQ
cBdYtEbVeATLDMdAQ3ynh8guAW37/jUF/CnKFOIeuzeMwX3y/bZrDjEwnOmVkyeQ
mli3gsQoZHAaYXYJcKmYE3/mI2gCoIOiscZfPcDbPSOzLzYV19LgZOcYfS7m2+xl
qsJQgAkRVgVFFzlFS78F2iGBTEJ2tH3gInxhP9e+uFgOv6jGkMuPWg71aHcI2Eqp
bLo7xevHVbVu8horeHE9L1UQ1QoxndkuWnPiDPdwaupyU9kD+Ex9u9+SaNfZN/9B
GiC1taDV6hQTabImUOU9x4rME4noqFUM7pPA/LRwGj7RFwC+NiM=
=0Pt+
-----END PGP SIGNATURE-----

--NHBuWLbeQWJ0REJ9--