On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote: > /* ALS and Proximity sensor */ > isl29028@44 { > - compatible = "isil,isl29028"; > + compatible = "isl,isl29028"; > reg = <0x44>; > interrupt-parent = <&gpio>; > interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>; > Does this intentionally list only the new string? If we want compatibility between new dts files and old kernels, this would need to be compatible = "isl,isl29028", compatible = "isil,isl29028"; I don't know if we want to go that far though, given that this is only a staging driver. Arnd -- 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