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;
From:   Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
To:     Cai Huoqing <caihuoqing@baidu.com>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>,
        linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org,
        Thierry Reding <thierry.reding@gmail.com>,
        Jonathan Hunter <jonathanh@nvidia.com>
Subject: Re: [PATCH] memory: tegra: Make use of the helper function devm_add_action_or_reset()
Date:   Fri, 24 Sep 2021 13:20:36 +0200
Message-Id: <163248239878.48775.4100179078920031712.b4-ty@canonical.com>
In-Reply-To: <20210922130002.586-1-caihuoqing@baidu.com>
References: <20210922130002.586-1-caihuoqing@baidu.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
Precedence: bulk

On Wed, 22 Sep 2021 21:00:01 +0800, Cai Huoqing wrote:
> The helper function devm_add_action_or_reset() will internally
> call devm_add_action(), and if devm_add_action() fails then it will
> execute the action mentioned and return the error code. So
> use devm_add_action_or_reset() instead of devm_add_action()
> to simplify the error handling, reduce the code.
> 
> 
> [...]

Applied, with simplified commit msg (no need to write too much for such simple
change), thanks!

[1/1] memory: tegra: Make use of the helper function devm_add_action_or_reset()
      commit: 1d8e0223bb52071ffc59391f0ebefa06ce5643d7

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>