Hi Dan, On Wed, 06 May 2015 16:04:59 -0400 Dan Williams wrote: > > diff --git a/include/asm-generic/pfn.h b/include/asm-generic/pfn.h > new file mode 100644 > index 000000000000..91171e0285d9 > --- /dev/null > +++ b/include/asm-generic/pfn.h > @@ -0,0 +1,51 @@ > +#ifndef __ASM_PFN_H > +#define __ASM_PFN_H > + > +#ifndef __pfn_to_phys > +#define __pfn_to_phys(pfn) ((dma_addr_t)(pfn) << PAGE_SHIFT) Why dma_addr_t and not phys_addr_t? i.e. it could use a comment if it is correct. -- Cheers, Stephen Rothwell sfr@canb.auug.org.au