Skip to contentSkip to conversation

Drivers: unisys: visorhba - style fix

PATCH Superseded Derek Robson 3 participants 3 messages Jun 9, 2017 – Jun 10, 2017 Staging

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Drivers: unisys: visorhba - style fix

Fixed style of permissions to octal.
Found using checkpatch

---
 drivers/staging/unisys/visorhba/visorhba_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/unisys/visorhba/visorhba_main.c +1 -1
index 2fd31c9762c6..a6e7a6bbc428 100644
@@ -1090,7 +1090,7 @@ static int visorhba_probe(struct visor_device *dev)
 		goto err_scsi_remove_host;
 	}
 	devdata->debugfs_info =
-		debugfs_create_file("info", S_IRUSR | S_IRGRP,
+		debugfs_create_file("info", 0440,
 				    devdata->debugfs_dir, devdata,
 				    &info_debugfs_fops);
 	if (!devdata->debugfs_info) {
Signature · 1 line
2.13.0
Acked-by on v1 1/1
#1 lore raw
Quoted 12 lines

-----Original Message-----
From: Derek Robson [mailto:[email protected]]
Sent: Friday, June 9, 2017 1:16 AM
To: Kershner, David A <[email protected]>; [email protected]; Binder, David Anthony <[email protected]>; Sell, Timothy C <[email protected]>; Wadgaonkar, Sameer Laxmikant <[email protected]>; Thompson, Bryan E. <[email protected]>; [email protected]
Cc: *S-Par-Maintainer <[email protected]>; [email protected]; [email protected]
Subject: [PATCH] Drivers: unisys: visorhba - style fix

Quoted 4 lines

Fixed style of permissions to octal.
Found using checkpatch

Quoted 20 lines
---
 drivers/staging/unisys/visorhba/visorhba_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/unisys/visorhba/visorhba_main.c +1 -2
index 2fd31c9762c6..a6e7a6bbc428 100644
@@ -1090,7 +1090,7 @@ static int visorhba_probe(struct visor_device *dev)
 		goto err_scsi_remove_host;
 	}
 	devdata->debugfs_info =
-		debugfs_create_file("info", S_IRUSR | S_IRGRP,
+		debugfs_create_file("info", 0440,
 				    devdata->debugfs_dir, devdata,
 				    &info_debugfs_fops);
 	if (!devdata->debugfs_info) {
--

2.13.0

#2 lore raw
In reply to Derek Robson

Drivers is never a valid subsystem. Do a git log --oneline <file> to look at what other people are doing. The style varies by subsystem so you should always run that command.

regards,
dan carpenter