[PATCH 44/83] staging: brcm80211: replaced typedef si_t with struct si_pub

[Roland Vossen]

Hello Julian,

the structure definition itself is already in a header file (aiutils.h). 
Probably you are referring to the forward declaration used in the patch?

Bye, Roland.

On 06/02/2011 01:58 AM, Julian Calaby wrote:
> On Wed, Jun 1, 2011 at 21:45, Roland Vossen<rvossen at broadcom.com>  wrote:
>> Code cleanup.
>>
>> Signed-off-by: Roland Vossen<rvossen at broadcom.com>
>> Reviewed-by: Arend van Spriel<arend at broadcom.com>
>>
>> diff --git a/drivers/staging/brcm80211/brcmfmac/wl_iw.c b/drivers/staging/brcm80211/brcmfmac/wl_iw.c
>> index 2cc9bc7..9819a35 100644
>> --- a/drivers/staging/brcm80211/brcmfmac/wl_iw.c
>> +++ b/drivers/staging/brcm80211/brcmfmac/wl_iw.c
>> @@ -27,7 +27,8 @@
>>   #include<dngl_stats.h>
>>   #include<dhd.h>
>>   #include<linux/ieee80211.h>
>> -typedef const struct si_pub si_t;
>> +
>> +struct si_pub;
>>
>>   #include<dngl_stats.h>
>>   #include<dhd.h>
>
> Should this struct definition be in a header file somewhere?