[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH V3] xen/arm: UART addresses are not always aligned to a page.
On Thu, 2013-10-31 at 19:52 +0000, Ian Campbell wrote: > On Thu, 2013-10-31 at 15:37 -0400, Jon Fraser wrote: > > diff --git a/xen/include/asm-arm/early_printk.h > > b/xen/include/asm-arm/early_printk.h > > index 5f7c529..707bbf7 100644 > > --- a/xen/include/asm-arm/early_printk.h > > +++ b/xen/include/asm-arm/early_printk.h > > @@ -14,6 +14,16 @@ > > > > #ifdef EARLY_PRINTK > > > > +/* need to add the uart address offset in page to the fixmap address */ > > +#define EARLY_UART_VIRTUAL_ADDRESS \ > > + (FIXMAP_ADDR(FIXMAP_CONSOLE) +(EARLY_UART_BASE_ADDRESS & ~PAGE_MASK)) > > + > > +#endif > > + > > +#ifndef __ASSEMBLY__ > > + > > +#ifdef EARLY_PRINTK > > + > > Is there something wrong with: > #ifdef EARLY_PRINT > > +#define EARLY_UART_VIETRUAL_ADDRESS ... > + > +#ifndef __ASSEMBLY__ > + > [... existing C code...] > + > +#endif /* !__ASSEMBLY__ */ > + > #endif /* EARLY_PRINTK */ > > Rather than two lots of early_printk ifdefery. > > Ian. > The dummy functions for early_printk and early_panic need to be visible for not EARLY_PRINTK, but not visible for __ASSEMBLY__. _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |