diff mbox series

memory: tegra: Prefer octal over symbolic permissions

Message ID 20230714150116.2823766-1-thierry.reding@gmail.com
State Accepted
Headers show
Series memory: tegra: Prefer octal over symbolic permissions | expand

Commit Message

Thierry Reding July 14, 2023, 3:01 p.m. UTC 
From: Thierry Reding <treding@nvidia.com>

checkpatch recommends using octal permissions instead of symbolic
permissions. Switch the debugfs files to use the former to silence
these warnings.

Signed-off-by: Thierry Reding <treding@nvidia.com>
---
 drivers/memory/tegra/tegra186-emc.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Krzysztof Kozlowski July 18, 2023, 6:53 p.m. UTC | #1 
On Fri, 14 Jul 2023 17:01:16 +0200, Thierry Reding wrote:
> checkpatch recommends using octal permissions instead of symbolic
> permissions. Switch the debugfs files to use the former to silence
> these warnings.
> 
> 

Applied, thanks!

[1/1] memory: tegra: Prefer octal over symbolic permissions
      https://git.kernel.org/krzk/linux-mem-ctrl/c/f54abe0132e405cd482b8cad355cd680e3aec810

Best regards,
diff mbox series

Patch

diff --git a/drivers/memory/tegra/tegra186-emc.c b/drivers/memory/tegra/tegra186-emc.c
index 83981ae3ea86..4007f4e16d74 100644
--- a/drivers/memory/tegra/tegra186-emc.c
+++ b/drivers/memory/tegra/tegra186-emc.c
@@ -212,12 +212,12 @@  static int tegra186_emc_get_emc_dvfs_latency(struct tegra186_emc *emc)
 	}
 
 	emc->debugfs.root = debugfs_create_dir("emc", NULL);
-	debugfs_create_file("available_rates", S_IRUGO, emc->debugfs.root,
-			    emc, &tegra186_emc_debug_available_rates_fops);
-	debugfs_create_file("min_rate", S_IRUGO | S_IWUSR, emc->debugfs.root,
-			    emc, &tegra186_emc_debug_min_rate_fops);
-	debugfs_create_file("max_rate", S_IRUGO | S_IWUSR, emc->debugfs.root,
-			    emc, &tegra186_emc_debug_max_rate_fops);
+	debugfs_create_file("available_rates", 0444, emc->debugfs.root, emc,
+			    &tegra186_emc_debug_available_rates_fops);
+	debugfs_create_file("min_rate", 0644, emc->debugfs.root, emc,
+			    &tegra186_emc_debug_min_rate_fops);
+	debugfs_create_file("max_rate", 0644, emc->debugfs.root, emc,
+			    &tegra186_emc_debug_max_rate_fops);
 
 	return 0;
 }