Hi Vineet, On Thu, 2016-12-22 at 16:25 -0800, Vineet Gupta wrote: > On 12/22/2016 06:09 AM, Alexey Brodkin wrote: > > diff --git a/arch/arc/include/asm/linkage.h b/arch/arc/include/asm/linkage.h > > index b29f1a9fd6f7..3a5f13d65ee1 100644 > > --- a/arch/arc/include/asm/linkage.h > > +++ b/arch/arc/include/asm/linkage.h > > @@ -28,7 +28,7 @@ > > ?/* annotation for data we want in DCCM - if enabled in .config */ > > ?.macro ARCFP_CODE > > ?#ifdef CONFIG_ARC_HAS_ICCM > > - .section .text.arcfp, "ax", at progbits > > + .section .text..arcfp, "ax", at progbits > > Why not turn this around and call it arcfp.text and arcfp.data etc - just like > done for other special sections such as sched, cpuidle, lock... Just so we are > consistent with normal kernel convention ! Right that looks much better! Will do so in v2. -Alexey
