"Nori, Sekhar" <[email protected]> writes:

> On Tue, Dec 08, 2009 at 05:09:37, Kevin Hilman wrote:
>> Sekhar Nori <[email protected]> writes:
>>
>
> [...]
>
>> >
>> > diff --git a/arch/arm/mach-davinci/Kconfig b/arch/arm/mach-davinci/Kconfig
>> > index 033bfed..0ebe185 100644
>> > --- a/arch/arm/mach-davinci/Kconfig
>> > +++ b/arch/arm/mach-davinci/Kconfig
>> > @@ -91,10 +91,14 @@ config MACH_DAVINCI_DM6467_EVM
>> >     bool "TI DM6467 EVM"
>> >     default ARCH_DAVINCI_DM646x
>> >     depends on ARCH_DAVINCI_DM646x
>> > +   select MACH_DAVINCI_DM6467TEVM
>> >     help
>> >       Configure this option to specify the whether the board used
>> >       for development is a DM6467 EVM
>> >
>> > +config MACH_DAVINCI_DM6467TEVM
>> > +   bool
>> > +
>>
>> I think you should add a comment here about the fact that this is
>> always auto-selected when DM6467 EVM is selected and is required for
>> the machine_is_* stuff to work.
>>
>
> Kevin,
>
> It seems this is already committed :)

doh.

> I could send a separate patch adding the comment if required.
>

Yes, that would be fine, then I'll fold them together before pushing
upstream.

Kevin
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to