Em Mon, 9 Jan 2017 14:23:33 +0100 Hans Verkuil <hverkuil@xxxxxxxxx> escreveu: > See the v4 series for details: > > https://www.spinics.net/lists/linux-media/msg108737.html > > Regards, > > Hans > > The following changes since commit 40eca140c404505c09773d1c6685d818cb55ab1a: > > [media] mn88473: add DVB-T2 PLP support (2016-12-27 14:00:15 -0200) > > are available in the git repository at: > > git://linuxtv.org/hverkuil/media_tree.git delta > > for you to fetch changes up to e6f199d01e7b8bc4436738b6c666fda31b9f3340: > > st-delta: debug: trace stream/frame information & summary (2017-01-09 14:16:45 +0100) > > ---------------------------------------------------------------- > Hugues Fruchet (10): > Documentation: DT: add bindings for ST DELTA > ARM: dts: STiH410: add DELTA dt node > ARM: multi_v7_defconfig: enable STMicroelectronics DELTA Support > MAINTAINERS: add st-delta driver > st-delta: STiH4xx multi-format video decoder v4l2 driver > st-delta: add memory allocator helper functions > st-delta: rpmsg ipc support > st-delta: EOS (End Of Stream) support > st-delta: add mjpeg support > st-delta: debug: trace stream/frame information & summary There is something wrong on this driver... even after applying all patches, it complains that there's a for there that does nothing: drivers/media/platform/sti/delta/delta-v4l2.c:322 register_decoders() warn: we never enter this loop drivers/media/platform/sti/delta/delta-v4l2.c: In function 'register_decoders': drivers/media/platform/sti/delta/delta-v4l2.c:322:16: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits] for (i = 0; i < ARRAY_SIZE(delta_decoders); i++) { ^ On a first glance, it seems that the register_decoders() function is reponsible to register the format decoders that the hardware recognizes. If so, I suspect that this driver is deadly broken. Please be sure that the upstream driver works properly before submitting it upstream. Also, please fix the comments to match the Kernel standard. E. g. instead of: /* guard output frame count: * - at least 1 frame needed for display * - at worst 21 * ( max h264 dpb (16) + * decoding peak smoothing (2) + * user display pipeline (3) ) */ It should be: /* * guard output frame count: * - at least 1 frame needed for display * - at worst 21 * ( max h264 dpb (16) + * decoding peak smoothing (2) + * user display pipeline (3) ) */ There are several similar occurrences among this patch series. Thanks, Mauro Thanks, Mauro -- 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
