linux-kernel - Re: [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-prev] [thread-next>] [day] [month] [year] [list]

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

ping?

On Sunday 09 October 2022 13:15:29 Pali Rohár wrote:
> Gregory, ping?
> 
> On Thursday 18 August 2022 00:30:47 Pali Rohár wrote:
> > 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
> >