Message ID | 20230623100822.274706-1-sui.jingfeng@linux.dev |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp5672758vqr; Fri, 23 Jun 2023 03:38:40 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ7HeLbUIiXrMc+WJM8Iz1gv+hO9MbdmWn4NowUX+pkXbAiQyjrqlYsHRSSzySa07KOPpjap X-Received: by 2002:a05:6a20:2591:b0:123:828f:68d with SMTP id k17-20020a056a20259100b00123828f068dmr8755725pzd.13.1687516720108; Fri, 23 Jun 2023 03:38:40 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1687516720; cv=none; d=google.com; s=arc-20160816; b=0XnvrDYZZMJocLJKfxCcbQ+d8CwsGeWCnIq52HOr5yRXKzJiqD2r9kyEbhHpJy7Byc xvX66c8zED6YRFagTIT2GXNWcr79T/9lvrBh/qjQU/SYJ8FVqL7gIC+c1awwcvlH3H+f kOocTKpdr8u2rcm2ktlJ5XgajaP7fMtJP4G+BQrqfdQY7FL86uAlCsKLbTBetoRMuBXa ZniqDpesl2zLl1yxMCl67epvyEO0HpQBjUsL7C/1TYSSLBsD+oM2IeIOS0vH9ZXVccgU 6XT9v3CUg4DTP6r5kBBn56mfieFjBRrTG155TAh5s1b0yr/fdVR3Z6GQhojLGzDShiMl 1GMQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=WQvQ1/8ew72J0lPd28yNsu5tnN3fJmOWFY9fr+RpJdw=; b=z0AFpUilS9nzGJK8FJUZkQWzSWw435/ZNoOnHBFnYgsQ9Zd2yj+m388LVXxU3iTGX/ MKiX7lH0pyIvCneWEO75sYdq/94bhRpoUX+fCOEG0dyT7kaYXHCiAxvGmG9UMoZwqtJt Pqx/kHil6naoq+TSnujvn1ntEX40wh8U5E0dAsjlWNHtl6v798V+/kdA3iqrxMb0nj5E aeNDM0gdIrxOcxWwXPH5G5BuytFMKy5pCTicSCh0wREJF75GI9TVCkUBZtNTv8IXF+Ae nKGlkQjjw0VFdu1jzOqMxPVZxodKWlscKSK6mj2f2+S1puXbKsaucYLCoffgWE6kVp2A a/0g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linux.dev header.s=key1 header.b=jwmZ6pTy; 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=linux.dev Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id y4-20020a63ad44000000b005439d1acbbcsi8500034pgo.440.2023.06.23.03.38.27; Fri, 23 Jun 2023 03:38:40 -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=@linux.dev header.s=key1 header.b=jwmZ6pTy; 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=linux.dev Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231667AbjFWKRW (ORCPT <rfc822;maxin.john@gmail.com> + 99 others); Fri, 23 Jun 2023 06:17:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56728 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231518AbjFWKRQ (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Fri, 23 Jun 2023 06:17:16 -0400 Received: from out-33.mta0.migadu.com (out-33.mta0.migadu.com [IPv6:2001:41d0:1004:224b::21]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D7E76E75 for <linux-kernel@vger.kernel.org>; Fri, 23 Jun 2023 03:17:05 -0700 (PDT) X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1687514907; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=WQvQ1/8ew72J0lPd28yNsu5tnN3fJmOWFY9fr+RpJdw=; b=jwmZ6pTywJ2krzZsDq3HjK7YXRqLJREJOP/aKd1VPXwi8EHrGBlxcFsG7bmb3VuYIYDIqX H68rmmEGjFLohzjNfmCSXXweAjP17iBHfkqsmnTfWZbPRNe1kS2wZ3rw5cZ32sHeIFoSQX 9m5BB/xHZe/63cSH3icuf4ux1cjMN0k= From: Sui Jingfeng <sui.jingfeng@linux.dev> To: Lucas Stach <l.stach@pengutronix.de>, Russell King <linux+etnaviv@armlinux.org.uk>, Christian Gmeiner <christian.gmeiner@gmail.com>, David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch> Cc: etnaviv@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, loongson-kernel@lists.loongnix.cn, Sui Jingfeng <suijingfeng@loongson.cn> Subject: [PATCH v1 0/8] drm/etnaviv: Various cleanup Date: Fri, 23 Jun 2023 18:08:14 +0800 Message-Id: <20230623100822.274706-1-sui.jingfeng@linux.dev> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Migadu-Flow: FLOW_OUT 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_NONE,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: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1769489532204829593?= X-GMAIL-MSGID: =?utf-8?q?1769489532204829593?= |
Series |
drm/etnaviv: Various cleanup
|
|
Message
Sui Jingfeng
June 23, 2023, 10:08 a.m. UTC
From: Sui Jingfeng <suijingfeng@loongson.cn>
No functional change.
Sui Jingfeng (8):
drm/etnaviv: Using the size_t variable to store the number of pages
drm/etnaviv: Using the unsigned int type to count the number of pages
drm/etnaviv: Drop the second argument of the etnaviv_gem_new_impl()
drm/etnaviv: Remove surplus else after return
drm/etnaviv: Keep the curly brace aligned
drm/etnaviv: No indentation by double tabs
drm/etnaviv: Add dedicated functions to create and destroy platform
device
drm/etnaviv: Add a helper to get a pointer to the first available node
drivers/gpu/drm/etnaviv/etnaviv_drv.c | 100 +++++++++++++-------
drivers/gpu/drm/etnaviv/etnaviv_gem.c | 14 +--
drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c | 7 +-
3 files changed, 77 insertions(+), 44 deletions(-)
Comments
Hi, Dear etnaviv folks Would you like to review this cleanup patch set ? I am asking because I'm wondering that if I should re-spin my other patch from the code base which *with* this series applied or from the code base *without* this series applied. I think this series looks fine, is it acceptable? On 2023/6/23 18:08, Sui Jingfeng wrote: > From: Sui Jingfeng <suijingfeng@loongson.cn> > > No functional change. > > Sui Jingfeng (8): > drm/etnaviv: Using the size_t variable to store the number of pages > drm/etnaviv: Using the unsigned int type to count the number of pages > drm/etnaviv: Drop the second argument of the etnaviv_gem_new_impl() > drm/etnaviv: Remove surplus else after return > drm/etnaviv: Keep the curly brace aligned > drm/etnaviv: No indentation by double tabs > drm/etnaviv: Add dedicated functions to create and destroy platform > device > drm/etnaviv: Add a helper to get a pointer to the first available node > > drivers/gpu/drm/etnaviv/etnaviv_drv.c | 100 +++++++++++++------- > drivers/gpu/drm/etnaviv/etnaviv_gem.c | 14 +-- > drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c | 7 +- > 3 files changed, 77 insertions(+), 44 deletions(-) >
Hi Jingfeng, Am Montag, dem 17.07.2023 um 16:36 +0800 schrieb suijingfeng: > Hi, Dear etnaviv folks > > > Would you like to review this cleanup patch set ? > > I am asking because I'm wondering that > > if I should re-spin my other patch from the code base > > which *with* this series applied or from the code base > > *without* this series applied. > > > I think this series looks fine, is it acceptable? > I've gone through the series and left some comments. All patches that I didn't comment on look fine to me. I'm generally in favor of taking this series. Regards, Lucas > > On 2023/6/23 18:08, Sui Jingfeng wrote: > > From: Sui Jingfeng <suijingfeng@loongson.cn> > > > > No functional change. > > > > Sui Jingfeng (8): > > drm/etnaviv: Using the size_t variable to store the number of pages > > drm/etnaviv: Using the unsigned int type to count the number of pages > > drm/etnaviv: Drop the second argument of the etnaviv_gem_new_impl() > > drm/etnaviv: Remove surplus else after return > > drm/etnaviv: Keep the curly brace aligned > > drm/etnaviv: No indentation by double tabs > > drm/etnaviv: Add dedicated functions to create and destroy platform > > device > > drm/etnaviv: Add a helper to get a pointer to the first available node > > > > drivers/gpu/drm/etnaviv/etnaviv_drv.c | 100 +++++++++++++------- > > drivers/gpu/drm/etnaviv/etnaviv_gem.c | 14 +-- > > drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c | 7 +- > > 3 files changed, 77 insertions(+), 44 deletions(-) > > >