Skip to contentSkip to conversation

Fix Kconfig coding style from spaces to tab

PATCH Applied Anand Moon 2 participants 3 messages Aug 1, 2024 – Aug 2, 2024 Library code

lore
Share & tools
lore
author
#1 lore raw

PATCH 2/2 Fix Makefile coding style from spaces to tab

In reply to Anand Moon

Adjust indentation from spaces to tab (+optional two spaces) as in coding style with command like:

	$ sed -e 's/^        /\t/' -i */Makefile
---
 dts/Makefile      | 30 +++++++++++++++---------------
 firmware/Makefile |  4 ++--
 images/Makefile   |  2 +-
 lib/Makefile      |  2 +-
 4 files changed, 19 insertions(+), 19 deletions(-)
dts/Makefile +15 -15
index fb51acef7c..8695a87333 100644
@@ -5,18 +5,18 @@ CPP ?= cpp
 # Disable noisy checks by default
 ifeq ($(findstring 1,$(DTC_VERBOSE)),)
 DTC_FLAGS += -Wno-unit_address_vs_reg \
-        -Wno-unit_address_format \
-        -Wno-avoid_unnecessary_addr_size \
-        -Wno-alias_paths \
-        -Wno-graph_child_address \
-        -Wno-simple_bus_reg \
-        -Wno-unique_unit_address \
-        -Wno-pci_device_reg
+	-Wno-unit_address_format \
+	-Wno-avoid_unnecessary_addr_size \
+	-Wno-alias_paths \
+	-Wno-graph_child_address \
+	-Wno-simple_bus_reg \
+	-Wno-unique_unit_address \
+	-Wno-pci_device_reg
 endif
 
 ifneq ($(findstring 2,$(DTC_VERBOSE)),)
 DTC_FLAGS += -Wnode_name_chars_strict \
-        -Wproperty_name_chars_strict
+	-Wproperty_name_chars_strict
 endif
 
 MAKEFLAGS += -rR --no-print-directory
@@ -130,15 +130,15 @@ quiet_cmd_clean    = CLEAN   $(obj)
 dtc-tmp = $(subst $(comma),_,$(dot-target).dts.tmp)
 
 dtc_cpp_flags  = -Wp,-MD,$(depfile).pre.tmp -nostdinc		\
-                 -Iinclude -I$(src) -Isrc -Itestcase-data	\
-                 -undef -D__DTS__
+	         -Iinclude -I$(src) -Isrc -Itestcase-data	\
+	         -undef -D__DTS__
 
 quiet_cmd_dtc = DTC     $@
 cmd_dtc = $(CPP) $(dtc_cpp_flags) -x assembler-with-cpp -o $(dtc-tmp) $< ; \
-        $(DTC) -O dtb -o $@ -b 0 \
-                -i $(src) $(DTC_FLAGS) \
-                -d $(depfile).dtc.tmp $(dtc-tmp) ; \
-        cat $(depfile).pre.tmp $(depfile).dtc.tmp > $(depfile)
+	$(DTC) -O dtb -o $@ -b 0 \
+	        -i $(src) $(DTC_FLAGS) \
+	        -d $(depfile).dtc.tmp $(dtc-tmp) ; \
+	cat $(depfile).pre.tmp $(depfile).dtc.tmp > $(depfile)
 
 $(obj)/%.dtb: $(src)/%.dts FORCE
 	$(call if_changed_dep,dtc)
@@ -148,7 +148,7 @@ all_arch: $(ARCH_DTB)
 	@:
 
 RCS_FIND_IGNORE := \( -name SCCS -o -name BitKeeper -o -name .svn -o -name CVS \
-                   -o -name .pc -o -name .hg -o -name .git \) -prune -o
+	           -o -name .pc -o -name .hg -o -name .git \) -prune -o
 
 PHONY += clean_arch
 clean_arch: __clean-files = $(ARCH_DTB)
firmware/Makefile +2 -2
index d5e4cee859..7265c55c42 100644
@@ -125,9 +125,9 @@ $(patsubst %.gen.o,$(obj)/%.gen.o, $(obj-pbl-y)): $(obj)/%.gen.o: $(fwdep-requir
 pbl-y += $(pbl-fwext-y)
 
 targets := $(patsubst $(obj)/%,%, \
-                                $(shell find $(obj) -name \*.gen.S 2>/dev/null))
+	                        $(shell find $(obj) -name \*.gen.S 2>/dev/null))
 targets += $(patsubst $(obj)/%,%, \
-                                $(shell find $(obj) -name \*.extgen.S 2>/dev/null))
+	                        $(shell find $(obj) -name \*.extgen.S 2>/dev/null))
 
 # just to build a built-in.o. Otherwise compilation fails when no
 # firmware is built.
images/Makefile +1 -1
index c0105609ee..9007911278 100644
@@ -236,5 +236,5 @@ clean-files := *.pbl *.pblb *.elf *.map start_*.imximg *.img barebox.z start_*.k
 	start_*.kwbuartimg *.socfpgaimg *.mlo *.t20img *.t20img.cfg *.t30img \
 	*.t30img.cfg *.t124img *.t124img.cfg *.mlospi *.mlo *.mxsbs *.mxssd *.rkimg \
 	start_*.simximg start_*.usimximg start_*.pimximg start_*.psimximg *.zynqimg \
