On Fri, Jun 11, 2010 at 5:51 AM, Bruno Randolf <b...@einfach.org> wrote:
> +/** ath5k_frame_receive_ok() - Do we want to receive this frame or not?
> + *
> + * Check if we want to further process this frame or not. Also update
> + * statistics. Return 1 if we want this frame, 0 if not.
> + */
> +static int
> +ath5k_receive_frame_ok(struct ath5k_softc *sc, struct ath5k_rx_status *rs)
> +{

Please use bool here.  This function does a lot, I wonder if it can't be
further split?

-- 
Bob Copeland %% www.bobcopeland.com
_______________________________________________
ath5k-devel mailing list
ath5k-devel@lists.ath5k.org
https://lists.ath5k.org/mailman/listinfo/ath5k-devel

Reply via email to