Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934472AbcLMSnl (ORCPT ); Tue, 13 Dec 2016 13:43:41 -0500 Received: from foss.arm.com ([217.140.101.70]:59252 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932250AbcLMSnk (ORCPT ); Tue, 13 Dec 2016 13:43:40 -0500 Subject: Re: [PATCH V8 3/3] irqchip: qcom: Add IRQ combiner driver To: Joe Perches , Agustin Vega-Frias References: <1480460259-8585-1-git-send-email-agustinv@codeaurora.org> <1480460259-8585-4-git-send-email-agustinv@codeaurora.org> <78bfd90d47200347628f7dd98451122f@codeaurora.org> <1481653317.29291.19.camel@perches.com> Cc: linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, rjw@rjwysocki.net, lenb@kernel.org, tglx@linutronix.de, jason@lakedaemon.net, timur@codeaurora.org, cov@codeaurora.org, agross@codeaurora.org, harba@codeaurora.org, jcm@redhat.com, msalter@redhat.com, mlangsdo@redhat.com, ahs3@redhat.com, astone@redhat.com, graeme.gregory@linaro.org, guohanjun@huawei.com, charles.garcia-tobin@arm.com From: Marc Zyngier X-Enigmail-Draft-Status: N1110 Organization: ARM Ltd Message-ID: <413b214c-146e-a1ca-7e84-6fc96959d7c7@arm.com> Date: Tue, 13 Dec 2016 18:43:33 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.2.0 MIME-Version: 1.0 In-Reply-To: <1481653317.29291.19.camel@perches.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1499 Lines: 48 On 13/12/16 18:21, Joe Perches wrote: > On Tue, 2016-12-13 at 10:23 -0500, Agustin Vega-Frias wrote: >> On 2016-12-07 13:16, Marc Zyngier wrote: >>>> + } >>>> + >>>> + combiner->domain = irq_domain_create_linear( >>>> + pdev->dev.fwnode, combiner->nirqs, &domain_ops, combiner); >>> >>> On a single line, please. Do no listen to the checkpatch police that >>> will tell you otherwise. It really hurt my eyes to see this opening >>> bracket and *nothing* after that. >> >> Will do. > > It seems generally preferred to have at least one argument on the > same line as the function being called. > > So, here are some options: > > Maximally fill the lines to 80 columns with the value being set > and function call while aligning to open parenthesis > > combiner->domain = irq_domain_create_linear(pdev->dev.fwnode, > combiner->nirqs, > &domain_ops, combiner); I can live with something like this. > Use a separate line for the function call: > > combiner->domain = > irq_domain_create_linear(pdev->dev.fwnode, combiner->nirqs, > &domain_ops, combiner); But I find this one pretty horrid. > Or just ignore the 80 column limit wherever you deem appropriate. Which is my usual advise. I consider the 80 column rule a good way of limiting the complexity of code (if the nesting pushes you too far on the right of the screen, you're doing something wrong), but not for simple statements such as this one. Thanks, M. -- Jazz is not dead. It just smells funny...