On 2015/5/7 23:51, Peter Maydell wrote: > On 7 May 2015 at 10:29, Shannon Zhao <zhaoshengl...@huawei.com> wrote: >> > From: Shannon Zhao <shannon.z...@linaro.org> >> > >> > Add aml_interrupt() for describing device interrupt in resource template. >> > These can be used to generating DSDT table for ACPI on ARM. >> > + /* Interrupt Number */ >> > + build_append_byte(var->buf, extract32(irq, 0, 8)); /* bits[7:0] */ >> > + build_append_byte(var->buf, extract32(irq, 8, 8)); /* bits[15:8] */ >> > + build_append_byte(var->buf, extract32(irq, 16, 8)); /* bits[23:16] */ >> > + build_append_byte(var->buf, extract32(irq, 24, 8)); /* bits[31:24] */ > You used this twice in the previous patch and again here. Can > we factor it out so we can > build_append_uint32(var->buf, irq); >
Ok, thanks for your suggestion, will fix this. -- Shannon