Re: [PATCH] small sym-2 fix

[Gérard Roudier]

Hello,

On Tue, 20 Nov 2001, Anton Blanchard wrote:

> Hi,
>
> > Could you revert your change and give my patch below a try. Btw, you will
> > be in sync with my current sources. Booting with sym53c8xx=debug:1 will
> > let the driver print all memory allocations to the syslog. You may send me
> > the drivers messages related to these allocations for information.
>
> Thanks, it boots OK now. Do you still want a debug log?

I can guess the result.

> BTW on ppc64 we can have io port addresses > 32 bits so this change is
> required.

Linux/ppc64 looks strange invention to me. As you know IO base addresses
are limited to 32 bit in PCI. And, btw, 32 bits seems to work just fine
here as PPC is defined from the driver as using normal IO. But, IIRC, the
strange Linux/PPC invention only supports MMIO. :-)

If you want to play with _explicit_ MMIO, you just have to remove a couple
of line from sym53c8xx.h. Here they are:

  /*
   *  Use normal IO if configured. Forced for alpha and powerpc.
   *  Powerpc fails copying to on-chip RAM using memcpy_toio().
   *  Forced to MMIO for sparc.
   */
  #if defined(__alpha__)
  #define	SYM_CONF_IOMAPPED
  #elif defined(__powerpc__)
- #define	SYM_CONF_IOMAPPED
- #define SYM_OPT_NO_BUS_MEMORY_MAPPING
  #elif defined(__sparc__)
  #undef SYM_CONF_IOMAPPED
  #elif defined(CONFIG_SCSI_SYM53C8XX_IOMAPPED)
  #define	SYM_CONF_IOMAPPED
  #endif

Btw, I cannot guess the result here. You may want to really let me know
this time. :)

I cannot apply your patch as it is, since I want the driver to distinguish
between kernel fake cookies associated with base addresses and actual
values of those registers. This is needed, since some of these values must
be known from SCSI SCRIPTS and thus must fit the _reality_ and not any
kernel developpers' dream, could be the greatest ones:).

Thanks, anyway, for reporting this problem.

Regards,
  Gérard.

> diff -urN linuxppc_2_4_devel/drivers/scsi/sym53c8xx_2/sym_glue.h linuxppc_2_4_devel_work/drivers/scsi/sym53c8xx_2/sym_glue.h
> --- linuxppc_2_4_devel/drivers/scsi/sym53c8xx_2/sym_glue.h	Mon Nov 12 11:46:42 2001
> +++ linuxppc_2_4_devel_work/drivers/scsi/sym53c8xx_2/sym_glue.h	Tue Nov 20 16:35:14 2001
> @@ -463,7 +462,7 @@
>
>  	vm_offset_t	mmio_va;	/* MMIO kernel virtual address	*/
>  	vm_offset_t	ram_va;		/* RAM  kernel virtual address	*/
> -	u32		io_port;	/* IO port address		*/
> +	u_long		io_port;	/* IO port address		*/
>  	u_short		io_ws;		/* IO window size		*/
>  	int		irq;		/* IRQ number			*/