[PATCH 2/2] b43: implement BCMA bus ops

["Rafał Miłecki" <zajec5@gmail.com>]

W dniu 6 lipca 2011 17:40 u?ytkownik Michael B?sch <m@bues.ch> napisa?:
> On Tue, ?5 Jul 2011 23:54:08 +0200
> Rafa? Mi?ecki <zajec5@gmail.com> wrote:
>
>>
>> Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
>> ---
>> ?drivers/net/wireless/b43/bus.c ?| ?100 +++++++++++++++++++++++++++++++++++++++
>> ?drivers/net/wireless/b43/bus.h ?| ? ?3 +
>> ?drivers/net/wireless/b43/main.c | ? ?7 +++
>> ?3 files changed, 110 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/net/wireless/b43/bus.c b/drivers/net/wireless/b43/bus.c
>> index 4200713..f56604f 100644
>> --- a/drivers/net/wireless/b43/bus.c
>> +++ b/drivers/net/wireless/b43/bus.c
>> @@ -23,6 +23,106 @@
>> ?#include "b43.h"
>> ?#include "bus.h"
>>
>> +/* BCMA */
>> +#ifdef CONFIG_B43_BCMA
>> +static inline int b43_bus_bcma_bus_may_powerdown(struct b43_bus_dev *dev)
>> +{
>> + ? ? return 0; /* bcma_bus_may_powerdown(dev->bdev->bus); */
>> +}
>
> Why is this inline?
> You can't inline and take a pointer of the function at the same time.
> (The compiler will workaround this by generating multiple versions of
> the function, of which at least one is not inline).
> So remove it, please. Same below...

I didn't know about any such a limitation.

Just for sure: did you really mean "pointer of the function"? Cause my
function is taking pointer of the struct, not pointer of function.

-- 
Rafa?