Hi Sylwester, On Sun, Aug 4, 2013 at 8:33 PM, Sylwester Nawrocki <sylvester.nawrocki@xxxxxxxxx> wrote: > Hi Arun, > > > On 08/02/2013 05:02 PM, Arun Kumar K wrote: >> >> The hardware interface module finally sends the commands to the >> FIMC-IS firmware and runs the interrupt handler for getting the >> responses. >> >> Signed-off-by: Arun Kumar K<arun.kk@xxxxxxxxxxx> >> Signed-off-by: Kilyeon Im<kilyeon.im@xxxxxxxxxxx> >> --- [snip] >> +static int itf_get_state(struct fimc_is_interface *itf, >> + unsigned long state) >> +{ >> + int ret = 0; >> + unsigned long flags; >> + >> + spin_lock_irqsave(&itf->slock_state, flags); >> + ret = test_bit(state,&itf->state); > > > Shouldn't it be __test_bit() ? > __test_bit() is not availble ! In file include/asm-generic/bitops/non-atomic.h, all other ops are prefixed with __xxx(), but its just test_bit(). Regards Arun -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html