diff mbox series

[1/5] arm64: dts: qcom: sdm845-mtp: fix WiFi configuration

Message ID 20230825220854.1655162-1-dmitry.baryshkov@linaro.org
State Superseded
Headers show
Series [1/5] arm64: dts: qcom: sdm845-mtp: fix WiFi configuration | expand

Commit Message

Dmitry Baryshkov Aug. 25, 2023, 10:08 p.m. UTC
Enable the host-cap-8bit quirk on this device. It is required for the
WiFi to function properly.

Fixes: 022bccb840b7 ("arm64: dts: sdm845: Add WCN3990 WLAN module device node")
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm64/boot/dts/qcom/sdm845-mtp.dts | 2 ++
 1 file changed, 2 insertions(+)

Comments

Konrad Dybcio Aug. 26, 2023, 9 a.m. UTC | #1
On 26.08.2023 00:08, Dmitry Baryshkov wrote:
> We have switched most of devices to use mbn (squashed) firmware files
> instead of spit mdt+bNN. Even this DT uses modem.mbn and a630_zap.mbn.
> Let's switch adsp and cdsp firmware files to use .mbn format too.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
Looks like 845 was mbn from the start

https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/log/qcom/sdm845?qt=grep&q=
>  arch/arm64/boot/dts/qcom/sdm845-mtp.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> index 56f5bf52d9bd..7579ba66c6d8 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> @@ -114,7 +114,7 @@ trip-point {
>  
>  &adsp_pas {
>  	status = "okay";
> -	firmware-name = "qcom/sdm845/adsp.mdt";
> +	firmware-name = "qcom/sdm845/adsp.mbn";
>  };
>  
>  &apps_rsc {
> @@ -415,7 +415,7 @@ vreg_s3c_0p6: smps3 {
>  
>  &cdsp_pas {
>  	status = "okay";
> -	firmware-name = "qcom/sdm845/cdsp.mdt";
> +	firmware-name = "qcom/sdm845/cdsp.mbn";
>  };
>  
>  &gcc {
Konrad Dybcio Aug. 26, 2023, 9:02 a.m. UTC | #2
On 26.08.2023 00:08, Dmitry Baryshkov wrote:
> Enable two PCIe hosts support on Qualcomm SDM845 MTP board.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  arch/arm64/boot/dts/qcom/sdm845-mtp.dts | 81 +++++++++++++++++++++++++
>  1 file changed, 81 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> index aec3f358d426..9fb41d2453c8 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> @@ -533,6 +533,36 @@ &mss_pil {
>  	firmware-name = "qcom/sdm845/mba.mbn", "qcom/sdm845/modem.mbn";
>  };
>  
> +&pcie0 {
> +	status = "okay";
last

> +	perst-gpios = <&tlmm 35 GPIO_ACTIVE_LOW>;
> +
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&pcie0_default_state>;
flip

[...]

> +	pcie0_default_state: pcie0-default-state {
> +		clkreq-pins {
> +			pins = "gpio36";
> +			function = "pci_e0";
> +			bias-pull-up;
> +		};
> +
> +		perst-n-pins {
> +			pins = "gpio35";
> +			function = "gpio";
> +
> +			drive-strength = <2>;
drop newline

> +			output-low;
> +			bias-pull-down;
flip

Konrad
Dmitry Baryshkov Aug. 26, 2023, 1:57 p.m. UTC | #3
On Sat, 26 Aug 2023 at 12:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>
> On 26.08.2023 00:08, Dmitry Baryshkov wrote:
> > We have switched most of devices to use mbn (squashed) firmware files
> > instead of spit mdt+bNN. Even this DT uses modem.mbn and a630_zap.mbn.
> > Let's switch adsp and cdsp firmware files to use .mbn format too.
> >
> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> > ---
> Looks like 845 was mbn from the start
>
> https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/log/qcom/sdm845?qt=grep&q=

sdm845-mtp predates these files. So is there an r-b?

> >  arch/arm64/boot/dts/qcom/sdm845-mtp.dts | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> > index 56f5bf52d9bd..7579ba66c6d8 100644
> > --- a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> > +++ b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> > @@ -114,7 +114,7 @@ trip-point {
> >
> >  &adsp_pas {
> >       status = "okay";
> > -     firmware-name = "qcom/sdm845/adsp.mdt";
> > +     firmware-name = "qcom/sdm845/adsp.mbn";
> >  };
> >
> >  &apps_rsc {
> > @@ -415,7 +415,7 @@ vreg_s3c_0p6: smps3 {
> >
> >  &cdsp_pas {
> >       status = "okay";
> > -     firmware-name = "qcom/sdm845/cdsp.mdt";
> > +     firmware-name = "qcom/sdm845/cdsp.mbn";
> >  };
> >
> >  &gcc {
Konrad Dybcio Aug. 26, 2023, 2:01 p.m. UTC | #4
On 26.08.2023 15:57, Dmitry Baryshkov wrote:
> On Sat, 26 Aug 2023 at 12:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>>
>> On 26.08.2023 00:08, Dmitry Baryshkov wrote:
>>> We have switched most of devices to use mbn (squashed) firmware files
>>> instead of spit mdt+bNN. Even this DT uses modem.mbn and a630_zap.mbn.
>>> Let's switch adsp and cdsp firmware files to use .mbn format too.
>>>
>>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>>> ---
>> Looks like 845 was mbn from the start
>>
>> https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/log/qcom/sdm845?qt=grep&q=
> 
> sdm845-mtp predates these files. So is there an r-b?
Yes, I hit enter too fast

Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
index b3c27a524742..1516113391ed 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
+++ b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
@@ -716,6 +716,8 @@  &wifi {
 	vdd-1.8-xo-supply = <&vreg_l7a_1p8>;
 	vdd-1.3-rfa-supply = <&vreg_l17a_1p3>;
 	vdd-3.3-ch0-supply = <&vreg_l25a_3p3>;
+
+	qcom,snoc-host-cap-8bit-quirk;
 };
 
 /* PINCTRL - additions to nodes defined in sdm845.dtsi */