Re: [XEN v3 1/9] xen/ns16550: Remove unneeded truncation check in the DT init code

[Julien Grall <julien@xxxxxxx>]

Hi Ayan

On 08/02/2023 12:05, Ayan Kumar Halder wrote:
> In an earlier commit (7c1de0038895), "ns16550.io_size" was u32 and
> "io_size" was u64. Thus, the ASSERT() was needed to check if the values
> are the same.
> However, in a later commit (c9f8e0aee507), "ns16550.io_size" was changed
> to u64. Thus, the ASSERT() became redundant.
>
> So, now as "io_size" and "uart->io_size" are both u64, there will be no
> truncation. Thus, one can remove the ASSERT() and extra assignment.
>
> Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>
> Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
Acked-by: Julien Grall <jgrall@xxxxxxxxxx>

Cheers,

--
Julien Grall