Hello, Here is v2 with following comments addressed: * drop 1/4 patch from v1 and use relevant api to load firmware without PAS. * add some details on ARM9 role in video hardware. * abstract scm calls to set hardware state. * remove setting aperture range for firmware and vcodec context banks. * add misc code review comments related to return handling, unwanted cast, etc. Comments are welcome! Vikash Garodia (5): media: venus: add a routine to reset ARM9 media: venus: add a routine to set venus state venus: add check to make scm calls media: venus: add no TZ boot and shutdown routine venus: register separate driver for firmware device .../devicetree/bindings/media/qcom,venus.txt | 8 +- drivers/media/platform/qcom/venus/core.c | 58 +++++- drivers/media/platform/qcom/venus/core.h | 10 + drivers/media/platform/qcom/venus/firmware.c | 217 ++++++++++++++++++--- drivers/media/platform/qcom/venus/firmware.h | 12 +- drivers/media/platform/qcom/venus/hfi_venus.c | 13 +- drivers/media/platform/qcom/venus/hfi_venus_io.h | 5 + 7 files changed, 275 insertions(+), 48 deletions(-) -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project