advansys broken (at least) on ARM and MIPS

2008-01-03 Thread Martin Michlmayr
Commit 9d511a4b29de6764931343d03e493f2e04df0271 removed the depends
on BROKEN || X86_32 line from advansys' Kconfig entry.  I'm not sure
that was such a great idea - the module doesn't compile at least on
ARM and MIPS:

ARM:

  CC [M]  drivers/scsi/advansys.o
drivers/scsi/advansys.c:71:2: warning: #warning this driver is still not 
properly converted to the DMA
 API
drivers/scsi/advansys.c: In function ‘AdvBuildCarrierFreelist’:
drivers/scsi/advansys.c:6486: warning: ‘virt_to_bus’ is deprecated (declared at 
include/asm/memory.h:1
92)
drivers/scsi/advansys.c:6486: warning: ‘virt_to_bus’ is deprecated (declared at 
include/asm/memory.h:1
92)
drivers/scsi/advansys.c:6486: warning: ‘virt_to_bus’ is deprecated (declared at 
include/asm/memory.h:1
92)
...
drivers/scsi/advansys.c: In function ‘advansys_get_sense_buffer_dma’:
drivers/scsi/advansys.c:9885: error: implicit declaration of function 
‘dma_cache_sync’

MIPS:

  Building modules, stage 2.
  MODPOST 64 modules
ERROR: free_dma [drivers/scsi/advansys.ko] undefined!

-- 
Martin Michlmayr
http://www.cyrius.com/
-
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: advansys broken (at least) on ARM and MIPS

2008-01-03 Thread Kyle McMartin
On Thu, Jan 03, 2008 at 08:43:53AM +0100, Martin Michlmayr wrote:
 MIPS:
 
   Building modules, stage 2.
   MODPOST 64 modules
 ERROR: free_dma [drivers/scsi/advansys.ko] undefined!
 

needs to depend on GENERIC_ISA_DMA_API
-
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: advansys broken (at least) on ARM and MIPS

2008-01-03 Thread James Bottomley

On Thu, 2008-01-03 at 08:43 +0100, Martin Michlmayr wrote:
 Commit 9d511a4b29de6764931343d03e493f2e04df0271 removed the depends
 on BROKEN || X86_32 line from advansys' Kconfig entry.  I'm not sure
 that was such a great idea - the module doesn't compile at least on
 ARM and MIPS:
 
 ARM:
 
   CC [M]  drivers/scsi/advansys.o
 drivers/scsi/advansys.c:71:2: warning: #warning this driver is still not 
 properly converted to the DMA
  API
 drivers/scsi/advansys.c: In function ‘AdvBuildCarrierFreelist’:
 drivers/scsi/advansys.c:6486: warning: ‘virt_to_bus’ is deprecated (declared 
 at include/asm/memory.h:1
 92)
 drivers/scsi/advansys.c:6486: warning: ‘virt_to_bus’ is deprecated (declared 
 at include/asm/memory.h:1
 92)
 drivers/scsi/advansys.c:6486: warning: ‘virt_to_bus’ is deprecated (declared 
 at include/asm/memory.h:1
 92)
 ...
 drivers/scsi/advansys.c: In function ‘advansys_get_sense_buffer_dma’:
 drivers/scsi/advansys.c:9885: error: implicit declaration of function 
 ‘dma_cache_sync’

That error doesn't look to tbe the fault of the driver.
dma_cache_sync() is a piece of the DMA API it looks like ARM doesn't
implement?

James


-
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: advansys broken (at least) on ARM and MIPS

2008-01-03 Thread Martin Michlmayr
Adding Russell King to CC who can hopefully answer James' question
below.

* James Bottomley [EMAIL PROTECTED] [2008-01-03 09:21]:
 On Thu, 2008-01-03 at 08:43 +0100, Martin Michlmayr wrote:
  Commit 9d511a4b29de6764931343d03e493f2e04df0271 removed the depends
  on BROKEN || X86_32 line from advansys' Kconfig entry.  I'm not sure
  that was such a great idea - the module doesn't compile at least on
  ARM and MIPS:
  
  ARM:
  
CC [M]  drivers/scsi/advansys.o
  drivers/scsi/advansys.c:71:2: warning: #warning this driver is still not 
  properly converted to the DMA
   API
  drivers/scsi/advansys.c: In function ‘AdvBuildCarrierFreelist’:
  drivers/scsi/advansys.c:6486: warning: ‘virt_to_bus’ is deprecated 
  (declared at include/asm/memory.h:1
  92)
  drivers/scsi/advansys.c:6486: warning: ‘virt_to_bus’ is deprecated 
  (declared at include/asm/memory.h:1
  92)
  drivers/scsi/advansys.c:6486: warning: ‘virt_to_bus’ is deprecated 
  (declared at include/asm/memory.h:1
  92)
  ...
  drivers/scsi/advansys.c: In function ‘advansys_get_sense_buffer_dma’:
  drivers/scsi/advansys.c:9885: error: implicit declaration of function 
  ‘dma_cache_sync’
 
 That error doesn't look to tbe the fault of the driver.
 dma_cache_sync() is a piece of the DMA API it looks like ARM doesn't
 implement?
 
 James
 

-- 
Martin Michlmayr
http://www.cyrius.com/
-
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html