Re: [PATCH v16 07/18] cxl: Share HDM decoder register unpacking

[Srirangan Madhavan]

On 10/29/26 11:21 PM, Richard Cheng wrote:
> > +		end = base - 1;
> > +
> What about making "end = base - 1" the default value of "end" and then call
> "check_add_overflow()", this would preserve the same behavior while elimate one branch.
>
> Just I think we can add some comments describing it's for the case of zero-size HDM.
>
> Best regards,
> Richard Cheng.


Thanks, I applied this in v17. The end defaults to base - 1, with a
comment that it covers a zero-size HDM decoder, and check_add_overflow()
now runs only for a non-zero size. I kept your Reviewed-by.

--
Regards,
Srirangan