Re: [PATCH] Staging: greybus: Match alignment with open parenthesis.
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: Johan Hovold <johan@xxxxxxxxxx>
- Subject: Re: [PATCH] Staging: greybus: Match alignment with open parenthesis.
- From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
- Date: Sun, 23 Jul 2017 15:10:53 +0300
- In-reply-to: <20170723084743.GF2729@localhost>
- References: <1500799197-4326-1-git-send-email-shreeya.patel23498@gmail.com> <20170723084743.GF2729@localhost>
- User-agent: NeoMutt/20170113 (1.7.2)
On Sun, Jul 23, 2017 at 10:47:43AM +0200, Johan Hovold wrote:
> On Sun, Jul 23, 2017 at 02:09:57PM +0530, Shreeya Patel wrote:
> > Alignment should match with open parenthesis.
> > This fixes the coding style issue.
>
> No, it need not. This is a choice left up to the author, even if
> checkpatch happens to complain about it with the --strict option.
>
This is staging code. Don't try to fight checkpatch.pl in staging...
You're just making work for everyone else (me) who already just reviewed
this patch and I don't want to review it a few more times when the next
person sends the exact same patch.
regards,
dan carpenter
_______________________________________________
greybus-dev mailing list
greybus-dev@xxxxxxxxxxxxxxxx
https://lists.linaro.org/mailman/listinfo/greybus-dev
[Index of Archives]
[Asterisk App Development]
[PJ SIP]
[Gnu Gatekeeper]
[IETF Sipping]
[Info Cyrus]
[ALSA User]
[Fedora Linux Users]
[Linux SCTP]
[DCCP]
[Gimp]
[Yosemite News]
[Deep Creek Hot Springs]
[Yosemite Campsites]
[ISDN Cause Codes]
[Asterisk Books]