All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Marangi <ansuelsmth@gmail.com>
To: Andy Gross <agross@kernel.org>,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	Konrad Dybcio <konrad.dybcio@somainline.org>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Cc: Christian Marangi <ansuelsmth@gmail.com>,
	Jonathan McDowell <noodles@earth.li>
Subject: [PATCH 07/13] ARM: dts: qcom: reduce pci IO size to 64K for ipq8064
Date: Tue,  5 Jul 2022 15:39:11 +0200	[thread overview]
Message-ID: <20220705133917.8405-8-ansuelsmth@gmail.com> (raw)
In-Reply-To: <20220705133917.8405-1-ansuelsmth@gmail.com>

The current value for pci IO is problematic for ath10k wifi card
commonly connected to ipq8064 SoC.
The current value is probably a typo and is actually uncommon to find
1MB IO space even on a x86 arch. Also with recent changes to the pci
driver, pci1 and pci2 now fails to function as any connected device
fails any reg read/write. Reduce this to 64K as it should be more than
enough and 3 * 64K of total IO space doesn't exceed the IO_SPACE_LIMIT
hardcoded for the ARM arch.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Tested-by: Jonathan McDowell <noodles@earth.li>
---
 arch/arm/boot/dts/qcom-ipq8064.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom-ipq8064.dtsi
