* [PATCH/TRIVIAL 2/4] drm/i915: Move newline to end of error message
2014-10-23 9:52 [PATCH/TRIVIAL 1/4] cpufreq: cpufreq-dt: Move newline to end of error message Geert Uytterhoeven
@ 2014-10-23 9:52 ` Geert Uytterhoeven
2014-10-23 9:52 ` [PATCH/TRIVIAL 3/4] iwlegacy: 4965-rs: Remove bogus colon after newline from debug message Geert Uytterhoeven
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Geert Uytterhoeven @ 2014-10-23 9:52 UTC (permalink / raw)
To: Jiri Kosina; +Cc: linux-kernel, Geert Uytterhoeven, David Airlie
Currently the error message is needlessly splitted across two lines.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: David Airlie <airlied@linux.ie>
---
drivers/gpu/drm/i915/i915_irq.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 3201986bf25ebaf6..359957656b5a38d0 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -2595,7 +2595,7 @@ static irqreturn_t gen8_irq_handler(int irq, void *arg)
}
if (pipe_iir & GEN8_DE_PIPE_IRQ_FAULT_ERRORS) {
- DRM_ERROR("Fault errors on pipe %c\n: 0x%08x",
+ DRM_ERROR("Fault errors on pipe %c: 0x%08x\n",
pipe_name(pipe),
pipe_iir & GEN8_DE_PIPE_IRQ_FAULT_ERRORS);
}
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH/TRIVIAL 3/4] iwlegacy: 4965-rs: Remove bogus colon after newline from debug message
2014-10-23 9:52 [PATCH/TRIVIAL 1/4] cpufreq: cpufreq-dt: Move newline to end of error message Geert Uytterhoeven
2014-10-23 9:52 ` [PATCH/TRIVIAL 2/4] drm/i915: " Geert Uytterhoeven
@ 2014-10-23 9:52 ` Geert Uytterhoeven
2014-10-23 9:52 ` [PATCH/TRIVIAL 4/4] uwb: " Geert Uytterhoeven
2014-11-19 8:06 ` [PATCH/TRIVIAL 1/4] cpufreq: cpufreq-dt: Move newline to end of error message Viresh Kumar
3 siblings, 0 replies; 5+ messages in thread
From: Geert Uytterhoeven @ 2014-10-23 9:52 UTC (permalink / raw)
To: Jiri Kosina; +Cc: linux-kernel, Geert Uytterhoeven, John W. Linville
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: John W. Linville <linville@tuxdriver.com>
---
drivers/net/wireless/iwlegacy/4965-rs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/iwlegacy/4965-rs.c b/drivers/net/wireless/iwlegacy/4965-rs.c
index eaaeea19d8c5bcc9..bac60b2bc3f014a5 100644
--- a/drivers/net/wireless/iwlegacy/4965-rs.c
+++ b/drivers/net/wireless/iwlegacy/4965-rs.c
@@ -1678,7 +1678,7 @@ il4965_rs_stay_in_table(struct il_lq_sta *lq_sta, bool force_search)
lq_sta->total_success > lq_sta->max_success_limit ||
(!lq_sta->search_better_tbl && lq_sta->flush_timer &&
flush_interval_passed)) {
- D_RATE("LQ: stay is expired %d %d %d\n:",
+ D_RATE("LQ: stay is expired %d %d %d\n",
lq_sta->total_failed, lq_sta->total_success,
flush_interval_passed);
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH/TRIVIAL 4/4] uwb: Remove bogus colon after newline from debug message
2014-10-23 9:52 [PATCH/TRIVIAL 1/4] cpufreq: cpufreq-dt: Move newline to end of error message Geert Uytterhoeven
2014-10-23 9:52 ` [PATCH/TRIVIAL 2/4] drm/i915: " Geert Uytterhoeven
2014-10-23 9:52 ` [PATCH/TRIVIAL 3/4] iwlegacy: 4965-rs: Remove bogus colon after newline from debug message Geert Uytterhoeven
@ 2014-10-23 9:52 ` Geert Uytterhoeven
2014-11-19 8:06 ` [PATCH/TRIVIAL 1/4] cpufreq: cpufreq-dt: Move newline to end of error message Viresh Kumar
3 siblings, 0 replies; 5+ messages in thread
From: Geert Uytterhoeven @ 2014-10-23 9:52 UTC (permalink / raw)
To: Jiri Kosina; +Cc: linux-kernel, Geert Uytterhoeven, linux-usb
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: linux-usb@vger.kernel.org
---
drivers/uwb/rsv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/uwb/rsv.c b/drivers/uwb/rsv.c
index 0887ae982783b7df..536ad42b0a4b1d7e 100644
--- a/drivers/uwb/rsv.c
+++ b/drivers/uwb/rsv.c
@@ -213,7 +213,7 @@ void uwb_rsv_backoff_win_timer(unsigned long arg)
bow->total_expired = 0;
bow->window = UWB_DRP_BACKOFF_WIN_MIN >> 1;
}
- dev_dbg(dev, "backoff_win_timer total_expired=%d, n=%d\n: ", bow->total_expired, bow->n);
+ dev_dbg(dev, "backoff_win_timer total_expired=%d, n=%d\n", bow->total_expired, bow->n);
/* try to relocate all the "to be moved" relocations */
uwb_rsv_handle_drp_avail_change(rc);
@@ -234,7 +234,7 @@ void uwb_rsv_backoff_win_increment(struct uwb_rc *rc)
bow->window <<= 1;
bow->n = prandom_u32() & (bow->window - 1);
- dev_dbg(dev, "new_window=%d, n=%d\n: ", bow->window, bow->n);
+ dev_dbg(dev, "new_window=%d, n=%d\n", bow->window, bow->n);
/* reset the timer associated variables */
timeout_us = bow->n * UWB_SUPERFRAME_LENGTH_US;
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH/TRIVIAL 1/4] cpufreq: cpufreq-dt: Move newline to end of error message
2014-10-23 9:52 [PATCH/TRIVIAL 1/4] cpufreq: cpufreq-dt: Move newline to end of error message Geert Uytterhoeven
` (2 preceding siblings ...)
2014-10-23 9:52 ` [PATCH/TRIVIAL 4/4] uwb: " Geert Uytterhoeven
@ 2014-11-19 8:06 ` Viresh Kumar
3 siblings, 0 replies; 5+ messages in thread
From: Viresh Kumar @ 2014-11-19 8:06 UTC (permalink / raw)
To: Geert Uytterhoeven, Linux PM list
Cc: Jiri Kosina, linux-kernel, Rafael J. Wysocki
On Thu, Oct 23, 2014 at 3:22 PM, Geert Uytterhoeven
<geert+renesas@glider.be> wrote:
> Currently the error message is needlessly splitted across two lines.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Cc: Rafael J. Wysocki <rjw@rjwysocki.net>
> ---
> drivers/cpufreq/cpufreq-dt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/cpufreq/cpufreq-dt.c b/drivers/cpufreq/cpufreq-dt.c
> index 92c162af5045a2da..685f59598137ea03 100644
> --- a/drivers/cpufreq/cpufreq-dt.c
> +++ b/drivers/cpufreq/cpufreq-dt.c
> @@ -192,7 +192,7 @@ static int cpufreq_init(struct cpufreq_policy *policy)
>
> ret = allocate_resources(policy->cpu, &cpu_dev, &cpu_reg, &cpu_clk);
> if (ret) {
> - pr_err("%s: Failed to allocate resources\n: %d", __func__, ret);
> + pr_err("%s: Failed to allocate resources: %d\n", __func__, ret);
> return ret;
> }
Please add me and linux-pm in cc next time.
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
^ permalink raw reply [flat|nested] 5+ messages in thread