From patchwork Tue Nov 28 00:58:43 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andre Przywara X-Patchwork-Id: 13470436 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 4A679C07CB1 for ; Tue, 28 Nov 2023 01:01:18 +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:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=xJFzKkSnRAaSDPAYudQFOLwdb+Tce/501nwD0wOnHs8=; b=gcZBT+WR1G7O0Y Kk7L6jPoy/HSiFmvCP2wb9Ml0/nmRc4WAz85OVoUgww8rT3HkE9Nz3l/ktVBV0yjw5r0TYQ2jhzN+ JIX0ri9jKU7tB6BQNzLTxQx2+pyCPMAgH/rZ7ooBQLsTAWC4n5TjflWPxH2l9giTzeUx4cFG/89rd XP9QntGbS49quGMiPN2yb+V0VoPUFcPP8khIyN9INAyfQckiAhd/I4iplEZIJtj0iluXUYciZkD13 brTj4JB6wf04HD2Hoft9UB72e9nJ2fzvqfXg9MQuPYU1I1IFB1+YweMoxAkPOEPJXEvCNWjFfYmFn VyU4PsJJ9xlEgpbuNSfw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1r7mT1-003mjz-2V; Tue, 28 Nov 2023 01:00:47 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1r7mSx-003miL-1g for linux-arm-kernel@lists.infradead.org; Tue, 28 Nov 2023 01:00:45 +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 6774E2F4; Mon, 27 Nov 2023 17:01:24 -0800 (PST) Received: from localhost.localdomain (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 4BA9A3F6C4; Mon, 27 Nov 2023 17:00:34 -0800 (PST) From: Andre Przywara To: Vasily Khoruzhick , Yangtao Li , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland Cc: "Rafael J . Wysocki" , Daniel Lezcano , Zhang Rui , Lukasz Luba , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Martin Botka , Bob McChesney , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev Subject: [PATCH v3 0/6] Add support for H616 Thermal system Date: Tue, 28 Nov 2023 00:58:43 +0000 Message-Id: <20231128005849.19044-1-andre.przywara@arm.com> X-Mailer: git-send-email 2.35.8 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231127_170043_676849_26916753 X-CRM114-Status: GOOD ( 22.18 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, this is v3 of the series by Martin, now complemented by patches to fix the problem with way-too-high temperatures reported on some boards. This seems to be related to the firmware being run, because the vendor U-Boot contains a hack avoiding this problem. Some investigation and digging in BSP code later we identified that bit 16 in register 0x3000000 (SYS_CFG) needs to be cleared for the raw temperature register values to contain reasonable values. To achieve this, patch 1/6 exports this very register from the already existing syscon device. Patch 4/6 then adds code to the thermal driver to find the syscon device via a new DT property, and query its regmap to clear bit 16 in there. I am not fully convinced this is the best solution, but it works for me. What leaves a bit of a bitter taste is that the SRAM driver (the one exporting the regmap) also uses this register, to switch some SRAM C region to the video engine (VE). Experiments show that only bit 0 in this register is doing this job, so the current mask covering the 31 LSBs should probably be amended to only cover bit 0. Another solution could be to model this bit as an SRAM switch, and let the THS driver claim some (dummy?) SRAM region from the syscon/SRAM driver directly. While this sounds cleaner to some degree, I don't think there is really such a THS SRAM region, so it's not fully correct either. I would appreciate any feedback on this, happy to implement the other approach, if that's desired. The rest of the patches is mostly unchanged from Martin's v2, just updated and massaged the commit messages a bit. I also added patch 3/6 to document some so-far unknown register value. Please have a look! Cheers, Andre Changelog v2 .. v3: - rebase on top of v6.7-rc3 - add patches to clear bit 16 in SYS_CFG register 0x3000000 - add syscon to the binding documentation - add patch explaining the unknown control register value Changelog v1 .. v2: - Fix typos - Remove h616 calc and init functions - Use TEMP_CALIB_MASK insteaf of 0xffff - Adjust calibration function to new offset and scale - Add proper comment to bindings patch - Split delta calculations to 2 lines - Add parentheses around caldata[2|3] >> 12 - Negate bindings if for clocks Andre Przywara (3): soc: sunxi: sram: export register 0 for THS on H616 thermal: sun8i: explain unknown H6 register value thermal: sun8i: add syscon register access code Martin Botka (3): dt-bindings: thermal: sun8i: Add H616 THS controller thermal: sun8i: add support for H616 THS controller arm64: dts: allwinner: h616: Add thermal sensor and zones .../thermal/allwinner,sun8i-a83t-ths.yaml | 30 ++-- .../arm64/boot/dts/allwinner/sun50i-h616.dtsi | 88 ++++++++++ drivers/soc/sunxi/sunxi_sram.c | 5 + drivers/thermal/sun8i_thermal.c | 152 ++++++++++++++++-- 4 files changed, 252 insertions(+), 23 deletions(-)