On 26 June 2012 18:58, Blue Swirl <blauwir...@gmail.com> wrote: > On Mon, Jun 25, 2012 at 7:38 PM, Peter Maydell <peter.mayd...@linaro.org> > wrote: >> +static inline uint64_t field64(uint64_t value, int start, int length) > > start and length could be unsigned.
They could be, but is there any reason why they should be? set_bit(), clear_bit() etc use 'int' for bit numbers, so this is consistent with that. -- PMM