On Tuesday 21 April 2009, John Williams wrote:

> Some (most?) of the Xilinx drivers currently have this construct:

>

> #ifdef CONFIG_OF

>

> // probe using OF

>

> #else

If there are multiple ways of detecting the device, then

the driver should be compilable on any system that allows

either one.

At the very least, it should be restricted to CONFIG_HAS_IOMEM,

which is probably required for any of these, but not provided

on stuff like UML or s390.

Drivers that use of_* functions unconditionally need to

depend on CONFIG_OF.

Also, some of the xilinx drivers apparantly use DCR, which in turn is

only defined when you have CONFIG_PPC_DCR, and these

have so far only been used on powerpc. If other architectures

start using DCR (I hope that never happens), we will need a

global CONFIG_DCR option.

Arnd <><