Em Thu, 8 Mar 2018 07:39:09 -0300
Mauro Carvalho Chehab <mche...@kernel.org> 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

Reply via email to