* [PATCH v2 0/2] arm64: dts: qcom: ipq8074: add MicroSD node
@ 2024-01-10 15:10 Chukun Pan
2024-01-10 15:10 ` [PATCH v2 1/2] arm64: dts: qcom: ipq6018: add LDOA2 regulator Chukun Pan
2024-01-10 15:10 ` [PATCH v2 2/2] arm64: dts: qcom: ipq6018: enable sdhci node Chukun Pan
0 siblings, 2 replies; 9+ messages in thread
From: Chukun Pan @ 2024-01-10 15:10 UTC (permalink / raw)
To: Bjorn Andersson
Cc: Konrad Dybcio, Conor Dooley, Krzysztof Kozlowski, linux-arm-msm,
linux-kernel, devicetree, Chukun Pan
> Changes from v2:
> * arm64: dts: qcom: ipq6018: add LDOA2 regulator
> No Changes:
> * arm64: dts: qcom: ipq8074: add MicroSD node
--
2.25.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 1/2] arm64: dts: qcom: ipq6018: add LDOA2 regulator
2024-01-10 15:10 [PATCH v2 0/2] arm64: dts: qcom: ipq8074: add MicroSD node Chukun Pan
@ 2024-01-10 15:10 ` Chukun Pan
2024-01-11 12:19 ` Konrad Dybcio
2024-01-10 15:10 ` [PATCH v2 2/2] arm64: dts: qcom: ipq6018: enable sdhci node Chukun Pan
1 sibling, 1 reply; 9+ messages in thread
From: Chukun Pan @ 2024-01-10 15:10 UTC (permalink / raw)
To: Bjorn Andersson
Cc: Konrad Dybcio, Conor Dooley, Krzysztof Kozlowski, linux-arm-msm,
linux-kernel, devicetree, Chukun Pan, Robert Marko
Add LDOA2 regulator of MP5496 to support SDCC voltage scaling.
Suggested-by: Robert Marko <robimarko@gmail.com>
Signed-off-by: Chukun Pan <amadeus@jmu.edu.cn>
---
arch/arm64/boot/dts/qcom/ipq6018.dtsi | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
index 5e1277fea725..2f892d3bbbc6 100644
--- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
@@ -174,6 +174,12 @@ ipq6018_s2: s2 {
regulator-max-microvolt = <1062500>;
regulator-always-on;
};
+
+ ipq6018_l2: l2 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
};
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v2 2/2] arm64: dts: qcom: ipq6018: enable sdhci node
2024-01-10 15:10 [PATCH v2 0/2] arm64: dts: qcom: ipq8074: add MicroSD node Chukun Pan
2024-01-10 15:10 ` [PATCH v2 1/2] arm64: dts: qcom: ipq6018: add LDOA2 regulator Chukun Pan
@ 2024-01-10 15:10 ` Chukun Pan
2024-01-10 15:22 ` Krzysztof Kozlowski
1 sibling, 1 reply; 9+ messages in thread
From: Chukun Pan @ 2024-01-10 15:10 UTC (permalink / raw)
To: Bjorn Andersson
Cc: Konrad Dybcio, Conor Dooley, Krzysztof Kozlowski, linux-arm-msm,
linux-kernel, devicetree, Chukun Pan
Enable mmc device found on ipq6018 devices.
This node supports both eMMC and SD cards.
Tested with:
eMMC (HS200)
SD Card (SDR50/SDR104)
Signed-off-by: Chukun Pan <amadeus@jmu.edu.cn>
---
arch/arm64/boot/dts/qcom/ipq6018.dtsi | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
index 5e1277fea725..39fb38914a1e 100644
--- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
@@ -436,6 +436,28 @@ dwc_1: usb@7000000 {
};
};
+ sdhc: mmc@7804000 {
+ compatible = "qcom,ipq6018-sdhci", "qcom,sdhci-msm-v5";
+ reg = <0x0 0x7804000 0x0 0x1000>,
+ <0x0 0x7805000 0x0 0x1000>;
+ reg-names = "hc", "cqhci";
+
+ interrupts = <GIC_SPI 123 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 138 IRQ_TYPE_LEVEL_HIGH>;
+ interrupt-names = "hc_irq", "pwr_irq";
+
+ clocks = <&gcc GCC_SDCC1_AHB_CLK>,
+ <&gcc GCC_SDCC1_APPS_CLK>,
+ <&xo>;
+ clock-names = "iface", "core", "xo";
+ resets = <&gcc GCC_SDCC1_BCR>;
+ max-frequency = <192000000>;
+ mmc-ddr-1_8v;
+ mmc-hs200-1_8v;
+
+ status = "disabled";
+ };
+
blsp_dma: dma-controller@7884000 {
compatible = "qcom,bam-v1.7.0";
reg = <0x0 0x07884000 0x0 0x2b000>;
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: qcom: ipq6018: enable sdhci node
2024-01-10 15:10 ` [PATCH v2 2/2] arm64: dts: qcom: ipq6018: enable sdhci node Chukun Pan
@ 2024-01-10 15:22 ` Krzysztof Kozlowski
2024-01-11 3:10 ` Chukun Pan
0 siblings, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2024-01-10 15:22 UTC (permalink / raw)
To: Chukun Pan, Bjorn Andersson
Cc: Konrad Dybcio, Conor Dooley, Krzysztof Kozlowski, linux-arm-msm,
linux-kernel, devicetree
On 10/01/2024 16:10, Chukun Pan wrote:
> Enable mmc device found on ipq6018 devices.
> This node supports both eMMC and SD cards.
>
> Tested with:
> eMMC (HS200)
> SD Card (SDR50/SDR104)
How? It is disabled...
...
> +
> + clocks = <&gcc GCC_SDCC1_AHB_CLK>,
> + <&gcc GCC_SDCC1_APPS_CLK>,
> + <&xo>;
> + clock-names = "iface", "core", "xo";
> + resets = <&gcc GCC_SDCC1_BCR>;
> + max-frequency = <192000000>;
> + mmc-ddr-1_8v;
> + mmc-hs200-1_8v;
Aren't these three properties of the board?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: qcom: ipq6018: enable sdhci node
2024-01-10 15:22 ` Krzysztof Kozlowski
@ 2024-01-11 3:10 ` Chukun Pan
2024-01-11 8:01 ` Krzysztof Kozlowski
0 siblings, 1 reply; 9+ messages in thread
From: Chukun Pan @ 2024-01-11 3:10 UTC (permalink / raw)
To: krzysztof.kozlowski
Cc: amadeus, andersson, conor+dt, devicetree, konrad.dybcio,
krzysztof.kozlowski+dt, linux-arm-msm, linux-kernel
On 10/01/2024 16:10, Chukun Pan wrote:
> Changes from v2:
> * arm64: dts: qcom: ipq6018: add LDOA2 regulator
> No Changes:
> * arm64: dts: qcom: ipq8074: add MicroSD node
I wrote it wrong in cover-letter, it's ipq6018, not ipq8074.
No Changes:
arm64: dts: qcom: ipq6018: enable sdhci node
On 10/01/2024 16:22, Krzysztof wrote:
> How? It is disabled...
Disable it because not all devices have eMMC or MicroSD connected.
I enabled it on the dts of the device and tested it on my ipq6000.
> Aren't these three properties of the board?
These are just general properties.
Thanks,
Chukun
--
2.25.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: qcom: ipq6018: enable sdhci node
2024-01-11 3:10 ` Chukun Pan
@ 2024-01-11 8:01 ` Krzysztof Kozlowski
2024-01-11 11:20 ` Chukun Pan
0 siblings, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2024-01-11 8:01 UTC (permalink / raw)
To: Chukun Pan
Cc: andersson, conor+dt, devicetree, konrad.dybcio,
krzysztof.kozlowski+dt, linux-arm-msm, linux-kernel
On 11/01/2024 04:10, Chukun Pan wrote:
> Disable it because not all devices have eMMC or MicroSD connected.
> I enabled it on the dts of the device and tested it on my ipq6000.
>
>> Aren't these three properties of the board?
>
> These are just general properties.
? What is a general property? I repeat my question - these do not look
SoC specific, but board.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: qcom: ipq6018: enable sdhci node
2024-01-11 8:01 ` Krzysztof Kozlowski
@ 2024-01-11 11:20 ` Chukun Pan
0 siblings, 0 replies; 9+ messages in thread
From: Chukun Pan @ 2024-01-11 11:20 UTC (permalink / raw)
To: krzysztof.kozlowski
Cc: amadeus, andersson, conor+dt, devicetree, konrad.dybcio,
krzysztof.kozlowski+dt, linux-arm-msm, linux-kernel
> ? What is a general property? I repeat my question - these do not look
> SoC specific, but board.
Sorry, I understood it wrong. I will remove the 1.8v properties
in the next patch.
Thanks,
Chukun
--
2.25.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: qcom: ipq6018: add LDOA2 regulator
2024-01-10 15:10 ` [PATCH v2 1/2] arm64: dts: qcom: ipq6018: add LDOA2 regulator Chukun Pan
@ 2024-01-11 12:19 ` Konrad Dybcio
2024-01-14 8:30 ` Chukun Pan
0 siblings, 1 reply; 9+ messages in thread
From: Konrad Dybcio @ 2024-01-11 12:19 UTC (permalink / raw)
To: Chukun Pan, Bjorn Andersson
Cc: Conor Dooley, Krzysztof Kozlowski, linux-arm-msm, linux-kernel,
devicetree, Robert Marko
On 1/10/24 16:10, Chukun Pan wrote:
> Add LDOA2 regulator of MP5496 to support SDCC voltage scaling.
>
> Suggested-by: Robert Marko <robimarko@gmail.com>
> Signed-off-by: Chukun Pan <amadeus@jmu.edu.cn>
> ---
> arch/arm64/boot/dts/qcom/ipq6018.dtsi | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> index 5e1277fea725..2f892d3bbbc6 100644
> --- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> @@ -174,6 +174,12 @@ ipq6018_s2: s2 {
> regulator-max-microvolt = <1062500>;
> regulator-always-on;
> };
> +
> + ipq6018_l2: l2 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
Why is is always-on?
Konrad
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: qcom: ipq6018: add LDOA2 regulator
2024-01-11 12:19 ` Konrad Dybcio
@ 2024-01-14 8:30 ` Chukun Pan
0 siblings, 0 replies; 9+ messages in thread
From: Chukun Pan @ 2024-01-14 8:30 UTC (permalink / raw)
To: konrad.dybcio
Cc: amadeus, andersson, conor+dt, devicetree, krzysztof.kozlowski+dt,
linux-arm-msm, linux-kernel, robimarko
> Why is is always-on?
No need, I will remove it in the next patch.
Thanks,
Chukun
--
2.25.1
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2024-01-14 8:49 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-10 15:10 [PATCH v2 0/2] arm64: dts: qcom: ipq8074: add MicroSD node Chukun Pan
2024-01-10 15:10 ` [PATCH v2 1/2] arm64: dts: qcom: ipq6018: add LDOA2 regulator Chukun Pan
2024-01-11 12:19 ` Konrad Dybcio
2024-01-14 8:30 ` Chukun Pan
2024-01-10 15:10 ` [PATCH v2 2/2] arm64: dts: qcom: ipq6018: enable sdhci node Chukun Pan
2024-01-10 15:22 ` Krzysztof Kozlowski
2024-01-11 3:10 ` Chukun Pan
2024-01-11 8:01 ` Krzysztof Kozlowski
2024-01-11 11:20 ` Chukun Pan
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).