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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 8B5FFC4345F for ; Mon, 15 Apr 2024 00:15:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Z7E+rhsBaWJ/U1bsgsRsgQARYvDNEhCg9d/LBrQDVz4=; b=XDA+c3gI5OHGAy ZkocJST+KTjFii4pJt34wvRoAE6U5LQAqZC0wj4CfZwu0wI2e8d8iecpb9Y2pUvU0X+pP/kQuWAMU w3L1m5CShPC1KiZDfa0Pxf4jXcxewhBmIG/3jnsonatq95PwZ/pNWbl2mQ/hTOdaQFjscKXjvOMkO Bq7idpUzPwaOOlOfKCTtFmenX41RVsCRcVE9M1rcupnKTE+ZaUENsn/I27X6aK4T8MHQ1gtXgp1vI 6V2JHNOnd5OoApgRszIjBh/B1XO7Mhvp8UmFIKHBbtUSOc0WGFdzo4dVsEVx0Q6RDEwys6PM85YPs Qmo4HMxGXLb7SazQpHXA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rw9zp-00000006asa-3vBs; Mon, 15 Apr 2024 00:14:53 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rw9zm-00000006arg-0OZN for linux-arm-kernel@lists.infradead.org; Mon, 15 Apr 2024 00:14:52 +0000 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 03649339; Sun, 14 Apr 2024 17:15:13 -0700 (PDT) Received: from minigeek.lan (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 6A0C83F64C; Sun, 14 Apr 2024 17:14:41 -0700 (PDT) Date: Mon, 15 Apr 2024 01:14:35 +0100 From: Andre Przywara To: "Ryan Walklin" Cc: "Viresh Kumar" , "Yangtao Li" , "Viresh Kumar" , "Nishanth Menon" , "Stephen Boyd" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Chen-Yu Tsai" , "Jernej Skrabec" , "Samuel Holland" , "Rafael J . Wysocki" , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-sunxi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, "Brandon Cheo Fusi" , "Martin Botka" , "Martin Botka" , "Chris Morgan" , "Mark Rutland" , "Lorenzo Pieralisi" , "Sudeep Holla" Subject: Re: [PATCH v4 0/8] cpufreq: sun50i: Add Allwinner H616 support Message-ID: <20240415011435.2aeae1f1@minigeek.lan> In-Reply-To: <0e021133-f8c1-457d-9079-d9c972e0e69b@app.fastmail.com> References: <20240329141311.27158-1-andre.przywara@arm.com> <20240404064027.shjqvqih7s5rnv2i@vireshk-i7> <0e021133-f8c1-457d-9079-d9c972e0e69b@app.fastmail.com> Organization: Arm Ltd. X-Mailer: Claws Mail 4.2.0 (GTK 3.24.31; x86_64-slackware-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240414_171450_214241_768DA54F X-CRM114-Status: GOOD ( 16.42 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, 04 Apr 2024 20:44:02 +1300 "Ryan Walklin" wrote: Hi Ryan, > On Thu, 4 Apr 2024, at 7:40 PM, Viresh Kumar wrote: > > Is it okay to merge all the changes via the cpufreq tree ? > > I have tested this series with an H700-based board, and have at least one speed-bin (1.032GHz) is not supported although the governor attempts to enable it based on the opp-supported-hw bitmask, and I am unable to reach the 1.5GHz bin at 1.16v (or higher) despite it working on the vendor BSP (kernel panic at boot if enabled), so this may need some slight rework. Thanks for the report! So can you try to merge the 1.032 GHz OPP into the 1.008 GHz one? That would be beneficial anyways since this is the default frequency that U-Boot sets up. Should be: opp-1008000000 { .... opp-microvolt-speed5 = <900000>; opp-supported-hw = <0x3f>; .... As for the 1.5 GHz speed bin: We could leave that out for now if it causes trouble. But can you first state how you got the OPPs? I copied them from some table you dumped once on IRC, but it would be good to double check what the actual values are that the BSP kernel runs with. The values in the vendor DT are highly inconsistent, besides we don't know for sure which speed bin index the BSP is using and how this maps to our method. Cheers, Andre > I have reached out to Andre on IRC to debug. > > Ryan _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel