On 09/23/2018 06:34 PM, Akinobu Mita wrote:
> This patchset adds support for changing frame interval and runtime PM for
> video-i2c driver. This also adds an helper macro to v4l2 common
> internal API that is used to to find a suitable frame interval.
>
> There are a couple of unrelated changes that are included for simplifying
> driver initialization code and register accesses.
>
> * v2
> - Add Acked-by and Reviewed-by tags
> - Update commit log to clarify the use after free
> - Add thermistor and termperature register address definisions
> - Stop adding v4l2_find_closest_fract() in v4l2 common internal API
> - Add V4L2_FRACT_COMPARE() macro in v4l2 common internal API
> - Use V4L2_FRACT_COMPARE() to find suitable frame interval instead of
> v4l2_find_closest_fract()
1) What's wrong with v4l2_find_closest_fract()?
2) Please test this with the latest v4l2-compliance: I recently improved
the S_PARM checks, so I want to make sure this driver passes those tests.
Regards,
Hans
> - Add comment for register address definisions
>
> Akinobu Mita (6):
> media: video-i2c: avoid accessing released memory area when removing
> driver
> media: video-i2c: use i2c regmap
> media: v4l2-common: add V4L2_FRACT_COMPARE
> media: vivid: use V4L2_FRACT_COMPARE
> media: video-i2c: support changing frame interval
> media: video-i2c: support runtime PM
>
> drivers/media/i2c/video-i2c.c | 286
> ++++++++++++++++++++++-----
> drivers/media/platform/vivid/vivid-vid-cap.c | 9 +-
> include/media/v4l2-common.h | 5 +
> 3 files changed, 247 insertions(+), 53 deletions(-)
>
> Cc: Matt Ranostay <matt.ranos...@konsulko.com>
> Cc: Sakari Ailus <sakari.ai...@linux.intel.com>
> Cc: Hans Verkuil <hansv...@cisco.com>
> Cc: Mauro Carvalho Chehab <mche...@kernel.org>
>
