<div dir="ltr"><div>i'd like to get some feedback before sending this off to the regular mailing list.</div><div>i'm following Documentation/process/submitting-patches.rst and i think this</div><div>is OK, but i thought that before and was wrong.</div><div><br></div><div>i'm not even sure if i should be asking here first, or sending this to the list.</div><div><br></div><div>are there any problems with the cleanup patch below?</div><div>it fixes something that <a href="http://checkpatch.pl">checkpatch.pl</a> lists as an error for pointer location.<br></div><div><br></div>=========================================================================<br><div>From 3ce7c7a3333aecaad6c28e0e1c17ce5b554274b0 Mon Sep 17 00:00:00 2001<br>From: Daniel Watson <<a href="mailto:ozzloy@challenge-bot.com">ozzloy@challenge-bot.com</a>><br>Date: Mon, 6 Jun 2022 11:31:31 -0700<br>Subject: [PATCH] staging: r8188eu: fix declaration to match 'foo *bar'<br><br>put '*' next to variable to match kernel style from checkpatch<br><br>Signed-off-by: Daniel Watson <<a href="mailto:ozzloy@challenge-bot.com">ozzloy@challenge-bot.com</a>><br>---<br> drivers/staging/r8188eu/include/rtw_io.h | 2 +-<br> 1 file changed, 1 insertion(+), 1 deletion(-)<br><br>diff --git a/drivers/staging/r8188eu/include/rtw_io.h b/drivers/staging/r8188eu/include/rtw_io.h<br>index 6910e2b430e24..0e390eda5b4d3 100644<br>--- a/drivers/staging/r8188eu/include/rtw_io.h<br>+++ b/drivers/staging/r8188eu/include/rtw_io.h<br>@@ -283,7 +283,7 @@ void free_io_queue(struct adapter *adapter);<br> void async_bus_io(struct io_queue *pio_q);<br> void bus_sync_io(struct io_queue *pio_q);<br> u32 _ioreq2rwmem(struct io_queue *pio_q);<br>-void dev_power_down(struct adapter * Adapter, u8 bpwrup);<br>+void dev_power_down(struct adapter *Adapter, u8 bpwrup);<br> <br> #define PlatformEFIOWrite1Byte(_a,_b,_c)            \<br>    rtw_write8(_a,_b,_c)<br>-- <br>2.34.1<br>=========================================================================<br><br><br></div><div><br></div><div>here's who i plan to send to:</div><div>Larry Finger <<a href="mailto:Larry.Finger@lwfinger.net" target="_blank">Larry.Finger@lwfinger.net</a>></div>Phillip Potter <<a href="mailto:phil@philpotter.co.uk" target="_blank">phil@philpotter.co.uk</a>><br>Greg Kroah-Hartman <<a href="mailto:gregkh@linuxfoundation.org" target="_blank">gregkh@linuxfoundation.org</a>><br><div><a href="mailto:linux-staging@lists.linux.dev" target="_blank">linux-staging@lists.linux.dev</a></div><div><br></div><div>should i include this list?<br></div><div><a href="mailto:linux-kernel@vger.kernel.org" target="_blank">linux-kernel@vger.kernel.org</a></div><div>it is called a list of last resort here <a href="https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html" target="_blank">https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html</a></div><div>but it is in the output of <a href="http://get-maintainer.pl">get-maintainer.pl</a> for this patch.</div><div><br></div><div>thanks!<br></div></div>