linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* pull-request: iwlwifi-next 2012-11-21
@ 2012-11-21 14:25 Johannes Berg
  2012-11-21 18:21 ` John W. Linville
  0 siblings, 1 reply; 4+ messages in thread
From: Johannes Berg @ 2012-11-21 14:25 UTC (permalink / raw)
  To: John Linville; +Cc: linux-wireless

John,

In his quest to try to understand the current iwlwifi problems (like
stuck queues etc.) Emmanuel has first cleaned up the PCIe code, I'm
including his changes in this pull request. Other than that I only have
a small cleanup from Sachin Kamat to remove a duplicate include and a
bugfix to turn off MFP if software crypto is enabled, but this isn't
really interesting as MFP isn't supported right now anyway.

Please pull.

Thanks,
johannes

The following changes since commit d18aa87fbfe80f33076942d11f19c9d813e835b1:

  iwlwifi: return commands with error on FW error (2012-11-09 20:55:14 +0100)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/iwlwifi/iwlwifi-next.git for-john

for you to fetch changes up to eea54c8ec971d4759c541dba351477dafc39ce54:

  iwlwifi: remove effectless assignment (2012-11-21 11:30:39 +0100)

----------------------------------------------------------------
Emmanuel Grumbach (14):
      iwlwifi: don't warn if transport's allocation failed
      iwlwifi: don't enable interrupt as a W/A when MSI is enabled
      iwlwifi: add comments for the PCIe transport statuses
      iwlwifi: rename functions in transport layer
      iwlwifi: init the replenish work in rx_init
      iwlwifi: continue clean up - pcie/rx.c
      iwlwifi: continue clean up - pcie/trans.c
      iwlwifi: continue clean up - pcie/tx.c
      iwlwifi: merge 2 functions in reclaim flow
      iwlwifi: make iwl_queue_used return bool
      iwlwifi: more cleanup in pcie/rx.c
      iwlwifi: make iwl_pcie_rxq_inc_wr_ptr static
      iwlwifi: update the RB_TIMEOUT to 0x11
      iwlwifi: remove effectless assignment

Johannes Berg (1):
      iwlwifi: disallow MFP with software crypto

Sachin Kamat (1):
      iwlwifi: Remove duplicate inclusion of iwl-trans.h in pcie/drv.c

 drivers/net/wireless/iwlwifi/dvm/mac80211.c  |    9 +-
 drivers/net/wireless/iwlwifi/dvm/main.c      |    2 -
 drivers/net/wireless/iwlwifi/iwl-config.h    |    2 +-
 drivers/net/wireless/iwlwifi/iwl-fh.h        |    2 +-
 drivers/net/wireless/iwlwifi/pcie/drv.c      |    1 -
 drivers/net/wireless/iwlwifi/pcie/internal.h |  104 +--
 drivers/net/wireless/iwlwifi/pcie/rx.c       |  357 ++++++--
 drivers/net/wireless/iwlwifi/pcie/trans.c    | 1018 ++---------------------
 drivers/net/wireless/iwlwifi/pcie/tx.c       | 1146 ++++++++++++++++++++------
 9 files changed, 1299 insertions(+), 1342 deletions(-)



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: pull-request: iwlwifi-next 2012-11-21
  2012-11-21 14:25 pull-request: iwlwifi-next 2012-11-21 Johannes Berg
@ 2012-11-21 18:21 ` John W. Linville
  2012-11-21 18:37   ` John W. Linville
  0 siblings, 1 reply; 4+ messages in thread
From: John W. Linville @ 2012-11-21 18:21 UTC (permalink / raw)
  To: Johannes Berg; +Cc: linux-wireless

Hmmm...building wireless-next with an allyesconfig:

  CC      drivers/net/wireless/iwlwifi/pcie/trans.o
drivers/net/wireless/iwlwifi/pcie/trans.c: In function ‘iwl_trans_pcie_alloc’:
drivers/net/wireless/iwlwifi/pcie/trans.c:1288:3: error: label ‘out_pci_release_regions’ used but not defined
drivers/net/wireless/iwlwifi/pcie/trans.c:1281:3: error: label ‘out_pci_disable_device’ used but not defined
drivers/net/wireless/iwlwifi/pcie/trans.c:1258:3: error: label ‘out_no_pci’ used but not defined
drivers/net/wireless/iwlwifi/pcie/trans.c: At top level:
drivers/net/wireless/iwlwifi/pcie/trans.c:1306:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1307:12: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1308:12: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1309:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1310:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1311:16: error: expected ‘)’ before ‘->’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1315:2: error: expected identifier or ‘(’ before ‘do’
drivers/net/wireless/iwlwifi/pcie/trans.c:1315:2: error: expected identifier or ‘(’ before ‘while’
drivers/net/wireless/iwlwifi/pcie/trans.c:1316:2: error: expected identifier or ‘(’ before ‘do’
drivers/net/wireless/iwlwifi/pcie/trans.c:1316:2: error: expected identifier or ‘(’ before ‘while’
drivers/net/wireless/iwlwifi/pcie/trans.c:1318:16: error: expected ‘)’ before ‘->’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1321:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1322:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1330:2: error: expected identifier or ‘(’ before ‘if’
drivers/net/wireless/iwlwifi/pcie/trans.c:1333:2: error: expected identifier or ‘(’ before ‘return’
drivers/net/wireless/iwlwifi/pcie/trans.c:1335:20: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1337:24: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1339:23: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1341:11: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
drivers/net/wireless/iwlwifi/pcie/trans.c:1343:2: error: expected identifier or ‘(’ before ‘return’
drivers/net/wireless/iwlwifi/pcie/trans.c:1344:1: error: expected identifier or ‘(’ before ‘}’ token
drivers/net/wireless/iwlwifi/pcie/trans.c: In function ‘iwl_trans_pcie_alloc’:
drivers/net/wireless/iwlwifi/pcie/trans.c:1304:2: warning: control reaches end of non-void function [-Wreturn-type]
make[2]: *** [drivers/net/wireless/iwlwifi/pcie/trans.o] Error 1

On Wed, Nov 21, 2012 at 03:25:14PM +0100, Johannes Berg wrote:
> John,
> 
> In his quest to try to understand the current iwlwifi problems (like
> stuck queues etc.) Emmanuel has first cleaned up the PCIe code, I'm
> including his changes in this pull request. Other than that I only have
> a small cleanup from Sachin Kamat to remove a duplicate include and a
> bugfix to turn off MFP if software crypto is enabled, but this isn't
> really interesting as MFP isn't supported right now anyway.
> 
> Please pull.
> 
> Thanks,
> johannes
> 
> The following changes since commit d18aa87fbfe80f33076942d11f19c9d813e835b1:
> 
>   iwlwifi: return commands with error on FW error (2012-11-09 20:55:14 +0100)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/iwlwifi/iwlwifi-next.git for-john
> 
> for you to fetch changes up to eea54c8ec971d4759c541dba351477dafc39ce54:
> 
>   iwlwifi: remove effectless assignment (2012-11-21 11:30:39 +0100)
> 
> ----------------------------------------------------------------
> Emmanuel Grumbach (14):
>       iwlwifi: don't warn if transport's allocation failed
>       iwlwifi: don't enable interrupt as a W/A when MSI is enabled
>       iwlwifi: add comments for the PCIe transport statuses
>       iwlwifi: rename functions in transport layer
>       iwlwifi: init the replenish work in rx_init
>       iwlwifi: continue clean up - pcie/rx.c
>       iwlwifi: continue clean up - pcie/trans.c
>       iwlwifi: continue clean up - pcie/tx.c
>       iwlwifi: merge 2 functions in reclaim flow
>       iwlwifi: make iwl_queue_used return bool
>       iwlwifi: more cleanup in pcie/rx.c
>       iwlwifi: make iwl_pcie_rxq_inc_wr_ptr static
>       iwlwifi: update the RB_TIMEOUT to 0x11
>       iwlwifi: remove effectless assignment
> 
> Johannes Berg (1):
>       iwlwifi: disallow MFP with software crypto
> 
> Sachin Kamat (1):
>       iwlwifi: Remove duplicate inclusion of iwl-trans.h in pcie/drv.c
> 
>  drivers/net/wireless/iwlwifi/dvm/mac80211.c  |    9 +-
>  drivers/net/wireless/iwlwifi/dvm/main.c      |    2 -
>  drivers/net/wireless/iwlwifi/iwl-config.h    |    2 +-
>  drivers/net/wireless/iwlwifi/iwl-fh.h        |    2 +-
>  drivers/net/wireless/iwlwifi/pcie/drv.c      |    1 -
>  drivers/net/wireless/iwlwifi/pcie/internal.h |  104 +--
>  drivers/net/wireless/iwlwifi/pcie/rx.c       |  357 ++++++--
>  drivers/net/wireless/iwlwifi/pcie/trans.c    | 1018 ++---------------------
>  drivers/net/wireless/iwlwifi/pcie/tx.c       | 1146 ++++++++++++++++++++------
>  9 files changed, 1299 insertions(+), 1342 deletions(-)
> 
> 
> 

-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: pull-request: iwlwifi-next 2012-11-21
  2012-11-21 18:21 ` John W. Linville
