The changelog in Hayakawa-san's CardBus patches state that, in 1.3I,
the elink3 driver (ep) has an endian bug on macppc.
I haven't noticed a PR on this (but I haven't looked very hard).

Tsubai-san's fix is to use bus_space_{read,write}_multi_stream_{2,4}
when doing PIO to move packets on and off the card (see below) But
while those methods exist on macppc, they aren't't a defined part of
the bus_space(9) API.

Is the patch below from Tsubai-san a good way to fix this for 1.4?  If
not, what's a better way?

And, where is the endian-ness of a bus_space_{read,write}_multi_N
operation defined? :->.



Index: sys/dev/ic/elink3.c
diff -c sys/dev/ic/elink3.c:1.1.1.1 sys/dev/ic/elink3.c:1.1.1.1.4.1
*** sys/dev/ic/elink3.c:1.1.1.1	Wed Jan 20 22:27:51 1999
--- sys/dev/ic/elink3.c	Mon Mar 22 23:44:55 1999
***************
*** 128,133 ****
--- 128,140 ----
  int epdebug = 0;
  #endif
  
+ #ifndef __BUS_SPACE_HAS_STREAM_METHODS
+ #define bus_space_read_multi_stream_2	bus_space_read_multi_2
+ #define bus_space_read_multi_stream_4	bus_space_read_multi_4
+ #define bus_space_write_multi_stream_2	bus_space_write_multi_2
+ #define bus_space_write_multi_stream_4	bus_space_write_multi_4
+ #endif /* __BUS_SPACE_HAS_STREAM_METHODS */
+ 
  /*
   * Structure to map media-present bits in boards to ifmedia codes and
   * printable media names. Used for table-driven ifmedia initialization.
***************
*** 1118,1124 ****
  					    (void *)(mtod(m, u_long) + count);
  					m->m_len -= count;
  				}
! 				bus_space_write_multi_4(iot, ioh,
  				    txreg, mtod(m, u_int32_t *), m->m_len >> 2);
  				m->m_data = (void *)(mtod(m, u_long) +
  					(u_long)(m->m_len & ~3));
--- 1125,1131 ----
  					    (void *)(mtod(m, u_long) + count);
  					m->m_len -= count;
  				}
! 				bus_space_write_multi_stream_4(iot, ioh,
  				    txreg, mtod(m, u_int32_t *), m->m_len >> 2);
  				m->m_data = (void *)(mtod(m, u_long) +
  					(u_long)(m->m_len & ~3));
***************
*** 1141,1147 ****
  					    (void *)(mtod(m, u_long) + 1);
  					m->m_len -= 1;
  				}
! 				bus_space_write_multi_2(iot, ioh,
  				    txreg, mtod(m, u_int16_t *),
  				    m->m_len >> 1);
  			}
--- 1148,1154 ----
  					    (void *)(mtod(m, u_long) + 1);
  					m->m_len -= 1;
  				}
! 				bus_space_write_multi_stream_2(iot, ioh,
  				    txreg, mtod(m, u_int16_t *),
  				    m->m_len >> 1);
  			}
***************
*** 1602,1608 ****
  				remaining -= count;
  			}
  			if (remaining > 3) {
! 				bus_space_read_multi_4(iot, ioh,
  				    rxreg, (u_int32_t *) offset,
  				    remaining >> 2);
  				offset += remaining & ~3;
--- 1609,1615 ----
  				remaining -= count;
  			}
  			if (remaining > 3) {
! 				bus_space_read_multi_stream_4(iot, ioh,
  				    rxreg, (u_int32_t *) offset,
  				    remaining >> 2);
  				offset += remaining & ~3;
***************
*** 1621,1627 ****
  				offset += 1;
  			}
  			if (remaining > 1) {
! 				bus_space_read_multi_2(iot, ioh,
  				    rxreg, (u_int16_t *) offset,
  				    remaining >> 1);
  				offset += remaining & ~1;
--- 1628,1634 ----
  				offset += 1;
  			}
  			if (remaining > 1) {
! 				bus_space_read_multi_stream_2(iot, ioh,
  				    rxreg, (u_int16_t *) offset,
  				    remaining >> 1);
  				offset += remaining & ~1;