Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-06 Thread Julien Grall
Hi Stefano, On 04/02/2021 22:39, Stefano Stabellini wrote: After the discussion with Rob, it is clear that we have to add a check on the node name for "pcie" in dt_bus_pci_match. However, that wouldn't solve the problem reported by Elliot, because in this case the node name is "pci" not "pcie".

Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-04 Thread Stefano Stabellini
On Thu, 4 Feb 2021, Julien Grall wrote: > On 04/02/2021 00:13, Stefano Stabellini wrote: > > On Wed, 3 Feb 2021, Julien Grall wrote: > > > On Wed, 3 Feb 2021 at 22:18, Stefano Stabellini > > > wrote: > > > > > > But aside from PCIe, let's say that we know of a few nodes for which > > > > > > "reg"

Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-04 Thread Julien Grall
On 04/02/2021 00:13, Stefano Stabellini wrote: On Wed, 3 Feb 2021, Julien Grall wrote: On Wed, 3 Feb 2021 at 22:18, Stefano Stabellini wrote: But aside from PCIe, let's say that we know of a few nodes for which "reg" needs a special treatment. I am not sure it makes sense to proceed with pa

Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-03 Thread Stefano Stabellini
On Wed, 3 Feb 2021, Julien Grall wrote: > On Wed, 3 Feb 2021 at 22:18, Stefano Stabellini > wrote: > > > > But aside from PCIe, let's say that we know of a few nodes for which > > > > "reg" needs a special treatment. I am not sure it makes sense to proceed > > > > with parsing those nodes without

Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-03 Thread Julien Grall
On Wed, 3 Feb 2021 at 22:18, Stefano Stabellini wrote: > > > But aside from PCIe, let's say that we know of a few nodes for which > > > "reg" needs a special treatment. I am not sure it makes sense to proceed > > > with parsing those nodes without knowing how to deal with that. > > > > I believe t

Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-03 Thread Stefano Stabellini
On Wed, 3 Feb 2021, Julien Grall wrote: > On 03/02/2021 00:18, Stefano Stabellini wrote: > > On Tue, 2 Feb 2021, Julien Grall wrote: > > > On 02/02/2021 18:12, Julien Grall wrote: > > > > On 02/02/2021 17:47, Elliott Mitchell wrote: > > > > > The handle_device() function has been returning failure

Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-03 Thread Julien Grall
On 03/02/2021 00:18, Stefano Stabellini wrote: On Tue, 2 Feb 2021, Julien Grall wrote: On 02/02/2021 18:12, Julien Grall wrote: On 02/02/2021 17:47, Elliott Mitchell wrote: The handle_device() function has been returning failure upon encountering a device address which was invalid.  A devic

Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-02 Thread Stefano Stabellini
On Tue, 2 Feb 2021, Julien Grall wrote: > On 02/02/2021 18:12, Julien Grall wrote: > > On 02/02/2021 17:47, Elliott Mitchell wrote: > > > The handle_device() function has been returning failure upon > > > encountering a device address which was invalid.  A device tree which > > > had such an entry

Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-02 Thread Julien Grall
On 02/02/2021 18:12, Julien Grall wrote: Hi, On 02/02/2021 17:47, Elliott Mitchell wrote: The handle_device() function has been returning failure upon encountering a device address which was invalid.  A device tree which had such an entry has now been seen in the wild.  As it causes no failu

Re: [PATCH] xen/arm: domain_build: Ignore device nodes with invalid addresses

2021-02-02 Thread Julien Grall
Hi, On 02/02/2021 17:47, Elliott Mitchell wrote: The handle_device() function has been returning failure upon encountering a device address which was invalid. A device tree which had such an entry has now been seen in the wild. As it causes no failures to simply ignore the entries, ignore them