From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f42.google.com (mail-wm1-f42.google.com [209.85.128.42]) (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 9EB66396588 for ; Wed, 25 Mar 2026 21:34:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774474498; cv=none; b=u4LJBqE2QrfAsMGccrDSvAwR3OVEMEdbtC4G5P4Sjv8Zj/kKRrrkLfcUN6Q39l4IDWf3bCTXZxcMHjzyLxS1HyeF03IxB4oL3/Un1S1c4tDqcA0yBj0XXcMjrvCww/Dk1CENV54qE8k2Q/YAZ4uAcqWdaPFNyXBm/wpTEkYdLho= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774474498; c=relaxed/simple; bh=ZQYFyxFfCiVFYjcUR80pgTmR9yPRGHtpS9PpvOl0IsU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Or0N/Ezk1G/QTNyUgz/tpj/KQOAgQE+TO8er1lZ3ofekHST+5yTCmGA+FlrJ7QdSCg65PQph9hOsdnVxL0Af5JADmjWZqi9XfKcHguyUxTDnyUS9D8sMavFwTzCA1NTL/18VwJCWtWxllH7hPmahCeBZ6l9Mgnu1dS1nzUyThBE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=tV8tbweQ; arc=none smtp.client-ip=209.85.128.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="tV8tbweQ" Received: by mail-wm1-f42.google.com with SMTP id 5b1f17b1804b1-4838c15e3cbso2574335e9.3 for ; Wed, 25 Mar 2026 14:34:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1774474495; x=1775079295; 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=aaERduj73kZVleeRSp8mEl5F7YmtzogBhrXc9iUxk2g=; b=tV8tbweQX0y3LWpidA3+m0EzJuYJO+lYJYPZ0FlFdIIhbNJfks50q6oi4JOl20YbVI Kjheq5uH2RS3gttt9DlPiXRFp7iUwrD67JceW22rBRNtRGk4Wj861NTuJ/Tf57HpC7Wt SKGQLW8dG2UXbTY/YI6Q/GrehquF+Ws/D11EzQf7hSpGlG8YPHvsrSs30Q5su1TRDDdj +cwXWDP0P4X/M1VSU5+K2qu/32jBK/8L8iiOVKW/tQLSnNQV0xpT0sBTjPCkff9rn6Ka TNkJ6HbHyUIU579zCKrwjFFV4fdFnUc0WFZFsWbbWgvEUByiGAZHJ9q13TdnXG9e+aNn kEuQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774474495; x=1775079295; 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=aaERduj73kZVleeRSp8mEl5F7YmtzogBhrXc9iUxk2g=; b=FntZd+V53+dQq5jExGHfTMNB3HkN2N4Rg0iBqTmZLoBSG4Jf1uAwY/K9vcPHU2olEk 0kroiBcz08sIDbZ5NgzfviBrsijY3zDiNgCd7AW4EY+ZrS9DKcGoHEZANYfkou6ABxSj WUA3flwqRBXcDHYFRXqzRmITd07TDV/L8p/HJjptxEkCxyNtLixKjtydlqMNBLPGgNrP 8QRj1bNDhrQ34WO5l+wniyP2xKyp5kohk/QIxDWpYDudtWhX4diIjLpemFgXYIewSQm3 4mfMdL93KKFNkW0+C1LPwhzoGEO7IXiSKHtlJSTTj7zy2SeIc4n0UdoO/6fDK7iPkJOt H93g== X-Forwarded-Encrypted: i=1; AJvYcCUtI7J0vcMrgNMzR78+SnE3l55us5GHy7c2Mys3NebwquTWUDX6+0x6Tr92RAqnVIVNc83lNospDutwoSo=@vger.kernel.org X-Gm-Message-State: AOJu0YyD7/aQAq4o5EFb/MZdTSTFEZClX1MkmSpxmFOjpDZeQmrZLp6p eVknXIggbJ6faA2e9qiBo+0OfEudjjNDiMtJq9LK4JKQU2Lfe3fSNG37PGALnxM7pW4= X-Gm-Gg: ATEYQzzPLbmC8LDDqhVXEDDdzZNTOaXtNnlV6+oFXstBVKAFu+gUvMnoZHOIRXSzzYS /Vhb/0zOPdQI279L8EACq33fifasbtZ3557dvQ4uzdFgDG/hGBTwK5xy9uD45OsOsY0E8GS/JiE LeMoVuhoUqLde28gpW+FCG3+2uzPsGTNJaZmWdJTYAdyQzHrgQIgKnqdzIHOsSf4qJrWNdZnnjd urU7Lo+9zC3SSgVsh1auYlWVE+5LDMu20T3G69a8+82EIsVLhgOhVzOOjmv69QhelD+n1+lUkH/ iITs3KbVSLxlZtJk1CITYSojWbi4D+26ZVv/Qz1lgiPdV7uZTZ6htOxg0w9gF1hbbh3EPGcikMV a0czHhvL4kTFKRqMSvSPGk50gk1ZAacsU/cIvxGe22V9Huxxe+H+SPF4sVQZfzGNGjp/KGQfD0V Cvsf18wZIoZGtEnHKALyv9zlWHq9g= X-Received: by 2002:a05:600c:45c4:b0:485:3dfc:57a with SMTP id 5b1f17b1804b1-487160717d3mr75241795e9.32.1774474494591; Wed, 25 Mar 2026 14:34:54 -0700 (PDT) Received: from localhost ([41.210.147.106]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-43b9194311asm3404640f8f.10.2026.03.25.14.34.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 25 Mar 2026 14:34:53 -0700 (PDT) Date: Thu, 26 Mar 2026 00:34:48 +0300 From: Dan Carpenter To: Shuah Khan Cc: Prithvi , Andy Shevchenko , gregkh@linuxfoundation.org, abrahamadekunle50@gmail.com, straube.linux@gmail.com, b9788213@gmail.com, ethantidmore06@gmail.com, weibu@redadmin.org, knavaneeth786@gmail.com, ignacio.pena87@gmail.com, dharanitharan725@gmail.com, lukagejak5@gmail.com, samasth.norway.ananda@oracle.com, karanja99erick@gmail.com, s9430939@naver.com, suunj1331@gmail.com, ysinghcin@gmail.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, linux-kernel-mentees@lists.linux.dev, david.hunter.linux@gmail.com, khalid@kernel.org Subject: Re: [PATCH v2] staging: rtl8723bs: fix constant on left side of test checkpatch warnings Message-ID: References: <20260323162901.121947-1-activprithvi@gmail.com> <20260324130234.3776tp4ykuerffcg@inspiron> <20260324135551.grth7jfjyoh5krzr@inspiron> <20260324160229.f46h25ebd2mxcfiq@inspiron> <656d3bf5-8d97-454e-9691-540cdb79dab4@linuxfoundation.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="BvSJJhSL1hENOWF8" Content-Disposition: inline In-Reply-To: --BvSJJhSL1hENOWF8 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, Mar 25, 2026 at 11:57:38AM -0600, Shuah Khan wrote: > My primary concern with this change is that it 10 files changed, with > 26 insertions(+), 32 deletions(-) > > It can be difficult to find any regressions unless the changes are > tested. I understand it is staging repo, but if we relax the rules > for staging irrespective of the scope of change, it becomes lot > harder to find regressions later. We are essentially leaving the > job of testing to users. Also if ans when the driver is pulled into > mainline, it will inherit the regressions that crept in. > To me this is a mechanical patch (basically automatic, flip the if statement) from a newbie. These kinds of patches are seldom a problem. I took a sample of the patches for five years from 2021 to the end of 2025. We merged 54406 commits in staging. We had 401 Fixes tags. But then I decided I don't care about Kconfig problems so I cut it down to 380 bugs. Half of those (189 commits) were from when the driver was merged. We tend to not review code very much when it is first merged to staging. The other half (191 commits) were from us missing bugs during review. 191 bugs over five years is 40 bugs per year. I hand reviewed a bunch of the 191 commits. It's mostly senior developers and maintainers who introduce bugs. They are normally doing complicated things like adding features or re-working core parts of the driver. Quite a few of these patches were tested. The fallout from newbie patches is pretty rare and often really minor. A common thing is "We deleted code but we didn't delete everything." The number of serious mess ups is probably just a couple times per year out of the 40 total bugs per year. I've attached my script so you can check yourself. NEW means the bug was introduced by a new driver. OLD means, ideally it would have been caught in review. regards, dan carpenter --BvSJJhSL1hENOWF8 Content-Type: application/x-sh Content-Disposition: attachment; filename="old_v_new.sh" Content-Transfer-Encoding: quoted-printable #!/bin/bash=0A=0A# set -e=0A=0ASTART=3D2021-01-01=0AEND=3D2026-01-01=0A=0AH= ASHES_W_FIX=3D$(git log --no-merges --before=3D2026-01-01 --after=3D2021-01= -01 --grep Fixes: --format=3D'%h' drivers/staging/)=0A=0Afor HASH in $HASHE= S_W_FIX ; do=0A # I don't care about Kconfigs=0A if ! git log -1 --na= me-only --format=3D'' $HASH | egrep -q '\.[ch]' ; then=0A continue= =0A fi=0A=0A # head -n1 so we only look at the first hash (lazy)=0A = LINE=3D$(git log -1 $HASH | egrep ' Fixes:' | head -n1)=0A=0A TAG= =3D$(echo $LINE | cut -d : -f 2 | cut -d '(' -f 1)=0A # a couple tags ar= e bad. check if they exist=0A if ! git cat-file -e $TAG 2> /dev/null ; = then=0A continue=0A fi=0A=0A AUTHOR=3D$(git log -1 --format=3D= '%an' $TAG)=0A=0A if git log -1 --name-status $TAG | grep ^A | egrep '\.= [ch]$' | grep -q "A" ; then=0A echo NEW $HASH $LINE=0A else=0A = echo OLD $HASH $LINE=0A fi=0Adone=0A=0A=0A --BvSJJhSL1hENOWF8--