selectable_int: fix check on SI constructor

Specifying the number of bits *is* allowed, it should mean "override
the length". More unit tests need running to confirm.

diff --git a/src/openpower/decoder/selectable_int.py b/src/openpower/decoder/selectable_int.py
index 56ba07c3b78b1a9bf88dee0c37d755171732fcce..22985014e292f046f95decee20f0e7a820600d04 100644 (file)
--- a/src/openpower/decoder/selectable_int.py
+++ b/src/openpower/decoder/selectable_int.py
@@ -221,7 +221,9 @@ class SelectableInt:
     def __init__(self, value, bits=None):
         if isinstance(value, SelectableInt):
             if bits is not None:
-                raise ValueError(value)
+                # check if the bitlength is different. TODO, allow override?
+                if bits != value.bits:
+                    raise ValueError(value)
             bits = value.bits
             value = value.value
         elif isinstance(value, FieldSelectableInt):