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 <[email protected]>
Intel Finland Oy

Reply via email to