Received: by 2002:a05:7412:b101:b0:e2:908c:2ebd with SMTP id az1csp3286208rdb; Thu, 16 Nov 2023 05:48:06 -0800 (PST) X-Google-Smtp-Source: AGHT+IGj8h4ju82+VXr9n2z6NJa8lccX/7yt1f+iIjwIhDB7Tte1nLKf+HlQpLA7fRHmvOAWtChH X-Received: by 2002:a17:90b:1646:b0:25e:a8ab:9157 with SMTP id il6-20020a17090b164600b0025ea8ab9157mr17014909pjb.22.1700142486127; Thu, 16 Nov 2023 05:48:06 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700142486; cv=none; d=google.com; s=arc-20160816; b=Fye+AYwsh8aA5mrK8vhvZ7QQ8K7lCOrU24hJYjIGnMmlUoIM8H3enIwdoybodYumzX HdO+jFipaD/qD/a/yCrTyMzqJ2hExcwg06eRu9IKoww8aQdfbpMi6ZYJd4Qm5HIMabqJ wt+CqZltLomnrOTd58069xDT06swlV4z10geYWbyHuZNE3+BPDxnzgYEh2Y7ISNZmqMv PMxfEtv6ZYsSbzDGZ/bJZYJJfjUqI6ivUZ3cA7qoCkDzKd4SM5yzRb1+uApPumupkWdf DMTNDdKgUTzyM4l6KJBpXoWo8YrvGwlBNc4MHaWfZQDoKWWj0hs9GKIgVjjU1hHsZ8+H VdeA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=d3GOUGlPKpK0MwKblUSzhKhCMINcf4CyZw/m9LfeVRo=; fh=aOUCuNRk3iURWs6yG766tOkEYeeMgh4ugeCCoDdq5qM=; b=RmE5x0kIyLqw9zMDEecKG1cVyTfSoKfmQs8ECwbtHTqQEuh4Zh8YCt9H6z2FadCfOX 8+yCr+AXRT3F8XGxpKaYrwlNpSoDv5IVSpn/I/VXUDPazv4Rwii/EwoIkuMox0wDOkZd 2KejK8Zxv9pdH+Ru923D9lAjTffqMRS8UpmI8UevYana+hcI3S0wi0eSC8/eJgepMSfS 4PdiwmqyoBv+0g1cuf/Sr2EAb2k/pEGyD7vdsrpFgE/QBJoVfIaBhQMULIFXV+6ja5SF ZFr+THpoEWfv4cV7cwIDpIKRCB97K/GgksMc7XWEXK1brlIQU3bmyV58JQ5kAPV1mMVd w18w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=Ennw2qkv; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Return-Path: Received: from snail.vger.email (snail.vger.email. [2620:137:e000::3:7]) by mx.google.com with ESMTPS id w12-20020a17090a8a0c00b00280cd49099asi2074279pjn.46.2023.11.16.05.48.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 16 Nov 2023 05:48:06 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7; Authentication-Results: mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=Ennw2qkv; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id 218B8820E501; Thu, 16 Nov 2023 05:48:05 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345238AbjKPNsF (ORCPT + 99 others); Thu, 16 Nov 2023 08:48:05 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34072 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231149AbjKPNsE (ORCPT ); Thu, 16 Nov 2023 08:48:04 -0500 Received: from madras.collabora.co.uk (madras.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e5ab]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 45CC8C1; Thu, 16 Nov 2023 05:48:01 -0800 (PST) Received: from mercury (cola.collaboradmins.com [195.201.22.229]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: sre) by madras.collabora.co.uk (Postfix) with ESMTPSA id B4995660734A; Thu, 16 Nov 2023 13:47:59 +0000 (GMT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1700142479; bh=0F+bD5IJvC5DvD53eU1GEpAuHcSfiOhDXPjgQwy4gxU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Ennw2qkv+TWcuWKJb/JxsE2jdCn3IKctEidn6+LBnljCe+v1UPs4IrEDksAcfrbkP i/NA/dK8J1rHvXw6S3r3jO7jTHYPtSOfCi2G2oRm2RIbEzPmnhaLuelKA+pQpBL8y4 /q9u2CLaklIlN7g3XcC9aEPaUFMtdMzK64qKkNG73gPlvHkOavshCDgCZjM0C0pDDT rihxJ1eRSLOTm3TYuHngAsXEKQckU9bKtfP+3mlLODjkX3RHMIcBjiPZ+o4oTPs1uO aqzt89HdHbgYsIkMHYFYkhvOUUPkte3vR9J1J8ox5s3tAe4F9JIfGrBuMX9QC+4ah2 houZmc0+DWMJg== Received: by mercury (Postfix, from userid 1000) id DA79210613A1; Thu, 16 Nov 2023 14:47:57 +0100 (CET) Date: Thu, 16 Nov 2023 14:47:57 +0100 From: Sebastian Reichel To: Andy Yan Cc: Sascha Hauer , Andy Yan , 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, kever.yang@rock-chips.com, chris.obbard@collabora.com Subject: Re: [PATCH 09/11] drm/rockchip: vop2: Add support for rk3588 Message-ID: <20231116134757.zu7axb6cvriqjm6y@mercury.elektranox.org> References: <20231114112534.1770731-1-andyshrk@163.com> <20231114112855.1771372-1-andyshrk@163.com> <20231115090823.GY3359458@pengutronix.de> <8f2ebc81-51c5-44d5-b27b-633a6cc85d0d@rock-chips.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="fodj6ulte5c5gkb5" Content-Disposition: inline In-Reply-To: <8f2ebc81-51c5-44d5-b27b-633a6cc85d0d@rock-chips.com> 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: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Thu, 16 Nov 2023 05:48:05 -0800 (PST) --fodj6ulte5c5gkb5 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Thu, Nov 16, 2023 at 06:39:40PM +0800, Andy Yan wrote: > > > vop2->sys_grf =3D syscon_regmap_lookup_by_phandle(dev->of_node, "r= ockchip,grf"); > > This already lacks an error check, shame on me... > >=20 > > > + vop2->vop_grf =3D syscon_regmap_lookup_by_phandle(dev->of_node, "ro= ckchip,vop-grf"); > > > + vop2->vo1_grf =3D syscon_regmap_lookup_by_phandle(dev->of_node, "ro= ckchip,vo1-grf"); > > > + vop2->sys_pmu =3D syscon_regmap_lookup_by_phandle(dev->of_node, "ro= ckchip,pmu"); > > ... but please don't duplicate that. >=20 > It a little difficult to find a proper way to do the check, as not every = soc need all these phandles. >=20 > Do i need check it per soc? I suggest adding a u32 flags to struct vop2_data and then have something like this: if (vop2_data->flags & VOP2_HAS_VOP_GRF) { vop2->vop_grf =3D syscon_regmap_lookup_by_phandle(dev->of_node, "rockch= ip,vop-grf"); if (IS_ERR(vop2->vop_grf)) return dev_err_probe(dev, PTR_ERR(vop2->vop_grf) "cannot get vop-gr= f"); } if (vop2_data->flags & VOP2_HAS_VO1_GRF) { vop2->vo1_grf =3D syscon_regmap_lookup_by_phandle(dev->of_node, "rockch= ip,vo1-grf"); if (IS_ERR(vop2->vo1_grf)) return dev_err_probe(dev, PTR_ERR(vop2->vo1_grf) "cannot get vo1-gr= f"); } =2E.. Greetings, -- Sebastian --fodj6ulte5c5gkb5 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmVWHYUACgkQ2O7X88g7 +po82w//W3u/dHpgUXr+77xCVn2g3AVTWISnP6mEQ1VQkqATekZ8LpLJq7thz2pp yuq3rG6e35mWVHVex5KNFzElA0eKPlHZ6Yy41mVK5lFn5/B00gVTKyY1Ke4b5gHS ub3KYPmhEssqLioA4H+8w7clThuT6CClRrhH7snBTh+/Wq99ivsJTRvib3v5V1jE nqzw/UnvodV54WlytiYADIsuRbcURk1BJx7xNgjuBBGWVjtOoUKPoJ4gqM8VpDmz Ibc0H42k82E4XnWDbMZUaCEA2DqZOgNwlEAwX3lQGuJE6LYlUt/hRcU6+31AzdSh ch9mGOFXz+8/6OxyWOD4uktftyiYAKFNL7Hon+z3pxBqoc2Bt94pcUlnfFyJvPgQ WoaCQH4crpeYkPH4fo5QCyW5sUdtn12cM1ihF1JsUicAMmzsLcUagXKL7c+ObVrz vCyPZGQDXuwiiNLSvkmG6yRrmAQFtquboGBPZOTH46fFNuASkR9ZuGEDm0nIH9Mh AwWYqdlz9h/bg0vR/uXqro0DI0gx7TJwap870n9v5gvfK/e4MWM15W/VzbUv0Kxo TL1+LtgAtDGbGVeuI3DHY94dzYwsov6yrViwuFC/wIgllZlHgK5Ir9su9XCfVCt1 SP5ybo1ag2VdjC88XQBHSV5m7hoKHC0PEJzAhtjvSSjbazh2PsM= =DnDq -----END PGP SIGNATURE----- --fodj6ulte5c5gkb5--