Re: [PATCH v3 3/6] xen/arm: mpu: Define Xen start address for MPU systems

[Julien Grall <julien@xxxxxxx>]

On 15/10/2024 17:24, Ayan Kumar Halder wrote:
> Hi,
Hi Ayan & Luca,

> On 14/10/2024 19:53, Luca Fancellu wrote:
> > + Frediano for suggestion about header protection define name
> >
> > > +++ b/xen/arch/arm/include/asm/mpu/layout.h
> > > @@ -0,0 +1,33 @@
> > > +/* SPDX-License-Identifier: GPL-2.0-only */
> > > +
> > > +#ifndef __ARM_MPU_LAYOUT_H__
> > > +#define __ARM_MPU_LAYOUT_H__
> > I think we are moving away from this notation:
> > https://patchwork.kernel.org/project/xen-devel/patch/20241004081713.749031-6-frediano.ziglio@xxxxxxxxx/
> > Shall this be ASM___ARM__MPU__LAYOUT_H ? @Frediano
> I could see that the patch hasn't been Rb yet and as per 
> https://patchwork.kernel.org/project/xen-devel/cover/20241004081713.749031-1-frediano.ziglio@xxxxxxxxx/ , the new coding style is still under discussion.
> May be the Arm maintainers (Stefano, Julien, Bertrand, Michal) can 
> provide their opinion on this.
The current guideline is been debating within among the committers. For 
now, please ignore the guideline.
Cheers,

--
Julien Grall