* [PATCH] Staging: Android: Fixed coding issue return parenthesis in alarm_dev.c
@ 2013-12-28 16:03 Arthur Schwalbenberg
2013-12-28 17:48 ` Greg KH
0 siblings, 1 reply; 2+ messages in thread
From: Arthur Schwalbenberg @ 2013-12-28 16:03 UTC (permalink / raw)
To: gregkh
Cc: john.stultz, dan.carpenter, arthurs578, cruzjbishop, devel,
linux-kernel
This is a patch to the alarm_dev.c file that fixes up a return is not
a function warning found by the checkpatch.pl tool
Signed-off-by: Arthur Schwalbenberg <arthurs578@gmail.com>
---
drivers/staging/android/alarm-dev.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/android/alarm-dev.c b/drivers/staging/android/alarm-dev.c
index 647694f..96b2f53 100644
--- a/drivers/staging/android/alarm-dev.c
+++ b/drivers/staging/android/alarm-dev.c
@@ -68,8 +68,8 @@ static struct devalarm alarms[ANDROID_ALARM_TYPE_COUNT];
*/
static int is_wakeup(enum android_alarm_type type)
{
- return (type == ANDROID_ALARM_RTC_WAKEUP ||
- type == ANDROID_ALARM_ELAPSED_REALTIME_WAKEUP);
+ return type == ANDROID_ALARM_RTC_WAKEUP ||
+ type == ANDROID_ALARM_ELAPSED_REALTIME_WAKEUP;
}
--
1.8.3.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] Staging: Android: Fixed coding issue return parenthesis in alarm_dev.c
2013-12-28 16:03 [PATCH] Staging: Android: Fixed coding issue return parenthesis in alarm_dev.c Arthur Schwalbenberg
@ 2013-12-28 17:48 ` Greg KH
0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2013-12-28 17:48 UTC (permalink / raw)
To: Arthur Schwalbenberg
Cc: john.stultz, dan.carpenter, cruzjbishop, devel, linux-kernel
On Sat, Dec 28, 2013 at 11:03:28AM -0500, Arthur Schwalbenberg wrote:
> This is a patch to the alarm_dev.c file that fixes up a return is not
> a function warning found by the checkpatch.pl tool
>
> Signed-off-by: Arthur Schwalbenberg <arthurs578@gmail.com>
Why the odd indentation?
> ---
> drivers/staging/android/alarm-dev.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/android/alarm-dev.c b/drivers/staging/android/alarm-dev.c
> index 647694f..96b2f53 100644
> --- a/drivers/staging/android/alarm-dev.c
> +++ b/drivers/staging/android/alarm-dev.c
> @@ -68,8 +68,8 @@ static struct devalarm alarms[ANDROID_ALARM_TYPE_COUNT];
> */
> static int is_wakeup(enum android_alarm_type type)
> {
> - return (type == ANDROID_ALARM_RTC_WAKEUP ||
> - type == ANDROID_ALARM_ELAPSED_REALTIME_WAKEUP);
> + return type == ANDROID_ALARM_RTC_WAKEUP ||
> + type == ANDROID_ALARM_ELAPSED_REALTIME_WAKEUP;
> }
This change is already in linux-next, right? I can't apply the same
change more than once to the tree...
greg k-h
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-12-28 17:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-28 16:03 [PATCH] Staging: Android: Fixed coding issue return parenthesis in alarm_dev.c Arthur Schwalbenberg
2013-12-28 17:48 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox