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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 51D68C46467 for ; Wed, 11 Jan 2023 22:50:30 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 1455A10E82C; Wed, 11 Jan 2023 22:50:24 +0000 (UTC) Received: from mail-wm1-x32c.google.com (mail-wm1-x32c.google.com [IPv6:2a00:1450:4864:20::32c]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6754310E830 for ; Wed, 11 Jan 2023 22:50:20 +0000 (UTC) Received: by mail-wm1-x32c.google.com with SMTP id g19-20020a05600c4ed300b003d9eb1dbc0aso10816858wmq.3 for ; Wed, 11 Jan 2023 14:50:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=+sQGs0x5broQ060QblgxdkojwSBGm6c9FifhWg0Fh6c=; b=bCZ6q6Ng37YThePYLToZ7UZCUEVKTGd+x9Jav7Yuj9c44zg/kzbHK6bfbRDkFpqX2h AOhC3rMtzc2Mjd7BO7rHu6k35iRDmYm3sK5fTSbxvpa0LihB7BJNCS+SEbDhIFPmJEVs 9sIFR3IuM+9ogXWM4NUiVFhJCqmAdfcUnaQNo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=+sQGs0x5broQ060QblgxdkojwSBGm6c9FifhWg0Fh6c=; b=zY/hnIyK6kGZ9EVMYG5n4YyNR263yRT0D3lVQg+0GuCMveVS6Ee2c97Yj5pPfbVaVS 6sU7qgcaf7ysnj9yewAJ6ZJ1Cz4fhp8Ydm+wxOovZq2AT0NAJ+L3yb181PBVe6iME/tB tm6mSmQhglzaD0bEx9Z1gObS9f6Fr8OXmg2KgNiskGaJECm1q9sqqYa3+rXb8Y1jjTiN YjcWTp8wiW5bn/Txy4PVKKuTu1Y/yqJn+u2KDXqqirRVEDQf7eXe45fhRi10LJVNZLru kdyVgZB2EdmT/JXpKn+msc+xEj3TlTxXcLUBZXKzJmF4N/IHdnnS6hImIlOneyBUxCfl ApdA== X-Gm-Message-State: AFqh2kpF8ji0O6i8UiQ8wyJiB2+eIbOnsuyP4ODeL4YrAjuSG+lI6L5c j3jKsiQ347ZnBn+L4OoEDegWxw== X-Google-Smtp-Source: AMrXdXspeQJHJT9AVEAbvVxV9Me8z0LaalWZChGsvYCqMMnkIdi3e4pfpWQIJflhA0h2wwXyL1PQWg== X-Received: by 2002:a05:600c:4fcc:b0:3d9:f91c:4bfc with SMTP id o12-20020a05600c4fcc00b003d9f91c4bfcmr5990323wmq.31.1673477418756; Wed, 11 Jan 2023 14:50:18 -0800 (PST) Received: from phenom.ffwll.local ([2a02:168:57f4:0:efd0:b9e5:5ae6:c2fa]) by smtp.gmail.com with ESMTPSA id o21-20020a1c7515000000b003d995a704fdsm19544026wmc.33.2023.01.11.14.50.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 11 Jan 2023 14:50:17 -0800 (PST) Date: Wed, 11 Jan 2023 23:50:15 +0100 From: Daniel Vetter To: =?iso-8859-1?Q?Ma=EDra?= Canal Message-ID: References: <20230109105807.18172-1-mcanal@igalia.com> <20230109105807.18172-6-mcanal@igalia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230109105807.18172-6-mcanal@igalia.com> X-Operating-System: Linux phenom 5.19.0-2-amd64 Subject: Re: [Intel-gfx] [PATCH 5/5] drm/plane: Unexport drm_any_plane_has_format() X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: =?iso-8859-1?Q?Andr=E9?= Almeida , Thomas Zimmermann , intel-gfx@lists.freedesktop.org, Simon Ser , dri-devel@lists.freedesktop.org, Maxime Ripard , Melissa Wen , VMware Graphics Reviewers , Daniel Vetter , Rodrigo Vivi , Alex Deucher , David Airlie , Christian =?iso-8859-1?Q?K=F6nig?= , Zack Rusin Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Mon, Jan 09, 2023 at 07:58:08AM -0300, Maíra Canal wrote: > As the format validation is being dealt with exclusively > inside framebuffer_check(), there is no need to export the > drm_any_plane_has_format() symbol. > > Therefore, unexport the drm_any_plane_has_format() symbol, reinforcing > that format validation is being dealt with by the DRM API. > > Signed-off-by: Maíra Canal Usually we also delete the kerneldoc at that point, since that's mainly for driver authors and they don't need this anymore. With that Also please move the function decl to the drm_crtc_internal.h since this is no longer used outside of drm core/helper code. Reviewed-by: Daniel Vetter > --- > drivers/gpu/drm/drm_plane.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/gpu/drm/drm_plane.c b/drivers/gpu/drm/drm_plane.c > index 24e7998d1731..67c0ab60c7b6 100644 > --- a/drivers/gpu/drm/drm_plane.c > +++ b/drivers/gpu/drm/drm_plane.c > @@ -837,7 +837,6 @@ bool drm_any_plane_has_format(struct drm_device *dev, > > return false; > } > -EXPORT_SYMBOL(drm_any_plane_has_format); > > /* > * __setplane_internal - setplane handler for internal callers > -- > 2.39.0 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch