[PATCH v2] staging: wlan-ng: remove helper function prism2sta_inf_handover()

[PATCH v2] staging: wlan-ng: remove helper function prism2sta_inf_handover()

[Calvince Otieno]

prism2sta_inf_handover() function basically calls pr_debug() to print
a literal string. This can be done by the parent function directly.

Signed-off-by: Calvince Otieno <calvncce@gmail.com>
---
Patch version v2:
	- Replace pr_debug() utility function with the module specific netdev_dbg()

 drivers/staging/wlan-ng/prism2sta.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/wlan-ng/prism2sta.c b/drivers/staging/wlan-ng/prism2sta.c
index caeceb89c9fa..cf07cf04e20c 100644
--- a/drivers/staging/wlan-ng/prism2sta.c
+++ b/drivers/staging/wlan-ng/prism2sta.c
@@ -1697,7 +1697,8 @@ void prism2sta_ev_info(struct wlandevice *wlandev,
 	/* Dispatch */
 	switch (inf->infotype) {
 	case HFA384x_IT_HANDOVERADDR:
-		pr_debug("received infoframe:HANDOVER (unhandled)\n");
+		netdev_dbg(wlandev->netdev,
+				"received infoframe:HANDOVER (unhandled)\n");
 		break;
 	case HFA384x_IT_COMMTALLIES:
 		prism2sta_inf_tallies(wlandev, inf);
-- 
Calvince Otieno

Re: [PATCH v2] staging: wlan-ng: remove helper function prism2sta_inf_handover()

[Andi Shyti]

Hi Calvince,

...

> @@ -1697,7 +1697,8 @@ void prism2sta_ev_info(struct wlandevice *wlandev,
>  	/* Dispatch */
>  	switch (inf->infotype) {
>  	case HFA384x_IT_HANDOVERADDR:
> -		pr_debug("received infoframe:HANDOVER (unhandled)\n");
> +		netdev_dbg(wlandev->netdev,
> +				"received infoframe:HANDOVER (unhandled)\n");

but then you generate this:

CHECK: Alignment should match open parenthesis
#240: FILE: drivers/staging/wlan-ng/prism2sta.c:1701:
+               netdev_dbg(wlandev->netdev,
+                               "received infoframe:HANDOVER (unhandled)\n");

total: 0 errors, 0 warnings, 1 checks, 9 lines checked

:-)

Make it a habit... before sending a patch check it with
checkpatch, in order to be sure you are not introducing other
checkpatch errors.

Don't rush... take your time, check your changes carefully; use
checkpatch, compile test, run sparse, etc. Give others time to
see your patch and comment.

Andi

Re: [PATCH v2] staging: wlan-ng: remove helper function prism2sta_inf_handover()

[Dan Carpenter]

On Fri, Oct 13, 2023 at 01:50:23PM +0300, Calvince Otieno wrote:
> prism2sta_inf_handover() function basically calls pr_debug() to print
> a literal string. This can be done by the parent function directly.
> 
> Signed-off-by: Calvince Otieno <calvncce@gmail.com>
> ---
> Patch version v2:
> 	- Replace pr_debug() utility function with the module specific netdev_dbg()

This isn't a v2 patch, right?  This patch assumes that we applied the
v1 patch...

This patch is not white space damaged though so that's good.

regards,
dan carpenter

Re: [PATCH v2] staging: wlan-ng: remove helper function prism2sta_inf_handover()

[Calvince Otieno]

On Fri, Oct 13, 2023 at 3:12 PM Dan Carpenter <dan.carpenter@linaro.org> wrote:
>
> On Fri, Oct 13, 2023 at 01:50:23PM +0300, Calvince Otieno wrote:
> > prism2sta_inf_handover() function basically calls pr_debug() to print
> > a literal string. This can be done by the parent function directly.
> >
> > Signed-off-by: Calvince Otieno <calvncce@gmail.com>
> > ---
> > Patch version v2:
> >       - Replace pr_debug() utility function with the module specific netdev_dbg()
>
> This isn't a v2 patch, right?  This patch assumes that we applied the
> v1 patch...
>
> This patch is not white space damaged though so that's good.
>
> regards,
> dan carpenter
>
>

Hello Dan,
Does this response mean that the patch was accepted?

--
Kind regards,
Calvince Otieno

Re: [PATCH v2] staging: wlan-ng: remove helper function prism2sta_inf_handover()

[Greg Kroah-Hartman]

On Fri, Oct 13, 2023 at 07:04:56PM +0300, Calvince Otieno wrote:
> On Fri, Oct 13, 2023 at 3:12 PM Dan Carpenter <dan.carpenter@linaro.org> wrote:
> >
> > On Fri, Oct 13, 2023 at 01:50:23PM +0300, Calvince Otieno wrote:
> > > prism2sta_inf_handover() function basically calls pr_debug() to print
> > > a literal string. This can be done by the parent function directly.
> > >
> > > Signed-off-by: Calvince Otieno <calvncce@gmail.com>
> > > ---
> > > Patch version v2:
> > >       - Replace pr_debug() utility function with the module specific netdev_dbg()
> >
> > This isn't a v2 patch, right?  This patch assumes that we applied the
> > v1 patch...
> >
> > This patch is not white space damaged though so that's good.
> >
> > regards,
> > dan carpenter
> >
> >
> 
> Hello Dan,
> Does this response mean that the patch was accepted?

No, sorry.

Re: [PATCH v2] staging: wlan-ng: remove helper function prism2sta_inf_handover()

[Dan Carpenter]

On Fri, Oct 13, 2023 at 07:04:56PM +0300, Calvince Otieno wrote:
> On Fri, Oct 13, 2023 at 3:12 PM Dan Carpenter <dan.carpenter@linaro.org> wrote:
> >
> > On Fri, Oct 13, 2023 at 01:50:23PM +0300, Calvince Otieno wrote:
> > > prism2sta_inf_handover() function basically calls pr_debug() to print
> > > a literal string. This can be done by the parent function directly.
> > >
> > > Signed-off-by: Calvince Otieno <calvncce@gmail.com>
> > > ---
> > > Patch version v2:
> > >       - Replace pr_debug() utility function with the module specific netdev_dbg()
> >
> > This isn't a v2 patch, right?  This patch assumes that we applied the
> > v1 patch...
> >
> > This patch is not white space damaged though so that's good.
> >
> > regards,
> > dan carpenter
> >
> >
> 
> Hello Dan,
> Does this response mean that the patch was accepted?

Ugh...  No, sorry.  I suck at emails.  White space damaged is not good.
I left out the word not...  :/

You need to figure out what's going on there.  Some of your patches are
not white space damaged so sometimes you're doing it right but here the
spaces were replaced with tabs so it doesn't apply.

regards,
dan carpenter

Re: [PATCH v2] staging: wlan-ng: remove helper function prism2sta_inf_handover()

[Dan Carpenter]

On Mon, Oct 16, 2023 at 07:40:57AM +0300, Dan Carpenter wrote:
> On Fri, Oct 13, 2023 at 07:04:56PM +0300, Calvince Otieno wrote:
> > On Fri, Oct 13, 2023 at 3:12 PM Dan Carpenter <dan.carpenter@linaro.org> wrote:
> > >
> > > On Fri, Oct 13, 2023 at 01:50:23PM +0300, Calvince Otieno wrote:
> > > > prism2sta_inf_handover() function basically calls pr_debug() to print
> > > > a literal string. This can be done by the parent function directly.
> > > >
> > > > Signed-off-by: Calvince Otieno <calvncce@gmail.com>
> > > > ---
> > > > Patch version v2:
> > > >       - Replace pr_debug() utility function with the module specific netdev_dbg()
> > >
> > > This isn't a v2 patch, right?  This patch assumes that we applied the
> > > v1 patch...
> > >
> > > This patch is not white space damaged though so that's good.
> > >
> > > regards,
> > > dan carpenter
> > >
> > >
> > 
> > Hello Dan,
> > Does this response mean that the patch was accepted?
> 
> Ugh...  No, sorry.  I suck at emails.  White space damaged is not good.
> I left out the word not...  :/
> 
> You need to figure out what's going on there.  Some of your patches are
> not white space damaged so sometimes you're doing it right but here the
> spaces were replaced with tabs so it doesn't apply.

Wow...  I absolutely suck at writing emails and *reading emails*.  This
wasn't the white space damaged patch.  This was the okay one.

However you sent a v2 patch in the wrong way.  You just posted a fixup
instead of resending the patch.  You've sent v2 patches before but let
me send you my blog link anyway...

https://staticthinking.wordpress.com/2022/07/27/how-to-send-a-v2-patch/

regards,
dan carpenter