Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Peng Fan <peng.fan@nxp.com>
To:     "Peng Fan (OSS)" <peng.fan@oss.nxp.com>,
        Aisheng Dong <aisheng.dong@nxp.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        "wsa@kernel.org" <wsa@kernel.org>
CC:     "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "festevam@gmail.com" <festevam@gmail.com>,
        dl-linux-imx <linux-imx@nxp.com>,
        "linux-i2c@vger.kernel.org" <linux-i2c@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Clark Wang <xiaoning.wang@nxp.com>
Subject: RE: [PATCH V2 0/7] i2c-imx-lpi2c: add IPG clock
Thread-Topic: [PATCH V2 0/7] i2c-imx-lpi2c: add IPG clock
Thread-Index: AQHYsW876SxYc4rxXUGbKvyYc8nHU626pHbA
Date:   Mon, 22 Aug 2022 08:46:31 +0000
Message-ID: <DU0PR04MB9417E7BD5F1FB7A8E00BAA3F88719@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20220816125526.2978895-1-peng.fan@oss.nxp.com>
In-Reply-To: <20220816125526.2978895-1-peng.fan@oss.nxp.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?eSzAauO7FAqf5pJk8O9yTh4lerqZcR1O5vYL77GEy1cVwOK8Qy1BAv4k2W+l?=
 =?us-ascii?Q?Q6zMh+lQ6HpvmYT4MQpBu8u4JgjTomAod5cqEgqd2biy21t+jdcLPSKG0EoK?=
 =?us-ascii?Q?un6WNoXn7GCVkIU4ju2esiBXs4grzUBddjlKPYAOWOuuN1k/6osrkv16SasR?=
 =?us-ascii?Q?wU1pxA7cNxnoG99R0KjS6gztfGqSJzOusFHR0Sd13EuYHC/amWJ4tHM0HJuH?=
 =?us-ascii?Q?ItEfzS3tzNmdTRi4EaytP3C1T++JYMHk/KyceCswdJYA9b7bzKQ8de97C+vt?=
 =?us-ascii?Q?DZd6L1XNkoDesZNjNUz5NMaWZ9MujyMsdkWa4/P3Z7xN0tkgBx/0emZB7JzG?=
 =?us-ascii?Q?8PoKN52SvesgYfKRt/B5xiH8lfOP7xxC3iL6CBmcohnsuHfa99yELHSDIoZC?=
 =?us-ascii?Q?4le7Z3YV9avCDDMxiQ9Cj2cf8B+ccCLOu8k8JwN+0E8G1wCL2i/BWlE0XT8F?=
 =?us-ascii?Q?O7xnYuf93Lxq+9nwbEiaSbJEKnYBnKk21V32kyKJLdMMzmEEAe8D45Tw+vCc?=
 =?us-ascii?Q?re1KnN5EfRnHUey1cwYYd62zOhxU6DwHcagloEicTvWtCmwt2wfkDtLMTrGK?=
 =?us-ascii?Q?3sm0nKq6Cuh/FlFjSjrHWoAMlglOJ3r1gHvIfkTwmWHuKZ5TZQ1wohsVjgaz?=
 =?us-ascii?Q?Y09RxgxVCcZxL5c31My9OkswTj1kAiTU7U4qaHObwDwwelK0c4W5Mvp42dpC?=
 =?us-ascii?Q?HLNDzZNlQ0WxjZ0rBZvnF6hTrz41JSiRma3wp9Tox8l+i3Xe0AqZnWMRCWQy?=
 =?us-ascii?Q?1B1YcSwP/GOWz3rMqORXshOdgTGw/CFLZMrTAfiTJ8qeRHjmnxsWEoEkaOIV?=
 =?us-ascii?Q?80h3oZ7OiyIV+TfYuS7Ta1UjSVfZ6JAu+LUqiHxFGkvrG3orKCbb6JfOzc1O?=
 =?us-ascii?Q?HwjwnJTsk2JD/IPLwSmkUYBsGzrsNcaZRpQ1oMXxI29jArr8/onymZdYTWoa?=
 =?us-ascii?Q?Vj4aK2mvl3zJ0EsZdftKz+FXMiUgc1AeMFUyY4FhDQ6fH2zOTPi5VubSuasZ?=
 =?us-ascii?Q?k260aipntdXZC63G+IAahE5U4FkW2DPGI1fUWtLxhXxHb26+t2U113P5lv5O?=
 =?us-ascii?Q?+zPb1zhSq90HF6HwDlGi/PRq/aTb7llx2QDhI/PVwmXJzhOGAaLVoBCVsWf+?=
 =?us-ascii?Q?BJxiIuHpTn+P4NqBiugLJCLyPWrL/vypjexjaOUgFU7y2AVVRUwQI83LNyIm?=
 =?us-ascii?Q?F6xDmL3J9yMzqTqixlEmLJZ6LdKlD407KOlS1ICTaLdgYhpSba0jkmcQWznh?=
 =?us-ascii?Q?U7JAGM748nGcSSW40d6WMxNv/Ad7xz5T1ggKEAiJcUKgYjnnNy8MviDEYhKX?=
 =?us-ascii?Q?0fO1M6doXZbuc5YA1nt84x0cLKxHtgvkHWpISJhqXH0UKniv3hX5apqG8kmE?=
 =?us-ascii?Q?ViQiKG9HM+TG2wtBjIpG15d7x0mzDsCSzEdXrhHn6jF+K3e2NNrs3qLRO2mQ?=
 =?us-ascii?Q?MFwi7M4qE2a4DfwwVgll9EsYnZaXSX+ehykaTP9H3RZVX+S/IUS69SWnSuiY?=
 =?us-ascii?Q?tyuGXsSfQXlVa6IfckiLzxWdoYQ5NLeI1eO3b8Xkg7OX8tOqdEr7VQYjPKH0?=
 =?us-ascii?Q?bdRv0I35HMpL07nTOI++AQcK3B9CrFD8QcfGbrG6?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: df147d29-8199-4150-122d-08da841acfd7
