[PATCH 2/5] [POWERPC] QE: add support for QE USB clocks routing

Timur Tabi timur at freescale.com
Fri Apr 18 06:49:59 EST 2008


Dale Farnsworth wrote:

> I find "is_brg" much more meaningful than "clk < QE_CLK1".
> 
> Maybe the computer doesn't need it, but it's helpful to a
> reader of the code.

That's what comments are for.  I'm all for self-documenting code, but not
creating additional code to avoid comments.

-- 
Timur Tabi
Linux kernel developer at Freescale



More information about the Linuxppc-dev mailing list