Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp2046842pxb; Sat, 27 Feb 2021 08:37:11 -0800 (PST) X-Google-Smtp-Source: ABdhPJx3VsX8YCOMc0SO46mmu15X78Yh5TANcmxT9BthIMvEaqVODsBiVXxlmIR8iFWTBuo+A9Zl X-Received: by 2002:a17:906:3ad0:: with SMTP id z16mr8625416ejd.72.1614443831415; Sat, 27 Feb 2021 08:37:11 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1614443831; cv=none; d=google.com; s=arc-20160816; b=PqI3oGuIV21iJ5dWtK58kGY4xzjUe1dg2cyxYff3v5+SOnDw8a0wg8pnEw+MpWw/xu BpM1NVOskqR8MlHkpYRhSGKL/fAkrccs4xwWN48gsRxdJtIo8duCk0HazpcM45uAvtCS OP+4OjcagIVGHNJz6CIhkigoARfuKPlYBfBTPaZYYurs4MkbQAjnkRsqmYZvTq96r8+J zxnp2Gk6vsMN1nJ7VfBzZBBpunyVuS06/qJoQqKIjhKc+HeTRgjngdk8ps4Ynq+fV9hA waH+uOgXCBAAn7Xb9ztxVWRtjmaaB8ktHgce4aYHNJa8ReDMQcG4JHaf+rZWzN9Fi97s 0D+A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=DXHL+udM4GSAeXTXZWwVsRe41aWGUJWN+RtgcVX6N38=; b=Tds3KYgwwWr2yHz6BiGIOPuwDQCBa6NzDjOjQ2IwprxeXW/OQ9i9gRbMhnq8urKE1R khfDXKGABslanNDz98p0wfpp2Bn36WR1P5p86HGoSaiiI1eFLXfTvLWzGYPdfBqcuRpR 5HrCs/q5oezPusj7DPjWKiZOnwD3R5JnELD6acVWSiQ9QxAb5tpYgUi0zHhvXhUTsIkL YC4YDRsgd/NMUfDaa9vWT+oujiVrzrenXyQYR81BgYWjrigP/VeSxKZgQdHair8qN8+M qfb/9ijVsQ3+dKLLmS/mY/h2EgqsbQBfESEDKPMYcH+Hmxga7cFiQ7m98lZgnebjUfQu 5zxQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=IMTwSCAa; 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 eb13si8559477edb.8.2021.02.27.08.36.48; Sat, 27 Feb 2021 08:37:11 -0800 (PST) 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=IMTwSCAa; 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 S230063AbhB0QgU (ORCPT + 99 others); Sat, 27 Feb 2021 11:36:20 -0500 Received: from mail.kernel.org ([198.145.29.99]:60780 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229999AbhB0QgS (ORCPT ); Sat, 27 Feb 2021 11:36:18 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id 15C0F64E56; Sat, 27 Feb 2021 16:35:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1614443736; bh=pThp2YUgioKYQbON2ROnR85+avA0x0gvTYbNYvNV6vY=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=IMTwSCAaYbowhVS5Ca4ty45IBUiAriKlT4GDmENMgW5L1Jr+FGS+UjN+dOJiUTXG7 yAafWli/v4hfgzkeXop+a1V+VnlOrmxoAkSPE+D3Ws3nX3aVd4HS/9fL031ziYkwo/ 2pWeZet15MgSqd02hEAz7gnTLXlpxq+Avldx991O+q9Dh+14OtD8FS0T2kMhoT/mDJ QA5i5jBMGQbEge+2fuzdpco7COx3SWE+STQHy0D3q14+yW+M+rnue5NcsUJLdYXtHX eKmFWMzh8xg6Rk75ORhGMKCvJU7Q/Zx7baZnbLi6ucseadHg305WDxar4KsYKZoL4L F+ttLs9BETv8A== Received: by mail-ed1-f43.google.com with SMTP id bd6so1609104edb.10; Sat, 27 Feb 2021 08:35:35 -0800 (PST) X-Gm-Message-State: AOAM532187yUpS9HP2geGEwCIF7GVpLH5e+mw60ePng5YA2DbRLYo7BT VhUe0v1C5KIi6RukFKWZkQdE9a+xU/Tm/Wirrrc= X-Received: by 2002:a05:6402:10ce:: with SMTP id p14mr8645979edu.348.1614443734715; Sat, 27 Feb 2021 08:35:34 -0800 (PST) MIME-Version: 1.0 References: <20200828153100.19006-1-krzk@kernel.org> In-Reply-To: From: Krzysztof Kozlowski Date: Sat, 27 Feb 2021 17:35:23 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH] devfreq: rk3399_dmc: Simplify with dev_err_probe() To: Chanwoo Choi Cc: MyungJoo Ham , Kyungmin Park , Chanwoo Choi , Linux PM list , linux-kernel Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 29 Aug 2020 at 15:10, Chanwoo Choi wrote: > > On Sat, Aug 29, 2020 at 12:31 AM Krzysztof Kozlowski wrote: > > > > Common pattern of handling deferred probe can be simplified with > > dev_err_probe(). Less code and the error value gets printed. > > > > Signed-off-by: Krzysztof Kozlowski > > --- > > drivers/devfreq/rk3399_dmc.c | 20 ++++++-------------- > > 1 file changed, 6 insertions(+), 14 deletions(-) > > > > diff --git a/drivers/devfreq/rk3399_dmc.c b/drivers/devfreq/rk3399_dmc.c > > index 027769e39f9b..35b3542f1f7b 100644 > > --- a/drivers/devfreq/rk3399_dmc.c > > +++ b/drivers/devfreq/rk3399_dmc.c > > @@ -324,22 +324,14 @@ static int rk3399_dmcfreq_probe(struct platform_device *pdev) > > mutex_init(&data->lock); > > > > data->vdd_center = devm_regulator_get(dev, "center"); > > - if (IS_ERR(data->vdd_center)) { > > - if (PTR_ERR(data->vdd_center) == -EPROBE_DEFER) > > - return -EPROBE_DEFER; > > - > > - dev_err(dev, "Cannot get the regulator \"center\"\n"); > > - return PTR_ERR(data->vdd_center); > > - } > > + if (IS_ERR(data->vdd_center)) > > + return dev_err_probe(dev, PTR_ERR(data->vdd_center), > > + "Cannot get the regulator \"center\"\n"); > > > > data->dmc_clk = devm_clk_get(dev, "dmc_clk"); > > - if (IS_ERR(data->dmc_clk)) { > > - if (PTR_ERR(data->dmc_clk) == -EPROBE_DEFER) > > - return -EPROBE_DEFER; > > - > > - dev_err(dev, "Cannot get the clk dmc_clk\n"); > > - return PTR_ERR(data->dmc_clk); > > - } > > + if (IS_ERR(data->dmc_clk)) > > + return dev_err_probe(dev, PTR_ERR(data->dmc_clk), > > + "Cannot get the clk dmc_clk\n"); > > > > data->edev = devfreq_event_get_edev_by_phandle(dev, 0); > > if (IS_ERR(data->edev)) > > -- > > 2.17.1 > > > > Applied it. Thanks. Hi Chanwoo, Do you know what happened with this patch? You replied that it is applied but I cannot find it in the Linus' or next trees. Best regards, Krzysztof