"msi-map-mask" is a required property for all Qcom PCIe controllers as it
would allow all PCIe devices under a bus to share the same MSI identifier.
Without this property, each device has to use a separate MSI identifier
which is not possible due to platform limitations.
Currently, this is not an issue since only one device is connected to the
bus on boards making use of this SoC.
Fixes: a33a532b3b1e ("arm64: dts: qcom: sm8650: Use GIC-ITS for PCIe0 and PCIe1")
Signed-off-by: Manivannan Sadhasivam <[email protected]>
---
arch/arm64/boot/dts/qcom/sm8650.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sm8650.dtsi b/arch/arm64/boot/dts/qcom/sm8650.dtsi
index d488b3b3265e..12ba839f215e 100644
--- a/arch/arm64/boot/dts/qcom/sm8650.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8650.dtsi
@@ -2277,6 +2277,7 @@ &mc_virt SLAVE_EBI1 QCOM_ICC_TAG_ALWAYS>,
/* Entries are reversed due to the unusual ITS DeviceID encoding */
msi-map = <0x0 &gic_its 0x1401 0x1>,
<0x100 &gic_its 0x1400 0x1>;
+ msi-map-mask = <0xff00>;
linux,pci-domain = <0>;
num-lanes = <2>;
@@ -2404,6 +2405,7 @@ &mc_virt SLAVE_EBI1 QCOM_ICC_TAG_ALWAYS>,
/* Entries are reversed due to the unusual ITS DeviceID encoding */
msi-map = <0x0 &gic_its 0x1481 0x1>,
<0x100 &gic_its 0x1480 0x1>;
+ msi-map-mask = <0xff00>;
linux,pci-domain = <1>;
num-lanes = <2>;
---
base-commit: d37e1e4c52bc60578969f391fb81f947c3e83118
change-id: 20240216-sm8550-msi-map-fix-ab8fe5e24df1
Best regards,
--
Manivannan Sadhasivam <[email protected]>
On 16/02/2024 18:05, Manivannan Sadhasivam wrote:
> "msi-map-mask" is a required property for all Qcom PCIe controllers as it
> would allow all PCIe devices under a bus to share the same MSI identifier.
>
> Without this property, each device has to use a separate MSI identifier
> which is not possible due to platform limitations.
>
> Currently, this is not an issue since only one device is connected to the
> bus on boards making use of this SoC.
>
> Fixes: a33a532b3b1e ("arm64: dts: qcom: sm8650: Use GIC-ITS for PCIe0 and PCIe1")
> Signed-off-by: Manivannan Sadhasivam <[email protected]>
Acked-by: Krzysztof Kozlowski <[email protected]>
Best regards,
Krzysztof
On 16/02/2024 18:05, Manivannan Sadhasivam wrote:
> "msi-map-mask" is a required property for all Qcom PCIe controllers as it
> would allow all PCIe devices under a bus to share the same MSI identifier.
>
> Without this property, each device has to use a separate MSI identifier
> which is not possible due to platform limitations.
>
> Currently, this is not an issue since only one device is connected to the
> bus on boards making use of this SoC.
>
> Fixes: a33a532b3b1e ("arm64: dts: qcom: sm8650: Use GIC-ITS for PCIe0 and PCIe1")
> Signed-off-by: Manivannan Sadhasivam <[email protected]>
> ---
> arch/arm64/boot/dts/qcom/sm8650.dtsi | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm8650.dtsi b/arch/arm64/boot/dts/qcom/sm8650.dtsi
> index d488b3b3265e..12ba839f215e 100644
> --- a/arch/arm64/boot/dts/qcom/sm8650.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8650.dtsi
> @@ -2277,6 +2277,7 @@ &mc_virt SLAVE_EBI1 QCOM_ICC_TAG_ALWAYS>,
> /* Entries are reversed due to the unusual ITS DeviceID encoding */
> msi-map = <0x0 &gic_its 0x1401 0x1>,
> <0x100 &gic_its 0x1400 0x1>;
> + msi-map-mask = <0xff00>;
>
> linux,pci-domain = <0>;
> num-lanes = <2>;
> @@ -2404,6 +2405,7 @@ &mc_virt SLAVE_EBI1 QCOM_ICC_TAG_ALWAYS>,
> /* Entries are reversed due to the unusual ITS DeviceID encoding */
> msi-map = <0x0 &gic_its 0x1481 0x1>,
> <0x100 &gic_its 0x1480 0x1>;
> + msi-map-mask = <0xff00>;
>
> linux,pci-domain = <1>;
> num-lanes = <2>;
>
> ---
> base-commit: d37e1e4c52bc60578969f391fb81f947c3e83118
> change-id: 20240216-sm8550-msi-map-fix-ab8fe5e24df1
>
> Best regards,
Reviewed-by: Neil Armstrong <[email protected]>
On Fri, Feb 16, 2024 at 10:35:21PM +0530, Manivannan Sadhasivam wrote:
> "msi-map-mask" is a required property for all Qcom PCIe controllers as it
> would allow all PCIe devices under a bus to share the same MSI identifier.
>
> Without this property, each device has to use a separate MSI identifier
> which is not possible due to platform limitations.
>
> Currently, this is not an issue since only one device is connected to the
> bus on boards making use of this SoC.
>
> Fixes: a33a532b3b1e ("arm64: dts: qcom: sm8650: Use GIC-ITS for PCIe0 and PCIe1")
> Signed-off-by: Manivannan Sadhasivam <[email protected]>
Reported-by: Johan Hovold <[email protected]>
Reviewed-by: Johan Hovold <[email protected]>
Johan
On Fri, 16 Feb 2024 22:35:21 +0530, Manivannan Sadhasivam wrote:
> "msi-map-mask" is a required property for all Qcom PCIe controllers as it
> would allow all PCIe devices under a bus to share the same MSI identifier.
>
> Without this property, each device has to use a separate MSI identifier
> which is not possible due to platform limitations.
>
> Currently, this is not an issue since only one device is connected to the
> bus on boards making use of this SoC.
>
> [...]
Applied, thanks!
[1/1] arm64: dts: sm8650: Add msi-map-mask for PCIe nodes
commit: dfc554d5217163f9d1c9d75d1380af1156df6eb3
Best regards,
--
Bjorn Andersson <[email protected]>