Hi Hoan, >> -static int cpc_read(struct cpc_reg *reg, u64 *val) >> +static int cpc_read(struct cpc_register_resource *reg_res, u64 *val) >> { >> int ret_val = 0; >> + void __iomem *vaddr = 0; >> + struct cpc_reg *reg = ®_res->cpc_entry.reg; >> + >> + if (reg_res->type == ACPI_TYPE_INTEGER) { >> + *val = reg_res->int_value; > It should be "*val = reg_res->cpc_entry.int_value;", isn't it ? > Yes, that's correct. I will fix it in the next version. Thanks! Interestingly, I don't think I saw an error or warning during compile:) Thanks, Prashanth -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html