On Tue, Apr 25, 2023 at 09:04 AM, Sami Mujawar wrote:

> 
> +/** RSI interface Version
> + See Section B4.4.3, RMM Specification, version A-bet0.
> + The width of the RsiInterfaceVersion fieldset is 64 bits.
> +*/
> +#define RSI_VER_MINOR_MASK 0x00FFULL
> +#define RSI_VER_MAJOR_MASK 0x7F00ULL
> +#define RSI_VER_MAJOR_SHIFT 16
> +

The RSI version masks should have been as below:
+#define RSI_VER_MINOR_MASK   0x0000FFFFULL
+#define RSI_VER_MAJOR_MASK   0x7FFF0000ULL

I will fix these in the next revision.

Regards,

Sami Mujawar