[Bridge] [PATCH] brcmfmac: drop Inter-Access Point Protocol packets by default

Kalle Valo kvalo at codeaurora.org
Wed Mar 14 16:10:21 UTC 2018


Rafał Miłecki <rafal at milecki.pl> writes:

>>> +	unsigned char *eth_data = skb_mac_header(skb) + ETH_HLEN;
>>> +#if !defined(CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS)
>>
>> #ifndef?
>
> I followed what is used in the include/linux/etherdevice.h. Is that a
> good exceuse? Could it be there any some good reason for #if defined()?

Don't know, maybe just a matter of taste? But it would be nice to know
the background behind #ifdef vs #if defined(), never figured it out why
two different forms.

-- 
Kalle Valo


More information about the Bridge mailing list