Received: by 2002:a05:7412:8d10:b0:f3:1519:9f41 with SMTP id bj16csp166912rdb; Tue, 5 Dec 2023 01:45:51 -0800 (PST) X-Google-Smtp-Source: AGHT+IG9XenNO2ToKTU3N9dqiwWNCZ4ieeYqECyicEWRFZE2wxWt30S/xAwIimYM8sBkq7Dg6fs2 X-Received: by 2002:a05:6a20:da95:b0:18f:97c:6156 with SMTP id iy21-20020a056a20da9500b0018f097c6156mr7518254pzb.83.1701769550866; Tue, 05 Dec 2023 01:45:50 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1701769550; cv=none; d=google.com; s=arc-20160816; b=tnkK26WDLUTUuUc4mJzR+U6S8hjHcygvtKIqsnCXH30v7zcMYIrWxelartjWLKvpFF rACEgOzh/V8OcOymBPhgRqk7t9cGaHjOhT3f7eOzk7VY567KQPhU8IzMWlD7MkXKxFx0 YwaccZced9KmTQCM31SRuZzZMHfCfPMBSbmDZltqMgIYa+tnzyn8DMD/dqxLuDTW4/k6 IxeN/s3BZ420ARSanGsXYO/TLZ8rbAolkorwz3boVaui/CLkVApzo6jfnX4W5gjJ+wdK 806Y76+xo5Qmo1rNI3Ofk4h3BJh77/zPlNMCVf+rsw6BoPOhFyMIVKNQ7mON7+kt4tPO RS+Q== 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:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=Qwe7/iDLwPEEdgYbXXnkzwx2HO9asx5VCXEH67b0F/w=; fh=2LdFoz7lQpY9fy9tL2U29gDaVmV5pFNOmSoRkDMmCMo=; b=jNs1AI4vtWP4+D8pyTHW1B2UV2mvw/SgZv806ZdJgRuhiOOqBP8CzwG5OFx/PzV9gO zo4Jsr2sW4nQnw595xI+dRxs9q6v7wDMd+QpLJ5OYfor84lxFqRFOb+QBpF4RsK2ksDH V49QaUTZhs+7fTehtkEtzH7bKtylD9ylj6O+RO57yxK7e65Q6AI+ckodliEV9sGgc96Y O2kn9OXtQ2cKoBmIdbAcvAcmWe/v8RrGOpxjpo1lcQvANIrVCML/vfc4Wjwfj/GYs2Za uRmaoNOE1r1GgDLSkL4nel5tZhW8f3XNQEhjO/W3w+CIpMO/TAdduc8UTZsX6AM5rlGq pMAw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@rock-chips.com header.s=default header.b=dGZ4BWZt; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=rock-chips.com Return-Path: Received: from morse.vger.email (morse.vger.email. [23.128.96.31]) by mx.google.com with ESMTPS id 142-20020a630194000000b005bd2b19dfc4si9746999pgb.493.2023.12.05.01.45.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 05 Dec 2023 01:45:50 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) client-ip=23.128.96.31; Authentication-Results: mx.google.com; dkim=pass header.i=@rock-chips.com header.s=default header.b=dGZ4BWZt; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=rock-chips.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by morse.vger.email (Postfix) with ESMTP id 9B974809926C; Tue, 5 Dec 2023 01:45:39 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at morse.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235005AbjLEJpO (ORCPT + 99 others); Tue, 5 Dec 2023 04:45:14 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33042 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235477AbjLEJo7 (ORCPT ); Tue, 5 Dec 2023 04:44:59 -0500 Received: from mail-m11876.qiye.163.com (mail-m11876.qiye.163.com [115.236.118.76]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DC93E124; Tue, 5 Dec 2023 01:44:54 -0800 (PST) DKIM-Signature: a=rsa-sha256; b=dGZ4BWZty1d0oIWgBZ6X8FBIyGDPCj743XxLuAfNp+xa34p6BSXVPPWnrai2cFtu/MHaXQF1gs5x6htnfVWl192MLEzMqyi5frolErK8Uhb06vt8nQrgBdjNV6XOr4AZV9ICA09I88dZL2kgtm/Es/DqW6YWSYfL+Qc52ZB/YV8=; c=relaxed/relaxed; s=default; d=rock-chips.com; v=1; bh=Qwe7/iDLwPEEdgYbXXnkzwx2HO9asx5VCXEH67b0F/w=; h=date:mime-version:subject:message-id:from; Received: from [172.16.12.141] (unknown [58.22.7.114]) by mail-m12779.qiye.163.com (Hmail) with ESMTPA id 59D887801C1; Tue, 5 Dec 2023 17:44:03 +0800 (CST) Message-ID: <87831dc3-2554-4b53-a9f8-6b61cf67732e@rock-chips.com> Date: Tue, 5 Dec 2023 17:44:03 +0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 11/14] drm/rockchip: vop2: Add support for rk3588 Content-Language: en-US To: Sascha Hauer , Andy Yan Cc: heiko@sntech.de, hjc@rock-chips.com, dri-devel@lists.freedesktop.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, krzysztof.kozlowski+dt@linaro.org, robh+dt@kernel.org, devicetree@vger.kernel.org, sebastian.reichel@collabora.com, kever.yang@rock-chips.com, chris.obbard@collabora.com References: <20231130122001.12474-1-andyshrk@163.com> <20231130122439.13374-1-andyshrk@163.com> <20231205092936.GW1057032@pengutronix.de> From: Andy Yan In-Reply-To: <20231205092936.GW1057032@pengutronix.de> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-HM-Spam-Status: e1kfGhgUHx5ZQUpXWQgPGg8OCBgUHx5ZQUlOS1dZFg8aDwILHllBWSg2Ly tZV1koWUFDSUNOT01LS0k3V1ktWUFJV1kPCRoVCBIfWUFZGU1DS1ZMGkpCGUoZGEgZTkxVEwETFh oSFyQUDg9ZV1kYEgtZQVlOQ1VJSVVMVUpKT1lXWRYaDxIVHRRZQVlPS0hVSk1PSU5JVUpLS1VKQl kG X-HM-Tid: 0a8c395cf962b24fkuuu59d887801c1 X-HM-MType: 1 X-HM-Sender-Digest: e1kMHhlZQR0aFwgeV1kSHx4VD1lBWUc6NRQ6PQw6PDw0FgMVDBpKIykY GQxPCRZVSlVKTEtKTE1CT09PSUJIVTMWGhIXVRoVHwJVAhoVOwkUGBBWGBMSCwhVGBQWRVlXWRIL WUFZTkNVSUlVTFVKSk9ZV1kIAVlBSElNTjcG X-Spam-Status: No, score=0.6 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, RCVD_IN_SORBS_WEB,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on morse.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (morse.vger.email [0.0.0.0]); Tue, 05 Dec 2023 01:45:39 -0800 (PST) Hi Sascha: On 12/5/23 17:29, Sascha Hauer wrote: > On Thu, Nov 30, 2023 at 08:24:39PM +0800, Andy Yan wrote: >> From: Andy Yan >> >> VOP2 on rk3588: >> >> Four video ports: >> VP0 Max 4096x2160 >> VP1 Max 4096x2160 >> VP2 Max 4096x2160 >> VP3 Max 2048x1080 >> >> 4 4K Cluster windows with AFBC/line RGB and AFBC-only YUV support >> 4 4K Esmart windows with line RGB/YUV support >> >> Signed-off-by: Andy Yan > > With the two nits below feel free to add my: > > Reviewed-by: Sascha Hauer > > Thanks for working on this. > >> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop2.h b/drivers/gpu/drm/rockchip/rockchip_drm_vop2.h >> index 8d7ff52523fb..8b16031eda52 100644 >> --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop2.h >> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop2.h >> @@ -13,9 +13,16 @@ >> >> #define VOP_FEATURE_OUTPUT_10BIT BIT(0) > > You could rename this to include "VP" for Video Port so it's not so > easily mixed up with the defines below. Yes, I have the same idea, maybe it's better to do the rename in a separate ? > >> >> +#define VOP2_FEATURE_HAS_SYS_GRF BIT(0) >> +#define VOP2_FEATURE_HAS_VO0_GRF BIT(1) >> +#define VOP2_FEATURE_HAS_VO1_GRF BIT(2) >> +#define VOP2_FEATURE_HAS_VOP_GRF BIT(3) >> +#define VOP2_FEATURE_HAS_SYS_PMU BIT(5) > > Should be BIT(4) Thanks for catching this, will fix in next version. > > Sascha >