On 09/13/2010 10:18 PM, matt mooney wrote: > Unnecessary cast from void* in assignment. Not very keen on this. The cast may not be strictly required, but it does document what's going on there. J > Signed-off-by: matt mooney <mfm@xxxxxxxxxxxxx> > --- > arch/x86/xen/mmu.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/x86/xen/mmu.c b/arch/x86/xen/mmu.c > index 42086ac..7436283 100644 > --- a/arch/x86/xen/mmu.c > +++ b/arch/x86/xen/mmu.c > @@ -737,7 +737,7 @@ pgd_t *xen_get_user_pgd(pgd_t *pgd) > > if (offset < pgd_index(USER_LIMIT)) { > struct page *page = virt_to_page(pgd_page); > - user_ptr = (pgd_t *)page->private; > + user_ptr = page->private; > if (user_ptr) > user_ptr += offset; > } -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html