Received: by 2002:a05:6358:9144:b0:117:f937:c515 with SMTP id r4csp2626053rwr; Fri, 28 Apr 2023 13:10:30 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ5OAHXtnQkt7iXLh/2jwKmXdY0aupXp76DHQfdyT4nPeDr9E5dOx/k3m+93L5o5LZvU8nxy X-Received: by 2002:a05:6a00:150f:b0:63f:1600:e711 with SMTP id q15-20020a056a00150f00b0063f1600e711mr8926802pfu.29.1682712629765; Fri, 28 Apr 2023 13:10:29 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1682712629; cv=none; d=google.com; s=arc-20160816; b=ppP4ixOKmJk8mA842qlFq+23UX8lw2eMV6+K02uyeeBNxFQFUveNM1mw5BtnImQL5a uS7b564luJNWbNUmD9R7Yf7FHC4n5JpYJyW8ffQAAVXkV6wdMGxn110amcHWolB73Y+W fygeigFpzJHiL2u41XffXY3bOFD769boQC1YckDZTBcOpB7MP+cNk+IFWeRf1vQ8tq7v 4jN2Mk/acHXe5hppwwgcoKCvyxPCfrV7pYMhMHmTiuAIgBfYYqp9+Kk8XCfHZfQsHpPp WPm3svSaPjL/BAHbz3oQgVJEtf0sNhLwEdnwlS5T2+nD4w6dnhvFywRfbgUaxiv+xRd/ Jvfw== 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-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=71Hj1US0pSM/IwfSeQssNoD3szbJK4N2OqLUHAmgjD4=; b=wleeQCpqiUVTFrBZtwD7/6m0sxcidMFkq0yl6whkt16KvL/c6gopwxwjyHm7M480O+ w0nRf4bmTdZLkTWx/e5I2MkYhP4DZ3vBdSK3ykzK7BrLHQSCssjpmgWsYQgm+k598Mfj eeB6NJhbbxByrNdrrvLxnlRUaSA6Q1IaRP/nPgCqhBWOZhUO4WQDh/OhaJJwL4BerpHq oQ9vZ9MUFeZ9dLO83DRklv10eGexV4GjAG0ZYzxC4CgFmr39AR5XQH72EaOYQCdOau2/ q7MYoh3vn6kp3+O6MCFOEBk2EAiVP8ligFCILjk3958WtYHw3uX38sfPDg5a+MfPTwGL xoDQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=X0iioVLU; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 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 out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id c18-20020aa79532000000b0062804ba6fb4si21565548pfp.58.2023.04.28.13.09.54; Fri, 28 Apr 2023 13:10:29 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=X0iioVLU; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1346338AbjD1UA7 (ORCPT + 99 others); Fri, 28 Apr 2023 16:00:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36588 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1345685AbjD1UA5 (ORCPT ); Fri, 28 Apr 2023 16:00:57 -0400 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 0181D212B; Fri, 28 Apr 2023 13:00:55 -0700 (PDT) Received: from notapiano (unknown [194.36.25.39]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: nfraprado) by madras.collabora.co.uk (Postfix) with ESMTPSA id ACF8F66032CF; Fri, 28 Apr 2023 21:00:50 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1682712054; bh=6i1JUq29VFHhcwxsShC8PZn9X8kOz1TV4HuL2Wb9A2A=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=X0iioVLUW03JDY5ecGWdF3jrGqspgJ31dBgi6MIyuHKvh809IXfDbjNkLqzXlt3Pg b4C6QSUOlxbf7vbKPiddSQft0WbwieHVXxlOwskGjfWnoFGAJs/cw1E3FTqiD8DRto c5tA469u0nmiDbw8vESjqzPYHRJa6i1onhQh2x6J9pZkMfZe7se0j7Hmj5NyjtUvFT 2LQF6TSTvW9n/hagf71iviV+PR9UIn1g7SUM4BeZftDCbQifMSZL9tWMcniVTlV9AQ XCc8s3k0kyHEFU0Sj4hwpTdcLBjgv+6aRAjdMnA67NWqg154TXkeLLerjmTXJjqimM h8Huc8mnGBJRA== Date: Fri, 28 Apr 2023 16:00:45 -0400 From: =?utf-8?B?TsOtY29sYXMgRi4gUi4gQS4=?= Prado To: Balsam CHIHI Cc: Chen-Yu Tsai , daniel.lezcano@linaro.org, angelogioacchino.delregno@collabora.com, rafael@kernel.org, amitk@kernel.org, rui.zhang@intel.com, matthias.bgg@gmail.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, rdunlap@infradead.org, ye.xingchen@zte.com.cn, p.zabel@pengutronix.de, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org, khilman@baylibre.com, james.lo@mediatek.com, rex-bc.chen@mediatek.com, Alexandre Bailon , Alexandre Mergnat Subject: Re: [PATCH 0/4] Add LVTS support for mt8192 Message-ID: <5a437b3c-e134-40df-830a-7ea0f21849fc@notapiano> References: <28e50e45-53cf-4a34-a81c-10f1faab2d4e@notapiano> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: 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 On Thu, Apr 27, 2023 at 04:08:13PM +0200, Balsam CHIHI wrote: > On Thu, Apr 27, 2023 at 1:20 AM Nícolas F. R. A. Prado > wrote: > > > > On Tue, Apr 25, 2023 at 01:28:39PM +0200, Balsam CHIHI wrote: > > > On Tue, Apr 25, 2023 at 12:00 PM Chen-Yu Tsai wrote: > > > > > > > > On Tue, Apr 25, 2023 at 6:21 AM Nícolas F. R. A. Prado > > > > wrote: > > > > > > > > > > On Tue, Mar 28, 2023 at 02:20:24AM +0200, Balsam CHIHI wrote: > > > > > > On Sat, Mar 25, 2023 at 5:33 AM Chen-Yu Tsai wrote: > > > > > > > > > > > > > > On Wed, Mar 22, 2023 at 8:48 PM Balsam CHIHI wrote: > > [..] > > > > > > > > > > > > Hi Chen-Yu Tsai, > > > > > > > > > > > > Thank you very much for helping me testing this suggestion. > > > > > > > > > > > > Indeed, calibration data is stored differently in the mt8192 compared to mt8195. > > > > > > So, the mt8192's support will be delayed for now, to allow further debugging. > > > > > > > > > > > > In the mean time, we will only continue to upstream the remaining > > > > > > mt8195's source code, so it will get full LVTS support. > > > > > > A new series will be submitted soon. > > > > > > > > > > Hi Balsam, > > > > > > > > > > like Chen-Yu mentioned, the calibration data is stored with 4 byte alignment for > > > > > MT8192, but the data that is split between non-contiguous bytes is for the > > > > > thermal controllers (called Resistor-Capacitor Calibration downstream) not the > > > > > sensors. The controller calibration isn't currently handled in this driver (and > > > > > downstream it also isn't used, since a current value is read from the controller > > > > > instead), so we can just ignore those. > > > > > > > > > > The patch below adjusts the addresseses for the sensors and gives me reasonable > > > > > reads, so the machine no longer reboots. Can you integrate it into your series? > > > > > > > > Not sure what I got wrong, but on my machine the VPU0 and VPU1 zone interrupts > > > > are still tripping excessively. The readings seem normal though. Specifically, > > > > it's bits 16 and 17 that are tripping. > > > > > > > > > > Hi Chen-Yu, > > > > > > Thank you for testing! > > > > > > As the readings are normal that proves that calibration data offsets > > > are correct. > > > would you like that I send the v2 of series to add mt8192 support? > > > Then we could deal with the interrupts later in a separate fix, > > > because the interrupt code in common for both SoC (mt8192 and mt8195)? > > > > > > Does Nícolas also have tripping interrupts? > > > On my side, I've got no interrupts tripping on mt8195. > > > > > > Any other suggestions (a question for everyone)? > > > > Hi, > > > > sorry for the delay. > > > > Indeed the interrupts are constantly tripping on mt8192 here as well. > > > > I do not see the same bits as Chen-Yu mentioned however, I see > > > > LVTS_MONINTSTS = 0x08070000 > > > > which corresponds to > > > > Hot threshold on sense point 3 > > high to normal offset on sense point 2 > > high offset on sense point 2 > > low offset on sense point 2 > > > > and it's the same on all controllers and domains here, which is weird. I noticed > > we have offset interrupts enabled even though we don't configure the values for > > those, but even after disabling them and clearing the status register, the > > interrupts keep triggering and the status is the same, so for some reason > > LVTS_MONINT doesn't seem to be honored. > > > > I also tried using the filtered mode instead of immediate for the sensors, and > > that together with disabling the extra interrupts, got me a zeroed > > LVTS_MONINTSTS. However no interrupts seem to be triggered at all (nor > > LVTS_MONINTSTS updated) when the temperature goes over the configured one in > > LVTS_HTHRE. > > > > I tried the driver on mt8195 (Tomato chromebook) as well, and it has the same > > LVTS_MONINTSTS = 0x08070000 > > even though the interrupts aren't being triggered, but in fact I don't see them > > triggering over the threshold either, so I suspect the irq number might be > > incorrectly described in the DT there. > > > > Do either of you have it working correctly on mt8195? > > > > Anyway, I'll keep digging and reply here when I find a solution. > > Hi Nícolas, > > Thank your for your time testing and investigating the interrupt issues! > > I only have an mt8195 based board (i1200-demo), and I could not > trigger any interrupt on it. > I whish that MediaTek could reply to this thread to give us more > information (I avoid disclosing MediaTek's internal information). > And now, it's clear that mt8192 interrupts does work at least (but not > properly, may be we could fix it at driver level). > > It's been a couple of days since I sent a v2 of the series that adds > LVTS support for mt8192 SoC (+ Suspend and Resume, + Doc update): > "https://lore.kernel.org/all/20230425133052.199767-1-bchihi@baylibre.com/". > I wish that it will be applied very soon, then we could patch the core driver. > > My colleagues "Alexandre Mergnat (amergnat@baylibre.com)" and > "Alexandre Bailon (abailon@baylibre.com)" are now part of this > project. > Please let them know of future information. > > Thanks again for suggesting solutions! Hi, finally managed to fix the issues. I had mis-read the interrupt status bits, which made things a whole lot more confusing... I CC'ed you on the series, but for the archive this is it: https://lore.kernel.org/all/20230428195347.3832687-1-nfraprado@collabora.com/ Please review/test it if you have the time. I have one extra comment regarding the mt8192 support, but I'll write it on the v2 of this series. Thanks, Nícolas