> -----Original Message----- > From: Arnd Bergmann [mailto:a...@arndb.de] > Sent: Tuesday, October 13, 2015 12:12 PM > To: Bjorn Helgaas > Cc: Wangzhou (B); Bjorn Helgaas; jingooh...@gmail.com; > pratyush.an...@gmail.com; li...@arm.linux.org.uk; > thomas.petazz...@free-electrons.com; Gabriele Paoloni; > lorenzo.pieral...@arm.com; james.mo...@arm.com; liviu.du...@arm.com; > ja...@lakedaemon.net; r...@kernel.org; gabriel.fernan...@linaro.org; > minghuan.l...@freescale.com; linux-...@vger.kernel.org; linux-arm- > ker...@lists.infradead.org; devicet...@vger.kernel.org; linux- > ker...@vger.kernel.org; zhangjukuo; qiuzhenfa; liudongdong (C); > qiujiang; xuwei (O); Liguozhu (Kenneth); Wangkefeng (Kevin); Rob > Herring > Subject: Re: [PATCH v10 4/6] PCI: hisi: Add PCIe host support for > HiSilicon SoC Hip05 > > On Monday 12 October 2015 16:35:45 Bjorn Helgaas wrote: > > > > > +{ > > > + u64 addr; > > > + struct device_node *msi_node; > > > + struct resource res; > > > + struct device_node *np = pp->dev->of_node; > > > + struct hisi_pcie *pcie = to_hisi_pcie(pp); > > > + > > > + msi_node = of_parse_phandle(np, "msi-parent", 0); > > > + if (!msi_node) { > > > + dev_err(pp->dev, "failed to find msi-parent\n"); > > > + return -EINVAL; > > > + } > > > + of_address_to_resource(msi_node, 0, &res); > > > > Does this use the "msi-parent" node in the same way as other drivers > > do? I'm sure there must be other places where we extract struct > > resource information from an "msi-parent" node, but I don't see them. > > > > I'm trying to verify that this isn't some kind of incompatible > > extension of the "msi-parent" property. I cc'd Arnd and Rob (DT > > experts). > > This is not ok, what this does is that it relies on a particular > implementation of the MSI controller and directly accesses its > registers.
Hi Arnd, thanks for reviewing. What we do is to retrieve the msi-parent physical address and we store it in our internal PCIe register locations... So we do not operate directly on the msi controller registers... So I wonder if the current implementation is Ok to retrieve the msi-parent address.... Thanks Gab > > Instead, it should reference only the msi irq domain and let the > driver for the MSI controller access the registers. Otherwise this > code has to be rewritten once the same PCI host code appears in > a machine that has a real GICv2m or GICv3. > > Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
