[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20231013094503.6xnohuoivlysmlpp@zenone.zhora.eu>
Date: Fri, 13 Oct 2023 11:45:03 +0200
From: Andi Shyti <andi.shyti@...nel.org>
To: Calvince Otieno <calvncce@...il.com>
Cc: outreachy@...ts.linux.dev, linux-kernel@...r.kernel.org,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Luke Koch <lu.ale.koch@...il.com>,
Bagas Sanjaya <bagasdotme@...il.com>,
Simon Horman <horms@...nel.org>, linux-staging@...ts.linux.dev
Subject: Re: [PATCH] staging: wlan-ng: remove helper function
prism2sta_inf_handover()
Hi Calvince,
...
> -/*
> - * prism2sta_inf_handover
> - *
> - * Handles the receipt of a Handover info frame. Should only be present
> - * in APs only.
> - *
> - * Arguments:
> - * wlandev wlan device structure
> - * inf ptr to info frame (contents in hfa384x order)
> - *
> - * Returns:
> - * nothing
> - *
> - * Side effects:
> - *
> - * Call context:
> - * interrupt
> - */
> -static void prism2sta_inf_handover(struct wlandevice *wlandev,
> - struct hfa384x_inf_frame *inf)
> -{
> - pr_debug("received infoframe:HANDOVER (unhandled)\n");
> -}
> -
> /*
> * prism2sta_inf_tallies
> *
> @@ -1724,7 +1697,7 @@ void prism2sta_ev_info(struct wlandevice *wlandev,
> /* Dispatch */
> switch (inf->infotype) {
> case HFA384x_IT_HANDOVERADDR:
> - prism2sta_inf_handover(wlandev, inf);
> + pr_debug("received infoframe:HANDOVER (unhandled)\n");
while I think the patch is correct, I believe you should take
this opportunity to use a better printing function. I am not a
network developer, but I believe network people would prefer
netdev_dbg() at this point.
The pr_*() family is one of the most generic printing method,
useful for debug and few other situations. Other printing
functions should be preferred because the provide more context.
Andi
Powered by blists - more mailing lists