On Tuesday 09 September 2014 13:52:54 Bartlomiej Zolnierkiewicz wrote: > I believe that this should use > > select GENERIC_PHY > > instead (just like all other PHY drivers except PHY_MIPHY365X which > also should be fixed to use select). Right. It would also be good to change 'depends on ARCH_STI' into 'depends on ARCH_STI || COMPILE_TEST', but only if it actually compiles on x86. Note that currently both providers and users of the PHY API use 'select GENERIC_PHY', which probably not what we want: only providers (like this one) should 'select' it, while drivers using this interface could either 'depends on GENERIC_PHY' or leave out the Kconfig dependency entirely. Adding 'depends on' for drivers using the interface might be cleaner, but can result in dependency loops. 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