From nobody Tue Jun 9 00:59:05 2026 Received: from mail-wr1-f52.google.com (mail-wr1-f52.google.com [209.85.221.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7D4573A0B2E for ; Mon, 25 May 2026 06:56:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.52 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779692206; cv=none; b=YciuJaK3tXf1LFmkd0U3h/Mfr7DDka0aMA5ABP7GWv4cUSiF6Tg8Qv/nDmf86A8ZjNSvnIU+76KVOvbiL3BrS8xMbvkoDGtkDOXYY8f0WMi22ue1gVZsUcotGuWcOVvL09mlhgGnyu+p9MXL8fAZZwBunexKzu9F68oOdvfjchI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779692206; c=relaxed/simple; bh=jDVOMYhYewn6ATtZbKp3VJy/YN+xTXplywg9VZlCfOA=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition; b=JyQEein+fOYYccOcKueOBC+2aQxFLtsP6P6sKt9MYbk/3R7cbJiBjjA90mKLXIJUnWW9cMfMFctcRZe8NqD+sc6yiT/gcREfhP90cAhKqHvvFjI8qANjl0GyJbU8wxKd4rgS6YoZFQYqNo5dT8zmDWz+aiEJZKLJJztXkpsBoSQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Gxscjd6x; arc=none smtp.client-ip=209.85.221.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Gxscjd6x" Received: by mail-wr1-f52.google.com with SMTP id ffacd0b85a97d-459bf19e87bso5334654f8f.1 for ; Sun, 24 May 2026 23:56:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1779692200; x=1780297000; darn=vger.kernel.org; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=GW1L+sX0aQAPGDZcLmvDCGiKclqdN5QY1XJTp1+tXFo=; b=Gxscjd6x0aoKCADdgTqMaBa/5VQ/n4Ru5Ap84EhCmT6zDhO79/Pecodw3cfEpvvJlH 8Bb/fCyxbJYuRaZbIZzhLdxNQbvwR0YLy0DdXrgUwSzMP5jo1Vl8dd5NsVUolf73usi8 oKXW9PUBFyb+kt4/zlVQ/RLFR3Ie7pCN1sSzK0WTPeufj7+GQEPW+UWXp4cikoUIFw9N Q8O7giN6AKQISk0k/B1AZXxDWMbTtmqTVCDz+Zh0MhnqlNmvBiZUkQD9wG4s1nlwuE1j Eb+ChdSCebUTVSujtZ98K2mQhTnjYg4sjxPOg/3pqpq2NCRuLrx6fDPAZYFSdwt6Rs4U uIhA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779692200; x=1780297000; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=GW1L+sX0aQAPGDZcLmvDCGiKclqdN5QY1XJTp1+tXFo=; b=dWKHQ2AdCN54BEcpt2ROfucYZpzWFBvOE+GnJrybomon7gsYx0BzJzUhGeHJNb4tyE 0n/nzR1U3D4aJi3XZwxW2fBbOWelRz4nEW676hpF4nr9htlhPpFsROSNtLPx9FAVYKgp 4jOefl/BCHdh/SqZqP3vYgfva4JnxuBJ8KUY3ysjojbb1NKpWPBbsDB/08jG2zhyYlWf 1wxCTL3ce0/w+k0Q6FsUgVGubZ049Cuy3jx99n58QIVXkcN1ReNa+6CT6g3Uc7JEBHkS 8JUwJDifl/IRQMWk9p1CK9ZFcTF6fPbagIGSGz7o28UUx4m79iKISFtPiTile5pZqtcs 1xkQ== X-Forwarded-Encrypted: i=1; AFNElJ8+9lc5X3TI8/mXlsh3YNCtAKZ3B3vGjFO1SgzU2aRU3tx9kMKYwQXYhT2U+qPO5mHpiXP5IjH+J5V8RRQ=@vger.kernel.org X-Gm-Message-State: AOJu0YxuCHrVMM0dKj0fyzqxuMa4CCGYYisgvf0SMlA33APgQXJzgm/c +o+DuyAcK2f7sZwwfh/ZDAkGS5oXY7NzzAlRQ8WUAwYUfvPvd6AWQQD4 X-Gm-Gg: Acq92OGDY5i1KwVSMMVwStraav87FvvTMS7seJTAYDw61/RAQcSgHrXfEE3OU3H2hH2 mh+P6inw/I2xY2xzDw3fsb9/YRF9H7+3c0R+8kiFvl0FnaYRjRWeWIwtagz9jfqJDPh0ua2vFsz /BARZHBLfxlxIS5IW7Y5c2Ft6gUjIxUkvqhWVS5bYDzF+kZJ64BXILZthL2zeIvsb0d5vcyWsqo tksoAp6DfuE3LsOKJF3ktcNiEvikrphF1tiINtd+c2kwN1imyX0IXD+UNUB5nFqngGkFQpsQqOC e1I67xnvxEBFOO69jJGe4Wk/c4clanEyayZVgKvJ5xVenYu1Fgn0XpsD51KmMat2HK5SpaeqUBm VY+q11mSbYdmaJfyySb9HcZN/RqAj5OFYrbsNAiq731+ruzL4a5HTX321vUn4qvaW03wrp4ISRd eMeqhfsMV28VA56po1BiH28o8WdcRbbqGPKQ== X-Received: by 2002:a5d:588d:0:b0:455:7c9f:a49e with SMTP id ffacd0b85a97d-45eb38c5508mr24586775f8f.25.1779692200400; Sun, 24 May 2026 23:56:40 -0700 (PDT) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-45eb6d5c32esm23804272f8f.26.2026.05.24.23.56.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 24 May 2026 23:56:39 -0700 (PDT) Date: Mon, 25 May 2026 09:56:36 +0300 From: Dan Carpenter To: Brian Norris Cc: Francesco Dolcini , Johannes Berg , Stefan Kerkmann , Kees Cook , linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: [PATCH] wifi: mwifiex: remove an unnecessary check Message-ID: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Disposition: inline X-Mailer: git-send-email haha only kidding Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" We know that "start_raw" is true so remove this check and pull the code in an tab. Signed-off-by: Dan Carpenter --- .../net/wireless/marvell/mwifiex/sta_cmd.c | 35 ++++++++----------- 1 file changed, 15 insertions(+), 20 deletions(-) diff --git a/drivers/net/wireless/marvell/mwifiex/sta_cmd.c b/drivers/net/w= ireless/marvell/mwifiex/sta_cmd.c index 70ce31d7c76e..623ddde8c8e5 100644 --- a/drivers/net/wireless/marvell/mwifiex/sta_cmd.c +++ b/drivers/net/wireless/marvell/mwifiex/sta_cmd.c @@ -1568,28 +1568,23 @@ int mwifiex_send_rgpower_table(struct mwifiex_priva= te *priv, const u8 *data, return -EINVAL; } =20 - if (start_raw) { - while ((*pos !=3D '\r' && *pos !=3D '\n') && - (token =3D strsep((char **)&pos, " "))) { - if (ptr - hostcmd->cmd >=3D - MWIFIEX_SIZE_OF_CMD_BUFFER) { - mwifiex_dbg( - adapter, ERROR, - "%s: hostcmd is larger than %d, aborting\n", - __func__, MWIFIEX_SIZE_OF_CMD_BUFFER); - return -ENOMEM; - } + while ((*pos !=3D '\r' && *pos !=3D '\n') && + (token =3D strsep((char **)&pos, " "))) { + if (ptr - hostcmd->cmd >=3D MWIFIEX_SIZE_OF_CMD_BUFFER) { + mwifiex_dbg(adapter, ERROR, + "%s: hostcmd is larger than %d, aborting\n", + __func__, MWIFIEX_SIZE_OF_CMD_BUFFER); + return -ENOMEM; + } =20 - ret =3D kstrtou8(token, 16, ptr); - if (ret < 0) { - mwifiex_dbg( - adapter, ERROR, - "%s: failed to parse hostcmd %d token: %s\n", - __func__, ret, token); - return ret; - } - ptr++; + ret =3D kstrtou8(token, 16, ptr); + if (ret < 0) { + mwifiex_dbg(adapter, ERROR, + "%s: failed to parse hostcmd %d token: %s\n", + __func__, ret, token); + return ret; } + ptr++; } } =20 --=20 2.53.0