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

From: Timur Tabi <[email-masked]>
Date: Thu, 17 Apr 2008 15:49:59 -0500

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