On 15.12.2022 20:32, Ayan Kumar Halder wrote: > As "io_size" and "uart->io_size" are both u64, so there will be no truncation. > Thus, one can remove the ASSERT() and extra assignment. > > 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. > > Signed-off-by: Ayan Kumar Halder <ayan.kumar.hal...@amd.com>
With the title improved to indicate what component is actually being touched Reviewed-by: Jan Beulich <jbeul...@suse.com> Jan