Received: by 2002:a05:6358:5282:b0:b5:90e7:25cb with SMTP id g2csp1995807rwa; Sun, 21 Aug 2022 23:23:30 -0700 (PDT) X-Google-Smtp-Source: AA6agR7D+9msW72mODpsG++r3L41koWS4izNMiKu4LZGWXlKg/5VzVEaJEcvhh6sa2kXPQGAN7MN X-Received: by 2002:a05:6402:44c:b0:445:f2f1:4add with SMTP id p12-20020a056402044c00b00445f2f14addmr15260236edw.257.1661149409858; Sun, 21 Aug 2022 23:23:29 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1661149409; cv=none; d=google.com; s=arc-20160816; b=ynH1rH5gK1a5bVSHz0iCYevW1sAQFpvGYBBgnTbkUlXbWXnzvycmHiCtOx98JNdjuS tH8VJ3GOv7jl0MXeU7+CMii3K8vZ7lcGB/kNK46uSS5blMEEU8Y4vCsdh2ZQrnWMJv6c XggcngiCwHUDgtjt7AagCAf3gmbMFJS+a+QCOqMlOzcMg+6FGL7vwJPPm8lkqOhZJe/K eSM6K/QPQwUKQ/vgj7fxUwMRjNmyfTuwSbwrUta0Lr8eeacqoPRp0Ihdoewa6Wfbzi2b qXrXyEsfGTgfl1phhLJiYoxzC8BbCu9JcClf9n+ppa00H+AdCR5vg+ADyHGGaWg6wN0A 8IRw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=A78rlXkUCQeapoCEKr+pARnpUSnk8hBoNI0D+HhTPxs=; b=mnOWzFPGvRnfVtZl09EiN7rWpjHvwVXlDgA92vbW8CsJ00FMBR05/5BeYL7fkiczpL Ah1ZmsRzpPjkujNMvgFfRJgKlF5qJna6VrfET/W3WmC/6W8x6/F/OEGDkNtEWR4kCBEN b/GV63g7/1YETxYeJ8/kRj0rywHv/H1Xg8XVx1p/F//q8u3gZThelmAiHxcbv+eEaP5x YVARBcQnjhubT5LaKhCDvpZBi6Ac8x6bb43HakV1Wj58ll2Mepi7ZYHKORdXsRWVzVmB nj3NXJpWBx5ramk2vX0ThFJOwaxW5CNqt++7YASyss8VFrx+WqmmQBPwrKKrmhzglevG FDeg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=vWe7EWIP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id mp16-20020a1709071b1000b00730bcba7f70si8837776ejc.635.2022.08.21.23.23.02; Sun, 21 Aug 2022 23:23:29 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=vWe7EWIP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230229AbiHVGJh (ORCPT + 99 others); Mon, 22 Aug 2022 02:09:37 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59210 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229656AbiHVGJf (ORCPT ); Mon, 22 Aug 2022 02:09:35 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7D46E24F05 for ; Sun, 21 Aug 2022 23:09:34 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 250D2B80E88 for ; Mon, 22 Aug 2022 06:09:33 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 949E7C433C1; Mon, 22 Aug 2022 06:09:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1661148571; bh=jkfZw4GZo+upzS4/44GEBKA5bFWYoet3Hu9G/RHcY3s=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=vWe7EWIPjr57tZHdahPOxSZbHiqx3NeDYX9EuhQYcVT4ZRfmE6c/7Pm/+FGwQIfke xhyga+2I80lQUGCyd7oGDfHBjU3jktlKIcm2MQloVdLCbnxL142l09vYZ1kLpd2nVF OAuuS//fXeq9M3ILrkaYRnFq3NPKlJ8xLZxUGACo= Date: Mon, 22 Aug 2022 08:09:27 +0200 From: Greg KH To: Yang Yingliang Cc: linux-kernel@vger.kernel.org, rafael@kernel.org Subject: Re: [PATCH -next] driver core: switch to use dev_err_probe() Message-ID: References: <20220819094656.1945653-1-yangyingliang@huawei.com> <1d4d1ef7-7386-cf8f-0a5a-04690ec27ed6@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1d4d1ef7-7386-cf8f-0a5a-04690ec27ed6@huawei.com> X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,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: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 22, 2022 at 10:10:22AM +0800, Yang Yingliang wrote: > Hi, > > On 2022/8/19 17:42, Greg KH wrote: > > On Fri, Aug 19, 2022 at 05:46:56PM +0800, Yang Yingliang wrote: > > > Use dev_err_probe() to simplify code and print error code. > > > > > > Signed-off-by: Yang Yingliang > > > --- > > > drivers/base/core.c | 8 +++----- > > > 1 file changed, 3 insertions(+), 5 deletions(-) > > > > > > diff --git a/drivers/base/core.c b/drivers/base/core.c > > > index 753e7cca0f40..e51a09f9d0ec 100644 > > > --- a/drivers/base/core.c > > > +++ b/drivers/base/core.c > > > @@ -2507,11 +2507,9 @@ static ssize_t uevent_store(struct device *dev, struct device_attribute *attr, > > > int rc; > > > rc = kobject_synth_uevent(&dev->kobj, buf, count); > > > - > > > - if (rc) { > > > - dev_err(dev, "uevent: failed to send synthetic uevent\n"); > > > - return rc; > > > - } > > > + if (rc) > > > + return dev_err_probe(dev, rc, > > > + "uevent: failed to send synthetic uevent\n"); > > I do not understand this at all, this is not on the probe path at all, > > so why is this function needed? > > > > Have you tested this? What was the resulting output before and after > > this change? > Yes, after this change, the error code will be print. You did not answer my question. Again, this is not on the probe() callback path, so why are you wanting to call dev_err_probe()? thanks, greg k-h