From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752489AbdGJEqh (ORCPT ); Mon, 10 Jul 2017 00:46:37 -0400 Received: from mail-pg0-f48.google.com ([74.125.83.48]:34942 "EHLO mail-pg0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750847AbdGJEqg (ORCPT ); Mon, 10 Jul 2017 00:46:36 -0400 Date: Mon, 10 Jul 2017 10:16:32 +0530 From: Viresh Kumar To: Mitchell Tasman Cc: Vaibhav Hiremath , Johan Hovold , Alex Elder , Greg Kroah-Hartman , devel@driverdev.osuosl.org, greybus-dev@lists.linaro.org, linux-kernel@vger.kernel.org Subject: Re: [greybus-dev] [PATCH v2] staging: greybus: arche: wrap over-length lines Message-ID: <20170710044632.GD2928@vireshk-i7> References: <20170709212809.14277-1-tasman@leaflabs.com> <20170710002506.15036-1-tasman@leaflabs.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170710002506.15036-1-tasman@leaflabs.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mitchell, On 09-07-17, 20:25, Mitchell Tasman wrote: > Adjust formatting of various statements to keep line length within > the 80 column limit preferred by the Linux kernel coding style. We try to follow that most of the time, but the end result should be easily readable. If it isn't, then we just ignore the rule. > Signed-off-by: Mitchell Tasman > --- > Changes in v2: Add back a missing space in a comment > > drivers/staging/greybus/arche-platform.c | 29 +++++++++++++++++++---------- > 1 file changed, 19 insertions(+), 10 deletions(-) > > diff --git a/drivers/staging/greybus/arche-platform.c b/drivers/staging/greybus/arche-platform.c > index eced2d2..eedd239 100644 > --- a/drivers/staging/greybus/arche-platform.c > +++ b/drivers/staging/greybus/arche-platform.c > @@ -172,15 +172,21 @@ static irqreturn_t arche_platform_wd_irq(int irq, void *devid) > if (arche_pdata->wake_detect_state == WD_STATE_BOOT_INIT) { > if (time_before(jiffies, > arche_pdata->wake_detect_start + > - msecs_to_jiffies(WD_COLDBOOT_PULSE_WIDTH_MS))) { > - arche_platform_set_wake_detect_state(arche_pdata, > - WD_STATE_IDLE); > + msecs_to_jiffies( > + WD_COLDBOOT_PULSE_WIDTH_MS))) { > + arche_platform_set_wake_detect_state( We don't break the lines like this in kernel to satisfy the 80 column width rule. Surely, some places would have similar code, but in general this isn't recommended. And that's why we never bothered about 80 column rule in this and below cases. > + arche_pdata, > + WD_STATE_IDLE); > } else { > - /* Check we are not in middle of irq thread already */ > + /* > + * Check we are not in middle of irq thread > + * already > + */ Yes, such changes would be fine. > if (arche_pdata->wake_detect_state != > WD_STATE_COLDBOOT_START) { > - arche_platform_set_wake_detect_state(arche_pdata, > - WD_STATE_COLDBOOT_TRIG); > + arche_platform_set_wake_detect_state( Not this. > + arche_pdata, > + WD_STATE_COLDBOOT_TRIG); > spin_unlock_irqrestore( > &arche_pdata->wake_lock, > flags); > @@ -199,8 +205,9 @@ static irqreturn_t arche_platform_wd_irq(int irq, void *devid) > * beyond 30msec, then it is coldboot else fallback > * to standby boot. > */ > - arche_platform_set_wake_detect_state(arche_pdata, > - WD_STATE_BOOT_INIT); > + arche_platform_set_wake_detect_state( Not this. > + arche_pdata, > + WD_STATE_BOOT_INIT); > } > } > > @@ -657,12 +664,14 @@ static SIMPLE_DEV_PM_OPS(arche_platform_pm_ops, > arche_platform_resume); > > static const struct of_device_id arche_platform_of_match[] = { > - { .compatible = "google,arche-platform", }, /* Use PID/VID of SVC device */ > + /* Use PID/VID of SVC device */ > + { .compatible = "google,arche-platform", }, This would be fine though. > { }, > }; > > static const struct of_device_id arche_combined_id[] = { > - { .compatible = "google,arche-platform", }, /* Use PID/VID of SVC device */ > + /* Use PID/VID of SVC device */ > + { .compatible = "google,arche-platform", }, > { .compatible = "usbffff,2", }, > { }, > }; -- viresh