2021-02-19 10:06:43

by Martin Kepplinger

[permalink] [raw]
Subject: [PATCH v3 0/5] arm64: dts: librem5 phone and devkit dts and config updates

hi,

Small updates to Purism hardware in order to maintain support.

revision history
----------------
v3: (thanks for the patience, Krzysztof)
* fix flash partition DT syntax

v2: (thank you, Krzysztof)
* fix flash partition DT syntax
* add one reviewed tag
https://lore.kernel.org/linux-arm-kernel/[email protected]/

v1:
https://lore.kernel.org/linux-arm-kernel/[email protected]/T/


Angus Ainslie (1):
arm64: dts: librem5: protect some partitions of the nor-flash

Guido Günther (3):
arm64: dts: imx8mq-librem5-devkit: Drop buck3 startup-ramp-delay
arm64: dts: librem5: Drop assigned-clocks from SAI2
arm64: defconfig: Enable devfreq support for i.MX8MQ

Martin Kepplinger (1):
arm64: dts: imx8mq-librem5-r2: set nearlevel to 120

.../dts/freescale/imx8mq-librem5-devkit.dts | 1 -
.../boot/dts/freescale/imx8mq-librem5-r2.dts | 2 +-
.../boot/dts/freescale/imx8mq-librem5.dtsi | 19 +++++++++++++++++--
arch/arm64/configs/defconfig | 2 ++
4 files changed, 20 insertions(+), 4 deletions(-)

--
2.20.1


2021-02-19 10:07:02

by Martin Kepplinger

[permalink] [raw]
Subject: [PATCH v3 1/5] arm64: dts: imx8mq-librem5-devkit: Drop buck3 startup-ramp-delay

From: Guido Günther <[email protected]>

The PMIC driver now sets appropriate default delays.

Signed-off-by: Guido Günther <[email protected]>
Signed-off-by: Martin Kepplinger <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dts | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dts b/arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dts
index dd217a0760e9..88508890063f 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dts
@@ -315,7 +315,6 @@
regulator-min-microvolt = <700000>;
regulator-max-microvolt = <1300000>;
regulator-boot-on;
- regulator-enable-ramp-delay = <200>;
rohm,dvs-run-voltage = <900000>;
};

--
2.20.1

2021-02-19 10:07:16

by Martin Kepplinger

[permalink] [raw]
Subject: [PATCH v3 2/5] arm64: dts: librem5: Drop assigned-clocks from SAI2

From: Guido Günther <[email protected]>

IMX8MQ_AUDIO_PLL1 and IMX8MQ_AUDIO_PLL2 are setup to the same rates
right on the clock controller.

Signed-off-by: Guido Günther <[email protected]>
Signed-off-by: Martin Kepplinger <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi | 2 --
1 file changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
index 06a4799b6aeb..c2bbbdeb93e3 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
@@ -1051,8 +1051,6 @@
assigned-clocks = <&clk IMX8MQ_CLK_SAI2>;
assigned-clock-parents = <&clk IMX8MQ_AUDIO_PLL1_OUT>;
assigned-clock-rates = <24576000>;
- assigned-clocks = <&clk IMX8MQ_AUDIO_PLL1>, <&clk IMX8MQ_AUDIO_PLL2>;
- assigned-clock-rates = <786432000>, <722534400>;
status = "okay";
};

--
2.20.1

2021-02-19 10:07:59

by Martin Kepplinger

[permalink] [raw]
Subject: [PATCH v3 3/5] arm64: defconfig: Enable devfreq support for i.MX8MQ

From: Guido Günther <[email protected]>

This is needed to make DRAM frequency scaling via devfreq as used
on the Librem 5 and Librem 5 Devkit.

With the interconnect support enabled we now need imx devfreq on top
to make this work.

Signed-off-by: Guido Günther <[email protected]>
Signed-off-by: Martin Kepplinger <[email protected]>
Reviewed-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm64/configs/defconfig | 2 ++
1 file changed, 2 insertions(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index d612f633b771..e9f3e47933d5 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -1022,6 +1022,8 @@ CONFIG_ARCH_TEGRA_186_SOC=y
CONFIG_ARCH_TEGRA_194_SOC=y
CONFIG_ARCH_TEGRA_234_SOC=y
CONFIG_TI_SCI_PM_DOMAINS=y
+CONFIG_ARM_IMX_BUS_DEVFREQ=m
+CONFIG_ARM_IMX8M_DDRC_DEVFREQ=m
CONFIG_EXTCON_PTN5150=m
CONFIG_EXTCON_USB_GPIO=y
CONFIG_EXTCON_USBC_CROS_EC=y
--
2.20.1

2021-02-19 10:08:23

by Martin Kepplinger

[permalink] [raw]
Subject: [PATCH v3 4/5] arm64: dts: librem5: protect some partitions of the nor-flash

From: Angus Ainslie <[email protected]>

These sections should be read only as they contain important data.

Signed-off-by: Angus Ainslie <[email protected]>
Signed-off-by: Martin Kepplinger <[email protected]>
---
.../boot/dts/freescale/imx8mq-librem5.dtsi | 17 +++++++++++++++++
1 file changed, 17 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
index c2bbbdeb93e3..d39ae27c8e42 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
@@ -258,6 +258,23 @@
compatible = "jedec,spi-nor";
reg = <0>;
spi-max-frequency = <1000000>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ partition@0 {
+ label = "protected0";
+ reg = <0x0 0x30000>;
+ read-only;
+ };
+ partition@30000 {
+ label = "protected1";
+ reg = <0x30000 0x10000>;
+ read-only;
+ };
+ partition@40000 {
+ label = "rw";
+ reg = <0x40000 0x1C0000>;
+ };
};
};

