Re: [PATCH 3/9] ASoC: Intel: document what Kconfig options do

[Andy Shevchenko]

On Thu, 2017-12-14 at 18:44 -0600, Pierre-Louis Bossart wrote:
> Document in comments what the options are supposed to mean, before
> clean-up in next patch.

> +config SND_SOC_INTEL_SST_ACPI
> +	tristate
> +	# This option controls ACPI-based probing on
> Haswell/Broadwell/
> +	#  Baytrail legacy and will be set when these platforms are
> enabled

I would rather start a new line (mind indentation as well) with list of
platforms:

# This option controls ACPI-based probing on
# Haswell/Broadwell/Baytrail legacy and will be set
# when these platforms are enabled

> +	# This option controls firmware download on
> Haswell/Broadwell/
> +	# Baytrail legacy and will be set when these platforms are
> enabled

Ditto,

-- 
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy