From patchwork Mon Jun 26 14:33:31 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sui Jingfeng X-Patchwork-Id: 112994 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp7539115vqr; Mon, 26 Jun 2023 07:52:23 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6FsWxJauV5N+eHN4UreK8Svnnf7ecRcfuzvbFKcxxHoY5z9i74KgZFDyqoKE/eR2pLGTOZ X-Received: by 2002:a05:6a20:7289:b0:11f:a55f:edf9 with SMTP id o9-20020a056a20728900b0011fa55fedf9mr31127665pzk.39.1687791143156; Mon, 26 Jun 2023 07:52:23 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1687791143; cv=none; d=google.com; s=arc-20160816; b=WjIp+qxwaQvws2fOIW/jBfH30bzvrg8qSPxInKfUOYnOZdQe8Bsugtnk56vXHGeRJX XrWb9YUv0demj4lQRDtfVmgEtIlQQkfGxPuK09fYr6m/seqdcZNPvP+bsWmf6teXj1WB 5aXlr1Lu/JypCX+d3VNnks/yFNzvUJZYHVhGpbCT1kVvgOhc4z/0t+YqwVVHu2R9kimq BiqzsnzPx692uEFAZz8y5SgqFrlzuWNZOI3JwLPDqLSGKXoUvTAch8mOQtetfqeNTOaC 69A/7XxnUc3FVmxJAcRHSPyNLNmRi0T5En9TVmMF7uVP6K28FK6yhn9VzD3/Vx/Vb8mf PtfA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=iSTsMgKoNUj2MkNLFj5vQ8HNI3EDBWysQ/B6RjB9JDo=; fh=96qtDnTU/URrOT6i7uaKTwabo9K0PbhXvr8DzFjxXtc=; b=w1LFUBzdGuNY3hk+tHR07I7vDz+naXZRKo/KekmRHlySxHEb0QhpNsrfn5iFg9C8mJ bNtS82jySkgYcENm3Y1A+s3KQivW/AW7td1zZwN4meIvluUiL7CJaEgo3Ea/E7O2JTIe NYOYkVletefKfnotNm5k9stpQdNdBEkvttsVlfNmsvEbRSNZ3zJ76iy0wAWO7Avk4sZh 5nzUU431wI7mmmpE4cG+H0kfc/ZrSGjgUnE1ySaHtKhZN3X+eTCDMlctGk3aXxTHY2Dj CQAzQeWfrJ0GOvqvDZsmaSH/bLg5q+flWN8kfa4TR010GV8qEUnRe7+zn7t1ND5RTKX+ 7IlQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id e5-20020a170902f10500b001ae4e665c60si4720378plb.156.2023.06.26.07.52.08; Mon, 26 Jun 2023 07:52:23 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231172AbjFZOdm (ORCPT + 99 others); Mon, 26 Jun 2023 10:33:42 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48494 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229520AbjFZOdh (ORCPT ); Mon, 26 Jun 2023 10:33:37 -0400 Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 145A0E7E; Mon, 26 Jun 2023 07:33:34 -0700 (PDT) Received: from loongson.cn (unknown [10.20.42.43]) by gateway (Coremail) with SMTP id _____8Cx5cS8oZlkjm8CAA--.3784S3; Mon, 26 Jun 2023 22:33:32 +0800 (CST) Received: from openarena.loongson.cn (unknown [10.20.42.43]) by localhost.localdomain (Coremail) with SMTP id AQAAf8CxvM67oZlk7cUJAA--.43395S3; Mon, 26 Jun 2023 22:33:31 +0800 (CST) From: Sui Jingfeng To: Thierry Reding , Mikko Perttunen , David Airlie , Daniel Vetter , Jonathan Hunter Cc: dri-devel@lists.freedesktop.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 2/2] drm/tegra: Remove surplus else after return Date: Mon, 26 Jun 2023 22:33:31 +0800 Message-Id: <20230626143331.640454-2-suijingfeng@loongson.cn> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230626143331.640454-1-suijingfeng@loongson.cn> References: <20230626143331.640454-1-suijingfeng@loongson.cn> MIME-Version: 1.0 X-CM-TRANSID: AQAAf8CxvM67oZlk7cUJAA--.43395S3 X-CM-SenderInfo: xvxlyxpqjiv03j6o00pqjv00gofq/ X-Coremail-Antispam: 1Uk129KBj93XoW7Cw15JF4DAF4fuw1xur1Utwc_yoW8XF13pa yag34YkrZ0qF4DZF12yF4j9wn5ZaySgFyIkFWDJa9avF1DXr10v348tFZrtFy5Gr97JFsI yF4qyw43A347AwcCm3ZEXasCq-sJn29KB7ZKAUJUUUUr529EdanIXcx71UUUUU7KY7ZEXa sCq-sGcSsGvfJ3Ic02F40EFcxC0VAKzVAqx4xG6I80ebIjqfuFe4nvWSU5nxnvy29KBjDU 0xBIdaVrnRJUUU9Fb4IE77IF4wAFF20E14v26r1j6r4UM7CY07I20VC2zVCF04k26cxKx2 IYs7xG6rWj6s0DM7CIcVAFz4kK6r126r13M28lY4IEw2IIxxk0rwA2F7IY1VAKz4vEj48v e4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_JFI_Gr1l84ACjcxK6xIIjxv20xvEc7CjxVAFwI 0_Gr0_Cr1l84ACjcxK6I8E87Iv67AKxVW8Jr0_Cr1UM28EF7xvwVC2z280aVCY1x0267AK xVW8Jr0_Cr1UM2kKe7AKxVWUXVWUAwAS0I0E0xvYzxvE52x082IY62kv0487Mc804VCY07 AIYIkI8VC2zVCFFI0UMc02F40EFcxC0VAKzVAqx4xG6I80ewAv7VC0I7IYx2IY67AKxVWU AVWUtwAv7VC2z280aVAFwI0_Gr0_Cr1lOx8S6xCaFVCjc4AY6r1j6r4UM4x0Y48IcxkI7V AKI48JMxAIw28IcxkI7VAKI48JMxC20s026xCaFVCjc4AY6r1j6r4UMxCIbckI1I0E14v2 6r1Y6r17MI8I3I0E5I8CrVAFwI0_Jr0_Jr4lx2IqxVCjr7xvwVAFwI0_JrI_JrWlx4CE17 CEb7AF67AKxVWUtVW8ZwCIc40Y0x0EwIxGrwCI42IY6xIIjxv20xvE14v26r1I6r4UMIIF 0xvE2Ix0cI8IcVCY1x0267AKxVW8JVWxJwCI42IY6xAIw20EY4v20xvaj40_Jr0_JF4lIx AIcVC2z280aVAFwI0_Gr0_Cr1lIxAIcVC2z280aVCY1x0267AKxVW8JVW8JrUvcSsGvfC2 KfnxnUUI43ZEXa7IU8l4iUUUUUU== X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1769777285656902049?= X-GMAIL-MSGID: =?utf-8?q?1769777285656902049?= else is not generally useful after return Signed-off-by: Sui Jingfeng --- drivers/gpu/drm/tegra/gem.c | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/tegra/gem.c b/drivers/gpu/drm/tegra/gem.c index 0ce22935fbd3..26d64f4545bd 100644 --- a/drivers/gpu/drm/tegra/gem.c +++ b/drivers/gpu/drm/tegra/gem.c @@ -180,15 +180,15 @@ static void *tegra_bo_mmap(struct host1x_bo *bo) struct iosys_map map; int ret; - if (obj->vaddr) { + if (obj->vaddr) return obj->vaddr; - } else if (obj->gem.import_attach) { + + if (obj->gem.import_attach) { ret = dma_buf_vmap_unlocked(obj->gem.import_attach->dmabuf, &map); return ret ? NULL : map.vaddr; - } else { - return vmap(obj->pages, obj->num_pages, VM_MAP, - pgprot_writecombine(PAGE_KERNEL)); } + + return vmap(obj->pages, obj->num_pages, VM_MAP, pgprot_writecombine(PAGE_KERNEL)); } static void tegra_bo_munmap(struct host1x_bo *bo, void *addr) @@ -198,10 +198,11 @@ static void tegra_bo_munmap(struct host1x_bo *bo, void *addr) if (obj->vaddr) return; - else if (obj->gem.import_attach) - dma_buf_vunmap_unlocked(obj->gem.import_attach->dmabuf, &map); - else - vunmap(addr); + + if (obj->gem.import_attach) + return dma_buf_vunmap_unlocked(obj->gem.import_attach->dmabuf, &map); + + vunmap(addr); } static struct host1x_bo *tegra_bo_get(struct host1x_bo *bo)