On Tue, Aug 07, 2007 at 12:26:04AM -0700, David Miller wrote: > > - struct sbus_dma *dma; > > + union { > > + struct sbus_dma *sbus_dma; > > + unsigned int x86_dma; > > + }; > > }; > > Feel free to make this a "void *dma_cookie" or similar. > It's private to the bus front-end. Hi Dave, Could I just clarify; would you prefer it to be a void *? I prefer the anonymous union that I have there right now, but I'm not particularly attached to it. In particular, I don't really care to be casting ints (x86) to pointers, but that's a matter of personal taste. -- "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." - To unsubscribe from this list: 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