From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3EA9F3FB2E; Thu, 23 Nov 2023 17:31:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="akWTPNe1" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 654A6C433CA; Thu, 23 Nov 2023 17:31:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1700760694; bh=rX3+rhCa3mVl/Lp4TWlVJWNaY+hrOxhxI3Sz8Cv/a14=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=akWTPNe1lV5OujfBvdQBx1ULUwSExbuy5jhZR82EF3ubUtj3kGG0XkWxzD7/JIfY7 hKXhuZ5kJz53G0ucX2cn3H7DlXDQTf+j3cMxx+6u3s45DCVnI9pxlIDOEyi6yxf3OC 83OeXzkt+3ugfyGj/fWZZDSYZcs7Gl7kQs8PFd9g= Date: Thu, 23 Nov 2023 17:31:32 +0000 From: Greg Kroah-Hartman To: Laurent Pinchart Cc: Umang Jain , linux-staging@lists.linux.dev, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org, Stefan Wahren , Dan Carpenter , Kieran Bingham , Phil Elwell , Dave Stevenson Subject: Re: [PATCH 6/9] staging: vc04_services: Drop vchiq_log_error() in favour of dev_dbg Message-ID: <2023112343-bobbed-throbbing-2c36@gregkh> References: <20231107095156.365492-1-umang.jain@ideasonboard.com> <20231107095156.365492-7-umang.jain@ideasonboard.com> <2023112322-wife-aspect-135b@gregkh> <20231123134941.GD16377@pendragon.ideasonboard.com> <2023112338-scrambler-utilize-a31d@gregkh> <20231123172825.GJ16377@pendragon.ideasonboard.com> Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231123172825.GJ16377@pendragon.ideasonboard.com> On Thu, Nov 23, 2023 at 07:28:25PM +0200, Laurent Pinchart wrote: > On Thu, Nov 23, 2023 at 01:53:42PM +0000, Greg Kroah-Hartman wrote: > > On Thu, Nov 23, 2023 at 03:49:41PM +0200, Laurent Pinchart wrote: > > > On Thu, Nov 23, 2023 at 01:02:45PM +0000, Greg Kroah-Hartman wrote: > > > > On Tue, Nov 07, 2023 at 04:51:53AM -0500, Umang Jain wrote: > > > > > Drop vchiq_log_error() macro which wraps dev_dbg(). Introduce the usage > > > > > of dev_dbg() directly. > > > > > > > > > > Add a new enum vchiq_log_type and log_type() helper to faciliate the > > > > > type of logging in dev_dbg(). This will help to determine the type of > > > > > logging("error", "warning", "debug", "trace") to dynamic debug. > > > > > > > > > > Signed-off-by: Umang Jain > > > > > --- > > > > > .../interface/vchiq_arm/vchiq_arm.c | 54 ++++---- > > > > > .../interface/vchiq_arm/vchiq_connected.c | 6 +- > > > > > .../interface/vchiq_arm/vchiq_core.c | 126 ++++++++++-------- > > > > > .../interface/vchiq_arm/vchiq_core.h | 23 +++- > > > > > .../interface/vchiq_arm/vchiq_dev.c | 47 ++++--- > > > > > 5 files changed, 143 insertions(+), 113 deletions(-) > > > > > > > > > > diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c > > > > > index 9fb3e240d9de..2cb2a6503058 100644 > > > > > --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c > > > > > +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c > > > > > @@ -696,8 +696,8 @@ int vchiq_initialise(struct vchiq_instance **instance_out) > > > > > > > > > > instance = kzalloc(sizeof(*instance), GFP_KERNEL); > > > > > if (!instance) { > > > > > - vchiq_log_error(state->dev, VCHIQ_CORE, > > > > > - "%s: error allocating vchiq instance\n", __func__); > > > > > + dev_dbg(state->dev, "%s: %s: %s: error allocating vchiq instance\n", > > > > > + log_cat(VCHIQ_CORE), log_type(ERROR), __func__); > > > > > > > > All dev_dbg() calls have __func__ in them automatically, you never need > > > > to duplicate it again as that's redundant :( > > > > > > Oh ? I didn't know that, and can't find it in the code. I may be missing > > > something though. Are you referring to the +f flag for dynamic debug > > > entries ? It won't work if dynamic debug isn't enabled though, but maybe > > > we don't care about that ? > > > > Yes, the "f" flag is what controls this, and if dynamic debug isn't > > enabled, you don't get any message here and we don't care about it :) > > You do if you #define DEBUG, that's one of the three options for > dev_dbg() (dynamic debug and no_printk() being the other two). Maybe > __func__ should be added to the dev_printk() version of dev_dbg() to > have a consistent behaviour. Drivers should NOT be defining DEBUG for anything in the tree, just use the normal interfaces, as no one will be selecting debug options from Kconfig. DEBUG is really only good for out-of-tree work. thanks, greg k-h