[PATCH] scsi: prevent ISA driver from building on PPC32

STALE2939d

3 messages, 2 authors, 2018-07-23 · open the first message on its own page

[PATCH] scsi: prevent ISA driver from building on PPC32

From: Randy Dunlap <hidden>
Date: 2018-07-21 19:58:21

From: Randy Dunlap <redacted>

Prevent drivers from building on PPC32 if they use isa_bus_to_virt(),
isa_virt_to_bus(), or isa_page_to_bus(), which are not available and
thus cause build errors.

../drivers/scsi/aha1542.c: In function 'aha1542_queuecommand':
../drivers/scsi/aha1542.c:461:30: error: implicit declaration of function 'isa_page_to_bus'; did you mean 'page_to_bus'? [-Werror=implicit-function-declaration]

Signed-off-by: Randy Dunlap <redacted>
Suggested-by: Michael Ellerman <mpe@ellerman.id.au>
---
 drivers/scsi/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--- linux-next-20180720.orig/drivers/scsi/Kconfig
+++ linux-next-20180720/drivers/scsi/Kconfig
@@ -428,7 +428,7 @@ config SCSI_AHA152X
 
 config SCSI_AHA1542
 	tristate "Adaptec AHA1542 support"
-	depends on ISA && SCSI && ISA_DMA_API
+	depends on ISA && SCSI && ISA_DMA_API && !PPC32
 	---help---
 	  This is support for a SCSI host adapter.  It is explained in section
 	  3.4 of the SCSI-HOWTO, available from

Re: [PATCH] scsi: prevent ISA driver from building on PPC32

From: Christoph Hellwig <hch@infradead.org>
Date: 2018-07-23 08:18:11

On Sat, Jul 21, 2018 at 12:58:21PM -0700, Randy Dunlap wrote:
From: Randy Dunlap <redacted>

Prevent drivers from building on PPC32 if they use isa_bus_to_virt(),
isa_virt_to_bus(), or isa_page_to_bus(), which are not available and
thus cause build errors.
Please don't introduce weird arch dependencies, and add a
CONFIG_ISA_VIRT_TO_BUS instead.

Re: [PATCH] scsi: prevent ISA driver from building on PPC32

From: Christoph Hellwig <hch@infradead.org>
Date: 2018-07-23 08:19:38

On Mon, Jul 23, 2018 at 01:18:11AM -0700, Christoph Hellwig wrote:
On Sat, Jul 21, 2018 at 12:58:21PM -0700, Randy Dunlap wrote:
quoted
From: Randy Dunlap <redacted>

Prevent drivers from building on PPC32 if they use isa_bus_to_virt(),
isa_virt_to_bus(), or isa_page_to_bus(), which are not available and
thus cause build errors.
Please don't introduce weird arch dependencies, and add a
CONFIG_ISA_VIRT_TO_BUS instead.
And in fact we have so few drivers that we should just kill off the
API entirely instead.  I'll take care of aha1542 in the next week or
so.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help