devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus DHCOM
@ 2022-12-15 23:57 Marek Vasut
  2022-12-15 23:57 ` [PATCH 2/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus PhyCore SoM Marek Vasut
  2023-01-01  2:24 ` [PATCH 1/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus DHCOM Shawn Guo
  0 siblings, 2 replies; 5+ messages in thread
From: Marek Vasut @ 2022-12-15 23:57 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Marek Vasut, Christoph Niedermaier, Fabio Estevam,
	Krzysztof Kozlowski, Peng Fan, Rob Herring, Shawn Guo,
	NXP Linux Team, kernel, devicetree

The "regulator-compatible" property is deprecated and unused, as the
match happens on the node name in Linux of_regulator_match() in case
the property is not present. Drop the deprecated property from DT.

Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Christoph Niedermaier <cniedermaier@dh-electronics.com>
Cc: Fabio Estevam <festevam@denx.de>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: kernel@dh-electronics.com
Cc: devicetree@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
 arch/arm64/boot/dts/freescale/imx8mp-dhcom-som.dtsi | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-dhcom-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-dhcom-som.dtsi
index 6e1192e751f81..21b1d75a9a1cf 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-dhcom-som.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-dhcom-som.dtsi
@@ -248,7 +248,6 @@ pmic: pmic@25 {
 		 */
 		regulators {
 			buck1: BUCK1 {	/* VDD_SOC (dual-phase with BUCK3) */
-				regulator-compatible = "BUCK1";
 				regulator-min-microvolt = <850000>;
 				regulator-max-microvolt = <1000000>;
 				regulator-ramp-delay = <3125>;
@@ -257,7 +256,6 @@ buck1: BUCK1 {	/* VDD_SOC (dual-phase with BUCK3) */
 			};
 
 			buck2: BUCK2 {	/* VDD_ARM */
-				regulator-compatible = "BUCK2";
 				regulator-min-microvolt = <850000>;
 				regulator-max-microvolt = <1000000>;
 				regulator-ramp-delay = <3125>;
@@ -266,7 +264,6 @@ buck2: BUCK2 {	/* VDD_ARM */
 			};
 
 			buck4: BUCK4 {	/* VDD_3V3 */
-				regulator-compatible = "BUCK4";
 				regulator-min-microvolt = <3300000>;
 				regulator-max-microvolt = <3300000>;
 				regulator-always-on;
@@ -274,7 +271,6 @@ buck4: BUCK4 {	/* VDD_3V3 */
 			};
 
 			buck5: BUCK5 {	/* VDD_1V8 */
-				regulator-compatible = "BUCK5";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <1800000>;
 				regulator-always-on;
@@ -282,7 +278,6 @@ buck5: BUCK5 {	/* VDD_1V8 */
 			};
 
 			buck6: BUCK6 {	/* NVCC_DRAM_1V1 */
-				regulator-compatible = "BUCK6";
 				regulator-min-microvolt = <1100000>;
 				regulator-max-microvolt = <1100000>;
 				regulator-always-on;
@@ -290,7 +285,6 @@ buck6: BUCK6 {	/* NVCC_DRAM_1V1 */
 			};
 
 			ldo1: LDO1 {	/* NVCC_SNVS_1V8 */
-				regulator-compatible = "LDO1";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <1800000>;
 				regulator-always-on;
@@ -298,7 +292,6 @@ ldo1: LDO1 {	/* NVCC_SNVS_1V8 */
 			};
 
 			ldo3: LDO3 {	/* VDDA_1V8 */
-				regulator-compatible = "LDO3";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <1800000>;
 				regulator-always-on;
@@ -306,13 +299,11 @@ ldo3: LDO3 {	/* VDDA_1V8 */
 			};
 
 			ldo4: LDO4 {	/* PMIC_LDO4 */
-				regulator-compatible = "LDO4";
 				regulator-min-microvolt = <3300000>;
 				regulator-max-microvolt = <3300000>;
 			};
 
 			ldo5: LDO5 {	/* NVCC_SD2 */
-				regulator-compatible = "LDO5";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <3300000>;
 			};
-- 
2.35.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 2/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus PhyCore SoM
  2022-12-15 23:57 [PATCH 1/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus DHCOM Marek Vasut
@ 2022-12-15 23:57 ` Marek Vasut
  2022-12-16  0:33   ` Fabio Estevam
  2023-01-01  2:24 ` [PATCH 1/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus DHCOM Shawn Guo
  1 sibling, 1 reply; 5+ messages in thread
From: Marek Vasut @ 2022-12-15 23:57 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Marek Vasut, Fabio Estevam, Jonas Kuenstler, Krzysztof Kozlowski,
	Peng Fan, Rob Herring, Shawn Guo, Teresa Remmet, NXP Linux Team,
	devicetree

The "regulator-compatible" property is deprecated and unused, as the
match happens on the node name in Linux of_regulator_match() in case
the property is not present. Drop the deprecated property from DT.

Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Fabio Estevam <festevam@denx.de>
Cc: Jonas Kuenstler <j.kuenstler@phytec.de>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Teresa Remmet <t.remmet@phytec.de>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: devicetree@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
 arch/arm64/boot/dts/freescale/imx8mp-phycore-som.dtsi | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-phycore-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-phycore-som.dtsi
index 79b290a002c19..ecc4bce6db97c 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-phycore-som.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-phycore-som.dtsi
@@ -99,7 +99,6 @@ pmic: pmic@25 {
 
 		regulators {
 			buck1: BUCK1 {
-				regulator-compatible = "BUCK1";
 				regulator-min-microvolt = <600000>;
 				regulator-max-microvolt = <2187500>;
 				regulator-boot-on;
@@ -108,7 +107,6 @@ buck1: BUCK1 {
 			};
 
 			buck2: BUCK2 {
-				regulator-compatible = "BUCK2";
 				regulator-min-microvolt = <600000>;
 				regulator-max-microvolt = <2187500>;
 				regulator-boot-on;
@@ -119,7 +117,6 @@ buck2: BUCK2 {
 			};
 
 			buck4: BUCK4 {
-				regulator-compatible = "BUCK4";
 				regulator-min-microvolt = <600000>;
 				regulator-max-microvolt = <3400000>;
 				regulator-boot-on;
@@ -127,7 +124,6 @@ buck4: BUCK4 {
 			};
 
 			buck5: BUCK5 {
-				regulator-compatible = "BUCK5";
 				regulator-min-microvolt = <600000>;
 				regulator-max-microvolt = <3400000>;
 				regulator-boot-on;
@@ -135,7 +131,6 @@ buck5: BUCK5 {
 			};
 
 			buck6: BUCK6 {
-				regulator-compatible = "BUCK6";
 				regulator-min-microvolt = <600000>;
 				regulator-max-microvolt = <3400000>;
 				regulator-boot-on;
@@ -143,7 +138,6 @@ buck6: BUCK6 {
 			};
 
 			ldo1: LDO1 {
-				regulator-compatible = "LDO1";
 				regulator-min-microvolt = <1600000>;
 				regulator-max-microvolt = <3300000>;
 				regulator-boot-on;
@@ -151,7 +145,6 @@ ldo1: LDO1 {
 			};
 
 			ldo2: LDO2 {
-				regulator-compatible = "LDO2";
 				regulator-min-microvolt = <800000>;
 				regulator-max-microvolt = <1150000>;
 				regulator-boot-on;
@@ -159,7 +152,6 @@ ldo2: LDO2 {
 			};
 
 			ldo3: LDO3 {
-				regulator-compatible = "LDO3";
 				regulator-min-microvolt = <800000>;
 				regulator-max-microvolt = <3300000>;
 				regulator-boot-on;
@@ -167,13 +159,11 @@ ldo3: LDO3 {
 			};
 
 			ldo4: LDO4 {
-				regulator-compatible = "LDO4";
 				regulator-min-microvolt = <800000>;
 				regulator-max-microvolt = <3300000>;
 			};
 
 			ldo5: LDO5 {
-				regulator-compatible = "LDO5";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <3300000>;
 				regulator-boot-on;
-- 
2.35.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH 2/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus PhyCore SoM
  2022-12-15 23:57 ` [PATCH 2/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus PhyCore SoM Marek Vasut
@ 2022-12-16  0:33   ` Fabio Estevam
  2022-12-16  0:36     ` Marek Vasut
  0 siblings, 1 reply; 5+ messages in thread
From: Fabio Estevam @ 2022-12-16  0:33 UTC (permalink / raw)
  To: Marek Vasut
  Cc: linux-arm-kernel, Jonas Kuenstler, Krzysztof Kozlowski, Peng Fan,
	Rob Herring, Shawn Guo, Teresa Remmet, NXP Linux Team, devicetree

Hi Marek,

On 15/12/2022 20:57, Marek Vasut wrote:
> The "regulator-compatible" property is deprecated and unused, as the
> match happens on the node name in Linux of_regulator_match() in case
> the property is not present. Drop the deprecated property from DT.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>

Thanks for the patch.

I have already sent the same fix:

https://lore.kernel.org/linux-arm-kernel/20221121162911.2377556-1-festevam@gmail.com/T/

Regards,

Fabio Estevam

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus PhyCore SoM
  2022-12-16  0:33   ` Fabio Estevam
@ 2022-12-16  0:36     ` Marek Vasut
  0 siblings, 0 replies; 5+ messages in thread
From: Marek Vasut @ 2022-12-16  0:36 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: linux-arm-kernel, Jonas Kuenstler, Krzysztof Kozlowski, Peng Fan,
	Rob Herring, Shawn Guo, Teresa Remmet, NXP Linux Team, devicetree

On 12/16/22 01:33, Fabio Estevam wrote:
> Hi Marek,

Hi,

> On 15/12/2022 20:57, Marek Vasut wrote:
>> The "regulator-compatible" property is deprecated and unused, as the
>> match happens on the node name in Linux of_regulator_match() in case
>> the property is not present. Drop the deprecated property from DT.
>>
>> Signed-off-by: Marek Vasut <marex@denx.de>
> 
> Thanks for the patch.
> 
> I have already sent the same fix:
> 
> https://lore.kernel.org/linux-arm-kernel/20221121162911.2377556-1-festevam@gmail.com/T/

Ah nice, thanks !

Please drop this patch then.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 1/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus DHCOM
  2022-12-15 23:57 [PATCH 1/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus DHCOM Marek Vasut
  2022-12-15 23:57 ` [PATCH 2/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus PhyCore SoM Marek Vasut
@ 2023-01-01  2:24 ` Shawn Guo
  1 sibling, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2023-01-01  2:24 UTC (permalink / raw)
  To: Marek Vasut
  Cc: linux-arm-kernel, Christoph Niedermaier, Fabio Estevam,
	Krzysztof Kozlowski, Peng Fan, Rob Herring, NXP Linux Team,
	kernel, devicetree

On Fri, Dec 16, 2022 at 12:57:09AM +0100, Marek Vasut wrote:
> The "regulator-compatible" property is deprecated and unused, as the
> match happens on the node name in Linux of_regulator_match() in case
> the property is not present. Drop the deprecated property from DT.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>

Applied, thanks!

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-01-01  2:24 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-12-15 23:57 [PATCH 1/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus DHCOM Marek Vasut
2022-12-15 23:57 ` [PATCH 2/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus PhyCore SoM Marek Vasut
2022-12-16  0:33   ` Fabio Estevam
2022-12-16  0:36     ` Marek Vasut
2023-01-01  2:24 ` [PATCH 1/2] arm64: dts: imx8mp: Drop deprecated regulator-compatible from i.MX8M Plus DHCOM Shawn Guo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).