-        *.image *.swapped *.missing-firmware
+	*.image *.swapped *.missing-firmware
 clean-files += pbl.lds
lib/Makefile +1 -1
index 0dde52527f..b3d024035c 100644
@@ -101,7 +101,7 @@ CFLAGS_ubsan.o := -fno-stack-protector
 libfdt_files = fdt.o fdt_ro.o fdt_wip.o fdt_rw.o fdt_sw.o fdt_strerror.o \
 	                      fdt_empty_tree.o
 $(foreach file, $(libfdt_files), \
-        $(eval CFLAGS_$(file) = -I $(srctree)/scripts/dtc/libfdt) \
+	$(eval CFLAGS_$(file) = -I $(srctree)/scripts/dtc/libfdt) \
 	$(eval CFLAGS_$(file:%.o=%.pbl.o) = -I $(srctree)/scripts/dtc/libfdt))
 
 obj-pbl-$(CONFIG_LIBFDT) += $(libfdt_files)
Signature · 1 line
2.44.0

Anand Moon

author
#0 lore raw

PATCH 1/2 Fix Kconfig coding style from spaces to tab

Adjust indentation from spaces to tab (+optional two spaces) as in coding style with command like:

	$ sed -e 's/^        /\t/' -i */Kconfig
---
 arch/Kconfig     |  2 +-
 commands/Kconfig | 10 +++++-----
 lib/Kconfig      | 16 ++++++++--------
 3 files changed, 14 insertions(+), 14 deletions(-)
arch/Kconfig +1 -1
index 2212b89288..ad47dbf78d 100644
@@ -17,4 +17,4 @@ config ARCH_HAS_CTRLC
 # by default.
 #
 config ARCH_DMA_DEFAULT_COHERENT
-        bool
+	bool
commands/Kconfig +5 -5
index 5b512f1bba..b72344f021 100644
@@ -97,10 +97,10 @@ config CMD_DEVLOOKUP
 
 	  devlookup [-v VAR] /dev/DEVICE [parameter]
 
-          Detects the device behind a device file and outputs it,
-          unless a second argument is given. In that case the device
-          parameter with that name is looked up. Specifying -v VARIABLE
-          will write output to VARIABLE instead of printing it.
+	  Detects the device behind a device file and outputs it,
+	  unless a second argument is given. In that case the device
+	  parameter with that name is looked up. Specifying -v VARIABLE
+	  will write output to VARIABLE instead of printing it.
 
 config CMD_DEVUNBIND
 	tristate
@@ -2331,7 +2331,7 @@ config CMD_OF_PROPERTY
 
 	  Valid formats for values:
 	  <0x00112233 4 05> - an array of cells. cells not beginning with a digit are
-                    interpreted as node paths and converted to phandles
+	            interpreted as node paths and converted to phandles
 	  [00 11 22 .. nn]  - byte stream
 	  If the value does not start with '<' or '[' it is interpreted as string
 
lib/Kconfig +8 -8
index df9ba6ccc9..44133a54f3 100644
@@ -45,22 +45,22 @@ config XZ_DECOMPRESS
 	select XZ_DEC_ARM64
 
 config XZ_DEC_X86
-        bool
+	bool
 
 config XZ_DEC_POWERPC
-        bool
+	bool
 
 config XZ_DEC_IA64
-        bool
+	bool
 
 config XZ_DEC_ARM
-        bool
+	bool
 
 config XZ_DEC_ARMTHUMB
-        bool
+	bool
 
 config XZ_DEC_SPARC
-        bool
+	bool
 
 config XZ_DEC_ARM64
 	bool
@@ -152,7 +152,7 @@ config CRC_CCITT
 	  functions require M here.
 
 config CRC8
-        bool "CRC8 function"
+	bool "CRC8 function"
 	help
 	  This option provides CRC8 function. Drivers may select this
 	  when they need to do cyclic redundancy check according CRC8
@@ -204,7 +204,7 @@ config GENERIC_LIB_MULDI3
 	bool
 
 config NLS
-        bool "Native language support"
+	bool "Native language support"
 
 config BLOBGEN
 	select BASE64
Signature · 1 line
2.44.0
Applied
#2 lore raw
Quoted 5 lines · Anand Moon

Adjust indentation from spaces to tab (+optional two spaces) as in coding style with command like:

	$ sed -e 's/^        /\t/' -i */Kconfig

Applied, thanks!

[1/2] Fix Kconfig coding style from spaces to tab
https://git.pengutronix.de/cgit/barebox/commit/?id=c1d7a1cca0e5 (link may not be stable)
[2/2] Fix Makefile coding style from spaces to tab
https://git.pengutronix.de/cgit/barebox/commit/?id=e58395c76b2e (link may not be stable)

Best regards,

Signature · 1 line
Sascha Hauer <[email protected]>