Skip to content

Commit 5e63dfe

Browse files
krzkthierryreding
authored andcommitted
soc/tegra: pmc: Use str_enable_disable-like helpers
Replace ternary (condition ? "enable" : "disable") syntax with helpers from string_choices.h because: 1. Simple function call with one argument is easier to read. Ternary operator has three arguments and with wrapping might lead to quite long code. 2. Is slightly shorter thus also easier to read. 3. It brings uniformity in the text - same string. 4. Allows deduping by the linker, which results in a smaller binary file. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Link: https://lore.kernel.org/r/20250114203638.1013670-1-krzysztof.kozlowski@linaro.org Signed-off-by: Thierry Reding <treding@nvidia.com>
1 parent 2014c95 commit 5e63dfe

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

drivers/soc/tegra/pmc.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@
4747
#include <linux/seq_file.h>
4848
#include <linux/slab.h>
4949
#include <linux/spinlock.h>
50+
#include <linux/string_choices.h>
5051
#include <linux/syscore_ops.h>
5152

5253
#include <soc/tegra/common.h>
@@ -1181,7 +1182,7 @@ static int powergate_show(struct seq_file *s, void *data)
11811182
continue;
11821183

11831184
seq_printf(s, " %9s %7s\n", pmc->soc->powergates[i],
1184-
status ? "yes" : "no");
1185+
str_yes_no(status));
11851186
}
11861187

11871188
return 0;

0 commit comments

Comments
 (0)