[RFC PATCH] arm: msm: moved Kconfig machine entry into mach-msm
Russell King - ARM Linux
linux at arm.linux.org.uk
Fri Oct 22 11:25:33 EDT 2010
On Fri, Oct 22, 2010 at 08:20:00AM -0700, Daniel Walker wrote:
> On Fri, 2010-10-22 at 08:49 +0100, Russell King - ARM Linux wrote:
> > On Thu, Oct 21, 2010 at 11:37:35AM -0700, Daniel Walker wrote:
> > > Has anyone suggested this? It seems like it might reduce conflicts
> > > in the Kconfig file.
> >
> > How? The choice normally has one entry for each mach- directory.
> > You're still going to have 'source' for each mach- directory, so
> > the chances of conflicts are no different.
>
> The source lines would be mostly static..
In practice, the entries here are mostly static already - the only
time that something gets added there is when a new class of machines
gets added.
> We wouldn't have to modify
> Kconfig to add new "select" statements to are entries. People end up
> having to send you patches which do something then modify the Kconfig
> (or in some cases they don't send them to you which is worse) in order
> to reduce conflicts. With this change they wouldn't need to do that.
As I say, the only time that these entries get touched is when a new
class of machines gets added - and given that people can't read (proven
by the number of times people get their entries out of order), they
should come through my tree.
> > Plus it increases the file count, makes things more indirect, and
> > therefore harder to follow.
>
> True. It's a trade off , but we already have this for the other menu
> options further down the Kconfig.
Actually, it'll increase the chances of a conflict when a change is made
to it - because more machine classes will be visible to any particular
patch. If we have two similarly named machine classes added at the same
time.
I think you're trying to make a mountain out of a mole hill. This
hasn't been a problem to date.
More information about the linux-arm-kernel
mailing list