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 X-Spam-Level: X-Spam-Status: No, score=-10.1 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C5AF5C43461 for ; Mon, 17 May 2021 10:24:58 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6C780611ED for ; Mon, 17 May 2021 10:24:58 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6C780611ED Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 8EF9D6E067; Mon, 17 May 2021 10:24:57 +0000 (UTC) Received: from mail-wr1-x42d.google.com (mail-wr1-x42d.google.com [IPv6:2a00:1450:4864:20::42d]) by gabe.freedesktop.org (Postfix) with ESMTPS id D330F6E921 for ; Mon, 17 May 2021 10:24:55 +0000 (UTC) Received: by mail-wr1-x42d.google.com with SMTP id h4so5761448wrt.12 for ; Mon, 17 May 2021 03:24:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20150623.gappssmtp.com; s=20150623; h=subject:to:cc:references:from:organization:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=1uqNXUpVyuomhEWX3uzRubVqH//B/oaEGgllzNXN/bQ=; b=s71N+Em/VMij/lL/5g5CtNDBocNukAPXqlIoUo7Uyt+f+cF+yWkr9s7CQ3qv/6lQPI Yb1UdBUD14KQw3qsG84gfZ96qWozHhlyj7gbdOp2qWx4DWj6jGhPCTil3B0F67k9zQhu woOh/nF21F8Uo06a9hXOu+1I53OjT9b/Zbzgp6kVbk6yE3CAfVAbVEeiXlwImzDwFIc+ N0PFWdi+jEVZvATjC+4QXCHSMbFwCiSdRb1s3FrIlrTivn17VUtNPX5piWM1g1l1KXyX ifPebyVmh9iKuncI9eKLJoRmBW2/lCPjw5lDbjpCJsNHIA/nrzNh32PJgOhE6GRa/1bL p6yw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:organization :message-id:date:user-agent:mime-version:in-reply-to :content-language:content-transfer-encoding; bh=1uqNXUpVyuomhEWX3uzRubVqH//B/oaEGgllzNXN/bQ=; b=hi5ZhOuYEcV2u6fW87ApXczpLpkJ8qL5dDjVwYVhsWjvV60YX3GgWpUxXeZ4XRt4Oc wazOVUD73zWEBPsoaDu5KdVTsbSW729NULsZqwqXL5+GFDzZux/6fmOtSrPd1WD5944H rDRssZfe3P3e7hZ5uzOucLTFXAD1fyS+/Tj0ySXhYp8nqWnFfNMBqHUWRW13GEXWzJLt q/7sycSr92RLvnUK1funD9rApKCqxe2P0LEyvvjqGJSythf5bNIkXQocaJP9XVsV/G/T 5jvc9AlOOKJIYKGCDNhjzFILtHEZnAAa5joCnMV4jNaoeHV1wBAaHpe+gAHIEQ89dnbb DSLQ== X-Gm-Message-State: AOAM532tfM0AoTHRShJTi1qKIMVHAcGtWXvShyieQNB8XASMhB4frpLw IrSmyCLLcfnLE1r78dsax7dt/A== X-Google-Smtp-Source: ABdhPJyvDDcgOB1SFtKlawbsc996gmsnMdi83ARlyPr2D3PYLKd4sf/6Tow1BtYpLrEXi7LIhr3ebg== X-Received: by 2002:a05:6000:188d:: with SMTP id a13mr38290175wri.61.1621247094350; Mon, 17 May 2021 03:24:54 -0700 (PDT) Received: from ?IPv6:2a01:e0a:90c:e290:3044:ffc:eaa9:6a70? ([2a01:e0a:90c:e290:3044:ffc:eaa9:6a70]) by smtp.gmail.com with ESMTPSA id y21sm21092169wmi.15.2021.05.17.03.24.51 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 17 May 2021 03:24:53 -0700 (PDT) Subject: Re: [PATCH v13 0/4] drm/panfrost: Add support for mt8183 GPU To: Steven Price , Ezequiel Garcia References: <20210421052855.1279713-1-drinkcat@chromium.org> <373d0803-8658-9413-2f51-1e9804c39126@baylibre.com> From: Neil Armstrong Organization: Baylibre Message-ID: <8e35886c-7b51-aa93-5f9c-df0fae635828@baylibre.com> Date: Mon, 17 May 2021 12:24:51 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Nicolas Boichat , Tomeu Vizoso , fshao@chromium.org, David Airlie , hsinyi@chromium.org, Linux Kernel Mailing List , dri-devel , devicetree , Rob Herring , "moderated list:ARM/Mediatek SoC support" , Alyssa Rosenzweig , Boris Brezillon , Matthias Brugger , hoegsberg@chromium.org, linux-arm-kernel Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On 14/05/2021 17:27, Steven Price wrote: > On 14/05/2021 15:48, Neil Armstrong wrote: >> On 13/05/2021 16:55, Ezequiel Garcia wrote: >>> Hi Neil, >>> >>> On Mon, 26 Apr 2021 at 06:59, Neil Armstrong wrote: >>>> >>>> Hi, >>>> >>>> On 21/04/2021 07:28, Nicolas Boichat wrote: >>>>> Hi! >>>>> >>>>> This is just a rebase of the v11, untested (but it seems like >>>>> Neil Armstrong recently tested it), with small changes in >>>>> binding and dts. v11 cover follows: >>>>> >>>>> Follow-up on the v5 [1], things have gotten significantly >>>>> better in the last year, thanks to the efforts on Bifrost >>>>> support by the Collabora team (and probably others I'm not >>>>> aware of). >>>>> >>>>> I've been testing this series on a MT8183/kukui device, with a >>>>> chromeos-5.10 kernel [2], and got basic Chromium OS UI up with >>>>> mesa 20.3.2 (lots of artifacts though). >>>>> >>>>> devfreq is currently not supported, as we'll need: >>>>> - Clock core support for switching the GPU core clock (see 2/4). >>>>> - Platform-specific handling of the 2-regulator (see 3/4). >>>>> >>>>> Since the latter is easy to detect, patch 3/4 just disables >>>>> devfreq if the more than one regulator is specified in the >>>>> compatible matching table. >>>>> >>>>> [1] https://patchwork.kernel.org/project/linux-mediatek/cover/20200306041345.259332-1-drinkcat@chromium.org/ >>>>> [2] https://crrev.com/c/2608070 >>>>> >>>>> Changes in v13: >>>>> - devfreq: Fix conflict resolution mistake when rebasing, didn't >>>>> even compile. Oops. >>>>> >>>>> Changes in v12: >>>>> - binding: Fix min/maxItems logic (Rob Herring) >>>>> - Add gpu node to mt8183-pumpkin.dts as well (Neil Armstrong). >>>>> >>>>> Changes in v11: >>>>> - binding: power-domain-names not power-domainS-names >>>>> - mt8183*.dts: remove incorrect supply-names >>>>> >>>>> Changes in v10: >>>>> - Fix the binding to make sure sram-supply property can be provided. >>>>> >>>>> Changes in v9: >>>>> - Explain why devfreq needs to be disabled for GPUs with >1 >>>>> regulators. >>>>> >>>>> Changes in v8: >>>>> - Use DRM_DEV_INFO instead of ERROR >>>>> >>>>> Changes in v7: >>>>> - Fix GPU ID in commit message >>>>> - Fix GPU ID in commit message >>>>> >>>>> Changes in v6: >>>>> - Rebased, actually tested with recent mesa driver. >>>>> - Add gpu regulators to kukui dtsi as well. >>>>> - Power domains are now attached to spm, not scpsys >>>>> - Drop R-B. >>>>> - devfreq: New change >>>>> - Context conflicts, reflow the code. >>>>> - Use ARRAY_SIZE for power domains too. >>>>> >>>>> Changes in v5: >>>>> - Rename "2d" power domain to "core2" >>>>> - Rename "2d" power domain to "core2" (keep R-B again). >>>>> - Change power domain name from 2d to core2. >>>>> >>>>> Changes in v4: >>>>> - Add power-domain-names description >>>>> (kept Alyssa's reviewed-by as the change is minor) >>>>> - Add power-domain-names to describe the 3 domains. >>>>> (kept Alyssa's reviewed-by as the change is minor) >>>>> - Add power domain names. >>>>> >>>>> Changes in v3: >>>>> - Match mt8183-mali instead of bifrost, as we require special >>>>> handling for the 2 regulators and 3 power domains. >>>>> >>>>> Changes in v2: >>>>> - Use sram instead of mali_sram as SRAM supply name. >>>>> - Rename mali@ to gpu@. >>>>> >>>>> Nicolas Boichat (4): >>>>> dt-bindings: gpu: mali-bifrost: Add Mediatek MT8183 >>>>> arm64: dts: mt8183: Add node for the Mali GPU >>>>> drm/panfrost: devfreq: Disable devfreq when num_supplies > 1 >>>>> drm/panfrost: Add mt8183-mali compatible string >>>>> >>>>> .../bindings/gpu/arm,mali-bifrost.yaml | 30 ++++- >>>>> arch/arm64/boot/dts/mediatek/mt8183-evb.dts | 5 + >>>>> .../arm64/boot/dts/mediatek/mt8183-kukui.dtsi | 5 + >>>>> .../boot/dts/mediatek/mt8183-pumpkin.dts | 5 + >>>>> arch/arm64/boot/dts/mediatek/mt8183.dtsi | 105 ++++++++++++++++++ >>>>> drivers/gpu/drm/panfrost/panfrost_devfreq.c | 9 ++ >>>>> drivers/gpu/drm/panfrost/panfrost_drv.c | 10 ++ >>>>> 7 files changed, 168 insertions(+), 1 deletion(-) >>>>> >>>> >>>> Seems this version is ready to be applied if we get a review on the DT ? >>>> >>>> Mathias ? could you have a look ? >>>> >>> >>> Given Rob has Acked the DT bindings, I think it's OK to apply patches >>> 1, 3 and 4 via drm-misc, letting Mediatek people sort out the DT changes. >>> >>> My two unsolicited cents :-) > > You make a convincing point - and if everyone is happy for the DT > changes to be handled separately I don't see a reason for the other > patches to be held up. > >> Yeah sure, is there a panfrost maintainer in the room ? I can apply them if you ack me. > > I seem to be applying most Panfrost changes these days, so I'll save you > the effort and push 1,3,4 to drm-misc-next. Thanks ! Neil > > Thanks, > > Steve >