From: Abraham Samuel Adekunle <abrahamadekunle50@gmail.com>
To: julia.lawall@inria.fr, gregkh@linuxfoundation.org
Cc: linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev,
dan.carpenter@linaro.org, andy@kernel.org,
Abraham Samuel Adekunle <abrahamadekunle50@gmail.com>
Subject: [PATCH v6 0/2] Cleanup duplicate NULL tests on a value
Date: Fri, 18 Apr 2025 15:41:55 +0000 [thread overview]
Message-ID: <cover.1744990405.git.abrahamadekunle50@gmail.com> (raw)
Changes in v5:
- Splitted patch v4 into a patchset with two patches so as to
proper explanation of each change due to different code patterns
in the change as suggested by Julia Lawall.
- Modified commit messages in each patch to properly explain the changes
show the initial NULL tests on the value.
- Added Subject title to each patch to be specific to the value being NULL tested.
Changed in v4:
- Separated initially integrated suggested change
"use modulo % 4096 over & 0xfff" to a different patch.
Changes in v3:
- Changed other cases to use modulo (% 4096) over (& 0xofff).
- Modified commit message to reflect these changes.
Changes in v2:
- Dropped patch files for media drivers from patchset as it is
not meant for outreachy applicants.
- Added full-stop aign to text in commit message.
- Made code more readable by adding a line break.
- Changed cases to use modulo (% 4096) over (& 0xfff).
Changes in v1
- Patch for drivers/staging/media/av7110/sp8870.c and
- drivers/staging/media/atomisp/pci/atomisp_gmin_platform.c
is split into two different patches in the patchset for use by the different
driver maintainers.
- Added subject title for each of the separated patches.
- Patch 1: Removed unnecessary curly braces {} initially inserted.
- Patch 2: Unnecessary {} was also removed for v1.
Abraham Samuel Adekunle (2):
staging: rtl8723bs: Remove duplicate NULL tests on pstat
staging: rtl8723bs: Prevent duplicate NULL tests on psta pointer
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 2 +-
drivers/staging/rtl8723bs/core/rtw_xmit.c | 62 +++++++++----------
2 files changed, 31 insertions(+), 33 deletions(-)
--
2.34.1
next reply other threads:[~2025-04-18 15:42 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-18 15:41 Abraham Samuel Adekunle [this message]
2025-04-18 15:41 ` [PATCH v6 1/2] staging: rtl8723bs: Remove duplicate NULL tests on pstat Abraham Samuel Adekunle
2025-04-19 16:23 ` Andy Shevchenko
2025-04-18 15:41 ` [PATCH v6 2/2] staging: rtl8723bs: Prevent duplicate NULL tests on psta pointer Abraham Samuel Adekunle
2025-04-19 16:25 ` Andy Shevchenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1744990405.git.abrahamadekunle50@gmail.com \
--to=abrahamadekunle50@gmail.com \
--cc=andy@kernel.org \
--cc=dan.carpenter@linaro.org \
--cc=gregkh@linuxfoundation.org \
--cc=julia.lawall@inria.fr \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.