lkml.org 
[lkml]   [2016]   [Nov]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 3/4] timekeeping: Ignore the bogus sleep time if pm_trace is enabled

* John Stultz <john.stultz@linaro.org> wrote:

> +static int pm_trace_notify(struct notifier_block *nb,
> + unsigned long mode, void *_unused)
> +{
> + switch (mode) {
> + case PM_POST_HIBERNATION:
> + case PM_POST_SUSPEND:
> + if (pm_trace_rtc_abused) {
> + pm_trace_rtc_abused = false;
> + pr_warn("Possible incorrect RTC due to pm_trace,"
> + "please use ntp-date or rdate to reset.\n");

Please don't break user-visible strings just to pacify checkpatch!

The bogus linebreak above hides a type in the user string:

Possible incorrect RTC due to pm_trace,please use ntp-date or rdate to reset.

(There's a missing space after the comma.)

Best practice is to preserve the continuous nature of the user string in the code.

In addition to that, please quote suggested command names, i.e. something like:

Possible incorrect RTC due to pm_trace, please use 'ntp-date' or 'rdate' to reset it.

> --- a/drivers/rtc/rtc-cmos.c
> +++ b/drivers/rtc/rtc-cmos.c
> @@ -191,6 +191,13 @@ static inline void cmos_write_bank2(unsigned char val, unsigned char addr)
>
> static int cmos_read_time(struct device *dev, struct rtc_time *t)
> {
> + /*
> + * If pmtrace abused the RTC for storage tell the caller that it is
> + * unusable.
> + */
> + if (!pm_trace_rtc_valid())
> + return -EIO;

Please standardize the spelling of 'pm_trace', as there's 3 variants present in
this patch alone:

'pm_trace'
'pm trace'
'pmtrace'

(Not to mention pm-trace.h - but that's a pre-existing inconsistency unrelated to
your patch.)

Thanks,

Ingo

\
 
 \ /
  Last update: 2016-11-21 09:18    [W:2.033 / U:1.244 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site