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;
Date:   Tue, 25 Apr 2023 17:06:37 +0200
From:   Andrew Lunn <andrew@lunn.ch>
To:     =?utf-8?B?5byg572R?= <m202171703@hust.edu.cn>
Cc:     Peter Korsgaard <peter@korsgaard.com>,
        hust-os-kernel-patches@googlegroups.com, linux-i2c@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: Re: [PATCH v3] i2c: ocores: use devm_ managed clks
Message-ID: <b5c00122-0fe0-4020-9036-e4cc37d1b51a@lunn.ch>
References: <fc8ed989-68e6-4fd4-a818-ae077bf5e6aa@lunn.ch>
 <20230422123253.137368-1-silver_code@hust.edu.cn>
 <cf543f92-af0d-4862-bea6-53a358ee9a31@lunn.ch>
 <85952de.43bfe.187b8e24612.Coremail.m202171703@hust.edu.cn>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <85952de.43bfe.187b8e24612.Coremail.m202171703@hust.edu.cn>
Precedence: bulk

On Tue, Apr 25, 2023 at 10:47:29PM +0800, 张网 wrote:
> Hi Andrew,
> 
> I would like to express my sincere gratitude for taking the time and effort to review 
> my submitted patch. I understand that reviewing can be a time-consuming process 
> and I truly appreciate your dedication.
> 
> As we move forward, I would like to inquire about the first version[1] of the patch I submitted. 
> As clk_disable_unprepare() has checks for error pointer and NULL already, I think there is no 
> need to add the check. So both the first version of the patch and this one can work on this 
> branch.
> 
> If there are any further changes or revisions needed, please do not hesitate to let me know. 
> I am committed to learning and improving, and I welcome any feedback you may have. 
> Thank you again for your support and guidance throughout this process.
> 
> Best regards,
> Wang Zhang
> ---
> [1] http://patchwork.ozlabs.org/project/linux-i2c/patch/20230416071854.58335-1-silver_code@hust.edu.cn/

So this patch is about the IRQ being an error code, and doing the
correct cleanup.

With the change to devm_clk_get_ there is no need to disabled the
clock, it will be done automatically. This means there is no cleanup
the driver needs to do itself. So the patch is no longer needed.

    Andrew