Received: by 2002:a05:6a10:6d10:0:0:0:0 with SMTP id gq16csp901727pxb; Fri, 22 Apr 2022 13:52:00 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxVilJHMq/0h657S/W//323SGJe12eTJEmY/TDdhsDU7ObJTSuSRdldRrWw5lH62n+KGh5z X-Received: by 2002:a17:902:b092:b0:156:a40f:f716 with SMTP id p18-20020a170902b09200b00156a40ff716mr6435418plr.72.1650660720764; Fri, 22 Apr 2022 13:52:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1650660720; cv=none; d=google.com; s=arc-20160816; b=YPAiFylVRnHoJcQeCu9ROau07HB4TZuM/69ppSvdm5xVozDTZyR/EaeQjvM2M97CEj hBaFWou7O4iAuDGep1ZKeRiiayWm6P2b7rHEqtuyQph0hvWVdNyT/UfagLH6v2PldHhp 4iY/eZJzQaXf65GRY2azK4IZaVaM7wdbdQcn4/2MhnMUrHlDku8+616nqCd+i8h/WFnX wmCmI9Zo3CTgrgibQmqAT5/IX36AaJPjZ/M4sxlXl38Lp74UKLuMj1udfRvEDCzG+m9w 8bPL+VzfsgD88Ks/B143TTOTX7hPGYswU325WvdLlaKgTcRpgDRq0Oz3CPFh8ZLkKWEi VU1A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:date:references :in-reply-to:subject:cc:to:from; bh=ICt5XunHqsELIdqZvSPeKFkS3NdL0ipmf4jJkro4jDI=; b=vbAxDdY10X7vZsirCP+M7+SGE+FfvUP4YCec3dqURyaxYB7MD4MKjtHMlfJ12DmQiC rqPKmohx/iqRK0sIGFLl7GXaZv18K9qJP6N0Ur6uGNN2XxA4XawG3RDLjjTSxSzWDgWV YQCE9KMkUI3e+Pig5P60KNCwfoboT8y9ZFyjRcXVa8rzGwMmPvgP0Dmp6FkkvsktTCZC Zy6Vqil0XhAi5B9uIjj4UHOfwxkKJ99nW6vDXAdvcnsrGMScHJN6/I7WVe4MlvmM6Hj4 +ZbiqyK3xnhe4YPXq7rWAaftGk9vy1QUSFi5GipVNswiYJYPglGyC4df2ei2c2gwmZB9 q2qw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net. [2620:137:e000::1:18]) by mx.google.com with ESMTPS id c185-20020a6335c2000000b003aa5942bf0bsi9030742pga.745.2022.04.22.13.51.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 22 Apr 2022 13:52:00 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 8A731215738; Fri, 22 Apr 2022 12:44:24 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232902AbiDUToq (ORCPT + 99 others); Thu, 21 Apr 2022 15:44:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49850 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231993AbiDUTop (ORCPT ); Thu, 21 Apr 2022 15:44:45 -0400 Received: from mail-pl1-f172.google.com (mail-pl1-f172.google.com [209.85.214.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5E5AE4B87B for ; Thu, 21 Apr 2022 12:41:54 -0700 (PDT) Received: by mail-pl1-f172.google.com with SMTP id j8so5994918pll.11 for ; Thu, 21 Apr 2022 12:41:54 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:in-reply-to:references:date :message-id:mime-version; bh=ICt5XunHqsELIdqZvSPeKFkS3NdL0ipmf4jJkro4jDI=; b=bPVCUHYgiFTsRFDjkSN9Wir7iTk2Lrk+/qPV44ewQiZA/21E2gvgx+7C60y9BUIGC6 MyHbIdQI5Mi1iowaWVNrPa5PDdMJXdXEwe6RM8oMxJmfLhoP8VPzZBqht0ui6+8KqTHX zqzHDjtfcCKYsx9nup0pIjxdlrrOu3ioNlYUjwUq+dKSXt6DqOFqTTdVwikeOFxJTemm 8HmtlCo1ds70ckXvIKadaniUKzShIDkOIc2tG7yyCg/BD7Q62lp0wagwbNYBRBRxablj 5vmMKJqKe+QHUIvsS+dNee5i7s5IEoX8o2Pe2jqkhjLkACzR96JoAqyBCnRwHBmbuJgZ dm1A== X-Gm-Message-State: AOAM531X8qOi64uiHH9RsitGuWrMP6WbtFUYDccHxB6c1Ljb6vGbOqQ4 ha34WKwSq5qfKSOTCRgpaXWTnw== X-Received: by 2002:a17:902:b596:b0:158:f23a:c789 with SMTP id a22-20020a170902b59600b00158f23ac789mr927819pls.57.1650570113700; Thu, 21 Apr 2022 12:41:53 -0700 (PDT) Received: from localhost (c-71-197-186-152.hsd1.wa.comcast.net. [71.197.186.152]) by smtp.gmail.com with ESMTPSA id v14-20020a17090a6b0e00b001d2bff34228sm3703433pjj.9.2022.04.21.12.41.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 21 Apr 2022 12:41:53 -0700 (PDT) From: Kevin Hilman To: Roger Lu , Matthias Brugger , Enric Balletbo Serra , Rob Herring , Nicolas Boichat , Stephen Boyd , Philipp Zabel Cc: Fan Chen , HenryC Chen , Xiaoqing Liu , Charles Yang , Angus Lin , Mark Rutland , Nishanth Menon , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, Project_Global_Chrome_Upstream_Group@mediatek.com, Guenter Roeck , Jia-wei Chang Subject: Re: [PATCH v24 0/7] soc: mediatek: SVS: introduce MTK SVS In-Reply-To: <3d463c8b099fdb1c9a0df9e615a8ca1d8a034120.camel@mediatek.com> References: <20220420102044.10832-1-roger.lu@mediatek.com> <7hczhbe3wn.fsf@baylibre.com> <3d463c8b099fdb1c9a0df9e615a8ca1d8a034120.camel@mediatek.com> Date: Thu, 21 Apr 2022 12:41:52 -0700 Message-ID: <7hsfq6ql4v.fsf@baylibre.com> MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,MAILING_LIST_MULTI, RDNS_NONE,SPF_HELO_NONE autolearn=no 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 Hi Roger, Roger Lu writes: > On Wed, 2022-04-20 at 16:22 -0700, Kevin Hilman wrote: [...] >> That being said, it would be really nice to see an integration tree >> where this was all tested on mainline (e.g. v5.17, or v5.18-rc) >> >> For example, I can apply this to v5.18-rc2 and boot on my mt8183-pumpkin >> board, it fails to probe[1] because there is no CCI node in the upstream >> mt8183.dtsi. >> >> I'm assuming this series is also not very useful without the CPUfreq >> series from Rex, so being able to test this, CCI and CPUfreq together on >> MT8183 on a mainline kernel would be very helpful. >> >> Kevin >> >> [1] >> [ 0.573332] mtk-svs 1100b000.svs: cannot find cci node >> [ 0.574061] mtk-svs 1100b000.svs: error -ENODEV: svs platform probe fail > > Just share. I've tested this series on below two platforms and it works as > expected. > - mt8183-Krane (kernel-v5.10) > - mt8192-Hayato (kernel-v5.4) Unfortunately testing on v5.4 and v5.10 with lots of other additional out-of-tree patches does not give much confidence that this series works with upstream, especially when I've given a few reasons why it will not work uptream. The examples I gave above for CCI and CPUs/cluster disable are good examples, but another one I forgot to mention is the dependency on Mali. The SVS driver will never probe because it also depens on a "mali" node, which doesn't exist upstream either (but panfrost does, and acutually loads/probes fine on v5.17/v5.18) so this should be fixed to work with upstream panfrost. IMO, in order for this to be merged upstream, it should at least have some basic validation with upstream, and so far I have not even been able to make it successfuly probe. To do that, you will need to either provide a list of the dependencies for testing this with mainline (e.g. CCI series, CPUfreq series, any DT changes), or even better, an integration tree based on recent mainline (e.g. v5.17 stable, or v5.18-rc) which shows all the patches (in addition to this series) used to validate this on mainline. Thanks, Kevin