Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp7229641rwd; Tue, 6 Jun 2023 08:00:59 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ7jXtZg7N+BbHaH2fub4QB9TIqohdeSvjMIs4om/S8FWtPMxdaiCja3B876WkcKXEr3EfsQ X-Received: by 2002:a05:620a:268d:b0:75e:b8ee:9a01 with SMTP id c13-20020a05620a268d00b0075eb8ee9a01mr2378513qkp.63.1686063659194; Tue, 06 Jun 2023 08:00:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1686063659; cv=none; d=google.com; s=arc-20160816; b=EoZ3jhd3fytX6Y7CZTdWbxiBlQoLzTDdaUTZTASSTfJ5vLDtdO7KFnGWNoy78cxcon 2lAls99nAu4P7c825fjaLT5aTINqACVcqLvXh5kKdwAX8r18+MTsmmvSCNUPj9tvH401 b7SKtuxFvD/skX+nbfvk9HicDirHhiIiy8fVoovMK3mszjWSv3KRtJ2ozda13zdqgSL5 iHES53Ln9GOdUCStUF+CtQ4YXEEvqfgLHWhM1gkeTpODHzsSOCQ2SJtBsnKEgexZhjCw wuaEyXK4r/FiyiBcZkd0gbUAsaVeQKTi7/WHAMz0J0watr8WdISyCjdo39i4sUvVFQTS 9ONg== 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=+k4HZQeXqQhONblJfWv+Zy6ExdWtIFzyyjTOH7aMm3c=; b=wWlaF+bZSX0mo+Fg5P351B6EkSVeY0+1PkM8jmWcU8WQUf/o2Q4/8bx6W+2qduJzk0 V1XKZSeHQXuPYNK8RhMpFIiMuR4mF11xaNXxV9r6WYV5xbOBkvdwBSpQVS+7VH02QW5B 6UuERsojwswoik3/37WWhZo8vWBmxjIbB9VJLCYCS3/s8yjex9IEZnBi0D5pdPZChBe1 uMAhD6sH7DihtZYL6HV8f7E8RTrJifh6OWyckkceMLJKqK8KEsOi1rUd5wdX36G2A66+ OcYzMcp3gMyLilbtXphN0tcqNCfdOhZdquBSO/vf5UvGKqYiNjbYsKsT3mwLhiTiu6X1 pP/g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass (test mode) header.i=@ideasonboard.com header.s=mail header.b=O4smJqs5; 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 Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id o15-20020a05620a130f00b0075ec95615bcsi1179602qkj.671.2023.06.06.08.00.39; Tue, 06 Jun 2023 08:00:59 -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 (test mode) header.i=@ideasonboard.com header.s=mail header.b=O4smJqs5; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238390AbjFFOmg (ORCPT + 99 others); Tue, 6 Jun 2023 10:42:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41340 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237908AbjFFOmX (ORCPT ); Tue, 6 Jun 2023 10:42:23 -0400 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7C15C2719 for ; Tue, 6 Jun 2023 07:40:20 -0700 (PDT) Received: from pendragon.ideasonboard.com (om126253223039.31.openmobile.ne.jp [126.253.223.39]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 23665AB; Tue, 6 Jun 2023 16:39:13 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1686062355; bh=qyqmnSxwDwLrbxk8+vjpFN2vWJrjzLzmUBFrL48Rkfw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=O4smJqs5dEKcDVHCfkFXhvvZxlZ37wGv8k7loP11UwO+eQEJjJv11rrjS/a9/JlDD L+RlpZsRVK8FkPZhlAJ+QiL+70D/6XtzH0RHbEBLfnJtZ8Itfu8ZfPf+Dzo+HFjYwZ qJvMdz7dyCVBQtyMGYZRL8EHzvAurt88+U0VvGZw= Date: Tue, 6 Jun 2023 17:39:37 +0300 From: Laurent Pinchart To: Siddh Raman Pant Cc: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Andrzej Hajda , Neil Armstrong , Robert Foss , Jonas Karlman , Jernej Skrabec , Jani Nikula , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Suraj Upadhyay Subject: Re: [PATCH v9 4/8] drm: Remove usage of deprecated DRM_NOTE Message-ID: <20230606143937.GD5197@pendragon.ideasonboard.com> References: <91d957dd92b41dda42da912397df195e3565866a.1686047727.git.code@siddh.me> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <91d957dd92b41dda42da912397df195e3565866a.1686047727.git.code@siddh.me> X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_PASS,SPF_PASS, T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED 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 Hi Siddh, Thank you for the patch. On Tue, Jun 06, 2023 at 04:15:18PM +0530, Siddh Raman Pant wrote: > drm_print.h says DRM_NOTE is deprecated in favor of drm_notice(). > > Signed-off-by: Siddh Raman Pant With the commit message fixed as mentioned in the review of an earlier patch in this series, Reviewed-by: Laurent Pinchart > --- > drivers/gpu/drm/drm_displayid.c | 2 +- > drivers/gpu/drm/drm_kms_helper_common.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/drm_displayid.c b/drivers/gpu/drm/drm_displayid.c > index 9edc111be7ee..27ffeee09e4f 100644 > --- a/drivers/gpu/drm/drm_displayid.c > +++ b/drivers/gpu/drm/drm_displayid.c > @@ -42,7 +42,7 @@ validate_displayid(const u8 *displayid, int length, int idx) > for (i = 0; i < dispid_length; i++) > csum += displayid[idx + i]; > if (csum) { > - DRM_NOTE("DisplayID checksum invalid, remainder is %d\n", csum); > + drm_notice(NULL, "DisplayID checksum invalid, remainder is %d\n", csum); > return ERR_PTR(-EINVAL); > } > > diff --git a/drivers/gpu/drm/drm_kms_helper_common.c b/drivers/gpu/drm/drm_kms_helper_common.c > index 0bf0fc1abf54..7a41373b67dc 100644 > --- a/drivers/gpu/drm/drm_kms_helper_common.c > +++ b/drivers/gpu/drm/drm_kms_helper_common.c > @@ -41,7 +41,7 @@ MODULE_LICENSE("GPL and additional rights"); > /* Backward compatibility for drm_kms_helper.edid_firmware */ > static int edid_firmware_set(const char *val, const struct kernel_param *kp) > { > - DRM_NOTE("drm_kms_helper.edid_firmware is deprecated, please use drm.edid_firmware instead.\n"); > + drm_notice(NULL, "drm_kms_helper.edid_firmware is deprecated, please use drm.edid_firmware instead.\n"); > > return __drm_set_edid_firmware_path(val); > } -- Regards, Laurent Pinchart