@ 2012-11-21 18:37   ` John W. Linville
  2012-11-21 19:43     ` John W. Linville
  0 siblings, 1 reply; 4+ messages in thread
From: John W. Linville @ 2012-11-21 18:37 UTC (permalink / raw)
  To: Johannes Berg; +Cc: linux-wireless

There was a merge conflict on that file, I may have screwed it up...

On Wed, Nov 21, 2012 at 01:21:32PM -0500, John W. Linville wrote:
> Hmmm...building wireless-next with an allyesconfig:
> 
>   CC      drivers/net/wireless/iwlwifi/pcie/trans.o
> drivers/net/wireless/iwlwifi/pcie/trans.c: In function ‘iwl_trans_pcie_alloc’:
> drivers/net/wireless/iwlwifi/pcie/trans.c:1288:3: error: label ‘out_pci_release_regions’ used but not defined
> drivers/net/wireless/iwlwifi/pcie/trans.c:1281:3: error: label ‘out_pci_disable_device’ used but not defined
> drivers/net/wireless/iwlwifi/pcie/trans.c:1258:3: error: label ‘out_no_pci’ used but not defined
> drivers/net/wireless/iwlwifi/pcie/trans.c: At top level:
> drivers/net/wireless/iwlwifi/pcie/trans.c:1306:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1307:12: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1308:12: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1309:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1310:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1311:16: error: expected ‘)’ before ‘->’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1315:2: error: expected identifier or ‘(’ before ‘do’
> drivers/net/wireless/iwlwifi/pcie/trans.c:1315:2: error: expected identifier or ‘(’ before ‘while’
> drivers/net/wireless/iwlwifi/pcie/trans.c:1316:2: error: expected identifier or ‘(’ before ‘do’
> drivers/net/wireless/iwlwifi/pcie/trans.c:1316:2: error: expected identifier or ‘(’ before ‘while’
> drivers/net/wireless/iwlwifi/pcie/trans.c:1318:16: error: expected ‘)’ before ‘->’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1321:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1322:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1330:2: error: expected identifier or ‘(’ before ‘if’
> drivers/net/wireless/iwlwifi/pcie/trans.c:1333:2: error: expected identifier or ‘(’ before ‘return’
> drivers/net/wireless/iwlwifi/pcie/trans.c:1335:20: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1337:24: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1339:23: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1341:11: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c:1343:2: error: expected identifier or ‘(’ before ‘return’
> drivers/net/wireless/iwlwifi/pcie/trans.c:1344:1: error: expected identifier or ‘(’ before ‘}’ token
> drivers/net/wireless/iwlwifi/pcie/trans.c: In function ‘iwl_trans_pcie_alloc’:
> drivers/net/wireless/iwlwifi/pcie/trans.c:1304:2: warning: control reaches end of non-void function [-Wreturn-type]
> make[2]: *** [drivers/net/wireless/iwlwifi/pcie/trans.o] Error 1
> 
> On Wed, Nov 21, 2012 at 03:25:14PM +0100, Johannes Berg wrote:
> > John,
> > 
> > In his quest to try to understand the current iwlwifi problems (like
> > stuck queues etc.) Emmanuel has first cleaned up the PCIe code, I'm
> > including his changes in this pull request. Other than that I only have
> > a small cleanup from Sachin Kamat to remove a duplicate include and a
> > bugfix to turn off MFP if software crypto is enabled, but this isn't
> > really interesting as MFP isn't supported right now anyway.
> > 
> > Please pull.
> > 
> > Thanks,
> > johannes
> > 
> > The following changes since commit d18aa87fbfe80f33076942d11f19c9d813e835b1:
> > 
> >   iwlwifi: return commands with error on FW error (2012-11-09 20:55:14 +0100)
> > 
> > are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/iwlwifi/iwlwifi-next.git for-john
> > 
> > for you to fetch changes up to eea54c8ec971d4759c541dba351477dafc39ce54:
> > 
> >   iwlwifi: remove effectless assignment (2012-11-21 11:30:39 +0100)
> > 
> > ----------------------------------------------------------------
> > Emmanuel Grumbach (14):
> >       iwlwifi: don't warn if transport's allocation failed
> >       iwlwifi: don't enable interrupt as a W/A when MSI is enabled
> >       iwlwifi: add comments for the PCIe transport statuses
> >       iwlwifi: rename functions in transport layer
> >       iwlwifi: init the replenish work in rx_init
> >       iwlwifi: continue clean up - pcie/rx.c
> >       iwlwifi: continue clean up - pcie/trans.c
> >       iwlwifi: continue clean up - pcie/tx.c
> >       iwlwifi: merge 2 functions in reclaim flow
> >       iwlwifi: make iwl_queue_used return bool
> >       iwlwifi: more cleanup in pcie/rx.c
> >       iwlwifi: make iwl_pcie_rxq_inc_wr_ptr static
> >       iwlwifi: update the RB_TIMEOUT to 0x11
> >       iwlwifi: remove effectless assignment
> > 
> > Johannes Berg (1):
> >       iwlwifi: disallow MFP with software crypto
> > 
> > Sachin Kamat (1):
> >       iwlwifi: Remove duplicate inclusion of iwl-trans.h in pcie/drv.c
> > 
> >  drivers/net/wireless/iwlwifi/dvm/mac80211.c  |    9 +-
> >  drivers/net/wireless/iwlwifi/dvm/main.c      |    2 -
> >  drivers/net/wireless/iwlwifi/iwl-config.h    |    2 +-
> >  drivers/net/wireless/iwlwifi/iwl-fh.h        |    2 +-
> >  drivers/net/wireless/iwlwifi/pcie/drv.c      |    1 -
> >  drivers/net/wireless/iwlwifi/pcie/internal.h |  104 +--
> >  drivers/net/wireless/iwlwifi/pcie/rx.c       |  357 ++++++--
> >  drivers/net/wireless/iwlwifi/pcie/trans.c    | 1018 ++---------------------
> >  drivers/net/wireless/iwlwifi/pcie/tx.c       | 1146 ++++++++++++++++++++------
> >  9 files changed, 1299 insertions(+), 1342 deletions(-)
> > 
> > 
> > 
> 
> -- 
> John W. Linville		Someday the world will need a hero, and you
> linville@tuxdriver.com			might be all we have.  Be ready.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: pull-request: iwlwifi-next 2012-11-21
  2012-11-21 18:37   ` John W. Linville
@ 2012-11-21 19:43     ` John W. Linville
  0 siblings, 0 replies; 4+ messages in thread
From: John W. Linville @ 2012-11-21 19:43 UTC (permalink / raw)
  To: Johannes Berg; +Cc: linux-wireless

OK, figured it out...

On Wed, Nov 21, 2012 at 01:37:56PM -0500, John W. Linville wrote:
> There was a merge conflict on that file, I may have screwed it up...
> 
> On Wed, Nov 21, 2012 at 01:21:32PM -0500, John W. Linville wrote:
> > Hmmm...building wireless-next with an allyesconfig:
> > 
> >   CC      drivers/net/wireless/iwlwifi/pcie/trans.o
> > drivers/net/wireless/iwlwifi/pcie/trans.c: In function ‘iwl_trans_pcie_alloc’:
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1288:3: error: label ‘out_pci_release_regions’ used but not defined
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1281:3: error: label ‘out_pci_disable_device’ used but not defined
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1258:3: error: label ‘out_no_pci’ used but not defined
> > drivers/net/wireless/iwlwifi/pcie/trans.c: At top level:
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1306:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1307:12: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1308:12: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1309:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1310:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1311:16: error: expected ‘)’ before ‘->’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1315:2: error: expected identifier or ‘(’ before ‘do’
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1315:2: error: expected identifier or ‘(’ before ‘while’
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1316:2: error: expected identifier or ‘(’ before ‘do’
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1316:2: error: expected identifier or ‘(’ before ‘while’
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1318:16: error: expected ‘)’ before ‘->’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1321:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1322:7: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘->’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1330:2: error: expected identifier or ‘(’ before ‘if’
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1333:2: error: expected identifier or ‘(’ before ‘return’
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1335:20: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1337:24: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1339:23: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1341:11: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘:’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1343:2: error: expected identifier or ‘(’ before ‘return’
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1344:1: error: expected identifier or ‘(’ before ‘}’ token
> > drivers/net/wireless/iwlwifi/pcie/trans.c: In function ‘iwl_trans_pcie_alloc’:
> > drivers/net/wireless/iwlwifi/pcie/trans.c:1304:2: warning: control reaches end of non-void function [-Wreturn-type]
> > make[2]: *** [drivers/net/wireless/iwlwifi/pcie/trans.o] Error 1
> > 
> > On Wed, Nov 21, 2012 at 03:25:14PM +0100, Johannes Berg wrote:
> > > John,
> > > 
> > > In his quest to try to understand the current iwlwifi problems (like
> > > stuck queues etc.) Emmanuel has first cleaned up the PCIe code, I'm
> > > including his changes in this pull request. Other than that I only have
> > > a small cleanup from Sachin Kamat to remove a duplicate include and a
> > > bugfix to turn off MFP if software crypto is enabled, but this isn't
> > > really interesting as MFP isn't supported right now anyway.
> > > 
> > > Please pull.
> > > 
> > > Thanks,
> > > johannes
> > > 
> > > The following changes since commit d18aa87fbfe80f33076942d11f19c9d813e835b1:
> > > 
> > >   iwlwifi: return commands with error on FW error (2012-11-09 20:55:14 +0100)
> > > 
> > > are available in the git repository at:
> > > 
> > >   git://git.kernel.org/pub/scm/linux/kernel/git/iwlwifi/iwlwifi-next.git for-john
> > > 
> > > for you to fetch changes up to eea54c8ec971d4759c541dba351477dafc39ce54:
> > > 
> > >   iwlwifi: remove effectless assignment (2012-11-21 11:30:39 +0100)
> > > 
> > > ----------------------------------------------------------------
> > > Emmanuel Grumbach (14):
> > >       iwlwifi: don't warn if transport's allocation failed
> > >       iwlwifi: don't enable interrupt as a W/A when MSI is enabled
> > >       iwlwifi: add comments for the PCIe transport statuses
> > >       iwlwifi: rename functions in transport layer
> > >       iwlwifi: init the replenish work in rx_init
> > >       iwlwifi: continue clean up - pcie/rx.c
> > >       iwlwifi: continue clean up - pcie/trans.c
> > >       iwlwifi: continue clean up - pcie/tx.c
> > >       iwlwifi: merge 2 functions in reclaim flow
> > >       iwlwifi: make iwl_queue_used return bool
> > >       iwlwifi: more cleanup in pcie/rx.c
> > >       iwlwifi: make iwl_pcie_rxq_inc_wr_ptr static
> > >       iwlwifi: update the RB_TIMEOUT to 0x11
> > >       iwlwifi: remove effectless assignment
> > > 
> > > Johannes Berg (1):
> > >       iwlwifi: disallow MFP with software crypto
> > > 
> > > Sachin Kamat (1):
> > >       iwlwifi: Remove duplicate inclusion of iwl-trans.h in pcie/drv.c
> > > 
> > >  drivers/net/wireless/iwlwifi/dvm/mac80211.c  |    9 +-
> > >  drivers/net/wireless/iwlwifi/dvm/main.c      |    2 -
> > >  drivers/net/wireless/iwlwifi/iwl-config.h    |    2 +-
> > >  drivers/net/wireless/iwlwifi/iwl-fh.h        |    2 +-
> > >  drivers/net/wireless/iwlwifi/pcie/drv.c      |    1 -
> > >  drivers/net/wireless/iwlwifi/pcie/internal.h |  104 +--
> > >  drivers/net/wireless/iwlwifi/pcie/rx.c       |  357 ++++++--
> > >  drivers/net/wireless/iwlwifi/pcie/trans.c    | 1018 ++---------------------
> > >  drivers/net/wireless/iwlwifi/pcie/tx.c       | 1146 ++++++++++++++++++++------
> > >  9 files changed, 1299 insertions(+), 1342 deletions(-)
> > > 
> > > 
> > > 
> > 
> > -- 
> > John W. Linville		Someday the world will need a hero, and you
> > linville@tuxdriver.com			might be all we have.  Be ready.
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 
> 
> -- 
> John W. Linville		Someday the world will need a hero, and you
> linville@tuxdriver.com			might be all we have.  Be ready.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-11-21 19:46 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-21 14:25 pull-request: iwlwifi-next 2012-11-21 Johannes Berg
2012-11-21 18:21 ` John W. Linville
2012-11-21 18:37   ` John W. Linville
2012-11-21 19:43     ` John W. Linville

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).