Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Tue, 19 Feb 2019 08:38:10 -0800
From:   Luis Chamberlain <mcgrof@kernel.org>
To:     yuankuiz@codeaurora.org
Cc:     gregkh@linuxfoundation.org, rafael@kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] firmware: hardcode the debug message for -ENOENT
Message-ID: <20190219163810.GI11489@garbanzo.do-not-panic.com>
References: <7e752055f72d67a4b963436d67965123@codeaurora.org>
 <e1e9df03b288daf7258ab9e0f7402ca1@codeaurora.org>
 <20190204233047.GT11489@garbanzo.do-not-panic.com>
 <d2f930db664d8433894a7bf20db32748@codeaurora.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <d2f930db664d8433894a7bf20db32748@codeaurora.org>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Sun, Feb 17, 2019 at 04:25:10PM +0800, yuankuiz@codeaurora.org wrote:
> On 2019-02-05 07:30 AM, Luis Chamberlain wrote:
> > On Mon, Jan 14, 2019 at 05:58:30PM +0800, yuankuiz@codeaurora.org wrote:
> > > Hi,
> > > 
> > >     Refined at below.
> > > 
> > > From bbd0d9c8f28eb78ca34353347c3d4092e88f000c Mon Sep 17 00:00:00 2001
> > 
> > This is all garbled, not sure why your patch looks all messed up.
> > 
> > Are you using git sendemail or something manual?
> > 
> >   Luis
> > 
> Done. Update it with resend it as below.

Sorry but I have not received it, can you resend?
While at it please address my comment below.

> From: John Zhao <yuankuiz@codeaurora.org>
> 
> When the return code of "-ENOENT" was printed inside
> of the debug message, which could be hardcoded meaningful.

This sentence doesn't read well. Please adjust it a bit to make
coherent sense, and resend it.

Perhaps you can use something like this:

We already issue a custom firmware debug print message when no file / path
was not found, however only the error code is returned. Stating clearly
that the file was not found is much more useful for debugging, so let's
be explicit about that.

  Luis

> 
> Signed-off-by: John Zhao <yuankuiz@codeaurora.org>
> ---
>  drivers/base/firmware_loader/main.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/base/firmware_loader/main.c
> b/drivers/base/firmware_loader/main.c
> index 8e9213b..7eaaf5e 100644
> --- a/drivers/base/firmware_loader/main.c
> +++ b/drivers/base/firmware_loader/main.c
> @@ -328,12 +328,12 @@ fw_get_filesystem_firmware(struct device *device,
> struct fw_priv *fw_priv)
>  		rc = kernel_read_file_from_path(path, &fw_priv->data, &size,
>  						msize, id);
>  		if (rc) {
> -			if (rc == -ENOENT)
> -				dev_dbg(device, "loading %s failed with error %d\n",
> -					 path, rc);
> -			else
> +			if (rc != -ENOENT)
>  				dev_warn(device, "loading %s failed with error %d\n",
>  					 path, rc);
> +			else
> +				dev_dbg(device, "loading %s failed for no such file or directory.\n",
> +					 path);
>  			continue;
>  		}
>  		dev_dbg(device, "direct-loading %s\n", fw_priv->fw_name);
> -- 
> 2.7.4