>-----Original Message-----
>From: Jonathan Cameron <[email protected]>
>Sent: 08 August 2025 14:29
>To: Shiju Jose <[email protected]>
>Cc: [email protected]; [email protected]; [email protected];
>[email protected]; Linuxarm <[email protected]>
>Subject: Re: [PATCH v5 1/7] hw/cxl/events: Update for rev3.2 common event
>record format
>
>On Thu, 7 Aug 2025 16:43:40 +0100
><[email protected]> wrote:
>
>> From: Shiju Jose <[email protected]>
>>
>> CXL spec 3.2 section 8.2.9.2.1 Table 8-55, Common Event Record format
>> has updated with optional Maintenance Operation Subclass, LD ID and ID
>> of the device head information.
>>
>> Add updates for the above optional parameters in the related CXL
>> events reporting and in the QMP commands to inject CXL events.
>>
>> In qapi/cxl.json added fields of Common Event Record to a common
>> structure CXLCommonEventBase.
>>
>> Signed-off-by: Jonathan Cameron <[email protected]>
>Hi Shiju,
>
>Sign off chain doesn't make sense. Drop my SoB. I guess this is because it
>went
>through my tree at some stage and hence was tagged by me. Given it's your
>patch, you should drop that from new versions.
Hi Jonathan,
Thanks. I will do.
>
>Jonathan
>
>
>> Signed-off-by: Shiju Jose <[email protected]>
Thanks,
Shiju
