aboutsummaryrefslogtreecommitdiffstats
path: root/src/broadcom/cle
diff options
context:
space:
mode:
authorEric Anholt <[email protected]>2018-07-30 11:41:15 -0700
committerEric Anholt <[email protected]>2018-07-30 14:29:01 -0700
commit89ac6fa4036da815b5cf4985f438cec73df67480 (patch)
tree28f33d348e354b1e42c1b7fb77e9ee2e140cc5e8 /src/broadcom/cle
parente146e3a795ecd070679b8dfd1ad7f370e9ec5665 (diff)
v3d: Add pack header support for f187 values.
V3D only has one of these (the top 16 bits of a float32) left in its CLs, but VC4 had many more. This gets us proper pretty-printing of the values instead of a large uint.
Diffstat (limited to 'src/broadcom/cle')
-rw-r--r--src/broadcom/cle/gen_pack_header.py7
-rw-r--r--src/broadcom/cle/v3d_decoder.c7
-rw-r--r--src/broadcom/cle/v3d_decoder.h1
-rw-r--r--src/broadcom/cle/v3d_packet_helpers.h9
-rw-r--r--src/broadcom/cle/v3d_packet_v33.xml12
5 files changed, 29 insertions, 7 deletions
diff --git a/src/broadcom/cle/gen_pack_header.py b/src/broadcom/cle/gen_pack_header.py
index 8ad54464cb8..f79ee42df5a 100644
--- a/src/broadcom/cle/gen_pack_header.py
+++ b/src/broadcom/cle/gen_pack_header.py
@@ -152,6 +152,8 @@ class Field(object):
type = 'bool'
elif self.type == 'float':
type = 'float'
+ elif self.type == 'f187':
+ type = 'float'
elif self.type == 'ufixed':
type = 'float'
elif self.type == 'sfixed':
@@ -317,6 +319,9 @@ class Group(object):
(value, start, end)
elif field.type == "float":
s = "#error %s float value mixed in with other fields" % name
+ elif field.type == "f187":
+ s = "__gen_uint(fui(%s) >> 16, %d, %d)" % \
+ (value, start, end)
elif field.type == "offset":
s = "__gen_offset(%s, %d, %d)" % \
(value, start, end)
@@ -370,6 +375,8 @@ class Group(object):
convert = "__gen_unpack_uint"
elif field.type == "float":
convert = "__gen_unpack_float"
+ elif field.type == "f187":
+ convert = "__gen_unpack_f187"
elif field.type == "offset":
convert = "__gen_unpack_offset"
elif field.type == 'ufixed':
diff --git a/src/broadcom/cle/v3d_decoder.c b/src/broadcom/cle/v3d_decoder.c
index 7eb963fcf18..22367ca6f76 100644
--- a/src/broadcom/cle/v3d_decoder.c
+++ b/src/broadcom/cle/v3d_decoder.c
@@ -316,6 +316,8 @@ string_to_type(struct parser_context *ctx, const char *s)
return (struct v3d_type) { .kind = V3D_TYPE_BOOL };
else if (strcmp(s, "float") == 0)
return (struct v3d_type) { .kind = V3D_TYPE_FLOAT };
+ else if (strcmp(s, "f187") == 0)
+ return (struct v3d_type) { .kind = V3D_TYPE_F187 };
else if (strcmp(s, "address") == 0)
return (struct v3d_type) { .kind = V3D_TYPE_ADDRESS };
else if (strcmp(s, "offset") == 0)
@@ -887,6 +889,11 @@ v3d_field_iterator_next(struct clif_dump *clif, struct v3d_field_iterator *iter)
__gen_unpack_float(iter->p, s, e));
break;
+ case V3D_TYPE_F187:
+ snprintf(iter->value, sizeof(iter->value), "%f",
+ __gen_unpack_f187(iter->p, s, e));
+ break;
+
case V3D_TYPE_ADDRESS: {
uint32_t addr =
__gen_unpack_uint(iter->p, s, e) << (31 - (e - s));
diff --git a/src/broadcom/cle/v3d_decoder.h b/src/broadcom/cle/v3d_decoder.h
index 10a7e50b87f..b5ead383bd8 100644
--- a/src/broadcom/cle/v3d_decoder.h
+++ b/src/broadcom/cle/v3d_decoder.h
@@ -99,6 +99,7 @@ struct v3d_type {
V3D_TYPE_UINT,
V3D_TYPE_BOOL,
V3D_TYPE_FLOAT,
+ V3D_TYPE_F187,
V3D_TYPE_ADDRESS,
V3D_TYPE_OFFSET,
V3D_TYPE_STRUCT,
diff --git a/src/broadcom/cle/v3d_packet_helpers.h b/src/broadcom/cle/v3d_packet_helpers.h
index 4720f643328..f340b790697 100644
--- a/src/broadcom/cle/v3d_packet_helpers.h
+++ b/src/broadcom/cle/v3d_packet_helpers.h
@@ -26,6 +26,7 @@
#include <stdbool.h>
#include <assert.h>
#include <math.h>
+#include <gallium/auxiliary/util/u_math.h>
#ifdef HAVE_VALGRIND
#include <valgrind.h>
@@ -205,3 +206,11 @@ __gen_unpack_float(const uint8_t *restrict cl, uint32_t start, uint32_t end)
return f->f;
}
+static inline float
+__gen_unpack_f187(const uint8_t *restrict cl, uint32_t start, uint32_t end)
+{
+ assert(end - start == 15);
+ uint32_t bits = __gen_unpack_uint(cl, start, end);
+ return uif(bits << 16);
+}
+
diff --git a/src/broadcom/cle/v3d_packet_v33.xml b/src/broadcom/cle/v3d_packet_v33.xml
index 1f6d467c92b..e4b7df002df 100644
--- a/src/broadcom/cle/v3d_packet_v33.xml
+++ b/src/broadcom/cle/v3d_packet_v33.xml
@@ -619,7 +619,7 @@
</packet>
<packet code="91" name="Sample State" min_ver="41">
- <field name="Coverage" size="16" start="16" type="uint"/> <!-- float-1-8-7 -->
+ <field name="Coverage" size="16" start="16" type="f187"/>
<field name="Mask" size="4" start="0" type="uint"/>
</packet>
@@ -672,16 +672,14 @@
</packet>
<packet name="Depth Offset" code="106" max_ver="33">
- <!-- these fields are both float-1-8-7 encoded (top 16 bits of a float32) -->
- <field name="Depth Offset Units" size="16" start="16" type="uint"/>
- <field name="Depth Offset Factor" size="16" start="0" type="uint"/>
+ <field name="Depth Offset Units" size="16" start="16" type="f187"/>
+ <field name="Depth Offset Factor" size="16" start="0" type="f187"/>
</packet>
<packet name="Depth Offset" code="106" min_ver="41">
<field name="Limit" size="32" start="32" type="float"/>
- <!-- these fields are both float-1-8-7 encoded (top 16 bits of a float32) -->
- <field name="Depth Offset Units" size="16" start="16" type="uint"/>
- <field name="Depth Offset Factor" size="16" start="0" type="uint"/>
+ <field name="Depth Offset Units" size="16" start="16" type="f187"/>
+ <field name="Depth Offset Factor" size="16" start="0" type="f187"/>
</packet>
<packet name="Clip Window" code="107">