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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3C386C77B7A for ; Tue, 6 Jun 2023 14:42:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231635AbjFFOmh (ORCPT ); Tue, 6 Jun 2023 10:42:37 -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> 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