X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Aug 2022 08:46:31.6560
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 8YA09tBAlprHfHBvaamhFO4Cf570/pf2UD3plEESWscJERPjVOOmlerm0ut1IqTY1qOF4/B20CRHALeE+xHquw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB7PR04MB4171
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> Subject: [PATCH V2 0/7] i2c-imx-lpi2c: add IPG clock

+ Wolfram Sang I2C maintainer.

Krzysztof,

Do you have time to give a look whether this patchset is ok for you?
Please forgive if this is too early ping. Some i.MX93 dts update pending

Thanks,
Peng.

>=20
> From: Peng Fan <peng.fan@nxp.com>
>=20
> V2:
>  use clk bulk API in driver to support backward compatibility.
>  Include a new patch, patch 1.
>=20
> The i.MX LPI2C needs PER and IPG clock, not just PER or IPG clock.
> The current driver/dts/bindings use one CLK. Although it works with with
> upstream kernel, but it not match the hardware design. If IPG clock is
> disabled, the LPI2C will not work.
>=20
> There are changes made to ARM32 i.MX7ULP dts, ARM64 i.MX8 dts, dt-
> bindings, and the lpi2c driver.
>=20
> The driver is updated to use bulk clk API to avoid break backward
> compatibility. But it is hard to avoid dtbs_check pass, because the dts a=
nd
> binding update are in separate patches.
>=20
> Peng Fan (7):
>   ARM: dts: imx7ulp: update the LPI2C clock-names
>   dt-bindings: i2c: i2c-imx-lpi2c: add ipg clk
>   dt-bindings: i2c: i2c-imx-lpi2c: add dmas property
>   dt-bindings: i2c: i2c-imx-lpi2c: add i.MX93
>   arm64: dts: imx8-ss-dma: add IPG clock for i2c
>   ARM: dts: imx7ulp: Add IPG clock for lpi2c
>   i2c: imx-lpi2c: use bulk clk API
>=20
>  .../bindings/i2c/i2c-imx-lpi2c.yaml           | 20 +++++++++++++---
>  arch/arm/boot/dts/imx7ulp.dtsi                | 10 ++++----
>  .../arm64/boot/dts/freescale/imx8-ss-dma.dtsi | 20 +++++++++-------
>  drivers/i2c/busses/i2c-imx-lpi2c.c            | 24 +++++++++----------
>  4 files changed, 47 insertions(+), 27 deletions(-)
>=20
> --
> 2.37.1