From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752509AbeEOIT0 (ORCPT ); Tue, 15 May 2018 04:19:26 -0400 Received: from lb2-smtp-cloud9.xs4all.net ([194.109.24.26]:34871 "EHLO lb2-smtp-cloud9.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752059AbeEOITY (ORCPT ); Tue, 15 May 2018 04:19:24 -0400 Subject: Re: [PATCH v2 00/29] Venus updates To: Stanimir Varbanov , Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Vikash Garodia References: <20180515075859.17217-1-stanimir.varbanov@linaro.org> From: Hans Verkuil Message-ID: <7d24ab33-84c2-0faa-a2d9-a82751a6bb6f@xs4all.nl> Date: Tue, 15 May 2018 10:19:19 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20180515075859.17217-1-stanimir.varbanov@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4wfEniIoTTKfKw7jU9w2J41HmaunvVArqZ/gxbGVjWZ//wkO8ALqspv0npTke9BdCUff3ZoWklAgBzt7NQoVmKx6AhpQZAEbeMj8isIRS7ivi2ozb1rwg2 h4XINB4q/BoDJ01AUfRgwMkCv52VKQwTe9y1FnzcQcMLR4EIawUnCqxc6a7AYDsxBDKAZNwKGEHmpEhJMky8HAS4qwdLfV7BR8n1G2tdOLETy0srcf1GqZUQ mogStz9MjgZcb95bjnV3tugt2Bf+x79PSYTwx1HSiPBvSAkhgB2R8GMTOWVY+tE5yJDxkD79rtIrQJxkvwFY696vcZH8zOvg6WllyfBGA5Jdd6mScktK51Tv 6QA70JCtxOIEU5u2DasbcRpfLmdyZXMtGhQ/QP7y78KhEX2v1V11YQIdS9sfMh2dzfhP4BMQ0iAZE2EXVyQdE+I+KYssdct+l8a7kWGvQ8TCCY3MrwrjYmWK 90OpQ1SOl4o/+2b6wBW4BW+y0pAEIEGX2vC1SA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Stanimir, On 05/15/18 09:58, Stanimir Varbanov wrote: > Hello, > > Here is v2 with following comments addressed: > > * reworked venus suspend 3xx and reuse it for 4xx. > * drop 10/28 patch from v1, i.e. call of session_continue when > buffer requirements are not sufficient. > * fixed kbuild test robot warning in 11/28 by allocating instance > variable from heap. > * spelling typo in 15/28. > * added Reviewed-by for DT changes. > * extended 28/28 HEVC support for encoder, now the profile and > level are selected properly. > > Comments are welcome! It all looks good, except for patch 27 (see my comments there). Once there is a v3 for patch 27 and I'm OK with it, then just let me know when it is ready in your opinion to be merged. Regards, Hans > > regards, > Stan > > Stanimir Varbanov (29): > venus: hfi_msgs: correct pointer increment > venus: hfi: preparation to support venus 4xx > venus: hfi: update sequence event to handle more properties > venus: hfi_cmds: add set_properties for 4xx version > venus: hfi: support session continue for 4xx version > venus: hfi: handle buffer output2 type as well > venus: hfi_venus: add halt AXI support for Venus 4xx > venus: hfi_venus: fix suspend function for venus 3xx versions > venus: hfi_venus: move set of default properties to core init > venus: hfi_venus: add suspend functionality for Venus 4xx > venus: venc,vdec: adds clocks needed for venus 4xx > venus: add common capability parser > venus: helpers: make a commmon function for power_enable > venus: core: delete not used flag for buffer mode > venus: helpers: rename a helper function and use buffer mode from caps > venus: add a helper function to set dynamic buffer mode > venus: add helper function to set actual buffer size > venus: delete no longer used bufmode flag from instance > venus: helpers: add buffer type argument to a helper > venus: helpers: add a new helper to set raw format > venus: helpers,vdec,venc: add helpers to set work mode and core usage > venus: helpers: extend set_num_bufs helper with one more argument > venus: helpers: add a helper to return opb buffer sizes > venus: vdec: get required input buffers as well > venus: vdec: new function for output configuration > venus: move frame size calculations in common place > venus: implementing multi-stream support > venus: add sdm845 compatible and resource data > venus: add HEVC codec support > > .../devicetree/bindings/media/qcom,venus.txt | 1 + > drivers/media/platform/qcom/venus/Makefile | 3 +- > drivers/media/platform/qcom/venus/core.c | 107 ++++ > drivers/media/platform/qcom/venus/core.h | 93 ++-- > drivers/media/platform/qcom/venus/helpers.c | 558 +++++++++++++++++++-- > drivers/media/platform/qcom/venus/helpers.h | 23 +- > drivers/media/platform/qcom/venus/hfi.c | 12 +- > drivers/media/platform/qcom/venus/hfi.h | 9 + > drivers/media/platform/qcom/venus/hfi_cmds.c | 64 ++- > drivers/media/platform/qcom/venus/hfi_helper.h | 112 ++++- > drivers/media/platform/qcom/venus/hfi_msgs.c | 401 +++------------ > drivers/media/platform/qcom/venus/hfi_parser.c | 291 +++++++++++ > drivers/media/platform/qcom/venus/hfi_parser.h | 45 ++ > drivers/media/platform/qcom/venus/hfi_venus.c | 95 +++- > drivers/media/platform/qcom/venus/hfi_venus_io.h | 25 + > drivers/media/platform/qcom/venus/vdec.c | 316 +++++++----- > drivers/media/platform/qcom/venus/venc.c | 211 ++++---- > 17 files changed, 1689 insertions(+), 677 deletions(-) > create mode 100644 drivers/media/platform/qcom/venus/hfi_parser.c > create mode 100644 drivers/media/platform/qcom/venus/hfi_parser.h >