request for feedback before sending this patch

[ozzloy <ozzloy@xxxxxxxxx>]

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@xxxxxxxxxxxxxxxxx>
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@xxxxxxxxxxxxxxxxx>
---
 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@xxxxxxxxxxxx>

Phillip Potter <phil@xxxxxxxxxxxxxxxx>
Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>

linux-staging@xxxxxxxxxxxxxxx

should i include this list?

linux-kernel@xxxxxxxxxxxxxxx

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!

_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies