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 63318CCF9E3 for ; Thu, 30 Oct 2025 19:46:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=FSZGYbOmCMoLcNoFG9BmYZt4BZZKr4wBKZWgl9N85BI=; b=N/XUt8/u+TudWU908IZj6JgVIs Hid7D+UfMyQDCW2mI6LSrkGnEfn33oVKS2G4f9kHJF5xNK7EDSGX1gJA3I402yiUzs/eCl9gftRJ2 Ud5ZhWfk/OVJBSVQ0ohLJu6KNE02QtZ4FDtLbt/0IbsjLcLT3obZUBMmyYZtEv4ppbmxoCa3jGOx3 d7mUMR+V+dvYowlK4W8+ZGjyIP6mkPeyveITRQM78weJPjSibdYCkneWfjSnrV5O8oQLWwF/G8KOC Q/CLqZ0QAocXl+54gcjUGLBJLL3yPaFqOhN2ge7WlT7f2clp/YeLR4PxB7L6X4EHyx9iDjKkUrIEC kD6WoZhQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vEYbM-00000004hfa-23sZ; Thu, 30 Oct 2025 19:46:30 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vEYbI-00000004hej-0p0B; Thu, 30 Oct 2025 19:46:25 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id 9481041B50; Thu, 30 Oct 2025 19:46:23 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id C2C70C4CEF1; Thu, 30 Oct 2025 19:46:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1761853583; bh=GV2XRPs93TwYftizCqQPTtAV3OHflrc4UhVM50o3qOI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=GYRixxz68G0B3WvSiTR45ogAq1hCd3hip6IbAr/vpLSvMQXPh5bqwKR3fHwvAiato k9lLxJ9Ol9IrjuTPLH4cv014bh8GtkT3rSxRQSzbQzx7GkYZcsUhrgxsizNlJgfS9i HssBlKBGGA9AOg0QdRsEjZoWkCmbgCCOZ4gdieHpstJL7HQCGPUCYACe6CBFZz0F4d 6UXn0RZmedORIu4k14ErbNgOuNfJAnYjpVCE+h4NoJ6JbM+yxnmxzHC2wvJucQp0YT VqQi6/6Mr7HsE+WssAKVJA6F01m+5MK27cfqtX7ZQzSTFWGDgznlUEZqL507BbeBkm CZB2fDXCDdykg== Date: Thu, 30 Oct 2025 19:46:15 +0000 From: Conor Dooley To: Jack Hsu Cc: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, jic23@kernel.org, dlechner@baylibre.com, nuno.sa@analog.com, andy@kernel.org, matthias.bgg@gmail.com, angelogioacchino.delregno@collabora.com, srini@kernel.org, ukleinek@kernel.org, gregkh@linuxfoundation.org, jirislaby@kernel.org, daniel.lezcano@linaro.org, tglx@linutronix.de, chunfeng.yun@mediatek.com, wim@linux-watchdog.org, linux@roeck-us.net, sean.wang@mediatek.com, zhiyong.tao@mediatek.com, andrew-ct.chen@mediatek.com, lala.lin@mediatek.com, jitao.shi@mediatek.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-pwm@vger.kernel.org, linux-serial@vger.kernel.org, linux-usb@vger.kernel.org, linux-watchdog@vger.kernel.org, Project_Global_Chrome_Upstream_Group@mediatek.com Subject: Re: [PATCH v6 10/11] arm64: dts: mediatek: add properties for MT6359 Message-ID: <20251030-boaster-chewing-61d458aa6c9e@spud> References: <20251030134541.784011-1-jh.hsu@mediatek.com> <20251030134541.784011-11-jh.hsu@mediatek.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="qjInnEs/qoh/Iqxq" Content-Disposition: inline In-Reply-To: <20251030134541.784011-11-jh.hsu@mediatek.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251030_124624_302965_256C7F22 X-CRM114-Status: GOOD ( 21.14 ) 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 --qjInnEs/qoh/Iqxq Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Oct 30, 2025 at 09:44:42PM +0800, Jack Hsu wrote: > Add properties of rtc fg (Fuel Gauge), external crystal > and auxadc definition for mt6359 pmic. >=20 > Signed-off-by: Jack Hsu > --- > arch/arm64/boot/dts/mediatek/mt6359.dtsi | 20 ++++++++++ > include/dt-bindings/iio/mt635x-auxadc.h | 50 ++++++++++++++++++++++++ > 2 files changed, 70 insertions(+) > create mode 100644 include/dt-bindings/iio/mt635x-auxadc.h >=20 > diff --git a/arch/arm64/boot/dts/mediatek/mt6359.dtsi b/arch/arm64/boot/d= ts/mediatek/mt6359.dtsi > index 467d8a4c2aa7..cc7053bdd292 100644 > --- a/arch/arm64/boot/dts/mediatek/mt6359.dtsi > +++ b/arch/arm64/boot/dts/mediatek/mt6359.dtsi > @@ -3,6 +3,8 @@ > * Copyright (C) 2022 MediaTek Inc. > */ > =20 > +#include > + > &pwrap { > pmic: pmic { > compatible =3D "mediatek,mt6359"; > @@ -302,6 +304,24 @@ mt6359_vsram_others_sshub_ldo: ldo_vsram_others_sshu= b { > =20 > mt6359rtc: rtc { > compatible =3D "mediatek,mt6358-rtc"; > + #address-cells =3D <1>; > + #size-cells =3D <1>; > + status =3D "disabled"; > + > + fginit: fginit { > + reg =3D <0 0x1>; > + bits =3D <0 8>; > + }; Uhhhh, what on earth is going on here? This is an RTC, what does fuelgauge stuff have to do with it? Did you test this at all? What does it even do? Very confused. > + > + fgsoc: fgsoc { > + reg =3D <1 0x1>; > + bits =3D <0 8>; > + }; > + > + ext32k: ext32k { > + reg =3D <2 0x1>; > + bits =3D <6 1>; > + }; > }; > }; > }; > diff --git a/include/dt-bindings/iio/mt635x-auxadc.h b/include/dt-binding= s/iio/mt635x-auxadc.h > new file mode 100644 > index 000000000000..69ba13a7b9ec > --- /dev/null > +++ b/include/dt-bindings/iio/mt635x-auxadc.h > @@ -0,0 +1,50 @@ > +/* SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause */ > +/* > + * Copyright (c) 2020 MediaTek Inc. > + */ > + > +#ifndef _DT_BINDINGS_MT635X_AUXADC_H > +#define _DT_BINDINGS_MT635X_AUXADC_H > + > +/* PMIC MT635x AUXADC channels */ > +#define AUXADC_BATADC 0x00 > +#define AUXADC_ISENSE 0x01 > +#define AUXADC_VCDT 0x02 > +#define AUXADC_BAT_TEMP 0x03 > +#define AUXADC_BATID 0x04 > +#define AUXADC_CHIP_TEMP 0x05 > +#define AUXADC_VCORE_TEMP 0x06 > +#define AUXADC_VPROC_TEMP 0x07 > +#define AUXADC_VGPU_TEMP 0x08 > +#define AUXADC_ACCDET 0x09 > +#define AUXADC_VDCXO 0x0a > +#define AUXADC_TSX_TEMP 0x0b > +#define AUXADC_HPOFS_CAL 0x0c > +#define AUXADC_DCXO_TEMP 0x0d > +#define AUXADC_VBIF 0x0e > +#define AUXADC_IMP 0x0f > +#define AUXADC_IMIX_R 0x10 > +#define AUXADC_VTREF 0x11 > +#define AUXADC_VSYSSNS 0x12 > +#define AUXADC_VIN1 0x13 > +#define AUXADC_VIN2 0x14 > +#define AUXADC_VIN3 0x15 > +#define AUXADC_VIN4 0x16 > +#define AUXADC_VIN5 0x17 > +#define AUXADC_VIN6 0x18 > +#define AUXADC_VIN7 0x19 What has this fine got to do with the node you're adding above? I don't understand why this is in this patch. There's random whitespace problems in it, and half the defines look bizarre. This seems like it should be in the same patch as some binding changes for an iio device? The mt635x auxadc binding doesn't even permit anything that would use any of the properties beyond this point. NAK. > + > +#define AUXADC_CHAN_MIN AUXADC_BATADC > +#define AUXADC_CHAN_MAX AUXADC_VIN7 > + > +#define ADC_PURES_100K (0) > +#define ADC_PURES_30K (1) > +#define ADC_PURES_400K (2) > +#define ADC_PURES_OPEN (3) > + > +#define ADC_PURES_100K_MASK (ADC_PURES_100K << 8) > +#define ADC_PURES_30K_MASK (ADC_PURES_30K << 8) > +#define ADC_PURES_400K_MASK (ADC_PURES_400K << 8) > +#define ADC_PURES_OPEN_MASK (ADC_PURES_OPEN << 8) > + > +#endif /* _DT_BINDINGS_MT635X_AUXADC_H */ > --=20 > 2.45.2 >=20 --qjInnEs/qoh/Iqxq Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCaQPAhwAKCRB4tDGHoIJi 0rESAQC+pk8ew/zg9l4Rp9iAnTnoGDOpKly9i4yhdfpXcv5AowEAvBfWYIfZ0eSw +rZuOzIx3kHSDbDpctohV+Wb6SErqQY= =lc/Z -----END PGP SIGNATURE----- --qjInnEs/qoh/Iqxq--