Re: [PATCH 7/9] drm/i915: Combine substrings for a message in gen6_drpc_info()

From: SF Markus Elfring
Date: Thu May 04 2017 - 16:48:41 EST


>> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
>> @@ -1529,8 +1529,8 @@ static int gen6_drpc_info(struct seq_file *m)
>>
>> forcewake_count = READ_ONCE(dev_priv->uncore.fw_domain[FW_DOMAIN_ID_RENDER].wake_count);
>> if (forcewake_count) {
>> - seq_puts(m, "RC information inaccurate because somebody "
>> - "holds a forcewake reference \n");
>> + seq_puts(m,
>> + "RC information inaccurate because somebody holds a forcewake reference.\n");
>
> And now you break the 80col rule. Blind adherence to checkpatch is impossible.

Have you got any other coding style preferences around the grepping
of longer message strings from such source code?

Regards,
Markus