On 09/11/05 06:26, Pavel Machek wrote:
> Hi!
>
>
>>+/* We know that the register wanted is in the range
>>+ * of the sliding window.
>>+ */
>>+#define ASD_READ_SW(ww, type, ord) \
>>+static inline type asd_read_##ww##_##ord (struct asd_ha_struct *asd_ha,\
>>+ u32 reg) \
>>+{ \
>>+ struct asd_ha_addrspace *io_handle = &asd_ha->io_handle[0]; \
>>+ u32 map_offs=(reg - io_handle-> ww##_base )+asd_mem_offs_##ww ();\
>>+ return asd_read_##ord (asd_ha, (unsigned long) map_offs); \
>>+}
>>+
>>+#define ASD_WRITE_SW(ww, type, ord) \
>>+static inline void asd_write_##ww##_##ord (struct asd_ha_struct *asd_ha,\
>>+ u32 reg, type val) \
>>+{ \
>>+ struct asd_ha_addrspace *io_handle = &asd_ha->io_handle[0]; \
>>+ u32 map_offs=(reg - io_handle-> ww##_base )+asd_mem_offs_##ww ();\
>>+ asd_write_##ord (asd_ha, (unsigned long) map_offs, val); \
>>+}
>>+
>>+ASD_READ_SW(swa, u8, byte);
>>+ASD_READ_SW(swa, u16, word);
>>+ASD_READ_SW(swa, u32, dword);
>>+
>>+ASD_READ_SW(swb, u8, byte);
>>+ASD_READ_SW(swb, u16, word);
>>+ASD_READ_SW(swb, u32, dword);
>>+
>>+ASD_READ_SW(swc, u8, byte);
>>+ASD_READ_SW(swc, u16, word);
>>+ASD_READ_SW(swc, u32, dword);
>>+
>>+ASD_WRITE_SW(swa, u8, byte);
>>+ASD_WRITE_SW(swa, u16, word);
>>+ASD_WRITE_SW(swa, u32, dword);
>>+
>>+ASD_WRITE_SW(swb, u8, byte);
>>+ASD_WRITE_SW(swb, u16, word);
>>+ASD_WRITE_SW(swb, u32, dword);
>>+
>>+ASD_WRITE_SW(swc, u8, byte);
>>+ASD_WRITE_SW(swc, u16, word);
>>+ASD_WRITE_SW(swc, u32, dword);
>
>
> This is certainly nice entry into "best abuse of macros" contest. Do
> you really need all those inline functions?
Hehehee, you may be right Pavel. I just wanted to minimize bugs which
could be introduced by syntax errors (not necessarily compilation errors).
Yes, reading byte/word/dword from HA memory is needed.
>>+static void __asd_write_reg_byte(struct asd_ha_struct *asd_ha, u32 reg, u8 val)
>>+{
>>+ struct asd_ha_addrspace *io_handle=&asd_ha->io_handle[0];
>>+ BUG_ON(reg >= 0xC0000000 || reg < ALL_BASE_ADDR);
>
>
> Will this work correctly with 2GB/2GB split kernels?
"reg" is an address in the host adapter's memory. Will this matter?
Luben
-
: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
