request for feedback before sending this patch

ozzloy ozzloy at gmail.com
Tue Jun 7 01:51:00 EDT 2022


i'd like to get some feedback before sending this off to the regular
mailing list.
i'm following Documentation/process/submitting-patches.rst and i think this
is OK, but i thought that before and was wrong.

i'm not even sure if i should be asking here first, or sending this to the
list.

are there any problems with the cleanup patch below?
it fixes something that checkpatch.pl lists as an error for pointer
location.

=========================================================================
>From 3ce7c7a3333aecaad6c28e0e1c17ce5b554274b0 Mon Sep 17 00:00:00 2001
From: Daniel Watson <ozzloy at challenge-bot.com>
Date: Mon, 6 Jun 2022 11:31:31 -0700
Subject: [PATCH] staging: r8188eu: fix declaration to match 'foo *bar'

put '*' next to variable to match kernel style from checkpatch

Signed-off-by: Daniel Watson <ozzloy at challenge-bot.com>
---
 drivers/staging/r8188eu/include/rtw_io.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/r8188eu/include/rtw_io.h
b/drivers/staging/r8188eu/include/rtw_io.h
index 6910e2b430e24..0e390eda5b4d3 100644
--- a/drivers/staging/r8188eu/include/rtw_io.h
+++ b/drivers/staging/r8188eu/include/rtw_io.h
@@ -283,7 +283,7 @@ void free_io_queue(struct adapter *adapter);
 void async_bus_io(struct io_queue *pio_q);
 void bus_sync_io(struct io_queue *pio_q);
 u32 _ioreq2rwmem(struct io_queue *pio_q);
-void dev_power_down(struct adapter * Adapter, u8 bpwrup);
+void dev_power_down(struct adapter *Adapter, u8 bpwrup);

 #define PlatformEFIOWrite1Byte(_a,_b,_c) \
  rtw_write8(_a,_b,_c)
-- 
2.34.1
=========================================================================



here's who i plan to send to:
Larry Finger <Larry.Finger at lwfinger.net>
Phillip Potter <phil at philpotter.co.uk>
Greg Kroah-Hartman <gregkh at linuxfoundation.org>
linux-staging at lists.linux.dev

should i include this list?
linux-kernel at vger.kernel.org
it is called a list of last resort here
https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html
but it is in the output of get-maintainer.pl for this patch.

thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20220606/01664a33/attachment.html>


More information about the Kernelnewbies mailing list