Hello, On Sun, 27 Mar 2016 16:03:48 +0200, Andrew Lunn wrote: > > +static irqreturn_t armada8k_pcie_irq_handler(int irq, void *arg) > > +{ > > + struct pcie_port *pp = arg; > > + struct armada8k_pcie *pcie = to_armada8k_pcie(pp); > > + void __iomem *base = pcie->base; > > + u32 val; > > + > > + val = readl(base + PCIE_GLOBAL_INT_CAUSE1_REG); > > + writel(val, base + PCIE_GLOBAL_INT_CAUSE1_REG); > > + > > + return IRQ_HANDLED; > > Maybe a comment as to why you are just throwing them away. I'll have a look into this. > > +static int armada8k_pcie_probe(struct platform_device *pdev) > > +{ > > + struct armada8k_pcie *pcie; > > + struct pcie_port *pp; > > + struct device *dev = &pdev->dev; > > + struct resource *base; > > + int ret; > > + > > + pcie = devm_kzalloc(dev, sizeof(*pcie), GFP_KERNEL); > > + if (!pcie) > > + return -ENOMEM; > > + > > + pcie->main_clk = devm_clk_get(dev, "main"); > > + if (!IS_ERR(pcie->main_clk)) > > + clk_prepare_enable(pcie->main_clk); > > + > > + pcie->lane_clk = devm_clk_get(dev, "port"); > > + if (!IS_ERR(pcie->lane_clk)) > > + clk_prepare_enable(pcie->lane_clk); > > Any need to handle -EPRODE_DEFERED here? Is this needed? The clocks are registered in of_clk_init(), i.e at time_init() time. This is way before the device drivers get probed, no? Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html