Received: by 2002:a05:6a10:17d3:0:0:0:0 with SMTP id hz19csp1930368pxb; Mon, 12 Apr 2021 09:54:17 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwwzEZq3RV9qH8J41jb0bzeeB8uWbdJsWM2ZyboR3RTA/e54+XOPN9/ouBrW8y2XJXoS+7n X-Received: by 2002:a17:906:4746:: with SMTP id j6mr16736339ejs.39.1618246457251; Mon, 12 Apr 2021 09:54:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1618246457; cv=none; d=google.com; s=arc-20160816; b=aZNVqZEh2v6Gzm/6PC/fgMx7t5o38JDL8G0JuQJ0sAe9nCcGqugF5JzQEIOLimBx+i WwOgCrQ5OfsGHRfyNlfV8t2hAQgSKr7Heky2Rdg6Kh1xJ6R7iNb+pSNXnnGYG9H6nk/m Zo900f8JnnhKpk3hrTEZHsMxpezkyfEgNUl88W1idy2dgY/TgRrNMLwwGB0NoglfAAFM jCP6KN5gL9jOIQbKcbNE7E8DWlV470ox1+bEdMAGq253dh64Ozyusi2vUTCfnm68hQBH fDQQlyxzW6RSKJH5VmUYHVTu/jVdjfK4F3A88nIug2nqtnk68rqQJhR0JF81kEuD8Zwn 7aeQ== 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:subject:reply-to:from:to :dkim-signature:date; bh=bur6pDDgjXw6skXkwkUN9ZOYk+3Tg1KFD6dBSPnGAeY=; b=oNTgdEl1CVGHEOeHRq7osTRknrBj92630OG/5GRlGBv7YGw8TBY+6DbqCLbUBwPeTd N1mq9IVM2qe9nm0jY3KaFsTh0QRtJ3B0Jr7LMegQSlGKEJgW+OQ3ljR9TisbmS8onxfs kb8lvbUzmmNni/nF3sR1V6b4OfekuAeyVnD/a1tAQVexG8B637mD31oElDT9bUaHXZlG HjXPy7N1sw7KJq9OGTu4/xMrTP0oGx1HhTDBCGXfotmKERwJTbipoLv1Q9eiBd8Q9m7E rcyZO0vCCMkhgD/Zyw3Y0AGPcCLPsGuAZ27L/5I7ibpKKYepAg25HGHWM241bMZiJymO QKSg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@bryanbrattlof.com header.s=protonmail3 header.b="l9jOGwG/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=bryanbrattlof.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id b25si7113085eja.29.2021.04.12.09.53.53; Mon, 12 Apr 2021 09:54:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@bryanbrattlof.com header.s=protonmail3 header.b="l9jOGwG/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=bryanbrattlof.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S244630AbhDLQsq (ORCPT + 99 others); Mon, 12 Apr 2021 12:48:46 -0400 Received: from mail-40136.protonmail.ch ([185.70.40.136]:30997 "EHLO mail-40136.protonmail.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1343763AbhDLQoG (ORCPT ); Mon, 12 Apr 2021 12:44:06 -0400 Date: Mon, 12 Apr 2021 16:43:31 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bryanbrattlof.com; s=protonmail3; t=1618245812; bh=bur6pDDgjXw6skXkwkUN9ZOYk+3Tg1KFD6dBSPnGAeY=; h=Date:To:From:Reply-To:Subject:In-Reply-To:References:From; b=l9jOGwG/5CoYVff8V7q5TX8HbZUX+omtZjQwP5pNTKJR2yuiTHx5ZNKnsSnfsXyAV FvGW1jNoSmOlzeSe5V5/IM7rT07luS8vMMauGVYEyFf/inaAISgM2KUEaDQoRXn09O U7EdlDkIOCEMDBYbka33WDTViGCxzdGVtu1VpYXj6GI1D5WmIZuFiRSnKrO0lLjHZK fWuhD8K/R7DfBonjjUKKtFYNi58CguG5wm7AmIfuzNqV2+umb/S23n0+w0E7UrlMK/ mb2bhiO1v9zwf6eNPivufhY8jFqe8lnfiI886yi3HkuSQMXrg8QoBN2FEZkOiZxzPa Uli54d2GOPsWw== To: Greg Kroah-Hartman , Hans de Goede , Larry Finger , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org From: Bryan Brattlof Reply-To: Bryan Brattlof Subject: NAK: [PATCH] staging: rtl8723bs: remove unnecessary adapter Message-ID: <20210412163414.5i53wyvguckj5uxz@bryanbrattlof.com> In-Reply-To: <20210412140054.27629-1-hello@bryanbrattlof.com> References: <20210412140054.27629-1-hello@bryanbrattlof.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-1.2 required=10.0 tests=ALL_TRUSTED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF shortcircuit=no autolearn=disabled version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on mailout.protonmail.ch Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Apr 12, 2021 at 02:01:49PM +0000, Bryan Brattlof wrote: > >The if2 adapter is defined in rtw_dvr_init() and in the dvobj_priv >structure but never used anywhere in the driver. This will remove all >definitions of if2 > >Signed-off-by: Bryan Brattlof >--- > drivers/staging/rtl8723bs/include/drv_types.h | 1 - > drivers/staging/rtl8723bs/os_dep/sdio_intf.c | 10 ++++------ > 2 files changed, 4 insertions(+), 7 deletions(-) > >diff --git a/drivers/staging/rtl8723bs/include/drv_types.h b/drivers/stagi= ng/rtl8723bs/include/drv_types.h >index cec8d5ac0e2e..35882dc2654e 100644 >--- a/drivers/staging/rtl8723bs/include/drv_types.h >+++ b/drivers/staging/rtl8723bs/include/drv_types.h >@@ -269,7 +269,6 @@ struct cam_entry_cache { > struct dvobj_priv { > =09/*-------- below is common data --------*/ > =09struct adapter *if1; /* PRIMARY_ADAPTER */ >-=09struct adapter *if2; /* SECONDARY_ADAPTER */ > > =09s32=09processing_dev_remove; > >diff --git a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c b/drivers/stagin= g/rtl8723bs/os_dep/sdio_intf.c >index 2b61a4bdd8a1..3545cb6ef886 100644 >--- a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c >+++ b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c >@@ -381,7 +381,7 @@ static int rtw_drv_init( > =09const struct sdio_device_id *id) > { > =09int status =3D _FAIL; >-=09struct adapter *if1 =3D NULL, *if2 =3D NULL; >+=09struct adapter *if1 =3D NULL; > =09struct dvobj_priv *dvobj; > > =09dvobj =3D sdio_dvobj_init(func); >@@ -395,17 +395,15 @@ static int rtw_drv_init( > =09/* dev_alloc_name && register_netdev */ > =09status =3D rtw_drv_register_netdev(if1); > =09if (status !=3D _SUCCESS) >-=09=09goto free_if2; >+=09=09goto free_if1; > > =09if (sdio_alloc_irq(dvobj) !=3D _SUCCESS) >-=09=09goto free_if2; >+=09=09goto free_if1; > > =09rtw_ndev_notifier_register(); > =09status =3D _SUCCESS; > >-free_if2: >-=09if (status !=3D _SUCCESS && if2) { >-=09} >+free_if1: > =09if (status !=3D _SUCCESS && if1) { > =09=09rtw_sdio_if1_deinit(if1); > =09} >--=20 >2.27.0 > Nack. There is a merge conflict when updating to next-20210412. I'll=20 resend v2 after I resolve the conflict. -- ~Bryan