[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <30468c15-16bf-4c20-bef3-35d58b9a31b1@gmail.com>
Date: Fri, 12 Sep 2025 16:23:11 +0200
From: Matthias Brugger <matthias.bgg@...il.com>
To: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
linux-mediatek@...ts.infradead.org, robh@...nel.org
Cc: herbert@...dor.apana.org.au, davem@...emloft.net, krzk+dt@...nel.org,
conor+dt@...nel.org, chunkuang.hu@...nel.org, p.zabel@...gutronix.de,
airlied@...il.com, simona@...ll.ch, maarten.lankhorst@...ux.intel.com,
mripard@...nel.org, tzimmermann@...e.de, jassisinghbrar@...il.com,
mchehab@...nel.org, chunfeng.yun@...iatek.com, vkoul@...nel.org,
kishon@...nel.org, sean.wang@...nel.org, linus.walleij@...aro.org,
lgirdwood@...il.com, broonie@...nel.org, andersson@...nel.org,
mathieu.poirier@...aro.org, daniel.lezcano@...aro.org, tglx@...utronix.de,
atenart@...nel.org, jitao.shi@...iatek.com, ck.hu@...iatek.com,
houlong.wei@...iatek.com, kyrie.wu@...iatek.corp-partner.google.com,
andy.teng@...iatek.com, tinghan.shen@...iatek.com, jiaxin.yu@...iatek.com,
shane.chien@...iatek.com, olivia.wen@...iatek.com, granquet@...libre.com,
eugen.hristev@...aro.org, arnd@...db.de, sam.shih@...iatek.com,
jieyy.yang@...iatek.com, frank-w@...lic-files.de, mwalle@...nel.org,
fparent@...libre.com, linux-crypto@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
dri-devel@...ts.freedesktop.org, linux-media@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-phy@...ts.infradead.org,
linux-gpio@...r.kernel.org, linux-remoteproc@...r.kernel.org,
linux-sound@...r.kernel.org
Subject: Re: [PATCH 37/38] arm64: dts: mediatek: mt8395-kontron-i1200: Fix
MT6360 regulator nodes
On 24/07/2025 10:39, AngeloGioacchino Del Regno wrote:
> All of the MT6360 regulator nodes were wrong and would not probe
> because the regulator names are supposed to be lower case, but
> they are upper case in this devicetree.
>
> Change all nodes to be lower case to get working regulators.
>
> Fixes: 94aaf79a6af5 ("arm64: dts: mediatek: add Kontron 3.5"-SBC-i1200")
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
Applied, thanks
> ---
> .../mediatek/mt8395-kontron-3-5-sbc-i1200.dts | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8395-kontron-3-5-sbc-i1200.dts b/arch/arm64/boot/dts/mediatek/mt8395-kontron-3-5-sbc-i1200.dts
> index 4985b65925a9..d16f545cbbb2 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8395-kontron-3-5-sbc-i1200.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8395-kontron-3-5-sbc-i1200.dts
> @@ -352,7 +352,7 @@ regulator {
> LDO_VIN2-supply = <&vsys>;
> LDO_VIN3-supply = <&vsys>;
>
> - mt6360_buck1: BUCK1 {
> + mt6360_buck1: buck1 {
> regulator-name = "emi_vdd2";
> regulator-min-microvolt = <600000>;
> regulator-max-microvolt = <1800000>;
> @@ -362,7 +362,7 @@ MT6360_OPMODE_LP
> regulator-always-on;
> };
>
> - mt6360_buck2: BUCK2 {
> + mt6360_buck2: buck2 {
> regulator-name = "emi_vddq";
> regulator-min-microvolt = <300000>;
> regulator-max-microvolt = <1300000>;
> @@ -372,7 +372,7 @@ MT6360_OPMODE_LP
> regulator-always-on;
> };
>
> - mt6360_ldo1: LDO1 {
> + mt6360_ldo1: ldo1 {
> regulator-name = "mt6360_ldo1"; /* Test point */
> regulator-min-microvolt = <1200000>;
> regulator-max-microvolt = <3600000>;
> @@ -380,7 +380,7 @@ mt6360_ldo1: LDO1 {
> MT6360_OPMODE_LP>;
> };
>
> - mt6360_ldo2: LDO2 {
> + mt6360_ldo2: ldo2 {
> regulator-name = "panel1_p1v8";
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> @@ -388,7 +388,7 @@ mt6360_ldo2: LDO2 {
> MT6360_OPMODE_LP>;
> };
>
> - mt6360_ldo3: LDO3 {
> + mt6360_ldo3: ldo3 {
> regulator-name = "vmc_pmu";
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <3300000>;
> @@ -396,7 +396,7 @@ mt6360_ldo3: LDO3 {
> MT6360_OPMODE_LP>;
> };
>
> - mt6360_ldo5: LDO5 {
> + mt6360_ldo5: ldo5 {
> regulator-name = "vmch_pmu";
> regulator-min-microvolt = <3300000>;
> regulator-max-microvolt = <3300000>;
> @@ -404,7 +404,7 @@ mt6360_ldo5: LDO5 {
> MT6360_OPMODE_LP>;
> };
>
> - mt6360_ldo6: LDO6 {
> + mt6360_ldo6: ldo6 {
> regulator-name = "mt6360_ldo6"; /* Test point */
> regulator-min-microvolt = <500000>;
> regulator-max-microvolt = <2100000>;
> @@ -412,7 +412,7 @@ mt6360_ldo6: LDO6 {
> MT6360_OPMODE_LP>;
> };
>
> - mt6360_ldo7: LDO7 {
> + mt6360_ldo7: ldo7 {
> regulator-name = "emi_vmddr_en";
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
Powered by blists - more mailing lists