From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 312F0C5AD4C for ; Thu, 23 Nov 2023 17:32:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=O555iSISXnOOFv246R/EfMBPvSl46TcvjeWfp/Ljy/Q=; b=Laph5KO9NvGnNk zLdc1KWIw/6IedVUOcF7Qm/gOLmQuaIJZtcjHaB8JfcsonxZvmsjr9O8O7wJkZeiB69gsLOf+EKqD jtQUvYmgk4CeKU1jPl0dXeeDMgIQzHQ78RR1KGg3SKalgluyHw5Jc3qzbM0DEPIUYW+OHlGujMC/b hXDIrOlRgpEinZi5w8aLs5ZIt3y5YK6NPK6kYHJZy5WylmskTWJasdt90tiewG44Lfm1p3gBZ8jQu eGVAsgZakYNpWzo2d34vVmPf7FxEXO5B1YFRtX4X73dy3lcSrEdkp/+q3j97/A0oDeUrVwJsIrzBA fjlQlueeYoDz/KSc/Piw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1r6DYB-005QZ3-2y; Thu, 23 Nov 2023 17:31:39 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1r6DY8-005QYV-0G; Thu, 23 Nov 2023 17:31:37 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 1C4DF623B2; Thu, 23 Nov 2023 17:31:35 +0000 (UTC) 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> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20231123172825.GJ16377@pendragon.ideasonboard.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231123_093136_207326_FDFC39D2 X-CRM114-Status: GOOD ( 27.49 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel