From patchwork Wed Jun 27 15:27:07 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stanimir Varbanov X-Patchwork-Id: 10492113 X-Patchwork-Delegate: agross@codeaurora.org Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 8B43A60230 for ; Wed, 27 Jun 2018 15:35:35 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id C45672953E for ; Wed, 27 Jun 2018 15:35:35 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id B8DC729544; Wed, 27 Jun 2018 15:35:35 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI, T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 5CDC02954B for ; Wed, 27 Jun 2018 15:35:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932837AbeF0Peq (ORCPT ); Wed, 27 Jun 2018 11:34:46 -0400 Received: from mail-wr0-f196.google.com ([209.85.128.196]:35860 "EHLO mail-wr0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934861AbeF0P2J (ORCPT ); Wed, 27 Jun 2018 11:28:09 -0400 Received: by mail-wr0-f196.google.com with SMTP id f16-v6so2477098wrm.3 for ; Wed, 27 Jun 2018 08:28:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=iP3OBq3KmmkdIHc4ClU6RmJI5t+xyWo9eNZ9y9WZFKg=; b=QAu+UGG+Kzcc915ZKNuhrLTocXs6qRLejPoTmNT4GfKzpIVQsDzz+OSgbT8adIPHfR igWEQyoHFwGXQR3MlIqMJHbmVIpfmS9WwFCYVB5NgfcPVS3/HBkDf82rKpuXxIhQ52UV j23TB05DJiq12ksZxvIckvxaSqLIVHamCZISw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=iP3OBq3KmmkdIHc4ClU6RmJI5t+xyWo9eNZ9y9WZFKg=; b=WWelE7BYZjlWgKA6x/lQnz2cHipdblIN0zcgUHHqOFqx5921Q4d5xQd9EEPripEFN9 zjYcjM6ZI40B2T+PJiKwPVWNPPf1qR7ANBV6JYZ/dsLZ+EbnoimYWDJJrJDj/kT3WT/t e7/aksSGcXuwRHGpYnsrytvN46Ma8EJxB+ICeLFOqI8pzy6jjM+CGDv90h+RG18V2FJK Aj1I7n6IFH85Umf6ly9dW+QgwB8y7KcKqJriv1YrpVMrotB/9rYM23+2YXhbDJ8KHiht J+JN1rWAMHE1kUxD+zB93kXMpHG2FJdviymEcc0kBS+/S0hPQ+Ddx/YU0Srwe/2++oC6 V+Lg== X-Gm-Message-State: APt69E25aF+Gwe8ljv6cJwbXpGcOIpDHBbBMRYPSGRbLGRa986jiCcmI ZV3lCwgGkdQUpZWdqQgFcVcgdA== X-Google-Smtp-Source: AAOMgpdDErl4TQ/cqKTZ/vru9EFick5+8i61YxWNnDuvRDGWcawn++VyyMY5Nf7PjZPZBxsiD6YIIA== X-Received: by 2002:adf:e881:: with SMTP id d1-v6mr2407375wrm.43.1530113288238; Wed, 27 Jun 2018 08:28:08 -0700 (PDT) Received: from mms-0440.qualcomm.mm-sol.com ([37.157.136.206]) by smtp.gmail.com with ESMTPSA id o203-v6sm2916075wmd.30.2018.06.27.08.28.07 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 27 Jun 2018 08:28:07 -0700 (PDT) From: Stanimir Varbanov To: Mauro Carvalho Chehab , Hans Verkuil Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Vikash Garodia , Tomasz Figa , Stanimir Varbanov Subject: [PATCH v4 09/27] venus: hfi_venus: move set of default properties to core init Date: Wed, 27 Jun 2018 18:27:07 +0300 Message-Id: <20180627152725.9783-10-stanimir.varbanov@linaro.org> X-Mailer: git-send-email 2.14.1 In-Reply-To: <20180627152725.9783-1-stanimir.varbanov@linaro.org> References: <20180627152725.9783-1-stanimir.varbanov@linaro.org> Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP This moves setting of default properties (firmware debug, idle indicator and low power mode) from session init to core init. All of those properties are need to be enabled/disabled early so that they could be used before the clients are even initialized. The other reason is to set idle indicator property early before we enter into venus_suspend function where we need to check for ARM9 WFI. Signed-off-by: Stanimir Varbanov Reviewed-by: Tomasz Figa --- drivers/media/platform/qcom/venus/hfi_venus.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/qcom/venus/hfi_venus.c b/drivers/media/platform/qcom/venus/hfi_venus.c index 72a8547eab39..7a83e967a8ea 100644 --- a/drivers/media/platform/qcom/venus/hfi_venus.c +++ b/drivers/media/platform/qcom/venus/hfi_venus.c @@ -1091,6 +1091,10 @@ static int venus_core_init(struct venus_core *core) if (ret) dev_warn(dev, "failed to send image version pkt to fw\n"); + ret = venus_sys_set_default_properties(hdev); + if (ret) + return ret; + return 0; } @@ -1135,10 +1139,6 @@ static int venus_session_init(struct venus_inst *inst, u32 session_type, struct hfi_session_init_pkt pkt; int ret; - ret = venus_sys_set_default_properties(hdev); - if (ret) - return ret; - ret = pkt_session_init(&pkt, inst, session_type, codec); if (ret) goto err;