Re: [PATCH v2] docs: kernel-parameters.txt: Fix whitespace

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/16/18 14:49, Thymo van Beers wrote:
> Some lines used spaces instead of tabs at line start.
> This can cause mangled lines in editors due to inconsistency.
> 
> Replace spaces for tabs where appropriate.
> 
> Signed-off-by: Thymo van Beers <thymovanbeers@xxxxxxxxx>
> ---
> Changes in v2:
>   - Rebase against docs-next
>   - Fix indentation modifications
> 
>  Documentation/admin-guide/kernel-parameters.txt | 136 ++++++++++++------------
>  1 file changed, 68 insertions(+), 68 deletions(-)
> 
> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> index 3487be79847c..f625f65c286f 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt

Most of the patch is OK IMO, but not the intel_pstate part:
The 2-space extra indents work fine here, while the extra tab makes a lot of the
lines go beyond the 80-column mark.

> @@ -1650,39 +1650,39 @@
>  			0	disables intel_idle and fall back on acpi_idle.
>  			1 to 9	specify maximum depth of C-state.
>  
> -	intel_pstate=  [X86]
> -		       disable
> -		         Do not enable intel_pstate as the default
> -		         scaling driver for the supported processors
> -		       passive
> -			 Use intel_pstate as a scaling driver, but configure it
> -			 to work with generic cpufreq governors (instead of
> -			 enabling its internal governor).  This mode cannot be
> -			 used along with the hardware-managed P-states (HWP)
> -			 feature.
> -		       force
> -			 Enable intel_pstate on systems that prohibit it by default
> -			 in favor of acpi-cpufreq. Forcing the intel_pstate driver
> -			 instead of acpi-cpufreq may disable platform features, such
> -			 as thermal controls and power capping, that rely on ACPI
> -			 P-States information being indicated to OSPM and therefore
> -			 should be used with caution. This option does not work with
> -			 processors that aren't supported by the intel_pstate driver
> -			 or on platforms that use pcc-cpufreq instead of acpi-cpufreq.
> -		       no_hwp
> -		         Do not enable hardware P state control (HWP)
> -			 if available.
> -		hwp_only
> -			Only load intel_pstate on systems which support
> -			hardware P state control (HWP) if available.
> -		support_acpi_ppc
> -			Enforce ACPI _PPC performance limits. If the Fixed ACPI
> -			Description Table, specifies preferred power management
> -			profile as "Enterprise Server" or "Performance Server",
> -			then this feature is turned on by default.
> -		per_cpu_perf_limits
> -			Allow per-logical-CPU P-State performance control limits using
> -			cpufreq sysfs interface
> +	intel_pstate=	[X86]
> +			disable
> +				Do not enable intel_pstate as the default
> +				scaling driver for the supported processors
> +			passive
> +				Use intel_pstate as a scaling driver, but configure it
> +				to work with generic cpufreq governors (instead of
> +				enabling its internal governor).  This mode cannot be
> +				used along with the hardware-managed P-states (HWP)
> +				feature.
> +			force
> +				Enable intel_pstate on systems that prohibit it by default
> +				in favor of acpi-cpufreq. Forcing the intel_pstate driver
> +				instead of acpi-cpufreq may disable platform features, such
> +				as thermal controls and power capping, that rely on ACPI
> +				P-States information being indicated to OSPM and therefore
> +				should be used with caution. This option does not work with
> +				processors that aren't supported by the intel_pstate driver
> +				or on platforms that use pcc-cpufreq instead of acpi-cpufreq.
> +			no_hwp
> +				Do not enable hardware P state control (HWP)
> +				if available.
> +			hwp_only
> +				Only load intel_pstate on systems which support
> +				hardware P state control (HWP) if available.
> +			support_acpi_ppc
> +				Enforce ACPI _PPC performance limits. If the Fixed ACPI
> +				Description Table, specifies preferred power management
> +				profile as "Enterprise Server" or "Performance Server",
> +				then this feature is turned on by default.
> +			per_cpu_perf_limits
> +				Allow per-logical-CPU P-State performance control limits using
> +				cpufreq sysfs interface
>  
>  	intremap=	[X86-64, Intel-IOMMU]
>  			on	enable Interrupt Remapping (default)
> @@ -2027,7 +2027,7 @@
>  			* [no]ncqtrim: Turn off queued DSM TRIM.
>  
>  			* nohrst, nosrst, norst: suppress hard, soft
> -                          and both resets.
> +			and both resets.

I would leave that line above indented like the one after "rstonce" below.

>  
>  			* rstonce: only attempt one reset during
>  			  hot-unplug link recovery


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



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux