Re: [Xen-devel] [XEN PATCH v3 1/2] xen/arm: Rename all early printk macro

[Julien Grall <julien@xxxxxxx>]

Hi,

On 11/03/2020 14:46, Anthony PERARD wrote:
> On Wed, Mar 11, 2020 at 01:57:37PM +0000, Julien Grall wrote:
> > Hi Anthony,
> >
> > On 09/03/2020 17:45, Anthony PERARD wrote:
> > > diff --git a/xen/arch/arm/arm32/head.S b/xen/arch/arm/arm32/head.S
> > > index e9d356f05c2b..2b593c5ef99a 100644
> > > --- a/xen/arch/arm/arm32/head.S
> > > +++ b/xen/arch/arm/arm32/head.S
> > > @@ -36,8 +36,8 @@
> > >    #define XEN_FIRST_SLOT      first_table_offset(XEN_VIRT_START)
> > >    #define XEN_SECOND_SLOT     second_table_offset(XEN_VIRT_START)
> > > -#if (defined (CONFIG_EARLY_PRINTK)) && (defined (EARLY_PRINTK_INC))
> > > -#include EARLY_PRINTK_INC
> > > +#if (defined (CONFIG_EARLY_PRINTK)) && (defined (CONFIG_EARLY_PRINTK_INC))
> > NIT: I would also take the opportunity to clean-up the line by remove the
> > extra () and the space before (. Something like:
> >
> > #if define(CONFIG_EARLY_PRINTK) && defined(CONFIG_EARLY_PRINTK_INC)
> >
> > [...]
> >
> > > diff --git a/xen/arch/arm/arm64/head.S b/xen/arch/arm/arm64/head.S
> > > index e5015f93a2d8..4d45ea3dac3c 100644
> > > --- a/xen/arch/arm/arm64/head.S
> > > +++ b/xen/arch/arm/arm64/head.S
> > > @@ -45,8 +45,8 @@
> > >    #define __HEAD_FLAGS            ((__HEAD_FLAG_PAGE_SIZE << 1) | \
> > >                                     (__HEAD_FLAG_PHYS_BASE << 3))
> > > -#if (defined (CONFIG_EARLY_PRINTK)) && (defined (EARLY_PRINTK_INC))
> > > -#include EARLY_PRINTK_INC
> > > +#if (defined (CONFIG_EARLY_PRINTK)) && (defined (CONFIG_EARLY_PRINTK_INC))
> > Same here.
> Those clean-up sounds good.
I will give a couple of days so Stefano can have an opportunity to 
comment. If I don't hear anything by Monday, I will commit it.
> > I am happy to fix both cases on commit:
> >
> > Acked-by: Julien Grall <jgrall@xxxxxxxxxx>
> Thanks,
Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel