On 11/21/2012 03:45 PM, Felipe Balbi wrote: > On Thu, Nov 15, 2012 at 04:34:07PM +0200, Roger Quadros wrote: >> For some platforms e.g. OMAP5, we cannot rely on USBHOST revision >> to determine the number of ports available. In such cases we have > > you need to make it clear *why* we can't. Imagine someone reading this 5 > years from now... he'll be all like: "why can't I find any documentation > about this OMAP5 ? Why was it so special that its revision register > wasn't enough to figure out number of ports ?" OK, i'll add a note like this "both OMAP5 and OMAP4 exhibit the same revision ID in the USBHOST_REVISION register, but in fact have different number of ports physically available on the SoC (i.e. 2 for OMAP4 and 3 for OMAP5 respectively). So we can't rely on REVISION register to determine number of ports for OMAP5 and depend on platform data/Device tree instead" cheers, -roger -- 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