On Oct 13, 2011, at 10:37 AM, David Laight wrote: > >> Kumar Gala wrote: >>>>>>> + phys_addr_t addr; >>>>> >>>>> Please add a comment here saying: >>>>> >>>>> 1) That 'addr' can be a virtual or physical address >>> The code and everything else makes that clear >> >> I'm sorry, but I have to strongly disagree here. It is *NOT* >> clear that a variable of type 'phys_addr_t' can hold something >> that is not a physical address. > > Since there is a discriminating field, could a union be used? > At a guess the type of the address is constrained between > produces and consumer??
Uugh. I'll add a comment to uio_mem. - k _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev