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