On Mon, 12 Sep 2011 17:23:02 +0200 (CEST) Francesco Gringoli <francesco.gring...@ing.unibs.it> wrote:
> On Sep 12, 2011, at 11:15 AM, Michael Büsch wrote: > > > switch (bin->opcode) { > >+ case 0x101: > >+ stmt->u.insn.name = "mul"; > >+ disasm_std_operand(stmt, 0, 0); > >+ disasm_std_operand(stmt, 1, 1); > >+ disasm_std_operand(stmt, 2, 2); > >+ break; > > > >There's some whitespace damage here. > > New patch (non-damaged) including changes to b43-asm/test.asm The patch is still damaged. It contains an extra space at the start of the context lines. Also, please make sure the patch can be applied with "git am", which means that it can be applied in the repository root directory with -p1 > =================================================================== > --- assembler/main.c 2011-08-21 14:16:33.000000000 +0200 > +++ assembler/main.c.orig 2011-09-12 16:47:26.000000000 +0200 ^^^ Need one more directory level here. If in doubt, use "git diff". > @@ -737,6 +737,9 @@ > unsigned int opcode; > > switch (insn->op) { > + case OP_MUL: > + do_assemble_insn(ctx, insn, 0x101); > + break; > case OP_ADD: > do_assemble_insn(ctx, insn, 0x1C0); > break; ^ extra whitespace here -- Greetings, Michael. _______________________________________________ b43-dev mailing list b43-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/b43-dev