[U-Boot] [PATCH] Improve DaVinci SPI speed

Delio Brignoli dbrignoli at audioscience.com
Tue May 25 11:31:48 CEST 2010


Hello Wolfgang,

On 24/05/2010, at 22:48, Wolfgang Denk wrote:
> In message <15866EAE-624B-4850-B10F-78AB99E3A1A2 at audioscience.com> you wrote:
>> 
>> On 21/05/2010, at 15:13, Wolfgang Denk wrote:
>>>> +				*rxp = buf_reg_val & 0xFF;	
>>>> +				rxp++;
>>>> +			}
>>> 
>>> Please change into:
>>> 
>>> 			if (rxp)
>>> 				*rxp++ = buf_reg_val & 0xFF;
>> 
>> Are you sure? Is folding that 3 line block into a one liner really worth
>> the loss of readability? I know what that means, so do you and many
>> others on this ML, but that's bound to raise a few eyebrows. A quick
> 
> Removing the need for braces is a good thing.

by making the code less intelligible, which is a bad thing.

> This is plain C, without any tricks or loopholes.

This is C, the hard-to-read kind, but you refuse to acknowledge that by stating
only your side of the argument, while I pointed out the tradeoff involved and my opinion.
I have no inclination to argue in this fashion :-)

Kind Regards
--
Delio

> 
> 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 at denx.de
> Build a system that even a fool can use and only a fool will want  to
> use it.




More information about the U-Boot mailing list