Johannes Stezenbach wrote:
> thomas schorpp wrote:
>
>>signed-off-by: t.schorpp@xxxxxx
>
>
> Thanks. I haven't had time to test this yet. Just a few comments:
>
>
>
>>@@ -212,11 +228,17 @@
>> }
>>
>> if (0 != av7110->current_input) {
>>+
>
>
> This adds trailing whitespace. (also in a couple of other places)
>
>
>>+ dprintk(1, "switching to analog TV: \n");
>
>
> printk with trailing space before the \n is also not cool
>
...
ok ok, ill read the kernel coding style documents.. i was in hurry.
y
tom
