linux-kernel - [PATCH 0/6] ARM: dts: pci-mvebu: Fix assigned-addresses for every PCIe Root Port

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 18 Aug 2022 00:30:47 +0200
From:   Pali Rohár <pali@...nel.org>
To:     Andrew Lunn <andrew@...n.ch>,
        Gregory Clement <gregory.clement@...tlin.com>,
        Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>,
        Rob Herring <robh+dt@...nel.org>
Cc:     linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Marek Behún <kabel@...nel.org>
Subject: [PATCH 0/6] ARM: dts: pci-mvebu: Fix assigned-addresses for every PCIe Root Port

Per IEEE Std 1275-1994 bindings documentation (to which kernel DT
bindings refers), DT property assigned-addresses contains BDF address
of resource. Currently more PCIe Root Port nodes have BDF address in
assigned-addresses which points to different PCIe Root Port nodes. This
obviously does not make sense as the address resource specified in
assigned-addresses of every PCIe Root Port describes address range of
internal registers which are specific for corresponding Marvell PCIe
Root Port. Fix this issue and align all BDF addresses in
assigned-addresses DT property to specify correct BDF address of the
current PCIe Root Port.

Note that current version of pci-mvebu.c controller driver, which
registers Marvell PCIe Root Ports, ignores BDF value in DT property
assigned-addresses. It expects that Root Port's assigned-addresses
contains address range of that root port. That is why driver currently
works without any issue and nobody spotted it. But if driver or
something else would do device tree validation then this issue should be
spotted and throws error. Also device tree files may be used by other
projects where drivers may require correct values.

This patch series aligns BDF address of every Marvell PCIe Root Port in
node name, config space in reg property and mem in assigned-address
property of internal registers resource.

Pali Rohár (6):
  ARM: dts: dove: Fix assigned-addresses for every PCIe Root Port
  ARM: dts: armada-370: Fix assigned-addresses for every PCIe Root Port
  ARM: dts: armada-xp: Fix assigned-addresses for every PCIe Root Port
  ARM: dts: armada-375: Fix assigned-addresses for every PCIe Root Port
  ARM: dts: armada-38x: Fix assigned-addresses for every PCIe Root Port
  ARM: dts: armada-39x: Fix assigned-addresses for every PCIe Root Port

 arch/arm/boot/dts/armada-370.dtsi        |  2 +-
 arch/arm/boot/dts/armada-375.dtsi        |  2 +-
 arch/arm/boot/dts/armada-380.dtsi        |  4 ++--
 arch/arm/boot/dts/armada-385.dtsi        |  6 +++---
 arch/arm/boot/dts/armada-39x.dtsi        |  6 +++---
 arch/arm/boot/dts/armada-xp-mv78230.dtsi |  8 ++++----
 arch/arm/boot/dts/armada-xp-mv78260.dtsi | 16 ++++++++--------
 arch/arm/boot/dts/dove.dtsi              |  2 +-
 8 files changed, 23 insertions(+), 23 deletions(-)

-- 
2.20.1