Hi Andreas, On Thu, Feb 25, 2016 at 11:08 AM, Andreas Fenkart <afenkart@xxxxxxxxx> wrote: > Signed-off-by: Andreas Fenkart <afenkart@xxxxxxxxx> > --- > drivers/net/wireless/marvell/mwifiex/scan.c | 52 +++++++++++++---------------- > 1 file changed, 23 insertions(+), 29 deletions(-) > > diff --git a/drivers/net/wireless/marvell/mwifiex/scan.c b/drivers/net/wireless/marvell/mwifiex/scan.c > index 43d7a92..276382e 100644 > --- a/drivers/net/wireless/marvell/mwifiex/scan.c > +++ b/drivers/net/wireless/marvell/mwifiex/scan.c > @@ -76,6 +76,18 @@ static u8 mwifiex_rsn_oui[CIPHER_SUITE_MAX][4] = { > { 0x00, 0x0f, 0xac, 0x04 }, /* AES */ > }; > > +static bool > +has_ieee_hdr(struct ieee_types_generic *ie, u8 key) > +{ > + return (ie && (ie->ieee_hdr.element_id == key)); Please remove all the parentheses in this line as they're all unnecessary. > +} > + > +static bool > +has_vendor_hdr(struct ieee_types_vendor_specific *ie, u8 key) > +{ > + return (ie && (ie->vend_hdr.element_id == key)); Here too. > +} > + > /* > * This function parses a given IE for a given OUI. > * Thanks, -- Julian Calaby Email: julian.calaby@xxxxxxxxx Profile: http://www.google.com/profiles/julian.calaby/ -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html