On 7/30/26 6:55 PM, Alison Schofield wrote:
> A committed zero-sized HDM decoder reports size 0, so 'cxl list'
> renders it identically to an ordinary unconfigured decoder ("state":
> "disabled"). The locked attribute that distinguishes firmware-committed
> decoders is not included in the JSON output.
>
> The library already reads the decoder's locked attribute and exports
> cxl_decoder_is_locked(); it was simply never emitted. Add a "locked"
> field to the decoder JSON, following the emit-only-when-true idiom used
> by the capability flags so existing output is unchanged for the common
> unlocked case.
>
> Signed-off-by: Alison Schofield <[email protected]>
Reviewed-by: Dave Jiang <[email protected]>
> ---
> cxl/json.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/cxl/json.c b/cxl/json.c
> index a92548828582..2214f7027d91 100644
> --- a/cxl/json.c
> +++ b/cxl/json.c
> @@ -870,6 +870,12 @@ struct json_object *util_cxl_decoder_to_json(struct
> cxl_decoder *decoder,
> json_object_object_add(jdecoder, "state", jobj);
> }
>
> + if (cxl_decoder_is_locked(decoder)) {
> + jobj = json_object_new_boolean(true);
> + if (jobj)
> + json_object_object_add(jdecoder, "locked", jobj);
> + }
> +
> if (cxl_port_is_endpoint(port)) {
> enum cxl_decoder_mode mode = cxl_decoder_get_mode(decoder);
>
