About Kconfig indentation fixes

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

 



Hello Masahiro,

Please see: -> https://paste.centos.org/view/06ed8bf0

===
diff --git a/drivers/firmware/Kconfig b/drivers/firmware/Kconfig
index b59e3041fd62..82ae68b13381 100644
--- a/drivers/firmware/Kconfig
+++ b/drivers/firmware/Kconfig
@@ -68,14 +68,14 @@ config EDD_OFF
          using the kernel parameter 'edd={on|skipmbr|off}'.
 
 config FIRMWARE_MEMMAP
-    bool "Add firmware-provided memory map to sysfs" if EXPERT
-    default X86
-    help
-      Add the firmware-provided (unmodified) memory map to /sys/firmware/memmap.
-      That memory map is used for example by kexec to set up parameter area
-      for the next kernel, but can also be used for debugging purposes.
+       bool "Add firmware-provided memory map to sysfs" if EXPERT
+       default X86
+       help
+         Add the firmware-provided (unmodified) memory map to /sys/firmware/memmap.
+         That memory map is used for example by kexec to set up parameter area
+         for the next kernel, but can also be used for debugging purposes.
 
-      See also Documentation/ABI/testing/sysfs-firmware-memmap.
+         See also Documentation/ABI/testing/sysfs-firmware-memmap.
  
 config DMIID
-    bool "Export DMI identification via sysfs to userspace"
-    depends on DMI
-    default y
+       bool "Export DMI identification via sysfs to userspace"
+       depends on DMI
+       default y
        help
          Say Y here if you want to query SMBIOS/DMI system identification
          information from userspace through /sys/class/dmi/id/ or if you want 
===

* IIUC majority of Kconfig entries use leading tab(\t) character(s) for indentation. But there are multiple Kconfig entries like above wherein indentation is without the leading tab(\t) character. Ex. above entries use 4 spaces ("   ") for indentation.

* I wanted to check
  - is it okay to send fix patches for such inconsistencies? They may lead to errors.
  - if Kconfig file has no assigned maintainer(s), should the patch go to -kbuild@ list?

* The Kconfig language text below does not say anything about indentation or usage of leading tab(\t) characters. Is that intentional?

  https://www.kernel.org/doc/html/latest/kbuild/kconfig-language.html#menu-dependencies

* Is there some method/rule to the indentation in Kconfig files/entries?


Thank you.
---
  -Prasad




[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux