From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f45.google.com (mail-wm1-f45.google.com [209.85.128.45]) (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 10FC13B6BFE for ; Thu, 28 May 2026 10:35:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779964561; cv=none; b=t8Sz5HRHWAhHXwZPXQ52wkPZGfb8iZo8OC/hkPbGoOBTLF2lI9nhNv8CMFSeTiH4ELc0Pa0YcmszDdMCFMKqE52ew73v/Mr9VXAo2UkQdK+13aUaqNwmwjuRlXNj9Ix/DpSy22Mujd5TXivokoc5LiEgM2PoHN5kYoB8no4EgCU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779964561; c=relaxed/simple; bh=Fg0V+oZAidj1nvbVQoP29tA6JrA291eDGVe7t9LD2ow=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=eRQHGApkqqVQpWBobIRjrLgO3Pye0yjsWAeWk5SK6/SEe94dTDPZNsmnsPF+dPURa5GQfKEk85s41Cym9xwIOMokyLGPH12q++6Yxjbx3wGd4UsPcMx+5ufeor6VXqyCAlUZmkify4I0vBYOcprwqr6ezlQ6cu5zykvWyWnV244= 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=pJb469dz; arc=none smtp.client-ip=209.85.128.45 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="pJb469dz" Received: by mail-wm1-f45.google.com with SMTP id 5b1f17b1804b1-4906238c62eso40039925e9.3 for ; Thu, 28 May 2026 03:35:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1779964558; x=1780569358; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=vJQU+eSTX518rgVEwjuFGD9upNQtN3nUPmcXZInkkD8=; b=pJb469dzLhoN+y/kbrBx5tVQ3VF6PCXcZdx1xG2dDtimAbdiFqi+YQg+EOp9KlPbkS D0wSXBqNe3KCaFCWQFXfrj1v5ov9izm9Ou7pDMZlpuM6dITRloO8aB/Cfuet6NeGCS62 MjN0X0bYTQHCusPIfU9fBC87edqMT/La6FdjrOXzx5opS2HFK9eulVULgdrVm5aBUPvs V9VJYZVInED42HGioVmeKL7u6i53Ur0JiWmOnFXzek6tnRGdcKMq9VWt57bZDl7Ye6NA A39CosycW7Q/J05WTjNhm8PEwzmRyj0rM2mG/3XuTjy8moI3voUL3SQnHbyg39S9vyK8 igOQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779964558; x=1780569358; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=vJQU+eSTX518rgVEwjuFGD9upNQtN3nUPmcXZInkkD8=; b=Pz1bavLEpswG6AeohXnZd+gXM2g2miG48fx+vGNJaFKXaur2pFjDvs6YIbwCLkwtjl 1N6tnu6ZYXpRp+/KNLS+FQT4xWFJDR6oPN0wQY6hrIgXtzC2B4GcStVoJnwgEWNO662d MG42fsijDbAaPkrrvjjw3jGHTol8IW0oe+xhZhVSfRy29IyXBBapuhniE8LotZqoqvbF iZWcpv6+ezJ9vKJGaptybUA2xxTlgAQ/sbWlGCm0fPXfDAV6rlxaLOepgcvRpYxGAHYj xkazryh+1aNFrI0lT0PVCGls95zDtkOZMdfYiJyjh9i37XqXkWor0Z+2tZ+QEcy1KGWV 8zvQ== X-Forwarded-Encrypted: i=1; AFNElJ/2ogBPpx2CDAqE5aXxzej21zPcsUtJEQIhm099bMTyNM2uULPlChseZCnFKa3tFKTKoJNamP+vyMIv2eQ=@vger.kernel.org X-Gm-Message-State: AOJu0YzP7Jh3hR8Nm+cpjTe5q6DzV2LUN/sSaTj5hTHCk3QJmwjokdzb ZnROm3+tY39/SEibclMTuqmMRkMSdsqMTGfGVW6dQWyNgD2BHxNhG3vm X-Gm-Gg: Acq92OFWVjMsyNcu5wJnRnZTj/MvfOa8nOC2gY2omw0vdlHm2kYUoUsDb+CmrkzKbDu ZF/n0X9krxClF1MxaQ8AgEzj4WpMlcEkftQZ+Z5+tqFukSBtaNKu6u8x5vf9mwomVnnvFOLaSy+ 7Qa60piQVLSdYYk8YB30IuPNvH4O1VtbSnNTBOft6gKZM9/iUs5i35pJFk/m6TNQGLhFvxyXqIR N4L5W5o3Bj3mXSmbGBoF/lujiz6OcMdy57NQs3h4leprC8ZgMQNS0H8yXPi8IQw1JCp0JP67Y96 oaFiV4Bt6ddDPkrC4AivT8Ijtq64ahVx7etMO+PkqVZpY1ixKfyoH1/NPq6oe4/GsqjSStSjf/r sUqFVdzFvwNaEVGslAkjhpfd1ed7E48yetJF/Ljkrca7r6OaMU3UGMDCOmlrS8xO5IIsuHxPBd+ SfsiZ0av0Vs5Pf0di5rJ9qSNMCAz7HyK2xBA== X-Received: by 2002:a05:600c:1550:b0:490:4f07:6d15 with SMTP id 5b1f17b1804b1-4904f076e3dmr375800285e9.17.1779964558385; Thu, 28 May 2026 03:35:58 -0700 (PDT) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4908eab0167sm13315625e9.16.2026.05.28.03.35.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 May 2026 03:35:57 -0700 (PDT) Date: Thu, 28 May 2026 13:35:54 +0300 From: Dan Carpenter To: Dawei Feng Cc: gregkh@linuxfoundation.org, omer.e.idrissi@gmail.com, hansg@kernel.org, hi@josie.lol, straube.linux@gmail.com, xela@viard.dev, ethantidmore06@gmail.com, liangjie@lixiang.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, jianhao.xu@seu.edu.cn, stable@vger.kernel.org, zilin@seu.edu.cn Subject: Re: [PATCH] staging: rtl8723bs: fix mismatched free of HalData in rtw_sdio_if1_init() Message-ID: References: <20260528101542.2395619-1-dawei.feng@seu.edu.cn> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260528101542.2395619-1-dawei.feng@seu.edu.cn> On Thu, May 28, 2026 at 06:15:42PM +0800, Dawei Feng wrote: > On Thu, May 28, 2026 at 15:30:18 Dan Carpenter wrote: > > > Manual inspection > > > confirms that the issue is still present in current mainline. > > > > > > An x86_64 allyesconfig build showed no new warnings. As we do not have > > > suitable RTL8723BS SDIO hardware to test with, no runtime testing was > > > able to be performed. > > > > to HERE should be put > > > > > > > > Fixes: 554c0a3abf21 ("staging: Add rtl8723bs sdio wifi driver") > > > Signed-off-by: Zilin Guan > > > Signed-off-by: Dawei Feng > > > --- > > ^^^ > > Here under the --- cut off line. We don't need this kind of meta > > commentary about testing in the permanent git log. Otherwise > > the patch is correct. > > Hi Dan, > > Thank you for the review and for pointing this out. > > The reason the manual inspection and testing commentary was placed above > the `---` line is that we were strictly following the example template > provided in Documentation/process/researcher-guidelines.rst. Ah, hm. Perhaps, the rules are changing. That's fine then. I would normally ask for a v2 but you copied from our template so that's on us. No need to resend. Reviewed-by: Dan Carpenter regards, dan carpenter