lkml.org 
[lkml]   [2017]   [Jan]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 01/37] PCI: dwc: dra7xx: Group all host related setup in add_pcie_port
    Date
    commit 150645b94348 ("PCI: dra7xx: Move struct pcie_port
    setup to probe function") moved host related setup to the probe
    function. However instead of cluttering the probe function with
    host related setup, group all host related setup in add_pcie_port
    function. This way when endpoint support is added, all the
    endpoint related setup can be added in a separate function.

    Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
    ---
    drivers/pci/dwc/pci-dra7xx.c | 13 ++++++-------
    1 file changed, 6 insertions(+), 7 deletions(-)

    diff --git a/drivers/pci/dwc/pci-dra7xx.c b/drivers/pci/dwc/pci-dra7xx.c
    index ec5617a..fb37e09 100644
    --- a/drivers/pci/dwc/pci-dra7xx.c
    +++ b/drivers/pci/dwc/pci-dra7xx.c
    @@ -288,9 +288,13 @@ static int __init dra7xx_add_pcie_port(struct dra7xx_pcie *dra7xx,
    struct platform_device *pdev)
    {
    int ret;
    - struct pcie_port *pp = &dra7xx->pp;
    - struct device *dev = pp->dev;
    + struct pcie_port *pp;
    struct resource *res;
    + struct device *dev = &pdev->dev;
    +
    + pp = &dra7xx->pp;
    + pp->dev = dev;
    + pp->ops = &dra7xx_pcie_host_ops;

    pp->irq = platform_get_irq(pdev, 1);
    if (pp->irq < 0) {
    @@ -374,7 +378,6 @@ static int __init dra7xx_pcie_probe(struct platform_device *pdev)
    void __iomem *base;
    struct resource *res;
    struct dra7xx_pcie *dra7xx;
    - struct pcie_port *pp;
    struct device *dev = &pdev->dev;
    struct device_node *np = dev->of_node;
    char name[10];
    @@ -384,10 +387,6 @@ static int __init dra7xx_pcie_probe(struct platform_device *pdev)
    if (!dra7xx)
    return -ENOMEM;

    - pp = &dra7xx->pp;
    - pp->dev = dev;
    - pp->ops = &dra7xx_pcie_host_ops;
    -
    irq = platform_get_irq(pdev, 0);
    if (irq < 0) {
    dev_err(dev, "missing IRQ resource\n");
    --
    1.7.9.5
    \
     
     \ /
      Last update: 2017-01-12 11:28    [W:4.111 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site