[U-Boot] [PATCH 3/9] openrisc: Add cpu files

Mike Frysinger vapier at gentoo.org
Tue Nov 22 21:54:03 CET 2011


On Monday 21 November 2011 23:46:41 Marek Vasut wrote:
> > On Mon, Nov 21, 2011 at 11:50:17PM +0100, Marek Vasut wrote:
> > > > +	l.addi	r3,r3,4
> > > > +	l.sfltu	r4,r5
> > > > +	l.bf	.L_reloc
> > > > +	 l.addi	r4,r4,4			/* delay slot */
> > > 
> > > The formating here doesn't seem right?
> > 
> > The extra space there is to denote that the instruction is
> > in a delay slot.
> > I think it brings more readability to the code, but if it
> > meets too much resistance I'll remove them.
> 
> You already have comment there, but let's see what the others think.

i think it's fine.  delay slots are a pita enough to deal with, so slight style 
tweaks that make this easier are a good thing.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20111122/f8fa8d17/attachment.pgp>


More information about the U-Boot mailing list