On 03/31/2012 01:48 PM, Mark Brown wrote: > On Fri, Mar 30, 2012 at 05:07:19PM -0600, Stephen Warren wrote: >> From: Stephen Warren <swarren@xxxxxxxxxx> >> >> Even though these options are typically only selected by other options, >> it's still useful to give them a brief description. > > Why do you find these short descriptions useful? They'll cause Kconfig > to prompt for them and... Honestly, the downstream patches for Tegra30 support that I was cherry-picking had descriptions for the Tegra30 options, so I needed to resolve the discrepancy When doing that, I figured why hide those options, even if they're typically auto-selected? Perhaps some out-of-tree ASoC machine driver needs those modules built. Still, I can go either way; do you want me to drop this, and remove the equivalent descriptions from the Tegra30 patches? >> - tristate >> + tristate "Tegra I2S driver" >> depends on SND_SOC_TEGRA >> help >> Say Y or M if you want to add support for codecs attached to the > > ...there's already a description in the long text anyway? -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
