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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DBA49C433EF for ; Thu, 14 Apr 2022 12:47:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241991AbiDNMt5 (ORCPT ); Thu, 14 Apr 2022 08:49:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39418 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229485AbiDNMty (ORCPT ); Thu, 14 Apr 2022 08:49:54 -0400 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 4240D91543; Thu, 14 Apr 2022 05:47:30 -0700 (PDT) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 0F6D3139F; Thu, 14 Apr 2022 05:47:30 -0700 (PDT) Received: from [10.1.32.20] (e122027.cambridge.arm.com [10.1.32.20]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D2D2A3F70D; Thu, 14 Apr 2022 05:47:27 -0700 (PDT) Message-ID: Date: Thu, 14 Apr 2022 13:47:25 +0100 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 v6 1/2] dt-bindings: Add DT schema for Arm Mali Valhall GPU Content-Language: en-GB To: AngeloGioacchino Del Regno , Nick Fan , Rob Herring , Matthias Brugger Cc: devicetree@vger.kernel.org, srv_heupstream@mediatek.com, David Airlie , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Project_Global_Chrome_Upstream_Group@mediatek.com, linux-mediatek@lists.infradead.org, alyssa.rosenzweig@collabora.com, wenst@chromium.org, linux-arm-kernel@lists.infradead.org References: <20220414025023.11516-1-Nick.Fan@mediatek.com> From: Steven Price In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 14/04/2022 12:51, AngeloGioacchino Del Regno wrote: > Il 14/04/22 04:50, Nick Fan ha scritto: >> Add devicetree schema for Arm Mali Valhall GPU >> >> Define a compatible string for the Mali Valhall GPU >> for MediaTek's SoC platform. >> >> Signed-off-by: Nick Fan > > Hello Nick, > Unfortunately, this binding is completely wrong. I think that's unfair, although there is room for improvement. > First of all, there's no arm,mali-valhall driver upstream - this will be > managed > by panfrost later, yes, but right now there's no support. We need a binding agreed upon before support can be added. > Then, you're also setting opp-microvolt in a way that will never (or, at > least, > not anytime soon) be supported by the upstream driver, as it manages > only one > supply for devfreq scaling. The mt8183 binding (already in tree) is very similar. The binding also should be describing the hardware not what the driver supports. There are indeed limitations in Panfrost for supporting multiple supplies, but that's something that needs improving in the driver not a reason to block a (presumably correct) description of the hardware. I can't comments on whether the specifics of the mt8192 are correct. > Besides, please don't push bindings that have no upstream driver, > especially if > these are for downstream drivers requiring proprietary components, while a > completely open source implementation is in the works. More constructively, Alyssa has already posted a patch (as part of the series adding driver support) which would extend the existing Bifrost bindings to (pre-CSF) Valhall: https://lore.kernel.org/dri-devel/20220211202728.6146-2-alyssa.rosenzweig@collabora.com/ I'm not sure I see the point of having a separate binding document for Valhall considering the (pre-CSF) hardware is the same from the kernel perspective. Thanks, Steve