[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] Elo touchpad 10 bytes emulator v2
From: |
Juan Quintela |
Subject: |
[Qemu-devel] Re: [PATCH] Elo touchpad 10 bytes emulator v2 |
Date: |
Tue, 30 Mar 2010 12:24:19 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Ricardo Ribalda Delgado <address@hidden> wrote:
> Hello Juan
>
> Thanks for your comments. About the indentation error... Do you have
> some kind of auto indent script(like the kernel code has). It is
> making me crazy trying to collaborate with a lot of projects an all of
> them with different styles.
Don't even start this discussion yet again :)
Short answer: we don't have.
Long answer: any indenter that would pass qemu code style will be turing
complete (at least) and possibly will also pass the Turing test.
>>> +#include <stdlib.h>
>>> +#include "../qemu-common.h"
>>> +#include "../qemu-char.h"
>>> +#include "../console.h"
>>
>> You can remove the "../" from those, Makefile sets correct include paths
>> for this to work.
>
>
> Ok. I used the mssmouse.c as reference. I can change that. (I guess
> that I should also replace "" with <>)
Everything uses "". I just looked when reviewing this patch at msmouse,
and that one could also take some cleanup. This happens a lot in qemu,
you search for another driver for inspiration, and Murphy gets just the
one with the wrong examples.
>> Why does the lenght of the FIFO changes here? I think this change in
>> independent of the rest of the patch (no knowledge about the 16550A to
>> know if it should be 16 or 32).
>
> I have to send a 2x10 bytes package, and it does not fit the the 16
> bytes buffer.... Any other suggestion about how to do it?
Nope, I am not a 16550A guru at all. No sure if your change will break
anything else or no, that is why I asked.
>>
>> Later, Juan.
>>
>
> Thanks again for your comments
You are welcome.
Later, Juan.