Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver
On 11/14/2016 03:59 PM, Stanimir Varbanov wrote: > Hi, > > On 11/11/2016 02:11 PM, Javier Martinez Canillas wrote: >> Hello Hans, >> >> On Fri, Nov 11, 2016 at 8:49 AM, Hans Verkuilwrote: >>> Hi Stanimir, >>> >>> Overall it looks good. As you saw, I do have some comments but nothing >>> major. >>> >>> One question: you use qcom as the directory name. How about using qualcomm? >>> >>> It's really not that much longer and a bit more obvious. >>> >>> Up to you, though. >>> >> >> It seems qcom is more consistent to the name used in most subsystems >> for Qualcomm: >> >> $ find -name *qcom >> ./arch/arm/mach-qcom >> ./arch/arm64/boot/dts/qcom >> ./Documentation/devicetree/bindings/soc/qcom >> ./sound/soc/qcom >> ./drivers/pinctrl/qcom >> ./drivers/soc/qcom >> ./drivers/clk/qcom >> >> $ find -name *qualcomm >> ./drivers/net/ethernet/qualcomm > > Also qcom is the vendor prefix used in [1] > > [1] Documentation/devicetree/bindings/vendor-prefixes.txt > qcom it is, then :-) Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver
Hi, On 11/11/2016 02:11 PM, Javier Martinez Canillas wrote: > Hello Hans, > > On Fri, Nov 11, 2016 at 8:49 AM, Hans Verkuilwrote: >> Hi Stanimir, >> >> Overall it looks good. As you saw, I do have some comments but nothing major. >> >> One question: you use qcom as the directory name. How about using qualcomm? >> >> It's really not that much longer and a bit more obvious. >> >> Up to you, though. >> > > It seems qcom is more consistent to the name used in most subsystems > for Qualcomm: > > $ find -name *qcom > ./arch/arm/mach-qcom > ./arch/arm64/boot/dts/qcom > ./Documentation/devicetree/bindings/soc/qcom > ./sound/soc/qcom > ./drivers/pinctrl/qcom > ./drivers/soc/qcom > ./drivers/clk/qcom > > $ find -name *qualcomm > ./drivers/net/ethernet/qualcomm Also qcom is the vendor prefix used in [1] [1] Documentation/devicetree/bindings/vendor-prefixes.txt -- regards, Stan -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver
Hello Hans, On Fri, Nov 11, 2016 at 8:49 AM, Hans Verkuilwrote: > Hi Stanimir, > > Overall it looks good. As you saw, I do have some comments but nothing major. > > One question: you use qcom as the directory name. How about using qualcomm? > > It's really not that much longer and a bit more obvious. > > Up to you, though. > It seems qcom is more consistent to the name used in most subsystems for Qualcomm: $ find -name *qcom ./arch/arm/mach-qcom ./arch/arm64/boot/dts/qcom ./Documentation/devicetree/bindings/soc/qcom ./sound/soc/qcom ./drivers/pinctrl/qcom ./drivers/soc/qcom ./drivers/clk/qcom $ find -name *qualcomm ./drivers/net/ethernet/qualcomm > Regards, > > Hans > Best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver
Hi Stanimir, Overall it looks good. As you saw, I do have some comments but nothing major. One question: you use qcom as the directory name. How about using qualcomm? It's really not that much longer and a bit more obvious. Up to you, though. Regards, Hans On 11/07/2016 06:33 PM, Stanimir Varbanov wrote: > Hi, > > Here is v3 of the Venus v4l2 video encoder/decoder driver. > > The changes since v2 are: > - return queued buffers on stream_on error. > - changed name of the driver vidc -> venus and reflect that in > querycap. > - fix video_device::release to point to video_device_release. > - tried to implement correctly g_selection for decoder and encoder. > - added Venus HFI 3xx basic support, to able to reuse driver on > msm8996. > - extend DT binding with reg-names and interrupt-names. > - parse DT IRQ and MEM resources by name. > - merge hfi_core,hfi_inst in venus_core and venus_inst structures. > - killed hfi_pkt_ops struct and use functions. > - various cleanups. > > The output of v4l2-compliance looks like: > > root@dragonboard-410c:/home/linaro# ./v4l2-compliance -d /dev/video0 > v4l2-compliance SHA : 405f0c21e0b52836d22c999aa4ee1f51d87998b2 > > Driver Info: > Driver name : qcom-venus > Card type : Qualcomm Venus video decoder > Bus info : platform:qcom-venus > Driver version: 4.4.23 > Capabilities : 0x84204000 > Video Memory-to-Memory Multiplanar > Streaming > Extended Pix Format > Device Capabilities > Device Caps : 0x04204000 > Video Memory-to-Memory Multiplanar > Streaming > Extended Pix Format > > Compliance test for device /dev/video0 (not using libv4l2): > > Required ioctls: > test VIDIOC_QUERYCAP: OK > > Allow for multiple opens: > test second video open: OK > test VIDIOC_QUERYCAP: OK > test VIDIOC_G/S_PRIORITY: OK > test for unlimited opens: OK > > Debug ioctls: > test VIDIOC_DBG_G/S_REGISTER: OK (Not Supported) > test VIDIOC_LOG_STATUS: OK (Not Supported) > > Input ioctls: > test VIDIOC_G/S_TUNER/ENUM_FREQ_BANDS: OK (Not Supported) > test VIDIOC_G/S_FREQUENCY: OK (Not Supported) > test VIDIOC_S_HW_FREQ_SEEK: OK (Not Supported) > test VIDIOC_ENUMAUDIO: OK (Not Supported) > test VIDIOC_G/S/ENUMINPUT: OK (Not Supported) > test VIDIOC_G/S_AUDIO: OK (Not Supported) > Inputs: 0 Audio Inputs: 0 Tuners: 0 > > Output ioctls: > test VIDIOC_G/S_MODULATOR: OK (Not Supported) > test VIDIOC_G/S_FREQUENCY: OK (Not Supported) > test VIDIOC_ENUMAUDOUT: OK (Not Supported) > test VIDIOC_G/S/ENUMOUTPUT: OK (Not Supported) > test VIDIOC_G/S_AUDOUT: OK (Not Supported) > Outputs: 0 Audio Outputs: 0 Modulators: 0 > > Input/Output configuration ioctls: > test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported) > test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported) > test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported) > test VIDIOC_G/S_EDID: OK (Not Supported) > > Control ioctls: > test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK > test VIDIOC_QUERYCTRL: OK > test VIDIOC_G/S_CTRL: OK > test VIDIOC_G/S/TRY_EXT_CTRLS: OK > test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK > test VIDIOC_G/S_JPEGCOMP: OK (Not Supported) > Standard Controls: 7 Private Controls: 0 > > Format ioctls: > test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK > test VIDIOC_G/S_PARM: OK (Not Supported) > test VIDIOC_G_FBUF: OK (Not Supported) > test VIDIOC_G_FMT: OK > test VIDIOC_TRY_FMT: OK > test VIDIOC_S_FMT: OK > test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported) > test Cropping: OK > test Composing: OK > test Scaling: OK > > Codec ioctls: > test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported) > test VIDIOC_G_ENC_INDEX: OK (Not Supported) > test VIDIOC_(TRY_)DECODER_CMD: OK (Not Supported) > > Buffer ioctls: > test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK > test VIDIOC_EXPBUF: OK > > Test input 0: > > > Total: 43, Succeeded: 43, Failed: 0, Warnings: 0 > > root@dragonboard-410c:/home/linaro# ./v4l2-compliance -d /dev/video1 > v4l2-compliance SHA : 405f0c21e0b52836d22c999aa4ee1f51d87998b2 > > Driver Info: > Driver name : vidc > Card type : video encoder > Bus info : platform:vidc > Driver version: 4.4.23 > Capabilities : 0x84204000 > Video Memory-to-Memory Multiplanar >
[PATCH v3 0/9] Qualcomm video decoder/encoder driver
Hi, Here is v3 of the Venus v4l2 video encoder/decoder driver. The changes since v2 are: - return queued buffers on stream_on error. - changed name of the driver vidc -> venus and reflect that in querycap. - fix video_device::release to point to video_device_release. - tried to implement correctly g_selection for decoder and encoder. - added Venus HFI 3xx basic support, to able to reuse driver on msm8996. - extend DT binding with reg-names and interrupt-names. - parse DT IRQ and MEM resources by name. - merge hfi_core,hfi_inst in venus_core and venus_inst structures. - killed hfi_pkt_ops struct and use functions. - various cleanups. The output of v4l2-compliance looks like: root@dragonboard-410c:/home/linaro# ./v4l2-compliance -d /dev/video0 v4l2-compliance SHA : 405f0c21e0b52836d22c999aa4ee1f51d87998b2 Driver Info: Driver name : qcom-venus Card type : Qualcomm Venus video decoder Bus info : platform:qcom-venus Driver version: 4.4.23 Capabilities : 0x84204000 Video Memory-to-Memory Multiplanar Streaming Extended Pix Format Device Capabilities Device Caps : 0x04204000 Video Memory-to-Memory Multiplanar Streaming Extended Pix Format Compliance test for device /dev/video0 (not using libv4l2): Required ioctls: test VIDIOC_QUERYCAP: OK Allow for multiple opens: test second video open: OK test VIDIOC_QUERYCAP: OK test VIDIOC_G/S_PRIORITY: OK test for unlimited opens: OK Debug ioctls: test VIDIOC_DBG_G/S_REGISTER: OK (Not Supported) test VIDIOC_LOG_STATUS: OK (Not Supported) Input ioctls: test VIDIOC_G/S_TUNER/ENUM_FREQ_BANDS: OK (Not Supported) test VIDIOC_G/S_FREQUENCY: OK (Not Supported) test VIDIOC_S_HW_FREQ_SEEK: OK (Not Supported) test VIDIOC_ENUMAUDIO: OK (Not Supported) test VIDIOC_G/S/ENUMINPUT: OK (Not Supported) test VIDIOC_G/S_AUDIO: OK (Not Supported) Inputs: 0 Audio Inputs: 0 Tuners: 0 Output ioctls: test VIDIOC_G/S_MODULATOR: OK (Not Supported) test VIDIOC_G/S_FREQUENCY: OK (Not Supported) test VIDIOC_ENUMAUDOUT: OK (Not Supported) test VIDIOC_G/S/ENUMOUTPUT: OK (Not Supported) test VIDIOC_G/S_AUDOUT: OK (Not Supported) Outputs: 0 Audio Outputs: 0 Modulators: 0 Input/Output configuration ioctls: test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported) test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported) test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported) test VIDIOC_G/S_EDID: OK (Not Supported) Control ioctls: test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK test VIDIOC_QUERYCTRL: OK test VIDIOC_G/S_CTRL: OK test VIDIOC_G/S/TRY_EXT_CTRLS: OK test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK test VIDIOC_G/S_JPEGCOMP: OK (Not Supported) Standard Controls: 7 Private Controls: 0 Format ioctls: test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK test VIDIOC_G/S_PARM: OK (Not Supported) test VIDIOC_G_FBUF: OK (Not Supported) test VIDIOC_G_FMT: OK test VIDIOC_TRY_FMT: OK test VIDIOC_S_FMT: OK test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported) test Cropping: OK test Composing: OK test Scaling: OK Codec ioctls: test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported) test VIDIOC_G_ENC_INDEX: OK (Not Supported) test VIDIOC_(TRY_)DECODER_CMD: OK (Not Supported) Buffer ioctls: test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK test VIDIOC_EXPBUF: OK Test input 0: Total: 43, Succeeded: 43, Failed: 0, Warnings: 0 root@dragonboard-410c:/home/linaro# ./v4l2-compliance -d /dev/video1 v4l2-compliance SHA : 405f0c21e0b52836d22c999aa4ee1f51d87998b2 Driver Info: Driver name : vidc Card type : video encoder Bus info : platform:vidc Driver version: 4.4.23 Capabilities : 0x84204000 Video Memory-to-Memory Multiplanar Streaming Extended Pix Format Device Capabilities Device Caps : 0x04204000 Video Memory-to-Memory Multiplanar Streaming Extended Pix Format Compliance test for device /dev/video1 (not using libv4l2): Required ioctls: test VIDIOC_QUERYCAP: OK Allow for multiple opens: test second video open: OK test VIDIOC_QUERYCAP: OK test VIDIOC_G/S_PRIORITY: OK test for unlimited opens: OK Debug ioctls: