| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Wed, 13 Jul 2022 09:52:42 +0200
From: Heiko Thiery <heiko.thiery@...il.com>
To: Frieder Schrempf <frieder@...s.de>
Cc: devicetree@...r.kernel.org,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
Rob Herring <robh+dt@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Shawn Guo <shawnguo@...nel.org>,
Frieder Schrempf <frieder.schrempf@...tron.de>,
Fabio Estevam <festevam@...il.com>,
Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
NXP Linux Team <linux-imx@....com>,
Pengutronix Kernel Team <kernel@...gutronix.de>
Subject: Re: [PATCH 4/6] arm64: dts: imx8mm-kontron: Use upper case PMIC
regulator names
Hi Frieder,
Am Mi., 13. Juli 2022 um 09:41 Uhr schrieb Frieder Schrempf <frieder@...s.de>:
>
> From: Frieder Schrempf <frieder.schrempf@...tron.de>
>
> Use upper case PMIC regulator names to comply with the bindings.
>
> Signed-off-by: Frieder Schrempf <frieder.schrempf@...tron.de>
> ---
> .../freescale/imx8mm-kontron-n801x-som.dtsi | 22 +++++++++----------
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi
> index 2d0661cce89b..2e3d51bbf92e 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi
> @@ -85,7 +85,7 @@ pca9450: pmic@25 {
>
> regulators {
> reg_vdd_soc: BUCK1 {
> - regulator-name = "buck1";
> + regulator-name = "BUCK1";
Wouldn't it be better to use the real signal or voltage rail names
used in the schematics?
Like it is done here as example:
https://elixir.bootlin.com/linux/v5.19-rc6/source/arch/arm64/boot/dts/freescale/imx8mn-evk.dts#L44
Or use a name that includes both information (signal + PMIC out):
https://elixir.bootlin.com/linux/v5.19-rc6/source/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi#L445
> regulator-min-microvolt = <800000>;
> regulator-max-microvolt = <850000>;
> regulator-boot-on;
> @@ -96,7 +96,7 @@ reg_vdd_soc: BUCK1 {
> };
>
> reg_vdd_arm: BUCK2 {
> - regulator-name = "buck2";
> + regulator-name = "BUCK2";
> regulator-min-microvolt = <850000>;
> regulator-max-microvolt = <950000>;
> regulator-boot-on;
> @@ -107,7 +107,7 @@ reg_vdd_arm: BUCK2 {
> };
>
> reg_vdd_dram: BUCK3 {
> - regulator-name = "buck3";
> + regulator-name = "BUCK3";
> regulator-min-microvolt = <850000>;
> regulator-max-microvolt = <950000>;
> regulator-boot-on;
> @@ -115,7 +115,7 @@ reg_vdd_dram: BUCK3 {
> };
>
> reg_vdd_3v3: BUCK4 {
> - regulator-name = "buck4";
> + regulator-name = "BUCK4";
> regulator-min-microvolt = <3300000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> @@ -123,7 +123,7 @@ reg_vdd_3v3: BUCK4 {
> };
>
> reg_vdd_1v8: BUCK5 {
> - regulator-name = "buck5";
> + regulator-name = "BUCK5";
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> regulator-boot-on;
> @@ -131,7 +131,7 @@ reg_vdd_1v8: BUCK5 {
> };
>
> reg_nvcc_dram: BUCK6 {
> - regulator-name = "buck6";
> + regulator-name = "BUCK6";
> regulator-min-microvolt = <1100000>;
> regulator-max-microvolt = <1100000>;
> regulator-boot-on;
> @@ -139,7 +139,7 @@ reg_nvcc_dram: BUCK6 {
> };
>
> reg_nvcc_snvs: LDO1 {
> - regulator-name = "ldo1";
> + regulator-name = "LDO1";
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> regulator-boot-on;
> @@ -147,7 +147,7 @@ reg_nvcc_snvs: LDO1 {
> };
>
> reg_vdd_snvs: LDO2 {
> - regulator-name = "ldo2";
> + regulator-name = "LDO2";
> regulator-min-microvolt = <800000>;
> regulator-max-microvolt = <900000>;
> regulator-boot-on;
> @@ -155,7 +155,7 @@ reg_vdd_snvs: LDO2 {
> };
>
> reg_vdda: LDO3 {
> - regulator-name = "ldo3";
> + regulator-name = "LDO3";
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> regulator-boot-on;
> @@ -163,7 +163,7 @@ reg_vdda: LDO3 {
> };
>
> reg_vdd_phy: LDO4 {
> - regulator-name = "ldo4";
> + regulator-name = "LDO4";
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <900000>;
> regulator-boot-on;
> @@ -171,7 +171,7 @@ reg_vdd_phy: LDO4 {
> };
>
> reg_nvcc_sd: LDO5 {
> - regulator-name = "ldo5";
> + regulator-name = "LDO5";
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <3300000>;
> };
> --
> 2.37.0
>
Powered by blists - more mailing lists