dtc: .quad asm directive generation
david at gibson.dropbear.id.au
Thu Mar 16 13:05:00 EST 2006
On Wed, Mar 15, 2006 at 06:59:24PM -0700, Mark A. Greer wrote:
> On Thu, Mar 16, 2006 at 11:49:48AM +1100, David Gibson wrote:
> > On Wed, Mar 15, 2006 at 05:00:59PM -0700, Mark A. Greer wrote:
> > > On Tue, Mar 14, 2006 at 06:06:57PM -0600, Jon Loeliger wrote:
> > > >
> > > > For 9 months nothing. Dave abdicates his DTC maintainer role to me.
> > >
> > > Jon,
> > >
> > > I can't find dtc under ~dgibson on ozlabs anymore. Do you have it
> > > somewhere now?
> > Ah, bother. I did arrange for one of the old addresses of the tree to
> > keep working, but forgot that it wasn't the one which had been widely
> > announced. Just a minute, I'll send an announcement to the lists.
> Okay, thanks guys.
> Here is a patch that fixes a problem I'm having with asm output. My
> host system is a x86 box so its 32-bit, little endian.
> Disclaimer: Everything below is AFAICT.
> The problem is that asm_emit_cell() was swapping its asm output when
> it shouldn't be (because the assembler will do the necessary swapping).
> The cell values (asm_emit_cell()) are different from the data values
> (asm_emit_data()) because the cell values are generated within the
> program and don't get swapped like the data values read from the dts file.
> They should be left as they are so that the assembler will swap them,
> if necessary. For example, when the property length field was 4,
> the asm output contained ".long 0x4000000" and sent the kernel prom.c
> dt parsing code into the weeds.
> The dtb output is correct.
> Did any of that make sense?
Ah, yes, that be32_to_cpu() is, indeed, complete crap.
> /me knows he's rambling...
> Anyway, here is a simple patch the fixes it for me (i.e., the
> cross-assembled asm output matches the dtb).
> diff -Nurp dtc/flattree.c dtc_new/flattree.c
> --- dtc/flattree.c 2006-02-24 10:57:56.000000000 -0700
> +++ dtc_new/flattree.c 2006-03-15 18:02:07.000000000 -0700
> @@ -121,7 +121,7 @@ static void asm_emit_cell(void *e, cell_
> FILE *f = e;
> - fprintf(f, "\t.long\t0x%x\n", be32_to_cpu(val));
> + fprintf(f, "\t.long\t0x%x\n", val);
> static void asm_emit_string(void *e, char *str, int len)
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
More information about the Linuxppc-dev