<div dir="ltr"><div>lol, when do you sleep?</div><div>you reply so quickly, and did so last time i tried too.<br></div><div>thanks for the feedback.  much appreciated!<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 6, 2022 at 10:59 PM Greg KH <<a href="mailto:greg@kroah.com">greg@kroah.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Jun 06, 2022 at 10:51:00PM -0700, ozzloy wrote:<br>
> i'd like to get some feedback before sending this off to the regular<br>
> mailing list.<br>
> i'm following Documentation/process/submitting-patches.rst and i think this<br>
> is OK, but i thought that before and was wrong.<br>
> <br>
> i'm not even sure if i should be asking here first, or sending this to the<br>
> list.<br>
> <br>
> are there any problems with the cleanup patch below?<br>
> it fixes something that <a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a> lists as an error for pointer<br>
> location.<br>
> <br>
> =========================================================================<br>
> >From 3ce7c7a3333aecaad6c28e0e1c17ce5b554274b0 Mon Sep 17 00:00:00 2001<br>
> From: Daniel Watson <<a href="mailto:ozzloy@challenge-bot.com" target="_blank">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" target="_blank">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<br>
> 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>
Looks correct to me, send away!<br>
<br>
> =========================================================================<br>
> <br>
> <br>
> <br>
> here's who i plan to send to:<br>
> Larry Finger <<a href="mailto:Larry.Finger@lwfinger.net" target="_blank">Larry.Finger@lwfinger.net</a>><br>
> 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>
> <a href="mailto:linux-staging@lists.linux.dev" target="_blank">linux-staging@lists.linux.dev</a><br>
<br>
Yes.<br>
<br>
> <br>
> should i include this list?<br>
> <a href="mailto:linux-kernel@vger.kernel.org" target="_blank">linux-kernel@vger.kernel.org</a><br>
<br>
If you really want to, sure.  It's not required.<br>
<br>
> it is called a list of last resort here<br>
> <a href="https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html" rel="noreferrer" target="_blank">https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html</a><br>
> but it is in the output of <a href="http://get-maintainer.pl" rel="noreferrer" target="_blank">get-maintainer.pl</a> for this patch.<br>
<br>
It's in the output of all runs of that tool, as you say, it's the last<br>
resort.<br>
<br>
good luck!<br>
<br>
greg k-h<br>
</blockquote></div>