Skip to content

Commit 0286cda

Browse files
committed
Devicetree: formating Macro expressions
Impliment linter to treat ANYTHING and ANYTHING(...) as "something already in parentheses after expansion" and warn patterns such as prop = <(MACRO(x))>; -> prop = <MACRO(x)> Signed-off-by: Kyle Micallef Bonnici <[email protected]>
1 parent d86b293 commit 0286cda

File tree

42 files changed

+58
-58
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

42 files changed

+58
-58
lines changed

boards/adi/sdp_k1/adi_sdp_k1.dts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444

4545
otghs_ulpi_phy: otghs_ulpis_phy {
4646
compatible = "usb-ulpi-phy";
47-
reset-gpios = <&gpiod 7 (GPIO_ACTIVE_LOW)>;
47+
reset-gpios = <&gpiod 7 GPIO_ACTIVE_LOW>;
4848
#phy-cells = <0>;
4949
};
5050

boards/ambiq/apollo4p_blue_kxr_evb/apollo4p_blue_kxr_evb.dts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,8 +177,8 @@
177177
};
178178

179179
zephyr_udc0: &usb {
180-
vddusb33-gpios = <&gpio0_31 13 (GPIO_PULL_UP)>;
181-
vddusb0p9-gpios = <&gpio0_31 15 (GPIO_PULL_UP)>;
180+
vddusb33-gpios = <&gpio0_31 13 GPIO_PULL_UP>;
181+
vddusb0p9-gpios = <&gpio0_31 15 GPIO_PULL_UP>;
182182
status = "okay";
183183
};
184184

boards/ambiq/apollo4p_evb/apollo4p_evb.dts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -186,8 +186,8 @@
186186
};
187187

188188
zephyr_udc0: &usb {
189-
vddusb33-gpios = <&gpio96_127 7 (GPIO_PULL_UP)>;
190-
vddusb0p9-gpios = <&gpio96_127 5 (GPIO_PULL_UP)>;
189+
vddusb33-gpios = <&gpio96_127 7 GPIO_PULL_UP>;
190+
vddusb0p9-gpios = <&gpio96_127 5 GPIO_PULL_UP>;
191191
status = "okay";
192192
};
193193

boards/ambiq/apollo510_evb/apollo510_evb.dts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
compatible = "zephyr,memory-region", "mmio-sram";
4040
reg = <0x20280000 0x100000>;
4141
zephyr,memory-region = "SRAM_NO_CACHE";
42-
zephyr,memory-attr = <(DT_MEM_ARM(ATTR_MPU_RAM_NOCACHE))>;
42+
zephyr,memory-attr = <DT_MEM_ARM(ATTR_MPU_RAM_NOCACHE)>;
4343
};
4444

4545
leds {
@@ -220,8 +220,8 @@
220220
};
221221

222222
zephyr_udc0: &usb {
223-
vddusb33-gpios = <&gpio64_95 27 (GPIO_PULL_UP)>;
224-
vddusb0p9-gpios = <&gpio64_95 26 (GPIO_PULL_UP)>;
223+
vddusb33-gpios = <&gpio64_95 27 GPIO_PULL_UP>;
224+
vddusb0p9-gpios = <&gpio64_95 26 GPIO_PULL_UP>;
225225
status = "okay";
226226
};
227227

boards/arduino/giga_r1/arduino_giga_r1_stm32h747xx_m7.dts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
device_type = "memory";
3131
reg = <0xc0000000 DT_SIZE_M(8)>;
3232
zephyr,memory-region = "SDRAM1";
33-
zephyr,memory-attr = <(DT_MEM_ARM(ATTR_MPU_RAM))>;
33+
zephyr,memory-attr = <DT_MEM_ARM(ATTR_MPU_RAM)>;
3434
};
3535

3636
aliases {

boards/arduino/portenta_h7/arduino_portenta_h7_stm32h747xx_m7.dts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
device_type = "memory";
4444
reg = <0xc0000000 DT_SIZE_M(8)>;
4545
zephyr,memory-region = "SDRAM1";
46-
zephyr,memory-attr = <(DT_MEM_ARM(ATTR_MPU_RAM))>;
46+
zephyr,memory-attr = <DT_MEM_ARM(ATTR_MPU_RAM)>;
4747
};
4848
};
4949

boards/arm/fvp_baser_aemv8r/fvp_baser_aemv8r_fvp_aemv8r_aarch64.dts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@
4242
compatible = "zephyr,memory-region", "mmio-dram";
4343
reg = <0x9a000000 0x66000000>;
4444
zephyr,memory-region = "DEVICE_REGION";
45-
zephyr,memory-attr = <(DT_MEM_ARM(ATTR_MPU_IO))>;
45+
zephyr,memory-attr = <DT_MEM_ARM(ATTR_MPU_IO)>;
4646
};
4747
};
4848
};

boards/arm/mps3/mps3_common.dtsi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@
8383
/* 0 - CONFIG_CORTEX_M_NULL_POINTER_EXCEPTION_PAGE_SIZE> */
8484
reg = <0x0 0x400>;
8585
zephyr,memory-region = "NULL_PTR_DETECT";
86-
zephyr,memory-attr = <(DT_MEM_ARM(ATTR_MPU_FLASH))>;
86+
zephyr,memory-attr = <DT_MEM_ARM(ATTR_MPU_FLASH)>;
8787
};
8888
/* DDR4 - 2G, alternates non-secure/secure every 256M */
8989
ddr4: memory@60000000 {

boards/arm/mps4/mps4_common.dtsi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@
9595
/* 0 - CONFIG_CORTEX_M_NULL_POINTER_EXCEPTION_PAGE_SIZE> */
9696
reg = <0x0 0x400>;
9797
zephyr,memory-region = "NULL_PTR_DETECT";
98-
zephyr,memory-attr = <(DT_MEM_ARM(ATTR_MPU_FLASH))>;
98+
zephyr,memory-attr = <DT_MEM_ARM(ATTR_MPU_FLASH)>;
9999
};
100100

101101
/* DDR4 - 2G, alternates non-secure/secure every 256M */

boards/fanke/fk743m5_xih6/fk743m5_xih6.dts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,14 +37,14 @@
3737
reg = <0x90000000 DT_SIZE_M(256)>;
3838
zephyr,memory-region = "EXTMEM";
3939
/* The ATTR_MPU_EXTMEM attribut causing a MPU FAULT */
40-
zephyr,memory-attr = <(DT_MEM_ARM(ATTR_MPU_IO))>;
40+
zephyr,memory-attr = <DT_MEM_ARM(ATTR_MPU_IO)>;
4141
};
4242

4343
sdram1: sdram@c0000000 {
4444
compatible = "zephyr,memory-region";
4545
reg = <0xc0000000 DT_SIZE_M(32)>;
4646
zephyr,memory-region = "SDRAM1";
47-
zephyr,memory-attr = <(DT_MEM_ARM(ATTR_MPU_RAM))>;
47+
zephyr,memory-attr = <DT_MEM_ARM(ATTR_MPU_RAM)>;
4848
};
4949
};
5050

0 commit comments

Comments
 (0)