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

From: Dan Carpenter
Date: Mon Oct 16 2023 - 00:42:12 EST


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@xxxxxxxxxx> 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@xxxxxxxxx>
> > > ---
> > > 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