linux-staging.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: Vivek BalachandharTN <vivek.balachandhar@gmail.com>
To: gregkh@linuxfoundation.org
Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org,
	vivek.balachandhar@gmail.com
Subject: [PATCH v3 00/15] rtl8723bs: cleanup and style improvements for better readability
Date: Mon, 29 Sep 2025 06:49:45 +0000	[thread overview]
Message-ID: <20250929065000.1010848-1-vivek.balachandhar@gmail.com> (raw)

This patch series includes 15 commits that clean up and improve
the style and formatting of the rtl8723bs driver in the staging tree.
The changes address spacing issues, indentation, comment formatting,
blank lines, and minor code clarity improvements.

No functional changes are introduced. All commits adhere to the Linux kernel
coding style guidelines to enhance code readability and maintainability.

The patches have been tested and are ready for review.

Changes in v2:
- Resent series as one properly threaded batch
- No content changes

Changes in v3:
- Rebased onto latest origin/staging-testing.
- Dropped the “remove unnecessary braces for single statement blocks” patch,
  which created empty-if (comment-only) blocks; feedback noted this was wrong.

Link to v2:
  https://lore.kernel.org/all/20250904021448.216461-1-vivek.balachandhar@gmail.com/

Vivek BalachandharTN (15):
  staging: rtl8723bs: fix spacing around operators
  staging: rtl8723bs: remove unnecessary space after type cast
  staging: rtl8723bs: fix overlong lines to conform with kernel coding
    style
  staging: rtl8723bs: align asterisk in block comment to fix formatting
  staging: rtl8723bs: fix logical continuation style by moving to
    previous line
  staging: rtl8723bs: fix indentation to align with open parenthesis
  staging: rtl8723bs: adding asterisks in multi-line block comments
  staging: rtl8723bs: remove space before semicolon
  staging: rtl8723bs: fix excessive indentation in nested if statement
  staging: rtl8723bs: fix unbalanced braces around conditional blocks
  staging: rtl8723bs: remove unnecessary parentheses in conditional
  staging: rtl8723bs: add braces to all arms of conditional statement
  staging: rtl8723bs: fix line ending with '('
  staging: rtl8723bs: place constant on right side of comparison
  staging: rtl8723bs: merge nested if conditions for clarity and tab
    problems

 drivers/staging/rtl8723bs/core/rtw_mlme.c | 415 +++++++++++++---------
 1 file changed, 250 insertions(+), 165 deletions(-)

-- 
2.39.5


             reply	other threads:[~2025-09-29  6:50 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-29  6:49 Vivek BalachandharTN [this message]
2025-09-29  6:49 ` [PATCH v3 01/15] staging: rtl8723bs: fix spacing around operators Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 02/15] staging: rtl8723bs: remove unnecessary space after type cast Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 03/15] staging: rtl8723bs: fix overlong lines to conform with kernel coding style Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 04/15] staging: rtl8723bs: align asterisk in block comment to fix formatting Vivek BalachandharTN
2025-09-29  6:58   ` Greg KH
2025-09-30  8:29     ` Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 05/15] staging: rtl8723bs: fix logical continuation style by moving to previous line Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 06/15] staging: rtl8723bs: fix indentation to align with open parenthesis Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 07/15] staging: rtl8723bs: adding asterisks in multi-line block comments Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 08/15] staging: rtl8723bs: remove space before semicolon Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 09/15] staging: rtl8723bs: fix excessive indentation in nested if statement Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 10/15] staging: rtl8723bs: fix unbalanced braces around conditional blocks Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 11/15] staging: rtl8723bs: remove unnecessary parentheses in conditional Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 12/15] staging: rtl8723bs: add braces to all arms of conditional statement Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 13/15] staging: rtl8723bs: fix line ending with '(' Vivek BalachandharTN
2025-09-29  6:49 ` [PATCH v3 14/15] staging: rtl8723bs: place constant on right side of comparison Vivek BalachandharTN
2025-09-29  6:50 ` [PATCH v3 15/15] staging: rtl8723bs: merge nested if conditions for clarity and tab problems Vivek BalachandharTN

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=20250929065000.1010848-1-vivek.balachandhar@gmail.com \
    --to=vivek.balachandhar@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).