Commit 9e0e4e11623e57c0f6635f5dc6972244ce511f77
1 parent
ac09281a02
Exists in
smarc-l5.0.0_1.0.0-ga
and in
5 other branches
ARM: at91: unused variable in at91_pm_verify_clocks
The code using the variable 'i' in this function is conditional which results in a harmless compiler warning. Using the IS_ENABLED macro instead of #ifdef makes the code look nicer and gets rid of the warning. Without this patch, building at91sam9263_defconfig results in: /home/arnd/linux-arm/arch/arm/mach-at91/pm.c: In function 'at91_pm_verify_clocks': /home/arnd/linux-arm/arch/arm/mach-at91/pm.c:137:6: warning: unused variable 'i' [-Wunused-variable] Signed-off-by: Arnd Bergmann <arnd@arndb.de> Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> Cc: Nicolas Ferre <nicolas.ferre@atmel.com>
Showing 1 changed file with 3 additions and 2 deletions Side-by-side Diff
arch/arm/mach-at91/pm.c
| ... | ... | @@ -153,7 +153,9 @@ |
| 153 | 153 | } |
| 154 | 154 | } |
| 155 | 155 | |
| 156 | -#ifdef CONFIG_AT91_PROGRAMMABLE_CLOCKS | |
| 156 | + if (!IS_ENABLED(CONFIG_AT91_PROGRAMMABLE_CLOCKS)) | |
| 157 | + return 1; | |
| 158 | + | |
| 157 | 159 | /* PCK0..PCK3 must be disabled, or configured to use clk32k */ |
| 158 | 160 | for (i = 0; i < 4; i++) { |
| 159 | 161 | u32 css; |
| ... | ... | @@ -167,7 +169,6 @@ |
| 167 | 169 | return 0; |
| 168 | 170 | } |
| 169 | 171 | } |
| 170 | -#endif | |
| 171 | 172 | |
| 172 | 173 | return 1; |
| 173 | 174 | } |