Hi Sunil, Thank you for the review.
> +EFI_STATUS > +EFIAPI > +FdtGetIntcAddressCells ( > + IN CONST VOID *Fdt, > + IN INT32 Node, > + OUT INT32 *AddressCells, OPTIONAL > + OUT INT32 *SizeCells OPTIONAL NIT: I might be wrong but alignment doesn't look correct. [SAMI] I wonder if this is an artifact of the email client. The alignment looks ok to me in visual studio code. That being said, I may have run uncrustify on Pierre's patch series as I am planning to get it merged soon. In either case I will get it addressed, thank you for the feedback. [/SAMI] Otherwise, LGTM. Reviewed-by: Sunil V L <suni...@ventanamicro.com <mailto:suni...@ventanamicro.com>> Thanks! Sunil -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#119772): https://edk2.groups.io/g/devel/message/119772 Mute This Topic: https://groups.io/mt/106770185/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