--
2.20.1

2021-02-19 10:09:59

by Martin Kepplinger

[permalink] [raw]
Subject: [PATCH v3 5/5] arm64: dts: imx8mq-librem5-r2: set nearlevel to 120

On Birch I can never reach 220 and hence the display would never
turn off. Tests suggest 120 to be a good threshold value for all Birch
devices.

Signed-off-by: Martin Kepplinger <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mq-librem5-r2.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-librem5-r2.dts b/arch/arm64/boot/dts/freescale/imx8mq-librem5-r2.dts
index d77fc5df3f06..73bd431cbd6a 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-librem5-r2.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mq-librem5-r2.dts
@@ -25,5 +25,5 @@
};

&proximity {
- proximity-near-level = <220>;
+ proximity-near-level = <120>;
};
--
2.20.1

2021-02-20 19:17:12

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v3 4/5] arm64: dts: librem5: protect some partitions of the nor-flash

On Fri, Feb 19, 2021 at 11:04:38AM +0100, Martin Kepplinger wrote:
> From: Angus Ainslie <[email protected]>
>
> These sections should be read only as they contain important data.
>
> Signed-off-by: Angus Ainslie <[email protected]>
> Signed-off-by: Martin Kepplinger <[email protected]>
> ---
> .../boot/dts/freescale/imx8mq-librem5.dtsi | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>

Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

2021-02-22 10:31:56

by Pavel Machek

[permalink] [raw]
Subject: Re: [PATCH v3 0/5] arm64: dts: librem5 phone and devkit dts and config updates

Hi!

> Small updates to Purism hardware in order to maintain support.

Please cc phone-devel@vger with phone stuff.

Best regards,
Pavel
--
http://www.livejournal.com/~pavelmachek


Attachments:
(No filename) (199.00 B)
signature.asc (188.00 B)
Digital signature
Download all attachments

2021-03-07 14:21:38

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH v3 4/5] arm64: dts: librem5: protect some partitions of the nor-flash

On Fri, Feb 19, 2021 at 11:04:38AM +0100, Martin Kepplinger wrote:
> From: Angus Ainslie <[email protected]>
>
> These sections should be read only as they contain important data.
>
> Signed-off-by: Angus Ainslie <[email protected]>
> Signed-off-by: Martin Kepplinger <[email protected]>
> ---
> .../boot/dts/freescale/imx8mq-librem5.dtsi | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
> index c2bbbdeb93e3..d39ae27c8e42 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
> @@ -258,6 +258,23 @@
> compatible = "jedec,spi-nor";
> reg = <0>;
> spi-max-frequency = <1000000>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> +
> + partition@0 {
> + label = "protected0";
> + reg = <0x0 0x30000>;
> + read-only;
> + };
> + partition@30000 {
> + label = "protected1";
> + reg = <0x30000 0x10000>;
> + read-only;
> + };

Please have a newline between nodes.

I fixed it up when applying.

Shawn

> + partition@40000 {
> + label = "rw";
> + reg = <0x40000 0x1C0000>;
> + };
> };
> };
>
> --
> 2.20.1
>

2021-03-07 15:23:09

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH v3 0/5] arm64: dts: librem5 phone and devkit dts and config updates

On Fri, Feb 19, 2021 at 11:04:34AM +0100, Martin Kepplinger wrote:
> Angus Ainslie (1):
> arm64: dts: librem5: protect some partitions of the nor-flash
>
> Guido G?nther (3):
> arm64: dts: imx8mq-librem5-devkit: Drop buck3 startup-ramp-delay
> arm64: dts: librem5: Drop assigned-clocks from SAI2
> arm64: defconfig: Enable devfreq support for i.MX8MQ
>
> Martin Kepplinger (1):
> arm64: dts: imx8mq-librem5-r2: set nearlevel to 120

Applied all, thanks.