index 9c32c637ea46..e13f6ae92e05 100644
--- a/arch/arm/boot/dts/qcom-ipq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-ipq8064.dtsi
@@ -920,7 +920,7 @@ pcie0: pci@1b500000 {
 			#address-cells = <3>;
 			#size-cells = <2>;
 
-			ranges = <0x81000000 0 0x0fe00000 0x0fe00000 0 0x00100000   /* downstream I/O */
+			ranges = <0x81000000 0 0x0fe00000 0x0fe00000 0 0x00010000   /* downstream I/O */
 				  0x82000000 0 0x08000000 0x08000000 0 0x07e00000>; /* non-prefetchable memory */
 
 			interrupts = <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>;
@@ -971,7 +971,7 @@ pcie1: pci@1b700000 {
 			#address-cells = <3>;
 			#size-cells = <2>;
 
-			ranges = <0x81000000 0 0x31e00000 0x31e00000 0 0x00100000   /* downstream I/O */
+			ranges = <0x81000000 0 0x31e00000 0x31e00000 0 0x00010000   /* downstream I/O */
 				  0x82000000 0 0x2e000000 0x2e000000 0 0x03e00000>; /* non-prefetchable memory */
 
 			interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>;
@@ -1022,7 +1022,7 @@ pcie2: pci@1b900000 {
 			#address-cells = <3>;
 			#size-cells = <2>;
 
-			ranges = <0x81000000 0 0x35e00000 0x35e00000 0 0x00100000   /* downstream I/O */
+			ranges = <0x81000000 0 0x35e00000 0x35e00000 0 0x00010000   /* downstream I/O */
 				  0x82000000 0 0x32000000 0x32000000 0 0x03e00000>; /* non-prefetchable memory */
 
 			interrupts = <GIC_SPI 71 IRQ_TYPE_LEVEL_HIGH>;
-- 
2.36.1


  parent reply	other threads:[~2022-07-05 14:13 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-05 13:39 [PATCH 00/13] Add ipq806x missing bindings Christian Marangi
2022-07-05 13:39 ` [PATCH 01/13] ARM: dts: qcom: add multiple missing pin definition for ipq8064 Christian Marangi
2022-07-06 12:48   ` Konrad Dybcio
2022-07-05 13:39 ` [PATCH 02/13] ARM: dts: qcom: add gsbi6 missing " Christian Marangi
2022-07-06  8:33   ` Krzysztof Kozlowski
2022-07-06 12:48   ` Konrad Dybcio
2022-07-05 13:39 ` [PATCH 03/13] ARM: dts: qcom: add missing rpm regulators and cells " Christian Marangi
2022-07-06  8:34   ` Krzysztof Kozlowski
2022-07-06 10:09     ` Christian Marangi
2022-07-06 11:55       ` Dmitry Baryshkov
2022-07-06 14:42       ` Krzysztof Kozlowski
2022-07-06 13:02   ` Konrad Dybcio
2022-07-06 12:52     ` Christian Marangi
2022-07-05 13:39 ` [PATCH 04/13] ARM: dts: qcom: disable smb208 regulators for ipq8064-rb3011 Christian Marangi
2022-07-06  8:34   ` Krzysztof Kozlowski
2022-07-06 10:11     ` Christian Marangi
2022-07-06 13:03   ` Konrad Dybcio
2022-07-06 12:56     ` Christian Marangi
2022-07-06 13:31       ` Konrad Dybcio
2022-07-06 13:19         ` Christian Marangi
2022-07-06 13:37           ` Konrad Dybcio
2022-07-06 13:25             ` Christian Marangi
2022-07-05 13:39 ` [PATCH 05/13] ARM: dts: qcom: add missing snps,dwmac compatible for gmac ipq8064 Christian Marangi
2022-07-06  8:35   ` Krzysztof Kozlowski
2022-07-06 13:04   ` Konrad Dybcio
2022-07-05 13:39 ` [PATCH 06/13] ARM: dts: qcom: enable usb phy by default for ipq8064 Christian Marangi
2022-07-06  8:36   ` Krzysztof Kozlowski
2022-07-06 13:04   ` Konrad Dybcio
2022-07-06 12:59     ` Christian Marangi
2022-07-06 13:33       ` Konrad Dybcio
2022-07-06 13:20         ` Christian Marangi
2022-07-05 13:39 ` Christian Marangi [this message]
2022-07-05 13:39 ` [PATCH 08/13] ARM: dts: qcom: fix dtc warning for missing #address-cells " Christian Marangi
2022-07-06  8:37   ` Krzysztof Kozlowski
2022-07-06 10:12     ` Christian Marangi
2022-07-06 14:45       ` Krzysztof Kozlowski
2022-07-05 13:39 ` [PATCH 09/13] ARM: dts: qcom: add smem node " Christian Marangi
2022-07-06  8:39   ` Krzysztof Kozlowski
2022-07-06 10:14     ` Christian Marangi
2022-07-06 10:37       ` Christian Marangi
2022-07-06 12:01       ` Dmitry Baryshkov
2022-07-06 14:46       ` Krzysztof Kozlowski
2022-07-06 13:11   ` Konrad Dybcio
2022-07-06 13:03     ` Christian Marangi
2022-07-06 13:41     ` Dmitry Baryshkov
2022-07-05 13:39 ` [PATCH 10/13] ARM: dts: qcom: add sic non secure " Christian Marangi
2022-07-06  8:39   ` Krzysztof Kozlowski
2022-07-05 13:39 ` [PATCH 11/13] ARM: dts: qcom: fix and add some missing gsbi " Christian Marangi
2022-07-06  8:40   ` Krzysztof Kozlowski
2022-07-05 13:39 ` [PATCH 12/13] ARM: dts: qcom: add speedbin efuse nvmem binding Christian Marangi
2022-07-06  8:40   ` Krzysztof Kozlowski
2022-07-05 13:39 ` [PATCH 13/13] ARM: dts: qcom: remove redundant binding from ipq8064 rb3011 dts Christian Marangi
2022-07-05 14:28 ` [PATCH 00/13] Add ipq806x missing bindings Krzysztof Kozlowski
2022-07-05 14:04   ` Christian Marangi
2022-07-05 15:55     ` Krzysztof Kozlowski
2022-07-05 15:40       ` Christian Marangi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220705133917.8405-8-ansuelsmth@gmail.com \
    --to=ansuelsmth@gmail.com \
    --cc=agross@kernel.org \
    --cc=bjorn.andersson@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=konrad.dybcio@somainline.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=noodles@earth.li \
    --cc=robh+dt@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.