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
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to