Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement

From	Juston Li <juston.h.li@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement
Date	2015-06-25 23:30 +0200
Message-ID	<pFmSC-3Rc-25@gated-at.bofh.it> (permalink)
References	<pEVIJ-6Iv-3@gated-at.bofh.it> <pEVIJ-6Iv-5@gated-at.bofh.it> <pFeim-8u1-15@gated-at.bofh.it> <pFhzA-4C6-5@gated-at.bofh.it> <pFm6e-2Hi-23@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Thu, Jun 25, 2015 at 1:31 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> Once you add the else statement, then that kind of doesn't make sense.
> Sudip is right.  It should be:
>
>         } else {
>                 /* commentary about else side */

Yeah, that does make more sense. I'll change it, thanks.

Patchset was responded by Greg's patch bot,  I'll resend when I figure out why.

Regards
Juston
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

Thread

Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement Juston Li <juston.h.li@gmail.com> - 2015-06-25 23:30 +0200
  Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-06-26 06:10 +0200

csiph-web