On Tuesday 21 February 2012 09:33:31 Tom Rini wrote: > On Mon, Feb 20, 2012 at 11:19:14PM -0500, Mike Frysinger wrote: > > On Monday 20 February 2012 16:53:40 Simon Glass wrote: > > > On Mon, Feb 20, 2012 at 12:07 PM, Tom Rini wrote: > > > > On Sun, Feb 19, 2012 at 02:15:30AM -0500, Mike Frysinger wrote: > > > >> On Saturday 18 February 2012 17:03:59 Simon Glass wrote: > > > >> > On Wed, Feb 15, 2012 at 5:57 AM, Aneesh V wrote: > > > >> > > -.globl reset_cpu > > > >> > > +.type reset_cpu, %function > > > >> > > +.global reset_cpu > > > >> > > > > >> > Should we introduce a macro to deal with this rather than writing > > > >> > it out each time? EXPORT()? > > > >> > > > >> we have it already with the linux/linkage.h header :) > > > > > > > > Well, unless my tree is out of date (or stuff is in-flight) we don't > > > > have the full compliment here. We have <linux/linkage.h> for all and > > > > <asm/linkage.h> for bfin. That said, yes, we should grab at least > > > > the ARM version and make use of ENTRY/END_FUNC ala the kernel. I'm > > > > behind on my "convert __attribute__((...)) to __attr" series already > > > > or I'd say more :) > > > > > > In case one of you is going to look at this, can we try to use > > > asm-generic as much as possible? > > > > i don't know what you mean ... we already have linux/linkage.h with > > sane defaults for pretty much everyone. the Blackfin asm/linkage.h is > > an empty file to satisfy building. > > Well, the kernel version for blackfin sets ALIGN/ALIGN_STR, so are they > out of sync?
the overall linkage.h concept is the same, but we've unified things better in the u-boot code. Linux's common linkage.h has x86-centric defaults which we specifically avoided in the u-boot version. we might want to tweak the ENDPROC() in u-boot's linkage.h to use % rather than @ since the latter is a comment char in arm asm and the former should work for all the arches i know of just the same as @. i expect the arm guys to submit a patch though at the same time they add a stub asm/linkage.h ;). -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot