All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] drivers: misc: kgdbts.c: Fix declaration warning.
@ 2021-03-13 21:58 Shubhankar Kuranagatti
  2021-03-23 12:54 ` Greg KH
  0 siblings, 1 reply; 2+ messages in thread
From: Shubhankar Kuranagatti @ 2021-03-13 21:58 UTC (permalink / raw)
  To: jason.wessel
  Cc: dianders, arnd, gregkh, kgdb-bugreport, linux-kernel, bkkarthik

A new line has been added after declaration. This is done to
maintain code uniformity.

Signed-off-by: Shubhankar Kuranagatti <shubhankarvk@gmail.com>
---
 drivers/misc/kgdbts.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/misc/kgdbts.c b/drivers/misc/kgdbts.c
index 945701bce553..3ee5acc1c391 100644
--- a/drivers/misc/kgdbts.c
+++ b/drivers/misc/kgdbts.c
@@ -474,6 +474,7 @@ static int emul_sstep_put(char *put_str, char *arg)
 {
 	if (!arch_needs_sstep_emulation) {
 		char *ptr = &put_str[11];
+
 		if (put_str[1] != 'T' || put_str[2] != '0')
 			return 1;
 		kgdb_hex2long(&ptr, &sstep_thread_id);
@@ -502,6 +503,7 @@ static int emul_sstep_put(char *put_str, char *arg)
 			return 1;
 		} else {
 			char *ptr = &put_str[11];
+
 			kgdb_hex2long(&ptr, &sstep_thread_id);
 		}
 		break;
@@ -740,6 +742,7 @@ static int validate_simple_test(char *put_str)
 static int run_simple_test(int is_get_char, int chr)
 {
 	int ret = 0;
+
 	if (is_get_char) {
 		/* Send an ACK on the get if a prior put completed and set the			* send ack variable
--
2.17.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 1/2] drivers: misc: kgdbts.c: Fix declaration warning.
  2021-03-13 21:58 [PATCH 1/2] drivers: misc: kgdbts.c: Fix declaration warning Shubhankar Kuranagatti
@ 2021-03-23 12:54 ` Greg KH
  0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2021-03-23 12:54 UTC (permalink / raw)
  To: Shubhankar Kuranagatti
  Cc: jason.wessel, dianders, arnd, kgdb-bugreport, linux-kernel, bkkarthik

On Sun, Mar 14, 2021 at 03:28:08AM +0530, Shubhankar Kuranagatti wrote:
> A new line has been added after declaration. This is done to
> maintain code uniformity.
> 
> Signed-off-by: Shubhankar Kuranagatti <shubhankarvk@gmail.com>
> ---
>  drivers/misc/kgdbts.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/misc/kgdbts.c b/drivers/misc/kgdbts.c
> index 945701bce553..3ee5acc1c391 100644
> --- a/drivers/misc/kgdbts.c
> +++ b/drivers/misc/kgdbts.c
> @@ -474,6 +474,7 @@ static int emul_sstep_put(char *put_str, char *arg)
>  {
>  	if (!arch_needs_sstep_emulation) {
>  		char *ptr = &put_str[11];
> +
>  		if (put_str[1] != 'T' || put_str[2] != '0')
>  			return 1;
>  		kgdb_hex2long(&ptr, &sstep_thread_id);
> @@ -502,6 +503,7 @@ static int emul_sstep_put(char *put_str, char *arg)
>  			return 1;
>  		} else {
>  			char *ptr = &put_str[11];
> +
>  			kgdb_hex2long(&ptr, &sstep_thread_id);
>  		}
>  		break;
> @@ -740,6 +742,7 @@ static int validate_simple_test(char *put_str)
>  static int run_simple_test(int is_get_char, int chr)
>  {
>  	int ret = 0;
> +
>  	if (is_get_char) {
>  		/* Send an ACK on the get if a prior put completed and set the			* send ack variable
> --
> 2.17.1

This patch is corrupted :(

Please fix up and resend the whole series properly threaded so our tools
can pick them both up at the same time.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-03-23 12:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-13 21:58 [PATCH 1/2] drivers: misc: kgdbts.c: Fix declaration warning Shubhankar Kuranagatti
2021-03-23 12:54 ` Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.