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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 11 Sep 2014 15:09:10 -0400 From: Murali Karicheri <m-karicheri2@...com> To: Murali Karicheri <m-karicheri2@...com> CC: <robh+dt@...nel.org>, <pawel.moll@....com>, <mark.rutland@....com>, <ijc+devicetree@...lion.org.uk>, <galak@...eaurora.org>, <bhelgaas@...gle.com>, <arnd@...db.de>, <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>, <linux-pci@...r.kernel.org>, <linux-arm-kernel@...ts.infradead.org> Subject: Re: [PATCH v3 1/2] PCI: keystone: remove handle of PCI mode configuration On 09/10/2014 01:12 PM, Murali Karicheri wrote: > Keystone PCI hardware supports both RC and EP modes and devcfg > register has bits to boot strap the device to either of these modes. > It seems proper to add this functionality to the boot loader rather > than in the driver as device will be operating in either mode, not > both any time. Currently the driver supports only RC mode and hence > register configuration in the driver is not needed and driver can > assume this is a RC hardware. > > Also update the DT documentation accordingly. > > Signed-off-by: Murali Karicheri<m-karicheri2@...com> > Acked-by: Arnd Bergmann<arnd@...db.de> > --- > v3 - No change > v2 - Added this separate patch to remove pci mode handling > .../devicetree/bindings/pci/pci-keystone.txt | 4 +--- > drivers/pci/host/pci-keystone.c | 21 ++------------------ > 2 files changed, 3 insertions(+), 22 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/pci-keystone.txt b/Documentation/devicetree/bindings/pci/pci-keystone.txt > index ceb3e24..bedacf0 100644 > --- a/Documentation/devicetree/bindings/pci/pci-keystone.txt > +++ b/Documentation/devicetree/bindings/pci/pci-keystone.txt > @@ -13,9 +13,7 @@ Required Properties:- > > compatibility: "ti,keystone-pcie" > reg: index 1 is the base address and length of DW application registers. > - index 2 is the base address and length of PCI mode configuration > - register. > - index 3 is the base address and length of PCI device ID register. > + index 2 is the base address and length of PCI device ID register. > > pcie_msi_intc : Interrupt controller device node for MSI IRQ chip > interrupt-cells: should be set to 1 > diff --git a/drivers/pci/host/pci-keystone.c b/drivers/pci/host/pci-keystone.c > index ff8ed25..f1119eb 100644 > --- a/drivers/pci/host/pci-keystone.c > +++ b/drivers/pci/host/pci-keystone.c > @@ -35,10 +35,6 @@ > #define MAX_MSI_HOST_IRQS 8 > #define MAX_LEGACY_HOST_IRQS 4 > > -/* RC mode settings masks */ > -#define PCIE_RC_MODE BIT(2) > -#define PCIE_MODE_MASK (BIT(1) | BIT(2)) > - > /* DEV_STAT_CTRL */ > #define PCIE_CAP_BASE 0x70 > > @@ -355,7 +351,6 @@ static int __init ks_pcie_probe(struct platform_device *pdev) > void __iomem *reg_p; > struct phy *phy; > int ret = 0; > - u32 val; > > ks_pcie = devm_kzalloc(&pdev->dev, sizeof(*ks_pcie), > GFP_KERNEL); > @@ -365,18 +360,6 @@ static int __init ks_pcie_probe(struct platform_device *pdev) > } > pp =&ks_pcie->pp; > > - /* index 2 is the devcfg register for RC mode settings */ > - res = platform_get_resource(pdev, IORESOURCE_MEM, 2); > - reg_p = devm_ioremap_resource(dev, res); > - if (IS_ERR(reg_p)) > - return PTR_ERR(reg_p); > - > - /* enable RC mode in devcfg */ > - val = readl(reg_p); > - val&= ~PCIE_MODE_MASK; > - val |= PCIE_RC_MODE; > - writel(val, reg_p); > - > /* initialize SerDes Phy if present */ > phy = devm_phy_get(dev, "pcie-phy"); > if (!IS_ERR_OR_NULL(phy)) { > @@ -385,8 +368,8 @@ static int __init ks_pcie_probe(struct platform_device *pdev) > return ret; > } > > - /* index 3 is to read PCI DEVICE_ID */ > - res = platform_get_resource(pdev, IORESOURCE_MEM, 3); > + /* index 2 is to read PCI DEVICE_ID */ > + res = platform_get_resource(pdev, IORESOURCE_MEM, 2); > reg_p = devm_ioremap_resource(dev, res); > if (IS_ERR(reg_p)) > return PTR_ERR(reg_p); Bjorn, These patches need to go onto host-keystone as well for v3.18. If you you have any comments, please let me know so that I can address them. I have incorporated comments from Arnd already. Thanks. Murali -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists