[PATCH v2 0/8] subject: staging: vt6655: Fix line wrapping in `RFvWriteWakeProgSyn`
From: Karolina Drobnik
Date: Mon Nov 01 2021 - 10:32:30 EST
This patch set is a series of small refactorings of the function
`RFvWriteWakeProgSyn`, now renamed to `rf_write_wake_prog_syn`.
The work here allowed to shorten lines marked by checkpatch.pl as
being too long by:
* introducing two new temporary variables (`idx` and `init_table`)
* rewriting a conditional to switch between two modes of
AL7320 initialization.
In addition to this, the patch set renames the function to align it with
the kernel coding style, updates the function description and removes two
unnecessary conditionals checking if the value of `init_count` is lower than
a predefined value.
Changes in v2:
* drop "staging: vt6655: Introduce `data` temporary variable" patch, suggested
by Julia Lawall <julia.lawall@xxxxxxxx>
* amend "staging: vt6655: Rewrite conditional in AL7320 initialization" patch
to use indexing and `init_table` temporary variable
* add two new patches to remove unnecessary checks for init count, suggested
by Mike Rapoport <mike.rapoport@xxxxxxxxx>:
- "staging: vt6655: Delete bogus check for `init_count` in AL2230"
- "staging: vt6655: Delete bogus check for `init_count` in AL7230"
Karolina Drobnik (8):
staging: vt6655: Introduce `idx` temporary variable
staging: vt6655: Use incrementation in `idx`
staging: vt6655: Remove unused `i` increments
staging: vt6655: Rewrite conditional in AL7320 initialization
staging: vt6655: Rename `RFvWriteWakeProgSyn` function
staging: vt6655: Update comment for `rf_write_wake_prog_syn`
staging: vt6655: Delete bogus check for `init_count` in AL2230
staging: vt6655: Delete bogus check for `init_count` in AL7230
drivers/staging/vt6655/channel.c | 2 +-
drivers/staging/vt6655/rf.c | 44 +++++++++++++-------------------
drivers/staging/vt6655/rf.h | 2 +-
3 files changed, 20 insertions(+), 28 deletions(-)
--
2.30.2