All of lore.kernel.org
 help / color / mirror / Atom feed
* changes to drivers/scsi/sym53c8xx_defs.h in the mips tree
@ 2005-01-29 16:37 Christoph Hellwig
  2005-01-30  1:18 ` Maciej W. Rozycki
  0 siblings, 1 reply; 3+ messages in thread
From: Christoph Hellwig @ 2005-01-29 16:37 UTC (permalink / raw)
  To: carstenl; +Cc: linux-mips

Hi Carsten,

do you remember what your changes to drivers/scsi/sym53c8xx_defs.h in
the mips tree are supposed to help with?

sym53c8xx_defs.h is only used for the ncr53c8xx driver, which despite
the name is only for NCR53c720 chips these days (supported on parisc,
and eisa/mca).  It has correct big endian support for parisc alrady.

Also the patch duplicates your changelog and copyright entry that's
in the file already.

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

* Re: changes to drivers/scsi/sym53c8xx_defs.h in the mips tree
  2005-01-29 16:37 changes to drivers/scsi/sym53c8xx_defs.h in the mips tree Christoph Hellwig
@ 2005-01-30  1:18 ` Maciej W. Rozycki
  2005-01-30 10:59   ` Christoph Hellwig
  0 siblings, 1 reply; 3+ messages in thread
From: Maciej W. Rozycki @ 2005-01-30  1:18 UTC (permalink / raw)
  To: Christoph Hellwig; +Cc: linux-mips

On Sat, 29 Jan 2005, Christoph Hellwig wrote:

> Hi Carsten,

 I'm afraid Carsten may no longer be reachable, but I may try to help in 
his stead.

> do you remember what your changes to drivers/scsi/sym53c8xx_defs.h in
> the mips tree are supposed to help with?

 It must have been the MIPS Technologies' Atlas development board which 
has a SYM53C810A incorporated, even though the resulting changes are 
specific to the board.  There appears to have been some sort of confusion 
about the semantics of i/o and mmio operations in mixed-endian 
configurations, which for the MIPS port actually is currently being 
resolved.

> sym53c8xx_defs.h is only used for the ncr53c8xx driver, which despite
> the name is only for NCR53c720 chips these days (supported on parisc,
> and eisa/mca).  It has correct big endian support for parisc alrady.

 I can verify whether the other driver operates correctly in the four 
possible configurations with an Atlas board.

 BTW, that SCSI_NCR_BIG_ENDIAN macro looks scary -- is the chip wired 
incorrectly (i.e. with byte lanes swapped at PCI) or are i/o and/or mmio 
operations simply broken for parisc?

> Also the patch duplicates your changelog and copyright entry that's
> in the file already.

 It looks like some sort of a merging mistake.

  Maciej

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

* Re: changes to drivers/scsi/sym53c8xx_defs.h in the mips tree
  2005-01-30  1:18 ` Maciej W. Rozycki
@ 2005-01-30 10:59   ` Christoph Hellwig
  0 siblings, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2005-01-30 10:59 UTC (permalink / raw)
  To: Maciej W. Rozycki; +Cc: linux-mips

On Sun, Jan 30, 2005 at 01:18:52AM +0000, Maciej W. Rozycki wrote:
>  BTW, that SCSI_NCR_BIG_ENDIAN macro looks scary -- is the chip wired 
> incorrectly (i.e. with byte lanes swapped at PCI) or are i/o and/or mmio 
> operations simply broken for parisc?

Many parisc systems have a NCR720 on the HP GSC bus, which actually
is accessed as big endian.

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

end of thread, other threads:[~2005-01-30 10:59 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-01-29 16:37 changes to drivers/scsi/sym53c8xx_defs.h in the mips tree Christoph Hellwig
2005-01-30  1:18 ` Maciej W. Rozycki
2005-01-30 10:59   ` Christoph Hellwig

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.