On Tue, Nov 10, 2015 at 11:42 AM, Chen, Yu C <yu.c.chen@xxxxxxxxx> wrote: > Hi, Lv > Sorry for my late feedback on the patch, one minor nit below: >> + start_pfn = page_to_pfn(virt_to_page(pointer)); >> + end_pfn = page_to_pfn(virt_to_page(pointer + length)); > Convert pointer to char* first? > end_pfn = page_to_pfn(virt_to_page((char*)pointer + length)); Why do you need that? void * + size_t = char * + size_t -- With Best Regards, Andy Shevchenko -- 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