[linux-dvb] Re: [PATCH] LGDT3302 Frontend Driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



castet.matthieu@xxxxxxx wrote:

>Hi,
>
>Selon Michael Krufky <mkrufky@xxxxxxx>:
>
>>Johannes Stezenbach wrote:
>>
>>>Michael Krufky wrote:
>>>      
>>>
>>>>+#define dprintk(args...) \
>>>>+do { \
>>>>+if (debug) printk(KERN_DEBUG "lgdt3302: " args); \
>>>>+} while (0)
>>>>
>>>Interesting formatting...
>>>      
>>>
>Yes but why do you need do {} while(0) stuff ?
>You don't need to declare a new block as you add only a if statement, so
>  
>
>>>+#define dprintk(args...) if (debug) printk(KERN_DEBUG "lgdt3302: " args)
>>>      
>>>
>will be a lot's of cleanner...
>
>
>Matthieu
>  
>
Agreed... I will add make that change.

-- 
Michael Krufky




[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux