[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 Crosthwaite |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver. |
Date: |
Sun, 5 May 2013 20:53:45 +1000 |
Hi Peter,
On Sun, May 5, 2013 at 8:47 PM, Peter Maydell <address@hidden> wrote:
> 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.
>
perhaps a quick:
#define extract16(a, b, c) (uint16_t)extract32((uint16_t)(a), (b), (c))
would keep the 16bit device code explicit and clean? Save on
dummy casts in certain situations as well (but not this one).
Regards,
Peter
> -- PMM
>
- Re: [Qemu-devel] [PATCH v2 1/4] Add i.MX FEC Ethernet driver, (continued)
- Re: [Qemu-devel] [PATCH v2 1/4] Add i.MX FEC Ethernet driver, Michael S. Tsirkin, 2013/05/06
- Re: [Qemu-devel] [PATCH v2 1/4] Add i.MX FEC Ethernet driver, Peter Maydell, 2013/05/06
- Re: [Qemu-devel] [PATCH v2 1/4] Add i.MX FEC Ethernet driver, Michael S. Tsirkin, 2013/05/06
- Re: [Qemu-devel] [PATCH v2 1/4] Add i.MX FEC Ethernet driver, Peter Maydell, 2013/05/06
- Re: [Qemu-devel] [PATCH v2 1/4] Add i.MX FEC Ethernet driver, Peter Crosthwaite, 2013/05/06
- Re: [Qemu-devel] [PATCH v2 1/4] Add i.MX FEC Ethernet driver, Peter Maydell, 2013/05/07
[Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver., Jean-Christophe DUBOIS, 2013/05/04
- Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver., Peter Crosthwaite, 2013/05/04
- Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver., Jean-Christophe DUBOIS, 2013/05/04
- Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver., Peter Maydell, 2013/05/05
- Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver.,
Peter Crosthwaite <=
- Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver., Peter Maydell, 2013/05/05
- Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver., Peter Crosthwaite, 2013/05/05
- Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver., Jean-Christophe DUBOIS, 2013/05/05
Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver., Andreas Färber, 2013/05/05
Re: [Qemu-devel] [PATCH v2 2/4] Add i.MX I2C controller driver., Jean-Christophe DUBOIS, 2013/05/05
[Qemu-devel] [PATCH v2 4/4] Add qtest support for i.MX I2C device emulation., Jean-Christophe DUBOIS, 2013/05/04