Re: [PATCH] TXx9 SPI controller driver (take 2)

To:	david-b@pacbell.net
Subject:	Re: [PATCH] TXx9 SPI controller driver (take 2)
From:	Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Date:	Sun, 01 Jul 2007 23:40:27 +0900 (JST)
Cc:	linux-mips@linux-mips.org, ralf@linux-mips.org, sshtylyov@ru.mvista.com, mlachwani@mvista.com, spi-devel-general@lists.sourceforge.net
In-reply-to:	<200706301046.20826.david-b@pacbell.net>
Original-recipient:	rfc822;linux-mips@linux-mips.org
References:	<200706300953.20156.david-b@pacbell.net> <20070701.023414.71085498.anemo@mba.ocn.ne.jp> <200706301046.20826.david-b@pacbell.net>
Sender:	linux-mips-bounce@linux-mips.org

On Sat, 30 Jun 2007 10:46:20 -0700, David Brownell <david-b@pacbell.net> wrote:
> > The compiler will optimize "1000000000 / 2 / spi->max_speed_hz" into
> > "500000000 / spi->max_speed_hz", so it can be treat as one "/", no?
> 
> Sure it's deterministic.  But that doesn't prevent me from
> needing a double-take to figure what it does ... it's best
> to avoid confusing idioms in code.  At the very least, put
> parentheses there ...

OK, I will use parentheses while I think 1000000000 is important part.
Well, maybe NSEC_PER_SEC would be better.
---
Atsushi Nemoto

<Prev in Thread]	Current Thread	[Next in Thread>
Re: [PATCH] TXx9 SPI controller driver (take 2), Atsushi Nemoto <= Re: [PATCH] TXx9 SPI controller driver (take 2), Atsushi Nemoto

Next by Date:	O2 RM7000 Issues, Kumba
Next by Thread:	Re: [PATCH] TXx9 SPI controller driver (take 2), Atsushi Nemoto
Indexes:	[Date] [Thread] [Top] [All Lists]