Em Thu, 8 Mar 2018 07:39:09 -0300 Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> escreveu: > Also, if this function starts returning NULL, I suspect that you also > need to change ov5640_s_frame_interval(), as currently it is called > at ov5640_s_frame_interval() as: > > sensor->current_mode = ov5640_find_mode(sensor, frame_rate, mode->width, > mode->height, true); > > without checking if the returned value is NULL. Setting > current_mode to NULL can cause oopses at ov5640_set_mode(). Actually, as ov5640_s_frame_interval() calls ov5640_try_fmt_internal() first. So, this should never happen. Thanks, Mauro