Received: by 2002:a6b:500f:0:0:0:0:0 with SMTP id e15csp1650049iob; Fri, 29 Apr 2022 09:43:08 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwzZLR2ihO5yyedeK4t+sjUQ/RUQNxLckH076HyEghlmr5a5ZP3wXF3MQ6nnw3IgZw6NPBa X-Received: by 2002:a65:4848:0:b0:39c:c393:688c with SMTP id i8-20020a654848000000b0039cc393688cmr225504pgs.376.1651250588415; Fri, 29 Apr 2022 09:43:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1651250588; cv=none; d=google.com; s=arc-20160816; b=aW8q4YW9xWVGllXfGZNbwT3NBvNM6X3Av9Fe82JhmH437xV6itBZuor4l9qW9L4+GO /6C46je0qsYfE+NL5kBtutmhQwg+sgt/doMNGP+Wgq3b0ZoBmAmahwudB7CBNrEgMwgs aDcDbzVQ7IKcFRTGIDfzPUULPHTraC0BnmxzeNYmxhGMsOzn85yJYGUyfLqRWhj12/fE HBea2wMftqMIPcUafMB/OaZYt4oxFaMKiM8+DoD6psCYRVrVOMicsCloFjXgpRLS++3T sz6Nn2TvREqn4IMHC0x8z8yQEmStu/OypdDHw1RGlI6ZO54cId125KWxU2fN5yhyn85J Oywg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:to:content-language:subject:user-agent:mime-version:date :message-id:dkim-signature; bh=rWD+nFd9qIZ4xdleJ2T+rOAtIdms6JGRH8x/6bfc2vc=; b=jTvNxo4fhc0hq3HgBSsONbGSMQENIaVxVGr7bCq6UHjb1uUjcTd2AXU4N3CMBtL7BD od7mVRdh/UQLyuXz41hLUdTjfRAQdPJCHGvuvdWgS9HMtthS0QBHbE2A5f1ix0k5RCGL hiscKJLoc96VdDR31O51DN+DSEH5/lne53nizVgiDqDEido/WRFdp8zBR4mZVADO7dfw p5bPaz++jYyKB6v7RYkmWYDGjL8u0+C5Iu7XnmU3zkBEHGHm05uHUu4vaXChDLZtHFTn eIDtaAkPuWlgNg2bjii6Sfpj0nPkErPphDSY4iP+OwAK7dgaYIEHoKsFIdYFUPf1CT8/ 3eUQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=Gfq++VVz; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=collabora.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id oo12-20020a17090b1c8c00b001ca8db710bcsi12694978pjb.179.2022.04.29.09.42.51; Fri, 29 Apr 2022 09:43:08 -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 header.i=@collabora.com header.s=mail header.b=Gfq++VVz; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=collabora.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1351176AbiD1Slc (ORCPT + 99 others); Thu, 28 Apr 2022 14:41:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54046 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232896AbiD1Slb (ORCPT ); Thu, 28 Apr 2022 14:41:31 -0400 Received: from bhuna.collabora.co.uk (bhuna.collabora.co.uk [46.235.227.227]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3E2B5BE9EE for ; Thu, 28 Apr 2022 11:38:16 -0700 (PDT) Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: dmitry.osipenko) with ESMTPSA id 26DAB1F45BA6 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1651171094; bh=LUoHkp961qm9ko1OBtouQWFIA8PMdZVSkHc/jw1XS44=; h=Date:Subject:To:References:From:In-Reply-To:From; b=Gfq++VVz0zfr6JFt1ax6Utc5gsxcHoJIm5n0+6KpdFYaHipbvdGcXdvSCeE4iToqp du8kN1/0uHwQg8uWiF4B+RQF5Sww9YSZq5/uKMtVA/iMrJpoVHRwu4GzEK3pbolvUJ jg1ZmnkE/osv2uVCh8aVrQt6C9IQJD2ZKuWW/JoS9vAPlelhwOMVOvLFQLl7P1FcXo 5BjvLzIPrhmEhbIj7WYtdUGHWEGPE3rQG5GyqHDmwLWPDVIdQ4bc52/kZKjn1F1+7s 0Yigqu3B//kV5MYZE6/sAzQ/uWp1VvY/pqlAFb2lGpLqAc3b5QYXQLKoaz7OW+Muev m55mWpLIa2xKQ== Message-ID: <9cb2e7fb-1597-999e-59b8-586b5fd7f04d@collabora.com> Date: Thu, 28 Apr 2022 21:38:10 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 Subject: Re: [PATCH v5 15/17] drm/shmem-helper: Make drm_gem_shmem_get_pages() private Content-Language: en-US To: David Airlie , Gerd Hoffmann , Gurchetan Singh , Chia-I Wu , Daniel Almeida , Gert Wollny , Gustavo Padovan , Daniel Stone , Tomeu Vizoso , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Rob Herring , Steven Price , Alyssa Rosenzweig , Rob Clark , Emil Velikov , Robin Murphy , Qiang Yu , Sumit Semwal , =?UTF-8?Q?Christian_K=c3=b6nig?= , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, virtualization@lists.linux-foundation.org, Dmitry Osipenko References: <20220424190424.540501-1-dmitry.osipenko@collabora.com> <20220424190424.540501-16-dmitry.osipenko@collabora.com> From: Dmitry Osipenko In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,SPF_HELO_PASS, SPF_PASS,UNPARSEABLE_RELAY 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 28.04.2022 15:33, Daniel Vetter пишет: > On Sun, Apr 24, 2022 at 10:04:22PM +0300, Dmitry Osipenko wrote: >> VirtIO-GPU driver was the only user of drm_gem_shmem_get_pages() >> and it now uses drm_gem_shmem_get_pages_sgt(). Make the get_pages() >> private to drm_gem_shmem_helper. >> >> Signed-off-by: Dmitry Osipenko >> --- >> drivers/gpu/drm/drm_gem_shmem_helper.c | 3 +-- >> include/drm/drm_gem_shmem_helper.h | 1 - >> 2 files changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/drivers/gpu/drm/drm_gem_shmem_helper.c b/drivers/gpu/drm/drm_gem_shmem_helper.c >> index 25e9bc2803ee..7ec5f8002f68 100644 >> --- a/drivers/gpu/drm/drm_gem_shmem_helper.c >> +++ b/drivers/gpu/drm/drm_gem_shmem_helper.c >> @@ -490,7 +490,7 @@ static int drm_gem_shmem_get_pages_locked(struct drm_gem_shmem_object *shmem) >> * Returns: >> * 0 on success or a negative error code on failure. >> */ > > We also delete the kerneldoc for functions not exported (kerneldoc is > geared towards driver writes). If there's anything critical the comment > explains about the internals, you can keep that as a normal C style > comment without the /** but generally there's no need for these anymore. Noted for v6.