On Tue, Feb 08, 2011 at 03:19:25AM +0300, Alexander Gordeev wrote: > Ð Mon, 7 Feb 2011 10:53:25 -0800 > Tirumala Marri <tmarri@xxxxxxx> ÐÐÑÐÑ: > > > dwc_read_reg32 is used nowhere throughout the code. One of dwc_read32 and > > dwc_read_reg32 should be removed IMO. There was once only dwc_read_reg32. In > > version 5 of your patchset I believe. Why did you add another function? > > AFAIK it is not correct to store pointers in u32 because they need 8 bytes > > on 64-bit archs. So it was ok with the old dwc_read_reg32. > > [Marri] If u32 is 8bytes isn't pointer type would be 8bytes as well. > > Sorry, I don't understand that. I think u32 is always 32bit = 4byte on > all archs. Right? Yes. Use an unsigned long if you want to hold a pointer correctly on all arches. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
