Hi, Thank you for the review! >> +required: >> + - compatible >> + - reg >> + - interrupt-controller >> + - "#interrupt-cells" >> + - msi-controller > > From the implementation of the driver, it looks like all properties are > required, except for brcm,msi-offset which has a fallback to the value 0. Yes, correct. Will update in next revision. ~Stan
