Hi Simon, On Fri, Dec 9, 2016 at 11:50 AM, Simon Horman <horms+renesas@xxxxxxxxxxxx> wrote: > Improve readability by listing the rmobile fallback compatibility string s/rmobile/generic/ > after the more-specific compatibility strings they provide a fallback for. > > This does not effect run-time behaviour as it is the order in the DTB that > determines which compatibility string is used. > > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > --- > drivers/irqchip/irq-renesas-intc-irqpin.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/irqchip/irq-renesas-intc-irqpin.c b/drivers/irqchip/irq-renesas-intc-irqpin.c > index 713177d97c7a..5fe1207a079c 100644 > --- a/drivers/irqchip/irq-renesas-intc-irqpin.c > +++ b/drivers/irqchip/irq-renesas-intc-irqpin.c > @@ -374,7 +374,6 @@ static const struct intc_irqpin_config intc_irqpin_rmobile = { > }; > > static const struct of_device_id intc_irqpin_dt_ids[] = { > - { .compatible = "renesas,intc-irqpin", }, > { .compatible = "renesas,intc-irqpin-r8a7778", > .data = &intc_irqpin_irlm_r8a777x }, > { .compatible = "renesas,intc-irqpin-r8a7779", > @@ -383,6 +382,7 @@ static const struct of_device_id intc_irqpin_dt_ids[] = { > .data = &intc_irqpin_rmobile }, > { .compatible = "renesas,intc-irqpin-sh73a0", > .data = &intc_irqpin_rmobile }, > + { .compatible = "renesas,intc-irqpin", }, BTW, I'm wondering what's the added value of keeping the generic "renesas,intc-irqpin" fallback. There is no user of the pure generic version only, and all SoC-specific versions are treated slightly different. > {}, > }; > MODULE_DEVICE_TABLE(of, intc_irqpin_dt_ids); Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- 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