Thanks,
Vaibhav Hiremath

> -----Original Message-----
> From: David Brownell [mailto:davi...@pacbell.net]
> Sent: Monday, January 12, 2009 5:24 AM
> To: Hiremath, Vaibhav
> Cc: linux-o...@vger.kernel.org; linux-media@vger.kernel.org;
> video4linux-l...@redhat.com; Jadav, Brijesh R; Shah, Hardik; Hadli,
> Manjunath; R, Sivaraj
> Subject: Re: [REVIEW PATCH 2/2] Added OMAP3EVM Multi-Media Daughter
> Card Support
> 
> On Tuesday 06 January 2009, hvaib...@ti.com wrote:
> > +config MACH_OMAP3EVM_DC
> > +       bool "OMAP 3530 EVM daughter card board"
> > +       depends on ARCH_OMAP3 && ARCH_OMAP34XX && MACH_OMAP3EVM
> 
> There can be other daughtercards, so the Kconfig text should
> say which specific card is being configured.  And it should
> probably use the "zero or one of these choices" syntax, so
> it's easier to include other options..
> 
[Hiremath, Vaibhav] I do agree with this point, but as of now I believe there 
is only one Daughter card which is available with us. And I would prefer to 
have menu option for the future when any new daughter card available and gets 
added to tree.

Any other opinions would be helpful?

> - Dave
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to