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 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 same thing b

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_dive

[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.