linux-kernel - [PATCH v2] Made the error messages a tad easier to read

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <SJ2P223MB1026A88F6556E5CCEAE52065F7A82@SJ2P223MB1026.NAMP223.PROD.OUTLOOK.COM>
Date: Mon, 22 Jul 2024 12:23:37 -0400
From: Steven Davis <goldside000@...look.com>
To: gregkh@...uxfoundation.org,
	christian.gromm@...rochip.com,
	parthiban.veerasooran@...rochip.com
Cc: linux-staging@...ts.linux.dev,
	linux-kernel@...r.kernel.org,
	Steven Davis <goldside000@...look.com>
Subject: [PATCH v2] Made the error messages a tad easier to read

I realized I forgot to add a description in the previous version. Whoopsie!
Anyway, this patch makes three error messages in the driver easier to read by capitalizing the first letters properly.
For example, "channel already linked" became "Channel already linked".
In addition, "expect" becomes "expected", as you would typically find in an error message.
My hopes are that this patch makes the errors slightly clearer.

Signed-off-by: Steven Davis <goldside000@...look.com>
---
 drivers/staging/most/video/video.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/most/video/video.c b/drivers/staging/most/video/video.c
index 6254a5df2502..2b3cdb1ce140 100644
--- a/drivers/staging/most/video/video.c
+++ b/drivers/staging/most/video/video.c
@@ -454,18 +454,18 @@ static int comp_probe_channel(struct most_interface *iface, int channel_idx,
 	struct most_video_dev *mdev = get_comp_dev(iface, channel_idx);
 
 	if (mdev) {
-		pr_err("channel already linked\n");
+		pr_err("Channel already linked\n");
 		return -EEXIST;
 	}
 
 	if (ccfg->direction != MOST_CH_RX) {
-		pr_err("wrong direction, expect rx\n");
+		pr_err("Wrong direction, expected rx\n");
 		return -EINVAL;
 	}
 
 	if (ccfg->data_type != MOST_CH_SYNC &&
 	    ccfg->data_type != MOST_CH_ISOC) {
-		pr_err("wrong channel type, expect sync or isoc\n");
+		pr_err("Wrong channel type, expected sync or isoc\n");
 		return -EINVAL;
 	}
 
-- 
2.45.2