From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 99F9CC4332F for ; Thu, 22 Dec 2022 11:09:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=TrRuEGFgFWcBK0Sgm8m0tH1Fs8lsQ62mZRL2ZQHQyj0=; b=steOn9D1pbIHgs RuODtJDLw8I+KfGbV7Zaqaf4568uj8UEHASIVOGJu8N6SAg5HlhDfvAUL50oPjTdb4UTz0/1tyG8T MNnQisrJ0dwyLMoSGhQnFjNBsjKPbV4zt+pvPrBYQ1+0rR1EtCi/HhInRRJUjXgmIj1u8OnYbr/ZU h7ZFAU19/lN9MDETf1wbWwg4J0ip4ScrH92HYKt1ZGTIawZbQ1rsIBJCgLMKbqVUlMArv575odIZO eVSPT/Z/lekUd5S1NpvmhC4zwTh5Svm9ZdvPNb+DtAvDDzg6R7wdsPR1klB9pFHmGRn0k9bIHBVga sSew4hVlScf7FkNSqeig==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p8JRI-00AjoV-4Y; Thu, 22 Dec 2022 11:08:40 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p8JR5-00Ajf7-7N for linux-arm-kernel@bombadil.infradead.org; Thu, 22 Dec 2022 11:08:27 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=Content-Transfer-Encoding:Content-Type :In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date:Message-ID: Sender:Reply-To:Content-ID:Content-Description; bh=+kvNcgxWeaa0hpCRxgK1Lm1JNjYaI3ecIkEuVNJzHBc=; b=BL7lrMuMuMbyt8OxnnzHv2tcEm 8GV7/a9Qv60XnEER9BvlAvwOAekvScheLev30wAn8s7/jGEvlHaK7yfTuiHESmKGZqzdwRUUsHmmx o6jYL6MmX5bZsrEtDXXJLS8OtEHTLfj8Am8ES51+cFebF2TX152MsVGOpwCyUOqZUmrHpIXIg7Qz2 rBWe+DZh0IniYNT5pqtjWIgAmU7DL7pV8opm188rRdroMw+GH8dvFzAKbS5iWxrTTUof7jqFARNMm eQZ6zDOYSrS/1YymbXOdoLIOpX1GuPtNJzQuiJUbqj9lkwgTS0pWVUdtKCtsahVXHc3yg8rD7D9fY P9q5+sYw==; Received: from mout.kundenserver.de ([212.227.126.131]) by desiato.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1p8JQz-00Dmam-0t for linux-arm-kernel@lists.infradead.org; Thu, 22 Dec 2022 11:08:24 +0000 Received: from [192.168.1.139] ([37.4.248.22]) by mrelayeu.kundenserver.de (mreue011 [212.227.15.167]) with ESMTPSA (Nemesis) id 1MacjC-1obzTr3ClG-00c7RZ; Thu, 22 Dec 2022 12:07:57 +0100 Message-ID: <8fcbdf54-98ae-6a06-ecaf-591a43b863a1@i2se.com> Date: Thu, 22 Dec 2022 12:07:56 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.2 Subject: Re: [PATCH v2 5/7] vc04_services: vchiq_arm: Drop VCHIQ_RETRY usage on disconnect To: Umang Jain , linux-staging@lists.linux.dev, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Kees Cook , Greg Kroah-Hartman , Florian Fainelli , Adrien Thierry , Dan Carpenter , Dave Stevenson , Kieran Bingham , Laurent Pinchart References: <20221219115725.11992-1-umang.jain@ideasonboard.com> <20221219115725.11992-6-umang.jain@ideasonboard.com> Content-Language: en-US From: Stefan Wahren In-Reply-To: <20221219115725.11992-6-umang.jain@ideasonboard.com> X-Provags-ID: V03:K1:ow/FZUP+RKjclWSiCtRIaLt7Bf7HqGbYjZOp9iWu0B/tP44u6ME OX1XZE9IOt7yjDKIPhox6tUfd0mvsbzBdeWXUEW/FbnRnRxlJfokQTdRyRkQzuZ1AD4HeWJ K58g7bsv6D8irvrrvSRSKMg3qyUYEfgfm6oNVudouzpxtRouV+smCpSQNQY7H+oYTgUAtiR RrZiVOK9YBcABgjeNjUeQ== UI-OutboundReport: notjunk:1;M01:P0:NIzVU/U6y2I=;ntISWqAzkY8Sol7nfKIpq0ArtHF OEAUdqhoO0+ln1uLQUUV3BATXP8NI6raoZsgxFYp+8KNkADoltuBDozq3eKQeXkUlBW2qC5VU uB3oCBgUsGDG2O3HwXYGuGBRmIRnyAkWaJxIZSS/ymtzKC/qCzQ3XXc8TqtIQekj/S64/hQzu 54/yrdQaUV6bF8mqgLii3khsYhTNvDwFeb1ByMP1+E9lodGgwGThuSkOFgYQPcS663BranzzU Ep1tPlUdS6xSto7F8EIB06WCxFtawf8Nx/r25qHt+UYvPBlnUhpnYOqYlQNeta95tPcHujcSM H5VbOCuLb9CypWEwWD+tRTOXjCHmSRb8tvVrw4QIcYabNQl8MjhmH0CCRP6EGc2zKhCZSklQ2 M0sGAYJrM76ib/yynCXJETnVXdxbsb7ig2fre1JITN1VZ5C4faTIiGIiY+98WU3cjxTKcQ0tE YBjrV+UJhFzuIya03PSxLZ7frSZ+M5Xj/mmbVB2xXSRZOUhVbSLWBKiMzkK6e6kG79bDJCZ90 qG9rS/oIHZ2P0Vlt7QQdLHgnwRY7NCvDS9wH1v3s9o1JVUV/Vwh6MQZMgfKPWaKFNdwQWokCj G9Mb3NuCb+FoCBmjqP6kDso00Hjxa+k2buzqwOTCaEeG+3yWfoDKk0zNssHgXWdE9WHhcTYGw zg5RMBsY2e4q2EhQXd9BHVpIBxyXASfZurv9QlOwWw== X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221222_110821_974919_3DCE07D0 X-CRM114-Status: GOOD ( 18.99 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Umang, Am 19.12.22 um 12:57 schrieb Umang Jain: > Drop the usage of VCHIQ_RETRY when the vchiq has connection status > VCHIQ_CONNSTATE_DISCONNECTED. Disconnected status will not be valid to > carry on a retry, replace the VCHIQ_RETRY with -ENOTCONN. > > This patch removes the usage of vCHIQ_RETRY completely and act as > intermediatory to address the TODO item: > * Get rid of custom function return values > for vc04_services/interface. > > Fixes: 71bad7f08641 ("staging: add bcm2708 vchiq driver") please drop this fixes tag since this commit doesn't fix a real issue and also shouldn't be applied to stable. > Signed-off-by: Umang Jain > --- > .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c > index 9c64d5de810e..ddb6d0f4daed 100644 > --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c > +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c > @@ -3641,7 +3641,7 @@ vchiq_loud_error_footer(void) > int vchiq_send_remote_use(struct vchiq_state *state) > { > if (state->conn_state == VCHIQ_CONNSTATE_DISCONNECTED) > - return VCHIQ_RETRY; > + return -ENOTCONN; > > return queue_message(state, NULL, MAKE_REMOTE_USE, NULL, NULL, 0, 0); > } > @@ -3649,7 +3649,7 @@ int vchiq_send_remote_use(struct vchiq_state *state) > int vchiq_send_remote_use_active(struct vchiq_state *state) > { > if (state->conn_state == VCHIQ_CONNSTATE_DISCONNECTED) > - return VCHIQ_RETRY; > + return -ENOTCONN; > > return queue_message(state, NULL, MAKE_REMOTE_USE_ACTIVE, > NULL, NULL, 0, 0); _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel