Return-path: Received: from mail-gx0-f174.google.com ([209.85.161.174]:36388 "EHLO mail-gx0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753325Ab1FETX0 (ORCPT ); Sun, 5 Jun 2011 15:23:26 -0400 Received: by gxk21 with SMTP id 21so1251113gxk.19 for ; Sun, 05 Jun 2011 12:23:25 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1306928768-7501-1-git-send-email-rvossen@broadcom.com> <1306928768-7501-44-git-send-email-rvossen@broadcom.com> <4DE90DCE.6060207@broadcom.com> <4DEB3FF3.8090904@broadcom.com> Date: Sun, 5 Jun 2011 20:23:25 +0100 Message-ID: (sfid-20110605_212329_740672_CD235784) Subject: Re: [PATCH 44/83] staging: brcm80211: replaced typedef si_t with struct si_pub From: julie Sullivan To: Julian Calaby Cc: Roland Vossen , "devel@linuxdriverproject.org" , "gregkh@suse.de" , "linux-wireless@vger.kernel.org" Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-wireless-owner@vger.kernel.org List-ID: > > #include > > #include > #include > -typedef const struct si_pub si_t; > + > +struct si_pub; > > #include > #include Is there any reason why dngl_stats.h and dhd.h are included twice in wl_iw.c? (if not, maybe you corrected this in another patch, I couldn't get the series to apply cleanly so there was no easy way of checking. If so sorry for the noise) Cheers Julie