On Mon, 12 Sep 2011 17:23:02 +0200 (CEST)
Francesco Gringoli <[email protected]> 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
[email protected]
http://lists.infradead.org/mailman/listinfo/b43-dev