From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Mon, 10 Oct 2011 17:00:46 -0500
Subject: [U-Boot] [PATCH v2 1/5] usb:gadget:s5p USB Device Controller
 (UDC) implementation
In-Reply-To: <20111010215447.0E5291408777@gemini.denx.de>
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>
	<20111010215447.0E5291408777@gemini.denx.de>
Message-ID: <4E936B0E.9000106@freescale.com>
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

On 10/10/2011 04:54 PM, Wolfgang Denk wrote:
> 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.

It wasn't for the fun of it, it was missing context leading to a
misunderstanding of what you were complaining about.

> 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?

Yes.

-Scott