From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wolfgang Denk <wd@denx.de>
Date: Mon, 10 Oct 2011 23:54:47 +0200
Subject: [U-Boot] [PATCH v2 1/5] usb:gadget:s5p USB Device Controller
	(UDC) implementation
In-Reply-To: <4E935BDC.7000109@freescale.com>
References: <1310567392-29082-1-git-send-email-l.majewski@samsung.com>
	<1310567392-29082-2-git-send-email-l.majewski@samsung.com>
	<20111006221048.39BEC140874A@gemini.denx.de>
	<20111007093950.17d94287@lmajewski.digital.local>
	<20111009194336.C84971408775@gemini.denx.de>
	<4E934EF5.9030302@freescale.com>
	<20111010203846.175291408753@gemini.denx.de>
	<4E935BDC.7000109@freescale.com>
Message-ID: <20111010215447.0E5291408777@gemini.denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Scott Wood,

In message <4E935BDC.7000109@freescale.com> you wrote:
>
> > Well, "user-visible strings" is definitely not the same thing as
> > "debug print code", at least not for me.
> 
> They're visible to a user that has #defined DEBUG.  They're something
> one might want to grep on.  checkpatch.pl explicitly considers

Strings yes. Code no.

> At some point you might want to consider actually writing down your
> interpretations of these things into a U-Boot coding style document, so
> at least we don't have to guess.

There is not much of interpretion.  Please stop discussing just for
the fun of it and switch on common sense.

This is bad:

			debug("This is a very, very long string just "
			      "to show what is meant by the CodingStyle "
			      "note about 'user-visible strings\n");
	
But this is bad, too:

				for (i = 0; i < 100; i++) {
					for (j = 0; j < 100; j++) {
						debug("f(i)=0x%08x, g(j)=0x%08x\n", some_function_doing_funny_things(i), some_other_function_doing_more_funny_things(j));
					}
				}

The first example is what the CodingStyle mentions: do not break
"user-visible strings".

The second example should clearly be reformatted, at least as:

				for (i = 0; i < 100; i++) {
					for (j = 0; j < 100; j++) {
						debug("f(i)=0x%08x, g(j)=0x%08x\n",
							some_function_doing_funny_things(i),
							some_other_function_doing_more_funny_things(j)
						);
					}
				}

or similar.

Agreed?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de
"How is this place run - is it an anarchy?"
"No, I wouldn't say so; it is not that well organised..."