Re: [PATCH v4 2/2] power: reset: at91-shdwc: add new shutdown controller driver

From	Alexandre Belloni <alexandre.belloni@free-electrons.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v4 2/2] power: reset: at91-shdwc: add new shutdown controller driver
Date	2016-03-16 13:10 +0100
Message-ID	<rdir0-kr-21@gated-at.bofh.it> (permalink)
References	<rbaQV-vD-7@gated-at.bofh.it> <rbaQV-vD-5@gated-at.bofh.it> <rcHhM-h8-25@gated-at.bofh.it> <rdhXY-8lR-15@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 16/03/2016 at 12:36:56 +0100, Nicolas Ferre wrote :
> I don't agree with you on the 2 last ones. In Documentation/CodingStyle,
> it is said that:
> 
> "Chapter 2: Breaking long lines and strings
> [..]
> Descendants are always substantially shorter than the parent and
> are placed substantially to the right."
> 
> Together with the 80 column requirement that I also fulfil, I think that
> my code is well aligned.
> 
> So I keep it like this ;-)
> 

checkpatch.pl --strict disagrees with you


-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH v4 2/2] power: reset: at91-shdwc: add new shutdown controller driver Nicolas Ferre <nicolas.ferre@atmel.com> - 2016-03-10 16:40 +0100
  Re: [PATCH v4 2/2] power: reset: at91-shdwc: add new shutdown  controller driver Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2016-03-14 21:30 +0100
    Re: [PATCH v4 2/2] power: reset: at91-shdwc: add new shutdown  controller driver Nicolas Ferre <nicolas.ferre@atmel.com> - 2016-03-16 12:40 +0100
      Re: [PATCH v4 2/2] power: reset: at91-shdwc: add new shutdown  controller driver Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2016-03-16 13:10 +0100

csiph-web