[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver.

From: Peter Maydell
Subject: Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver.
Date: Sun, 5 May 2013 11:47:09 +0100

On 5 May 2013 04:58, Jean-Christophe DUBOIS <address@hidden> wrote:
> On 05/05/2013 05:14 AM, Peter Crosthwaite wrote:
>> You could use an extract macro. extract32 is generally considered
>> better than shift and & logicvfor multi-bit fields. Ill have to look
>> up if extract16 exists (im away from tree for this review)
> no extract16() macro spotted.
> Should one be added?

There's no need for one -- just use extract32. The only
reason for having a separate extract64 is to avoid doing
64 bit arithmetic when we don't have to, I think.

-- PMM

reply via email to

[Prev in Thread] Current Thread [Next in Thread]