Hello Hans, Would be great if you could at least get this into 5.2 : media: atmel: atmel-isc: make try_fmt error less verbose I noticed several sensor drivers act in a way that makes this message appear a lot, when the sensors actually work fine. They just return errors in try_fmt if the format is not supported, which is not a proper way to do it, but oh well, we have to cope with that in the caller. Applications like gstreamer will call this many times from something like an enum_fmt iteration, which will eventually work, but, the atmel_isc will keep printing faulty messages, which is not exactly something desired in this situation. (or if it's not possible now then maybe as -rc patch later on...) Thanks, Eugen On 02.05.2019 16:13, Hans Verkuil wrote: > This previous PR https://patchwork.linuxtv.org/patch/55818/ is now split > in two, one with patches for 5.2 and one with stuff that can go into 5.3. > > This PR contains the patches with new features for 5.3. > > Regards, > > Hans > > The following changes since commit 7afa8db323e37b9174cf78a1c9ab0ae7a9f5e7dd: > > media: vsp1: Add support for missing 16-bit RGB555 formats (2019-04-25 11:07:05 -0400) > > are available in the Git repository at: > > git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.3a > > for you to fetch changes up to 73d1351fb7d7dafd88d15848bfacfe9f16b13351: > > media: coda: use v4l2_m2m_buf_copy_metadata (2019-05-02 15:04:53 +0200) > > ---------------------------------------------------------------- > Tag branch > > ---------------------------------------------------------------- > Eugen Hristev (4): > media: atmel: atmel-isc: reworked white balance feature > media: v4l2-ctrl: fix flags for DO_WHITE_BALANCE > media: atmel: atmel-isc: add support for DO_WHITE_BALANCE > media: atmel: atmel-isc: make try_fmt error less verbose > > Philipp Zabel (12): > media: coda: move register debugging to coda_debug level 3 > media: coda: move job ready message to coda_debug level 2 > media: coda: add coda_frame_type_char helper > media: coda: improve decoder job finished debug message > media: coda: demote s_ctrl debug messages to level 2 > media: coda: add menu strings to s_ctrl debug output > media: coda: update profile and level controls after sequence initialization > media: coda: add decoder MPEG-4 profile and level controls > media: v4l2-ctrl: add MPEG-2 profile and level controls > media: coda: add decoder MPEG-2 profile and level controls > media: coda: add lockdep asserts > media: coda: use v4l2_m2m_buf_copy_metadata > > Documentation/media/uapi/v4l/ext-ctrls-codec.rst | 56 +++++++++++++++++++ > drivers/media/platform/atmel/atmel-isc-regs.h | 6 ++- > drivers/media/platform/atmel/atmel-isc.c | 263 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------- > drivers/media/platform/coda/Makefile | 2 +- > drivers/media/platform/coda/coda-bit.c | 69 +++++++++++++++++++----- > drivers/media/platform/coda/coda-common.c | 129 +++++++++++++++++++++++++++++++++----------- > drivers/media/platform/coda/coda-mpeg2.c | 44 +++++++++++++++ > drivers/media/platform/coda/coda-mpeg4.c | 48 +++++++++++++++++ > drivers/media/platform/coda/coda.h | 14 +++++ > drivers/media/platform/coda/coda_regs.h | 2 +- > drivers/media/v4l2-core/v4l2-ctrls.c | 24 +++++++++ > include/uapi/linux/v4l2-controls.h | 18 +++++++ > 12 files changed, 597 insertions(+), 78 deletions(-) > create mode 100644 drivers/media/platform/coda/coda-mpeg2.c > create mode 100644 drivers/media/platform/coda/coda-mpeg4.c >
