On Tue, Jul 10, 2018 at 09:33:47PM +0200, Sergio Paracuellos wrote: > +static int mt7621_pcie_parse_dt(struct mt7621_pcie_port *port) > +{ > + struct device *dev = port->dev; > + struct device_node *node = dev->of_node; > + struct resource regs; > + const char *type; > + int err; > + > + type = of_get_property(node, "device_type", NULL); > + if (!type || strcmp(type, "pci")) { Instead of testing for pci, can you test for pcie? I always like to use the == 0 and != 0 with strcmp() because it's easier to think about type == pci or type != pcie. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel