On Fri, Apr 27, 2012 at 12:55:36PM +0000, Arnd Bergmann wrote: > On Friday 27 April 2012, Mark Brown wrote: > > There is one, that's the problem - the driver is defaulting to the same > > state as the main hw_random option (as is idiomatic for this subsystem). > > This means that if you've enabled any hw_random device this driver will > > be enabled by default. I would argue that this default directive only > > makes sense if there's an explicit dependency limiting the platforms it > > can build on. If it can build on any platform then omitting the default > > and just defaulting to N as is normal for drivers would be better. > How about making it default to (HW_RANDOM && ARCH_AT91) then? Yes, something like that would do the job too.