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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxx>  wrote:
Code cleanup.

Signed-off-by: Roland Vossen<rvossen@xxxxxxxxxxxx>
Reviewed-by: Arend van Spriel<arend@xxxxxxxxxxxx>

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?


_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux