Re: [patch] [media] radio-si476x: check different function pointers

2013-04-11 Thread Hans Verkuil
On Wed April 10 2013 13:40:51 Dan Carpenter wrote: This is a static checker where it complains if we check for one function pointer and then call a different function on the next line. In most cases, the code does the same thing before and after this patch. For example, when -phase_diversity

Re: [patch] [media] radio-si476x: check different function pointers

2013-04-11 Thread Andrey Smirnov
On Thu, Apr 11, 2013 at 11:24 AM, Hans Verkuil hverk...@xs4all.nl wrote: On Wed April 10 2013 13:40:51 Dan Carpenter wrote: This is a static checker where it complains if we check for one function pointer and then call a different function on the next line. In most cases, the code does the

[patch] [media] radio-si476x: check different function pointers

2013-04-10 Thread Dan Carpenter
This is a static checker where it complains if we check for one function pointer and then call a different function on the next line. In most cases, the code does the same thing before and after this patch. For example, when -phase_diversity is non-NULL then -phase_div_status is also non-NULL.