On 3/5/24 22:04, Dorine Tipo wrote:
Align the function parameters to silence a checkpatch warning.
This patch aligns parameters in functions calculate_ofdmr_parameter(),
card_set_phy_parameter(), card_update_tsf() and card_get_tsf_offset(),
for better readability.
Signed-off-by: Dorine Tipo <dorine.a.tipo@xxxxxxxxx>
---
Changelog:
V2: Modified subjects to be unique for each patch, as requested by
<gregkh@xxxxxxxxxxxxxxxxxxx>
Fixed the warnings in the patches as suggested by
<philipp.g.hortmann@xxxxxxxxx>
V3: Combined the vt6655 patches into one patch as suggested by
<dan.carpenter@xxxxxxxxxx>
V4: Removed the multiple signed-off-by as suggested by
<philipp.g.hortmann@xxxxxxxxx>
Omitted the cover letter as the changes have been combined into one
patch
Reworked the patch subject to be concise
V5: Fixed the commit message to "Align
the function parameters to silence a checkpatch warning." as
suggested by <dan.carpenter@xxxxxxxxxx>
V6: Fix trailing white spaces in the patch to silence checkpatch warning
V7: Edited the subject line to include file name as suggested by
<philipp.g.hortmann@xxxxxxxxx>
Hi Dorine,
what is the git repo you are using and which branch?
I cannot apply your patch. See below.
Check the git repo as shown below:
I hope we can find the reason... do you modify the patch with an other
editor?
Bye Philipp
kernel@matrix-ESPRIMO-P710:~/Documents/git/kernels/staging$ git remote
show origin
* remote origin
Fetch URL:
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
Push URL:
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
HEAD branch: master
Remote branches:
bus_cleanup tracked
greybus tracked
main tracked
master tracked
staging-linus tracked
staging-next tracked
staging-octeon tracked
staging-testing tracked
Local branches configured for 'git pull':
master merges with remote master
staging-next merges with remote staging-next
staging-testing merges with remote staging-testing
Local refs configured for 'git push':
master pushes to master (up to date)
staging-next pushes to staging-next (local out of date)
staging-testing pushes to staging-testing (up to date)
kernel@matrix-ESPRIMO-P710:~/Documents/git/kernels/staging$ git branch -a
arnd_berg
ath10k_shorted
checkout
master
rtl8192_next
staging-next
* staging-testing
remotes/origin/HEAD -> origin/master
remotes/origin/bus_cleanup
remotes/origin/greybus
remotes/origin/main
remotes/origin/master
remotes/origin/staging-linus
remotes/origin/staging-next
remotes/origin/staging-octeon
remotes/origin/staging-testing
kernel@matrix-ESPRIMO-P710:~/Documents/git/kernels/staging$ git status
On branch staging-testing
Your branch is up to date with 'origin/staging-testing'.
nothing to commit, working tree clean
kernel@matrix-ESPRIMO-P710:~/Documents/git/kernels/staging$ git pull
Already up to date.
kernel@matrix-ESPRIMO-P710:~/Documents/git/kernels/staging$ git apply
--reject --whitespace=fix ~/Downloads/20240305-\[PATCH\ v7\]\ staging_\
vt6655_\ Match\ open\ parentheses\ in\ card.c-14951.txt
Checking patch drivers/staging/vt6655/card.c...
error: while searching for:
* Return Value: none
*/
static void calculate_ofdmr_parameter(unsigned char rate,
u8 bb_type,
unsigned char *tx_rate,
unsigned char *rsv_time)
{
switch (rate) {
case RATE_6M:
error: patch failed: drivers/staging/vt6655/card.c:81
error: while searching for:
* Return Value: none
*/
bool card_update_tsf(struct vnt_private *priv, unsigned char rx_rate,
u64 bss_timestamp)
{
u64 local_tsf;
u64 tsf_offset = 0;
error: patch failed: drivers/staging/vt6655/card.c:288
error: while searching for:
if (bss_timestamp != local_tsf) {
tsf_offset = card_get_tsf_offset(rx_rate, bss_timestamp,
local_tsf);
/* adjust TSF, HW's TSF add TSF Offset reg */
tsf_offset = le64_to_cpu(tsf_offset);
iowrite32((u32)tsf_offset, priv->port_offset + MAC_REG_TSFOFST);
error: patch failed: drivers/staging/vt6655/card.c:297
error: while searching for:
* Return Value: true if succeed; otherwise false
*/
bool card_set_beacon_period(struct vnt_private *priv,
unsigned short beacon_interval)
{
u64 next_tbtt;
error: patch failed: drivers/staging/vt6655/card.c:321
Applying patch drivers/staging/vt6655/card.c with 4 rejects...
Rejected hunk #1.
Rejected hunk #2.
Rejected hunk #3.
Rejected hunk #4.
kernel@matrix-ESPRIMO-P710:~/Documents/git/kernels/staging$
