Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1421687 > unrolled thread

[PATCH] alarmtimer: fixed comments describing structure fields

Started byPratyush Patel <pratyushpatel.1995@gmail.com>
First post2016-06-14 11:10 +0200
Last post2016-06-17 23:00 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] alarmtimer: fixed comments describing structure fields Pratyush Patel <pratyushpatel.1995@gmail.com> - 2016-06-14 11:10 +0200
    Re: [PATCH] alarmtimer: fixed comments describing structure fields John Stultz <john.stultz@linaro.org> - 2016-06-17 23:00 +0200

#1421687 — [PATCH] alarmtimer: fixed comments describing structure fields

FromPratyush Patel <pratyushpatel.1995@gmail.com>
Date2016-06-14 11:10 +0200
Subject[PATCH] alarmtimer: fixed comments describing structure fields
Message-ID<rJSwa-2Nc-3@gated-at.bofh.it>
Updated struct alarm and struct alarm_timer descriptions.

Signed-off-by: Pratyush Patel <pratyushpatel.1995@gmail.com>
---
 include/linux/alarmtimer.h | 6 +++---
 kernel/time/alarmtimer.c   | 1 -
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/include/linux/alarmtimer.h b/include/linux/alarmtimer.h
index 52f3b7d..9d80312 100644
--- a/include/linux/alarmtimer.h
+++ b/include/linux/alarmtimer.h
@@ -26,10 +26,10 @@ enum alarmtimer_restart {
  * struct alarm - Alarm timer structure
  * @node:	timerqueue node for adding to the event list this value
  *		also includes the expiration time.
- * @period:	Period for recuring alarms
+ * @timer:	hrtimer used to schedule events while running
  * @function:	Function pointer to be executed when the timer fires.
- * @type:	Alarm type (BOOTTIME/REALTIME)
- * @enabled:	Flag that represents if the alarm is set to fire or not
+ * @type:	Alarm type (BOOTTIME/REALTIME).
+ * @state:	Flag that represents if the alarm is set to fire or not.
  * @data:	Internal data value.
  */
 struct alarm {
diff --git a/kernel/time/alarmtimer.c b/kernel/time/alarmtimer.c
index e840ed8..c3aad68 100644
--- a/kernel/time/alarmtimer.c
+++ b/kernel/time/alarmtimer.c
@@ -30,7 +30,6 @@
  * struct alarm_base - Alarm timer bases
  * @lock:		Lock for syncrhonized access to the base
  * @timerqueue:		Timerqueue head managing the list of events
- * @timer: 		hrtimer used to schedule events while running
  * @gettime:		Function to read the time correlating to the base
  * @base_clockid:	clockid for the base
  */
-- 
2.7.4

[toc] | [next] | [standalone]


#1425468

FromJohn Stultz <john.stultz@linaro.org>
Date2016-06-17 23:00 +0200
Message-ID<rL91U-31m-17@gated-at.bofh.it>
In reply to#1421687
On Tue, Jun 14, 2016 at 2:00 AM, Pratyush Patel
<pratyushpatel.1995@gmail.com> wrote:
> Updated struct alarm and struct alarm_timer descriptions.
>
> Signed-off-by: Pratyush Patel <pratyushpatel.1995@gmail.com>


Queued. Thanks for submitting this!
-john

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web