Re: [PATCH v2 2/3] hw/cxl: Add utility functions decoder interleave ways and target count.

["Philippe Mathieu-Daudé" <philmd@linaro.org>]

On 7/9/23 13:35, Jonathan Cameron wrote:
> As an encoded version of these key configuration parameters is available
> in a register, provide functions to extract it again so as to avoid
> the need for duplicating the storage.
> 
> Whilst here update the _enc() function to include additional values
> as defined in the CXL 3.0 specification. Whilst they are not
> currently used in the emulation, they may be in future and it is
> easier to compare with the specification if all values are covered.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
> v2: Thanks to Philippe Mathieu-Daudé
>   - Expand both enc() and dec() functions to include full set of values
>     defined in CXL r3.0
>   - Pushed implementation down into the .c file.
> ---
>   include/hw/cxl/cxl_component.h |  2 ++
>   hw/cxl/cxl-component-utils.c   | 59 ++++++++++++++++++++++++++++++----
>   2 files changed, 55 insertions(+), 6 deletions(-)

Hoping the values match the "CXL r3.0 Section 8.2.4.19.1 CXL
HDM Decoder Capability Register":

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>