diff mbox series

[v3,13/19] qemu/bitops.h: Add extract8 and extract16

Message ID 20200225124710.14152-14-alex.bennee@linaro.org
State Superseded
Headers show
Series testing & plugin updates | expand

Commit Message

Alex Bennée Feb. 25, 2020, 12:47 p.m. UTC
From: Yoshinori Sato <ysato@users.sourceforge.jp>


Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-Id: <20190607091116.49044-10-ysato@users.sourceforge.jp>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Message-Id: <20200212130311.127515-3-ysato@users.sourceforge.jp>
---
 include/qemu/bitops.h | 38 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)

-- 
2.20.1

Comments

Philippe Mathieu-Daudé Feb. 25, 2020, 2:04 p.m. UTC | #1
On 2/25/20 1:47 PM, Alex Bennée wrote:
> From: Yoshinori Sato <ysato@users.sourceforge.jp>

> 

> Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>

> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> Message-Id: <20190607091116.49044-10-ysato@users.sourceforge.jp>


^ This tag can go.

> Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

> Message-Id: <20200212130311.127515-3-ysato@users.sourceforge.jp>

> ---

>   include/qemu/bitops.h | 38 ++++++++++++++++++++++++++++++++++++++

>   1 file changed, 38 insertions(+)

> 

> diff --git a/include/qemu/bitops.h b/include/qemu/bitops.h

> index 02c1ce6a5d4..f55ce8b320b 100644

> --- a/include/qemu/bitops.h

> +++ b/include/qemu/bitops.h

> @@ -301,6 +301,44 @@ static inline uint32_t extract32(uint32_t value, int start, int length)

>       return (value >> start) & (~0U >> (32 - length));

>   }

>   

> +/**

> + * extract8:

> + * @value: the value to extract the bit field from

> + * @start: the lowest bit in the bit field (numbered from 0)

> + * @length: the length of the bit field

> + *

> + * Extract from the 8 bit input @value the bit field specified by the

> + * @start and @length parameters, and return it. The bit field must

> + * lie entirely within the 8 bit word. It is valid to request that

> + * all 8 bits are returned (ie @length 8 and @start 0).

> + *

> + * Returns: the value of the bit field extracted from the input value.

> + */

> +static inline uint8_t extract8(uint8_t value, int start, int length)

> +{

> +    assert(start >= 0 && length > 0 && length <= 8 - start);

> +    return extract32(value, start, length);

> +}

> +

> +/**

> + * extract16:

> + * @value: the value to extract the bit field from

> + * @start: the lowest bit in the bit field (numbered from 0)

> + * @length: the length of the bit field

> + *

> + * Extract from the 16 bit input @value the bit field specified by the

> + * @start and @length parameters, and return it. The bit field must

> + * lie entirely within the 16 bit word. It is valid to request that

> + * all 16 bits are returned (ie @length 16 and @start 0).

> + *

> + * Returns: the value of the bit field extracted from the input value.

> + */

> +static inline uint16_t extract16(uint16_t value, int start, int length)

> +{

> +    assert(start >= 0 && length > 0 && length <= 16 - start);

> +    return extract32(value, start, length);

> +}

> +

>   /**

>    * extract64:

>    * @value: the value to extract the bit field from

>
diff mbox series

Patch

diff --git a/include/qemu/bitops.h b/include/qemu/bitops.h
index 02c1ce6a5d4..f55ce8b320b 100644
--- a/include/qemu/bitops.h
+++ b/include/qemu/bitops.h
@@ -301,6 +301,44 @@  static inline uint32_t extract32(uint32_t value, int start, int length)
     return (value >> start) & (~0U >> (32 - length));
 }
 
+/**
+ * extract8:
+ * @value: the value to extract the bit field from
+ * @start: the lowest bit in the bit field (numbered from 0)
+ * @length: the length of the bit field
+ *
+ * Extract from the 8 bit input @value the bit field specified by the
+ * @start and @length parameters, and return it. The bit field must
+ * lie entirely within the 8 bit word. It is valid to request that
+ * all 8 bits are returned (ie @length 8 and @start 0).
+ *
+ * Returns: the value of the bit field extracted from the input value.
+ */
+static inline uint8_t extract8(uint8_t value, int start, int length)
+{
+    assert(start >= 0 && length > 0 && length <= 8 - start);
+    return extract32(value, start, length);
+}
+
+/**
+ * extract16:
+ * @value: the value to extract the bit field from
+ * @start: the lowest bit in the bit field (numbered from 0)
+ * @length: the length of the bit field
+ *
+ * Extract from the 16 bit input @value the bit field specified by the
+ * @start and @length parameters, and return it. The bit field must
+ * lie entirely within the 16 bit word. It is valid to request that
+ * all 16 bits are returned (ie @length 16 and @start 0).
+ *
+ * Returns: the value of the bit field extracted from the input value.
+ */
+static inline uint16_t extract16(uint16_t value, int start, int length)
+{
+    assert(start >= 0 && length > 0 && length <= 16 - start);
+    return extract32(value, start, length);
+}
+
 /**
  * extract64:
  * @value: the value to extract the bit field from