Messages in this thread | | | From | Arnd Bergmann <> | Subject | Re: [PATCH] AT91: add AT91SAM9X5 dummy configuration variable | Date | Tue, 5 Jul 2011 13:25:47 +0200 |
| |
On Tuesday 05 July 2011, Jean-Christophe PLAGNIOL-VILLARD wrote: > > You have convinced me. > > But I will have to remove the other dependencies that I mentioned before > > in the thread. > > > > We can drop this patch. > I prefer to hide the platform specific driver other wise we will have a huge > menu entry in Kconfig with unrelated drivers that can not be used at all on > the selected mach
I fear it's too late for that. You already have to decide on many thousands of configuration options, most of which are totally irrelevant for a particular system.
IMHO, the two much better ways to deal with SOC-specific device drivers are defconfig files and/or specific defaults, like
config ABX500_CORE bool "ST-Ericsson ABX500 Mixed Signal Circuit register functions" default ARCH_U300 || ARCH_U8500
This lets everybody choose whether they would like to build the driver or not, which is very useful for allyesconfig/allnoconfig/randconfig builds. At the same time, the default gets it right in almost all other cases.
Arnd
| |