* [PATCH] Drivers: staging: most: sound: Fixed styling issue.
@ 2021-02-09 12:50 ` Mukul Mehar
0 siblings, 0 replies; 8+ messages in thread
From: Mukul Mehar @ 2021-02-09 12:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Mukul Mehar
This patch fixes a warning, of the line ending with a '(',
generated by checkpatch.pl.
Signed-off-by: Mukul Mehar <mukulmehar02@gmail.com>
---
drivers/staging/most/sound/sound.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/most/sound/sound.c b/drivers/staging/most/sound/sound.c
index 3a1a59058042..4dd1bf95d1ce 100644
--- a/drivers/staging/most/sound/sound.c
+++ b/drivers/staging/most/sound/sound.c
@@ -228,12 +228,12 @@ static int playback_thread(void *data)
struct mbo *mbo = NULL;
bool period_elapsed = false;
- wait_event_interruptible(
- channel->playback_waitq,
- kthread_should_stop() ||
- (channel->is_stream_running &&
- (mbo = most_get_mbo(channel->iface, channel->id,
- &comp))));
+ wait_event_interruptible(channel->playback_waitq,
+ kthread_should_stop() ||
+ (channel->is_stream_running &&
+ (mbo = most_get_mbo(channel->iface,
+ channel->id,
+ &comp))));
if (!mbo)
continue;
--
2.25.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] Drivers: staging: most: sound: Fixed styling issue.
@ 2021-02-09 12:50 ` Mukul Mehar
0 siblings, 0 replies; 8+ messages in thread
From: Mukul Mehar @ 2021-02-09 12:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Mukul Mehar
This patch fixes a warning, of the line ending with a '(',
generated by checkpatch.pl.
Signed-off-by: Mukul Mehar <mukulmehar02@gmail.com>
---
drivers/staging/most/sound/sound.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/most/sound/sound.c b/drivers/staging/most/sound/sound.c
index 3a1a59058042..4dd1bf95d1ce 100644
--- a/drivers/staging/most/sound/sound.c
+++ b/drivers/staging/most/sound/sound.c
@@ -228,12 +228,12 @@ static int playback_thread(void *data)
struct mbo *mbo = NULL;
bool period_elapsed = false;
- wait_event_interruptible(
- channel->playback_waitq,
- kthread_should_stop() ||
- (channel->is_stream_running &&
- (mbo = most_get_mbo(channel->iface, channel->id,
- &comp))));
+ wait_event_interruptible(channel->playback_waitq,
+ kthread_should_stop() ||
+ (channel->is_stream_running &&
+ (mbo = most_get_mbo(channel->iface,
+ channel->id,
+ &comp))));
if (!mbo)
continue;
--
2.25.1
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] Drivers: staging: most: sound: Fixed styling issue.
2021-02-09 12:50 ` Mukul Mehar
@ 2021-02-09 13:00 ` Greg KH
-1 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2021-02-09 13:00 UTC (permalink / raw)
To: Mukul Mehar; +Cc: devel, linux-kernel
On Tue, Feb 09, 2021 at 06:20:37PM +0530, Mukul Mehar wrote:
> This patch fixes a warning, of the line ending with a '(',
> generated by checkpatch.pl.
>
> Signed-off-by: Mukul Mehar <mukulmehar02@gmail.com>
> ---
> drivers/staging/most/sound/sound.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
Any reason you didn't also cc: the other developer that
get_maintainer.pl tells you to for this file?
Please fix that up and resend, especially as that developer also sent a
different patch for this driver right before you, so someone's patch is
going to get confused...
thanks,
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Drivers: staging: most: sound: Fixed styling issue.
@ 2021-02-09 13:00 ` Greg KH
0 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2021-02-09 13:00 UTC (permalink / raw)
To: Mukul Mehar; +Cc: devel, linux-kernel
On Tue, Feb 09, 2021 at 06:20:37PM +0530, Mukul Mehar wrote:
> This patch fixes a warning, of the line ending with a '(',
> generated by checkpatch.pl.
>
> Signed-off-by: Mukul Mehar <mukulmehar02@gmail.com>
> ---
> drivers/staging/most/sound/sound.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
Any reason you didn't also cc: the other developer that
get_maintainer.pl tells you to for this file?
Please fix that up and resend, especially as that developer also sent a
different patch for this driver right before you, so someone's patch is
going to get confused...
thanks,
greg k-h
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] Drivers: staging: most: sound: Fixed styling issue.
@ 2021-02-09 13:06 ` Mukul Mehar
0 siblings, 0 replies; 8+ messages in thread
From: Mukul Mehar @ 2021-02-09 13:06 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, christian.gromm, Mukul Mehar
This patch fixes a warning, of the line ending with a '(',
generated by checkpatch.pl.
Signed-off-by: Mukul Mehar <mukulmehar02@gmail.com>
---
drivers/staging/most/sound/sound.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/most/sound/sound.c b/drivers/staging/most/sound/sound.c
index 3a1a59058042..4dd1bf95d1ce 100644
--- a/drivers/staging/most/sound/sound.c
+++ b/drivers/staging/most/sound/sound.c
@@ -228,12 +228,12 @@ static int playback_thread(void *data)
struct mbo *mbo = NULL;
bool period_elapsed = false;
- wait_event_interruptible(
- channel->playback_waitq,
- kthread_should_stop() ||
- (channel->is_stream_running &&
- (mbo = most_get_mbo(channel->iface, channel->id,
- &comp))));
+ wait_event_interruptible(channel->playback_waitq,
+ kthread_should_stop() ||
+ (channel->is_stream_running &&
+ (mbo = most_get_mbo(channel->iface,
+ channel->id,
+ &comp))));
if (!mbo)
continue;
--
2.25.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] Drivers: staging: most: sound: Fixed styling issue.
@ 2021-02-09 13:06 ` Mukul Mehar
0 siblings, 0 replies; 8+ messages in thread
From: Mukul Mehar @ 2021-02-09 13:06 UTC (permalink / raw)
To: gregkh; +Cc: devel, christian.gromm, linux-kernel, Mukul Mehar
This patch fixes a warning, of the line ending with a '(',
generated by checkpatch.pl.
Signed-off-by: Mukul Mehar <mukulmehar02@gmail.com>
---
drivers/staging/most/sound/sound.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/most/sound/sound.c b/drivers/staging/most/sound/sound.c
index 3a1a59058042..4dd1bf95d1ce 100644
--- a/drivers/staging/most/sound/sound.c
+++ b/drivers/staging/most/sound/sound.c
@@ -228,12 +228,12 @@ static int playback_thread(void *data)
struct mbo *mbo = NULL;
bool period_elapsed = false;
- wait_event_interruptible(
- channel->playback_waitq,
- kthread_should_stop() ||
- (channel->is_stream_running &&
- (mbo = most_get_mbo(channel->iface, channel->id,
- &comp))));
+ wait_event_interruptible(channel->playback_waitq,
+ kthread_should_stop() ||
+ (channel->is_stream_running &&
+ (mbo = most_get_mbo(channel->iface,
+ channel->id,
+ &comp))));
if (!mbo)
continue;
--
2.25.1
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] Drivers: staging: most: sound: Fixed styling issue.
2021-02-09 13:06 ` Mukul Mehar
@ 2021-02-09 17:27 ` Greg KH
-1 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2021-02-09 17:27 UTC (permalink / raw)
To: Mukul Mehar; +Cc: devel, christian.gromm, linux-kernel
On Tue, Feb 09, 2021 at 06:36:19PM +0530, Mukul Mehar wrote:
> This patch fixes a warning, of the line ending with a '(',
> generated by checkpatch.pl.
>
> Signed-off-by: Mukul Mehar <mukulmehar02@gmail.com>
> ---
> drivers/staging/most/sound/sound.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/most/sound/sound.c b/drivers/staging/most/sound/sound.c
> index 3a1a59058042..4dd1bf95d1ce 100644
> --- a/drivers/staging/most/sound/sound.c
> +++ b/drivers/staging/most/sound/sound.c
> @@ -228,12 +228,12 @@ static int playback_thread(void *data)
> struct mbo *mbo = NULL;
> bool period_elapsed = false;
>
> - wait_event_interruptible(
> - channel->playback_waitq,
> - kthread_should_stop() ||
> - (channel->is_stream_running &&
> - (mbo = most_get_mbo(channel->iface, channel->id,
> - &comp))));
> + wait_event_interruptible(channel->playback_waitq,
> + kthread_should_stop() ||
> + (channel->is_stream_running &&
> + (mbo = most_get_mbo(channel->iface,
> + channel->id,
> + &comp))));
This line is horrid, you can indent it further (line up the
most_get_mbo() parameters at the least.
Also you lost the extra indentation of the "(mbo " portion, can you make
sure that is there too?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Drivers: staging: most: sound: Fixed styling issue.
@ 2021-02-09 17:27 ` Greg KH
0 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2021-02-09 17:27 UTC (permalink / raw)
To: Mukul Mehar; +Cc: devel, christian.gromm, linux-kernel
On Tue, Feb 09, 2021 at 06:36:19PM +0530, Mukul Mehar wrote:
> This patch fixes a warning, of the line ending with a '(',
> generated by checkpatch.pl.
>
> Signed-off-by: Mukul Mehar <mukulmehar02@gmail.com>
> ---
> drivers/staging/most/sound/sound.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/most/sound/sound.c b/drivers/staging/most/sound/sound.c
> index 3a1a59058042..4dd1bf95d1ce 100644
> --- a/drivers/staging/most/sound/sound.c
> +++ b/drivers/staging/most/sound/sound.c
> @@ -228,12 +228,12 @@ static int playback_thread(void *data)
> struct mbo *mbo = NULL;
> bool period_elapsed = false;
>
> - wait_event_interruptible(
> - channel->playback_waitq,
> - kthread_should_stop() ||
> - (channel->is_stream_running &&
> - (mbo = most_get_mbo(channel->iface, channel->id,
> - &comp))));
> + wait_event_interruptible(channel->playback_waitq,
> + kthread_should_stop() ||
> + (channel->is_stream_running &&
> + (mbo = most_get_mbo(channel->iface,
> + channel->id,
> + &comp))));
This line is horrid, you can indent it further (line up the
most_get_mbo() parameters at the least.
Also you lost the extra indentation of the "(mbo " portion, can you make
sure that is there too?
thanks,
greg k-h
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-02-09 17:29 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-09 12:50 [PATCH] Drivers: staging: most: sound: Fixed styling issue Mukul Mehar
2021-02-09 12:50 ` Mukul Mehar
2021-02-09 13:00 ` Greg KH
2021-02-09 13:00 ` Greg KH
2021-02-09 13:06 Mukul Mehar
2021-02-09 13:06 ` Mukul Mehar
2021-02-09 17:27 ` Greg KH
2021-02-09 17:27 ` 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.